Pb affectation champs d'une struct passée en pointeur

Bonjour,

j'ai un ptit souci avec une struct passée en pointeur en argument d'une méthode
et en fait, elle n'est pas mise à jour au retour,
je sais pas trop comment ça se fait...

donc si vous pourriez me donner un ptit coup de pouce ;) ...

Alors déjà, attention,
ça peut paraître un peu barbare...
mais j'utilise des types définis dans une librairie C...


Donc, j'ai 2 structures :

Code:

---

1 2 3 4 5 6 7 8 9

struct Message{ unsigned char nature; char nomUtilisateur[MAX_SIZE]; unsigned short periode; unsigned char data;};   struct Unite{ unsigned short longueur; Message mess;};

---

J'ai une méthode qui me sert à remplir une structure 'Unite' :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

MaMethode(unsigned short *lgMess, Message *mess) { unsigned short lgData; unsigned char *donnees;   //...calcul de la longueur... *lgMess = sizeof(Message) - 1 + lgData;   // allocation de la structure du message mess = (Message*) malloc(*lgMess); (void) memset(mess, 0, *lgMess);   // affectation des champs de la structure mess->nature = ... StringCbCat(mess->nomUtilisateur, MAX_SIZE, "nomUtilisateur");   donnees = new unsigned char[lgData]; //... remplissage du tableau...   //copie dans le message memcpy(&mess->data, donnes, lgData);   delete[] donnees; }

---

Et donc, quand j'appelle cette méthode :

Code:

---

1 2 3

Unite monUnite;   MaMethode(&monUnite.longueur, &monUnite.mess);

---

eh ben, le champ 'longueur' est bien à jour
mais aucun champ de la struct 'mess':(

Pourtant, quand je rentre en debug,
ça paraît tout bien...


Alors alors??
z'avez des pistes?...

Dans ta fonction alloue de l'espace pour un autre objet "Message" que tu affecte à ton pointeur :

Code:

---

mess = (Message*) malloc(*lgMess);

---

Donc ton pointeur ne pointe plus sur l'objet que tu a passé en paramètre mais sur ce nouvel objet que tu viens de créer.

En plus tu fait des allocations avec malloc un peu plus loin avec new, c'est vraiement un melange de C / C++

Houla, tu veux passer quelques nuits blanches avec ce code pour comprendre les futurs bugs ?

Code:

---

*lgMess = sizeof(Message) - 1 + lgData;

---

lgData n'est pas initialisé.
sizeof(Message) fait 4 ou 8 (c'est un pointeur) mais certainement pas ce que tu penses.

Tu passes un pointeur sur Message en paramètre. Donc quand tu quittes ta fonction, le pointeur dans l'appelant n'est pas modifié. 2 manières de corriger cela, passer un pointeur de pointeur sur Message en paramètre ou alors retourner un pointeur sur Message en retour de fonction.

Citation:

---

Envoyé par Lucien63

Dans ta fonction alloue de l'espace pour un autre objet "Message" que tu affecte à ton pointeur :

Code:

---

mess = (Message*) malloc(*lgMess);

---

Donc ton pointeur ne pointe plus sur l'objet que tu a passé en paramètre mais sur ce nouvel objet que tu viens de créer.

---

ah ouais ok,
pff, j'suis trop nulle avec les pointeurs et les allocations mémoires:?

Donc, en fait, il vaudrait mieux que je passe un autre argument pour pour le tableau de données
et que je l'affecte au champ de ma structure ensuite, au retour de la méthode,
un truc dans le genre, c'est ça??

Citation:

---

Envoyé par Lucien63

En plus tu fait des allocations avec malloc un peu plus loin avec new, c'est vraiement un melange de C / C++

---

En ben, voui,
c'est bien ce que je disais
c'est un peu barbare:mouarf:

Salut,

toujours se ramener à quelque chose de simple pour voir si le principe marche!

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29

#include <iostream> using namespace std;   struct Message { int a, b; };   struct Unite { long l; Message mess; };   void MaMethode(long *l, Message *mess) { *l=5; mess->b = 0; mess->a = 8; }   int main() { Unite u; MaMethode(&u.l, &u.mess);   cout << u.l << u.mess.b << u.mess.a << endl; return 0; }

---

Donc en principe ton code marche (note qu'au lieu d'utiliser des pointeurs tu peux utiliser des références, cf faq).

En fait ton problème c'est que tu ne touches pas à l'adresse pointée par le mess passé en argument.

Je m'explique:

Code:

---

1 2 3 4 5 6 7 8

unsigned short lgData; unsigned char *donnees;   //...calcul de la longueur... *lgMess = sizeof(Message) - 1 + lgData;   // allocation de la structure du message mess = (Message*) malloc(*lgMess);

---

A la dernière ligne, tu changes le contenu de la variable mess, sans avoir jamais utilisé le contenu de cette variable avant. Ta fonction se comporte exactement comme celle-ci:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

MaMethode(unsigned short *lgMess) { unsigned short lgData; unsigned char *donnees;   //...calcul de la longueur... *lgMess = sizeof(Message) - 1 + lgData;   // allocation de la structure du message Message *mess = (Message*) malloc(*lgMess); ... }

---

Comment résoudre ce problème?
Bah en enlevant cette ligne:

Code:

---

1 2	// allocation de la structure du message mess = (Message*) malloc(*lgMess);

---

car *mess est déjà alloué statiquement car monUnité est bien allouée statiquement.

Edit:

Je me rends compte avec les posts ci-dessus que tu peux même enlever ces trois lignes:

Code:

---

1 2 3 4 5

*lgMess = sizeof(Message) - 1 + lgData;   // allocation de la structure du message mess = (Message*) malloc(*lgMess); (void) memset(mess, 0, *lgMess);

---

Et qu'il faut régler le problème avec lgData.

Citation:

---

Envoyé par ram-0000

sizeof(Message) fait 4 ou 8 (c'est un pointeur) mais certainement pas ce que tu penses.

---

Il me semble que Message est une structure !

oui, Message est une structure


en faisant

Code:

---

mess = (Message*) malloc(*lgMess);

---

je veux allouer une zone mémoire plus grande que celle de la structure
afin d'y mettre mon tableau data au bout...

Vous comprenez??...


C'était fait comme ça dans l'appli en C dont je m'inspire...

Mais le problème c'est que monUnite.mess est allouée statiquement.

C'est comme si tu faisais:

Code:

---

1 2	int a = 1; (&a) = malloc(sizeof(int));

---

C'est absurde de faire ça, n'est-ce pas?

Tu ne peux pas changer l'adresse de a puisque la variable est allouée statiquement.

Et d'ailleurs, Message est toujours de la même taille:

Code:

---

1 2 3 4 5

struct Message{ unsigned char nature; char nomUtilisateur[MAX_SIZE]; unsigned short periode; unsigned char data;};

---

Il n'a pas une taille variable, donc si tu veux en allouer un dynamiquement tu fais:

Code:

---

Message *pointeur = malloc(sizeof(Message));

---

ou en C++:

Code:

---

Message *pointeur = new Message;

---

Si tu veux rajouter un tableau data à la fin, il faut changer ta structure:

Code:

---

1 2 3 4 5 6

struct Message{ unsigned char nature; char nomUtilisateur[MAX_SIZE]; unsigned short periode; unsigned char * data; //un POINTEUR };

---

Et alors tu procéderais comme ça:

Code:

---

1 2	Message mess; mess->data = new data[lgData+1];

---

Ou alors dans un esprit C++, tu ferais de Message une classe et lui donnerais un constructeur, des fonctions membres et un destructeur pour gérer l'allocation et la libération de data.

Ou tu utiliserais carrément des string...

Citation:

---

Envoyé par Dreambeliever

Si tu veux rajouter un tableau data à la fin, il faut changer ta structure:

Code:

---

1 2 3 4 5 6

struct Message{ unsigned char nature; char nomUtilisateur[MAX_SIZE]; unsigned short periode; unsigned char * data; //un POINTEUR };

---

Et alors tu procéderais comme ça:

Code:

---

1 2	Message mess; mess->data = new data[lgData+1];

---

---

Je suis bien d'accord...

mais le souci,
c'est que ma variable de type Message,
une fois remplie,
je dois la passer à une des fonctions de la librairie C sous forme de tableau de char:?
pour cela, il faut donc que tous les champs de la structure soient contigus dans une même zone mémoire
et ça ne serait pas le cas avec le pointeur pour le champ data...

Donc alors faire comme ça:

Citation:

---

je viens de me rendre compte de ce que tu veux faire: allouer plus d'espace que prévu pour une structure et écrire ton data dans le surplus de mémoire.

C'est possible en C, en C++ aussi bien que fortement déconseillé en C++ (car alors on ne peut pas faire d'héritage, ...), mais pas dans ce cas là car ta variable mess est déjà allouée statiquement.

Si tu veux le faire il faut pouvoir l'allouer dynamiquement, c'est -à-dire changer la déclaration de la structure Unite pour mettre mess en pointeur (et alors utiliser un pointeur de pointeur en argument de MaMethode)

---

Pourrais-tu m'en dire plus
pleeaaaaaaase:zoubi:

En reprenant ton code et en utilisant une référence sur pointeur au lieu d'un double pointeur:

Code:

---

1 2 3 4 5 6 7 8 9 10 11

struct Unite{ unsigned short longueur; Message * mess; };   MaMethode(unsigned short *lgMess, Message *& mess) { ... }   MaMethode(&monUnite.longueur, monUnite.mess);

---

avec un double pointeur:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

struct Unite{ unsigned short longueur; Message * mess; };   MaMethode(unsigned short *lgMess, Message ** p_mess) { // utiliser (*p_mess) au lieu de mess ... }   MaMethode(&monUnite.longueur, &monUnite.mess);

---

:ccool:
yeah,

j'essaie ça demain
et vous tiens au courant :zoubi:

Muchas gracias!!

j'ai essayé avec une référence sur le pointeur,
ça marche nickel;)

trop cool,
depuis le temps que je galérais là-dessus...