Ecrire dans une variable globale depuis une procédure ?!?

Version imprimable

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:

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 :)

04/02/2006, 16h40
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...
04/02/2006, 20h10
Invité

Citation:

Envoyé par 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...

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? :roll:

Par convention, on dit habituellement qu'une procedure retourne rien (void), alors qu'une fonction retourne une valeur.
04/02/2006, 21h39
bigboomshakala

Re: Ecrire dans une variable globale depuis une procédure ?!

Citation:

Envoyé par 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.

salut

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

:merci:
05/02/2006, 00h00
Goundy

Citation:

Envoyé par voider

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

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?
05/02/2006, 00h17
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.
05/02/2006, 04h18
Invité

Citation:

Envoyé par Goundy

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).
05/02/2006, 11h29
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.

Voila la procédure :

Code:

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 :)

08/02/2006, 16h18
Jan Rendek

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

Code:

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 :D)

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
08/02/2006, 16h43
Blowih

salut,
essaye ce proto pour ta fonction

Code:

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
08/02/2006, 16h59
damien99

En effet le problème était ici, merci :)