Problème free() : Tableau de structures

Bonsoir,

Je réalise un petit programme dans lequel j'utilise un tableau de structures. Ce tableau est alloué dynamiquement, mais j'ai un souci : je ne peux pas écrire free(tab[i]), voici un exemple qui sera plus parlant :

Code:

---

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

//ma_struct *tab est initialisé et construit avant     int i = 0; int nb = 10; //Le nombre de structures dans mon tableau ma_struct *tmp = NULL;   tmp = (adr *) malloc(sizeof(adr) * nb);   if(tmp != NULL) { for(i = 0; i < nb; i++) { tmp[i] = tab[i]; free(tab[i]); // Cette ligne provoque l'erreur "incompatible type for argument 1 of ‘free’" } }

---

Je ne comprends pas pourquoi free(tab[i]) me renvoie cette erreur
"incompatible type for argument 1 of ‘free’"

Any idea ? :)

tab[i] c'est une valeur, c'est à dire que tu passes à free la valeur contenue à l'adresse (tab+i), qui est donc le pointeur sur *(tab+i) notation équivalente dans le langage C à tab[i].

il faut juste faire free(tab+i) pour libérer le pointeur sur tab[i]. En effet, l'argument reçu par free est un pointeur (une adresse mémoire) et non une valeur comme l'est tab[i]==*(tab+i).


Si tu ne comprends pas l'explication, la FAQ peut peut être répondre à ta question mieux que moi. Merci en tout cas de me dire si j'ai répondu à ta question.

il y a plusieurs choses incorrectes :

Code:

---

1 2 3 4

ma_struct *tmp = NULL; tmp = (adr *) malloc(sizeof(adr) * nb);

---

déjà devrait être :

Code:

---

1 2 3 4

ma_struct *tmp = NULL; tmp = (ma_struct *) malloc(sizeof(ma_struct) * nb);

---

Ensuite, si tab est alloué, tab[i] est un élément. Donc free(tab[i]) ne veut rien dire.

Ce devrait être free(tab)

Et si tab[i] a été alloué, alors écrire tmp = tab[i] en ayant alloué nb*ma_struct pour tmp est idiot et ne veux rien dire non plus.

Citation:

---

Envoyé par bit_o

Je ne comprends pas pourquoi free(tab[i]) me renvoie cette erreur
"incompatible type for argument 1 of ‘free’"

---

Quel est le type de tab ? Tu peux poster le code complet ?

Citation:

---

Envoyé par souviron34

Code:

---

1 2 3 4

ma_struct *tmp = NULL; tmp = (ma_struct *) malloc(sizeof(ma_struct) * nb);

---

---

Une façon compliquée d'écrire :

Code:

---

1 2	ma_struct *tmp = malloc (sizeof *tmp * nb);

---

Je ne voudrai pas polluer ce post, mais j'ai toujours cru que pour libérer la mémoire dévolue à un tableau, (1) free(tab) suffisait, et qu'il était inutile de libérer la mémoire pour chaque élément du tableau
(2)

Code:

---

1 2 3

for (i=0;i<TailleTableau;i++) free(tab+i);

---

Quelle est la version correcte ? (1) ou (2)

Celà dépend si tab[i] a été alloué par un malloc, ou non.
Si c'est le cas alors il faut la méthode 2 puis éventuellement la méthode 1 si tab a été alloué lui même par un malloc.

Citation:

---

Envoyé par Emmanuel Delahaye

Une façon compliquée d'écrire :

Code:

---

1 2	ma_struct *tmp = malloc (sizeof *tmp * nb);

---

---

oui je sais mais je gardais ce qu'avait fait le PO.

;)

Bonjour,

le problème de type (ma_struct et adr) tient dans le fait que j'ai voulu donné un exemple de mon code, et non mon code complet ;)

Ce que je veux faire c'est créer un tableau d'éléments de type ma_struct. Ce tableau je devrais l'agrandir donc j'utilise un tableau dynamique.
L'allocation doit ressembler à ça :

Code:

---

1 2	tmp = (ma_struct *) malloc(sizeof(ma_struct) * nb);

---

Ensuite, je veux recopier dans ce tableau tmp, mon tableau tab. Donc

Code:

---

1 2 3 4 5

for(i = 0; i < nb; i++) { tmp[i] = tab[i]; }

---

Enfin il faut que je libère la mémoire occupée par le premier tableau (tab). Et c'est la que se pose mon problème ...

Le fait d'écrire free(tab) va bien libérer la mémoire occupée par l'ensemble de mon tableau, n'est-ce-pas ?

Si tu veux agrandir ton tableau, fais simplement un realloc (si évidemment le précédent a bien été alloué par un malloc). Les valeurs pécédentes du tableau sont conservées par le realloc.

Citation:

---

Envoyé par bit_o

Le fait d'écrire free(tab) va bien libérer la mémoire occupée par l'ensemble de mon tableau, n'est-ce-pas ?

---

oui si ton premier tableau a été alloué par un malloc du même type que celui utilisé pour tmp.

LoL je ne savais plus trop ou j'étais avec ces structures ... :aie:
C'est bon tout marche parfaitement :)