Discussion :

[bit_o]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[kromartien]

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.

[souviron34]

il y a plusieurs choses incorrectes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
ma_struct *tmp = NULL; 
 
tmp = (adr *) malloc(sizeof(adr) * nb);

déjà devrait être :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Emmanuel Delahaye]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[souviron34]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

[bit_o]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[kromartien]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

[Trap D]

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.

[Emmanuel Delahaye]

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 ?