Problème struct et union

Bonjour,

pour faire une communication USB j'ai créer la structure suivant :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

//Structure pour les données mémoire struct Memoire { unsigned char iAdresse_I2C; unsigned int iAdresse; unsigned char iNb_Octet; unsigned char iDonnees[57]; };   union Donnees_Produit { //Pour l'affectation de toute la mémoire unsigned char pDonnees[63];   //Structure pour les données mémoire Memoire pMemoire; };

---

dans mon code j'affecte la structure mémoire de la structure Donnees_Produit avec les informations suivantes :
iAdresse_I2C = 0xA6
iAdresse = 0x12345678
iNb_Octet = 0x57
iDonnees = données à écrire

ce que j'attends c'est d'avoir dans mon tableau pDonnes[63] :

0xA6
0x78
0x56
0x34
0x12
0x57
...

or

pDonnes[63] contient
0xA6
0x00
0x00
0x00
0x78
0x56
0x34
0x12
0x57

j'ai l'impression que QT lors de la compilation change le type de données de la variable iAdresse_I2C

Savez-vous comment corriger ce problème ?

Merci par avance pour votre aide, je suis débutant sur QT.

Cordialement

Bonjour,

Qt n'y est pour rien. Le coupable c'est ton processeur. Comme il est difficile d'en changer, il faut comprendre le problème pour pouvoir le gérer.
La plupart des processeurs ont des contraintes pour accéder à une données qui dépendent de son type. Par exemple pour ton processeur, un int doit avoir une taille de 4, une adresse multiple de 4 (alignement=4) et stocke en premier l'octet de poids faible puis les autres en finissant par le poids fort (little endian). D'autres processeurs ont des int alignés 2, 4 ou 8, utilisent le big endian(comme les humain en terminant par les unités) ou leur taille fait 2, 4 ou 8 octets.
Une structure a l'alignement qui est le plus grand alignement de ses composants. Donc ici Memoire et Memoire::iAdresse doivent être à des adresses multiples de 4 et doivent avoir une taille multiple de 4. Pas d'autre solution que de rajouter un trou de 3 octets après iAdresse_i2c et un trou de 2 octets après iDonnees[]. Tu peux vérifier qu'un sizeof(Memoire) donne 63+3+2=68 qui est multiple de 4.
Le problème est d'imaginer que le buffer à transmettre à un équivalent simple en mémoire.
Je te propose 2 solutions.
La première est assez simple à écrire mais pose un énorme problème de compatibilité. Il suffit de déclarer la structure comme étant [[packed]] ou __attribute__((__packed__)) ou #pragma pack(1) ou ... (la syntaxe dépend de ton compilateur pas de Qt), le compilateur va compresser la structure pour y mettre un 'faux' int qui sera composé 4 octets comme un int. Quand le code voudra y écrire ou lire un int, le compilateur produira la code qui fera qu'on semblera y accéder comme un vrai int.
La seconde est plus complexe et consiste à mieux comprendre qu'un buffer à transmettre n'est pas une zone qui correspond forcément à des variables du langage (remarque que dans la plupart des autres langages ton idée n'aurait aucun équivalent.)

La première chose à faire, est de réorganiser ses données en fonction du type : les types qui prennent le + d'octets en premier et les plus petits en dernier.

Dans ton cas :mrgreen: (en 32 bits - 4 octets), si tu mets en premier iAdresse, ensuite ton tableau iDonnees[57] et enfin tes 2 autres membres iNb_Octet et iAdresse_I2C, ton bourrage ("padding" en anglais) devrait se produire à la fin de la structure (4 + 57 + 2 + 1 de paddding), au pire après ton tableau et à la fin (4 + 57 + 3 de padding + 2 + 2 de paddding).

Une autre solution (celle de mon V.D.D., voisin du dessus @dalfab), c'est de créer un tableau de 60, et de le transmettre qu'à partir de la case 3, parce qu'au début tu as mis tes 2 membres (<- cela fait une structure avec des décalages ("offset" en anglais) pour récupérer les bonnes valeurs)

Un exemple simple :wow: (non compilé), si tous tes membres sont de type char :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

typedef enum e_mem_member { MEM_MEMBER_BYTE = 0, MEM_MEMBER_I2C_ADDRESS = 1, MEM_MEMBER_END = 3 } t_mem_member;     typedef struct s_mem { unsigned int address; unsigned char data[60]; } t_mem;     void mem_set_member(s_mem* mem, t_mem_member member, char value) { if ((mem != NULL) && (member < MEM_MEMBER_END)) { *(mem->data + member) = value; } }

---

Bonjour

Un autre souci avec ton idée, c'est qu'elle est brillante mais interdite. Une union n'est pas faite à l'origine pour transformer une représentation des données mais juste pour gagner de la place (à l'époque ou la place était rare). Donc si tu es sûr de ne jamais utiliser a et b en même temps, tu as alors possibilité d'inclure a et b dans une union pour que ces deux variables occupent le même espace mémoire. Ensuite tu remplis "a", puis tu traites "a". Puis tu remplis "b", puis tu traites "b". Mais tu n'as pas le droit de remplir "a" puis d'aller regarder ce que ça donne dans "b" car ça donne un UB.

Donc si tu veux splitter ta structure dans un tableau de char, tout en prenant en compte (en plus) le fait que la norme précise juste qu'un int fait "au minimum" 4 octets (donc il a parfaitement le droit d'en faire plus et j'ai connu une fois un os où il en faisait 6 !!!), il te faut l'écrire de cette façon

Code:

---

1 2 3 4 5 6 7 8 9 10 11

//Structure pour les données mémoire struct s_Memoire { unsigned char iAdresse_I2C; unsigned int iAdresse; unsigned char iNb_Octet; unsigned char iDonnees[57]; } pMemoire;   unsigned char *pDonnees; pDonnees=malloc(sizeof(struct sMemoire)); memcpy(pDonnees, &pMemoire, sizeof(struct sMemoire));

---