Initialisation d'un tableau de 2 dimensions de structures

Bonjour,

j'ai récupérer du code source qui défini une structure I2C_UNIT:

Code:

---

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

struct I2C_UNIT { Hal_Queue_UnknownSize<I2C_WORD> m_dataQueue; size_t m_bytesTransferred; size_t m_bytesToTransfer; BOOL m_fRead;   BOOL I2C_Initialize(); BOOL I2C_Uninitialize(); BOOL I2C_Enqueue( I2C_ACTION* action); void I2C_Cancel( I2C_ACTION* action, bool signal ); };

---

Je souhaite allouer de la mémoire pour un tableau 2 dimensions de cette structure:

Code:

---

1 2 3 4 5

int j = 0;   I2C_UNIT** tActionU= (struct I2C_UNIT**)calloc(6,sizeof(struct I2C_UNIT*));   for(j = 0; j < 6; j++) *(tActionU+j)= (struct I2C_UNIT*)calloc(1,sizeof(struct I2C_UNIT));

---

Mon programme fonctionne correctement lorsque je n'alloue pas la mémoire, mais il reste bloquer sur ces lignes de codes lorsque je les ajoutent.

Auriez-vous une idée?

Cordialement,
Lionel.

BOnjour,

Il me semble que ta question releve plus du C++ que du C...

Cependant:

Code:

---

*(tActionU+j) = /* [...] */;

---

est incorrect à moi que tes structures ne soient de taille 1 octets (ce qui a priori n'est pas le cas!)

Le code correct à écrire est :

Code:

---

1 2 3

tActionU[j] = (struct I2C_UNIT*) calloc (1, sizeof(struct I2C_UNIT)); // ou *(tActionU + j * sizeof (struct I2C_UNIT)) = (struct I2C_UNIT*) calloc (1, sizeof(struct I2C_UNIT));

---

Voilà au moins une erreur...

Bonjour,
Si tu es en C++, l'utilisation calloc/malloc est une erreur -> préférer new/delete pour les constructeurs/destructeurs.
Si tu es en C++, il existe des choses comme std::vector pour éviter les allocations dynamiques explicites.
Si tu es en C++, utilise des pointeurs intelligents plutôt que des pointeurs nus.
Pourquoi utiliser un tableau à 2 dimensions puisque :
1/ Toutes les lignes sont de même taille ?
2/ Toutes les lignes ont une taille de 1 ?

Enfin, si tu es dans un environnement où tu ne peux utiliser ni std::vector, ni pointeur intelligent (ce qui me surprendrait puisque tu as des template et que tu fais de l'allocation dynamique) et que tu es obligé de passer par ces allocations non safe, ce devrait plutôt ressembler à :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

int j = 0;   I2C_UNIT** tActionU= new I2C_UNIT*[6];   for(j = 0; j < 6; j++) { tActionU[j] = new I2C_UNIT[1]; }   // destruction avec delete[] et non delete tout seul

---

@Fabllot : Et l'arithmétique des pointeurs ? *(tActionU+j) <=> tActionU[j]

Citation:

---

Envoyé par 3DArchi

@Fabllot : Et l'arithmétique des pointeurs ? *(tActionU+j) <=> tActionU[j]

---

Mea Culpa... J'ai confondu *(tActionU+j) avec (*tActionU)+j ...:aie:
Ceci dit, le passage par tableau (ou vector) evite ce genre d'erreur !