Que doit-on faire d'un pointeur récupéré à partir d'une fonction ?

Bonjour,

Soit la fonction ayant le prototype suivant :

Code:

---

char* pouet();

---

et une variable globale :

Code:

---

char* ptr;

---

Si dans un bloc, je l'utilise comme suit :

Code:

---

1 2 3 4 5

{ ... ptr = pouet(); .... }

---

Dois-je par la suite me soucier de la place que prennent les données pointées par ptr, si je n'en ai plus besoin ?

Merci.

dans un cas comme celui-là, il faut faire l'allocation mémoire dans ta fonction pouet
mais il faut pas oublier de désallouer le bloc une fois que tu ten sers pu, i.e faire un free(ptr)
évidement il faut le placer au bon endroit (et pas faire de free dans la fonction pouet)

1. Pourquoi une variable globale ?
2. Impossible de répondre sans savoir ce que fait pouet().

Ca n'est pas forcément plus clair que sur ton post précédent j'dois avouer :aie:

• On ignore ce que fait pouet()
• Il faut éviter autant que possible les variable globales et passer donc au maximum par les arguments
• Qu'entend tu par "se soucier de la place que prennent les données pointées lorsque tu n'en as plus besoin" ? Les libérer ? Si c'est ca, oui il faut libérer les espaces plus utilisés !
• Un exemple concret sur ce que tu veux faire serait mieux qu'un exemple factice même pas complet !

J'ai essayé de faire simple pour ne pas vous embrouiller, mais je m'aperçois qu'en fait, vous manquez d'éléments pour pouvoir répondre.

Je présente donc les choses de manière plus précise :

Soit la structure suivante :

Code:

---

1 2 3 4 5 6

typedef struct myStruct { .... char name[10]; .... } myStruct;

---

Soit une fonction qui renvoit un pointeur vers une "instance" de cette structure quand on lui communique un entier (référençant un jeu de données... enfin bref... peu importe) :

Code:

---

myStruct* getEntry(int);

---

Et soit la fonction qui renvoie un pointeur vers le champ name quand on lui passe en paramètre un pointeur vers une structure myStruct :

Code:

---

char* getName(*myStruct);

---

Ces deux fonctions sont issues d'une API, et je n'ai donc aucun contrôle sur leur définition.

J'ai ensuite la fonction suivante :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

char** creationTableau(char** tab) { myStruct *structEntry; tab = malloc (10*sizeof (*tab));   for (i = 0; i < max_node; i++) { structEntry = getEntry(i); tab[i] = getName(node_entry); } }

---

Comment corretement libérer la mémoire occupée par tab ?
Je n'ai pas de problème pour free(tab), mais c'est plutôt pour le contenu de tab...

Bonjour en algoritmique ca donnerai ca :

Code:

---

1 2 3 4 5

pour chaque element du tableau desallouer element   desallouer tableau

---

Citation:

---

Envoyé par oodini

Comment corretement libérer la mémoire occupée par tab ?
Je n'ai pas de problème pour free(tab), mais c'est plutôt pour le contenu de tab...

---

Tout dépend de comment est retournée la chaîne par AiNodeEntryGetName ! Si c'est une allocation qui est faite pour stocker la chaine, tu peux faire comme BainE l'a expliqué.

Citation:

---

Envoyé par Franck.H

Si c'est une allocation qui est faite pour stocker la chaine, tu peux faire comme BainE l'a expliqué.

---

Je ne sais pas si c'est une allocation qui est faite.
Tout ce que je sais de la fonction, vous le savez.
Je n'ai que le prototype du .h.

J'ai initialement essayé de faire comme l'explique BainE, mais ça plantait mon programme dans l'imbrication interne. J'en déduis donc qu'il n'y a aps allocation par malloc ou autre.

Dois-je en déduire que je n'ai pas à me soucier de la libération de mémoire des éléments, et que j'ai juste à faire un free(tab) ?

Citation:

---

Envoyé par oodini

Et soit la fonction qui renvoie un pointeur vers le champ name quand on lui passe en paramètre un pointeur vers une structure myStruct :

Code:

---

char* getName(*myStruct);

---

---

Tu veux dire char * getName(myStruct *) ?

Citation:

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11

char** creationTableau(char** tab) { myStruct *structEntry; tab = malloc (10*sizeof (*tab));   for (i = 0; i < max_node; i++) { structEntry = getEntry(i); tab[i] = getName(node_entry); } }

---

Comment corretement libérer la mémoire occupée par tab ?
Je n'ai pas de problème pour free(tab), mais c'est plutôt pour le contenu de tab...

---

1. Inutile de passer tab en paramètre puique tu ne le lis pas.
2. Impossible de répondre sans savoir ce que fait getEntry().
3. De toute évidence, getEntry() utilise une ou plusieurs variables globales...

Pour te libérer de l'influence de getEntry(), tu peux choisir de mettre une recopie des chaînes dans ton tableau plutôt que leurs adresse (en faisant tab[i] = strdup(getName(node_entry));, ou un équivalent plus standard).
Dans ce cas, tu sauras qu'il faut faire un free() sur chaque entrée du tableau avant de faire faire un free() sur le tableau lui-même.

Citation:

---

Envoyé par Médinoc

1. Inutile de passer tab en paramètre puique tu ne le lis pas.
2. Impossible de répondre sans savoir ce que fait getEntry().
3. De toute évidence, getEntry() utilise une ou plusieurs variables globales...

Pour te libérer de l'influence de getEntry(), tu peux choisir de mettre une recopie des chaînes dans ton tableau plutôt que leurs adresse (en faisant tab[i] = strdup(getName(node_entry));, ou un équivalent plus standard).
Dans ce cas, tu sauras qu'il faut faire un free() sur chaque entrée du tableau avant de faire faire un free() sur le tableau lui-même.

---

1. Ben je ne le lis pas, mais k'interveins dessus.

2. Dommage...

Et el fait que le free plante en suffit pas pour en déduire qu'il n'y a aps de désallocation à faire ?

1. Eh ben, tu le déclares à l'intérieur de la fonction. De toute façon, tu le retournes...
Je te rappelle que les variables sont passées par valeur. Si tu modifies directement le pointeur tab, ça n'aura aucun effet sur la fonction appelante...

2. Il pourrait falloir désallouer autre chose. Impossible de savoir si ce n'est pas documenté.

OK, merci d'insister sur le premier point, ça n'était pas superflu. :-)

Quant au second point, c'est bien le problème de cette API... Doc quasi inexistante.