Discussion :

[zentaf]

Bonjour à tous,

J'ai écrit une fonction char * str_ext(char *ch, int start, int end) qui extrait une sous chaine de la chaine d'origine selon les indices. La fonction retourne la sous chaine. J'utilise l'allocation dynamique avec malloc mais je suis obligé de ne pas faire de free() dans ma fonction, je laisse ça à l'utilisateur de ma fonction (chose que j'aime pas trop).

Pour ce souci, je me demande :

Si l'utilisateur utilise la fonction de la manière suivante,
est-ce qu'on doit faire un free() aussi? si oui comment?

Code :

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

1
2
3
char *ch = "Bonjour à tous !";
char s1[255]="";
strcpy(s1,str_ext(ch,0,6));

Rq: admettons que s1 est toujours suffisamment grande pour accueillir la sous chaine.

au lieu de :

Code :

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

s1 = str_ext(ch,0,6);

Merci,
zentaf.

[Metalman]

Ca va leaker...
Tu perds le pointeur qui a été alloué dans str_ext.
C'est pour ça qu'il "faudrait" faire comme strcpy : demander un tableau de la bonne taille "avant" d'exécuter ton code.

Mais le but du C et des str*, c'est justement que l'utilisateur gère à sa guise la mémoire ! ^^'

[Bktero]

est-ce qu'on doit faire un free() aussi?

Tu réponds toi même à ta question :

free()[...], je laisse ça à l'utilisateur de ma fonction

L'utilisateur ne fait pas de free() dans ton exemple, il y a donc une fuite mémoire.

si oui comment?

En passant par une variable intermédiaire pour stocker le pointeur retournée par la fonction. Ensuite, tu fais le strcpy() et enfin le free(). Les précautions pour utiliser ta fonction sont les mêmes pour que utiliser strdup().

[Franck.H]

Allocation dynamique = libération, tu ne peux pas passer à côté !

Maintenant, si tu fait une bibliothèque pour gérer les chaînes de caractères, si c'est un gros projet qui englobe peut-être même un plus gros projet, pourquoi ne pas mettre en place une liste chaînée qui garderais la trace de chaque allocations dynamiques. Au chargement du programme, tu appelles (au bon moment mais c'est à toi de le choisir suivant comment tu fais les initialisations primaires) la fonction atexit() dans la quelle tu renseigne une fonction qui va libérer chaque adresse contenue dans ta liste puis pour finir ta liste. Au moment de la fermeture du programme, si l'utilisateur à oublié de libérer certaines choses bin c'est le programme qui s'en occupe.

Bon je le concède, ça fait pas mal de boulot en plus mais tu peux t'en faire un module réutilisable et l'utiliser pour d'autres projets, je l'ai déjà fait à une époque et c'est prévu en interne à ma bibliothèque de chaînes de caractères (voir ma signature).

Sinon laisser le soin à l'utilisateur de le faire (de toute manière vaut mieux laisser un moyen de le faire) mais si après y'a des fuites mémoire ce sera pas de ta faute

[Neckara]

Bonjour,

A la fin de l'exécution du programme, toutes les ressources mémoires sont libérées ( si je ne dis pas de bêtises ) mais d'autres ressources peuvent ne pas l'être (sémaphore système V ?). Libérer la mémoire avec un atexit() me semble donc un peu superflu.

Surtout que cette méthode sera inefficace si on recouvre (avec par exemple exec()) le programme par un autre (la mémoire restera allouée il me semble).

Après, c'est aussi une question de méthode :
Dès qu'on récupère un pointeur, on se demande s'il faut le désallouer ou non, etc. Et si on a quand même des problèmes de désallocation, Valgrind est là pour nous aider

[Franck.H]

Pour la libération c'est selon le système donc il faut partir du principe qu'il ne le fait pas car la norme ne garantie rien à ce niveau