Discussion :

[damien99]

Salut,

J'ai créer une structure (voir ci-dessous) mais quant j'essaye de modifier la variable "statut" depuis une procédure la variable ne change pas de valeur. Par contre quant j'attribut une valeur a cette variable depuis le corps du programme pas de problème.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
typedef struct
{
   int num;
   SOCKET socket;
   int statut;
   char pseudo[13];
   SOCKADDR_IN csin;
}CLIENT;
 
CLIENT client1, client2, client3, client4;  //déclaration (variables globales)

Merci de m'aider

[Sve@r]

Bon, déjà la notion de procédure n'existe pas en C++ ni en C. Ptet que tu te crois en Pascal...

Ensuite, ben il n'y a aucun problème. Tu peux modifier "client1.statut" comme tu veux d'où tu veux. De même pour "client2.statut" et les autres...

[Invité]

Bon, déjà la notion de procédure n'existe pas en C++ ni en C. Ptet que tu te crois en Pascal...

De quoi tu jase?

Heh, bien sur que ca existe des procedures en c/c++. Tu aurais pas besoin d'un petit cours avant de repondre aux gens?

Par convention, on dit habituellement qu'une procedure retourne rien (void), alors qu'une fonction retourne une valeur.

[bigboomshakala]

Salut,

J'ai créer une structure (voir ci-dessous) mais quant j'essaye de modifier la variable "statut" depuis une procédure la variable ne change pas de valeur. Par contre quant j'attribut une valeur a cette variable depuis le corps du programme pas de problème.

salut

si tu nous montrais comment tu fais qu'on puisse voir ce qui cloche ?

[Goundy]

Heh, bien sur que ca existe des procedures en c/c++. Tu aurais pas besoin d'un petit cours avant de repondre aux gens?

pour moi il a raison, la notion de procédure n'existe pas en C/C++, même jamais utilisé, même si une fonction ne retournant rien peut être interpreté en tant que procédure,
aurai-tu besoins d'une bonne introduction avant de casser les gens?


Pour le monsieur, montre comment tu assigne une valeur à status, sinon pourquoi ne pas passer aux variables locales?

[Luc Hermitte]

Hyppothèses:
- variables locales qui masquent les variables globales
- paramètres passés par valeur

Et effectivement, on ne parle ni de "procédure" ni de "méthode" en C++. Des équivalents peuvent être réalisés à l'aide de fonctions toutes fois.

[Invité]

aurai-tu besoins d'une bonne introduction avant de casser les gens?

Hehe, au moins m'a tu fait sourire

Je maintient tout de meme mon point, peu importe comment on peut l'appeler, le concept de "procedure" existe en c++. Dans le sens ou (on me l'a toujours apprit/dit ainsi) une procedure est une fonction qui ne retourne pas de valeur (void).

[Rafy]

Je pense que le problème vient, comme l'a déjà dit Luc Hermitte
, c'est un passage de la structure, à la fonction qui est sensée la modifier, par valeur. Ainsi le compilateur va copier la structure dans la liste d'argument, et en fait la fonction va modifier la copie mais pas l'objet copié...
Il faut voir un passage par pointeur ou référence de la structure à modifier.
Il faudrai que tu nous donne le code de ta fonction.

[damien99]

Voila la procédure :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
/**********La prcédure**********/
void acceptation(HWND dlg, CLIENT cl)
{
   if(cl.statut==0)     {
      sinsize = sizeof(cl.csin);
      if((cl.socket = accept(sock, (SOCKADDR *)&cl.csin, &sinsize)) != INVALID_SOCKET)     {
         sprintf(buffer,"%cC%c",255,1);
         send(cl.socket,buffer,530,0);
         SetDlgItemText(dlg,120+cl.num,"Occupé");
         cl.statut=1;  /***quant je teste "cl.statut" est à 1***/
      }
   }
}
/**********Appel de la prcédure (dans le programme)**********/
acceptation(hDlg,client1); /*** client1.statut ne veut pas passer à 1 losrque qu'un client se connecte (quant il se connecte client1 biensur)***/
acceptation(hDlg,client2);/***idem***/
acceptation(hDlg,client3);/***idem***/
acceptation(hDlg,client4);/***idem***/

Pourquoi "cl.statut" dans la structure est = a 1et dans le prgramme les client1/2/3/4 ne sont pas a 1 :

Il y a aussi un autre problème, la ligne "SetDlgItemText(dlg,120+cl.num,"Occupé");" n'a aucun effet sur le programme. (ça c'est plutot du Visual mais si quelqu'un a une idée...)

Merci de m'aider

[Jan Rendek]

Ta fonction acceptation telle que définie accepte ses paramètres par copie.
Lorsque tu l'appelles à la ligne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

acceptation(hDlg,client1);

Le compilateur génère une copie de hDlg et de client1. C'est donc sur une copie de client1 et non pas sur la véritable variable client1 que acceptation() travaille.
Au retour de cette fonction, seul la copie à été modifiée et non l'original. Ce qui explique que client1.status ne soit pas modifié.

EDIT1: +1 à LH, puisque sont hypothèse 2 est la bonne (d'ailleurs je suppose que comme d'autres ici, il doit donner quelques cours de C++ débutant )

EDIT2: Pour ce qui est de l'ergotage procedure/fonction je trouve que pourrir un thread quand on à rien à apporter plus que moyen. On peut avec une bonne intention répondre à coté (je plaide coupable), mais je pense qu'un forum n'est pas une ressource WRITE ONLY

[Blowih]

salut,
essaye ce proto pour ta fonction

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

void acceptation(HWND dlg, CLIENT& cl)

le param cl est passé par référence grace au &, comme ça il n'y a pas de recopie de ton objet, il est directement accesible dans ta fonction et tu peux le modifier

[damien99]

En effet le problème était ici, merci