Discussion :

[mach1]

Bonjour,
j'ai lu qu'il était fortement conseillé de libérer les variables allouées dynamiquement en utilisant la fonction free, lorsque celles-ci n'étaient plus utilisées, et ceci en particulier juste avant que le programme se termine.
Toutefois, j'aimerais savoir ce qu'il en est dans le cas où le programme se termine avec un "statut d'erreur". C'est à dire, si avant de faire un exit(EXIT_FAILURE) il était recommandé de libérer toutes les variables avec free (ce qui compliquerait beaucoup le code dans certains cas).

Merci par avance.

[jeroman]

Salut

Bonjour,
j'ai lu qu'il était fortement conseillé de libérer les variables allouées dynamiquement en utilisant la fonction free, lorsque celles-ci n'étaient plus utilisées, et ceci en particulier juste avant que le programme se termine.
Toutefois, j'aimerais savoir ce qu'il en est dans le cas où le programme se termine avec un "statut d'erreur". C'est à dire, si avant de faire un exit(EXIT_FAILURE) il était recommandé de libérer toutes les variables avec free (ce qui compliquerait beaucoup le code dans certains cas).

Merci par avance.

C'est d'ailleurs une des raisons pour lesquelles je ne suis pas trop ami avec la fonction exit. J'ai l'impression que de plus en plus de codeurs utilisent cette fonction, par facilité (pour éviter structurer correctement leur programme) et de manière un peu abusive, dès le moindre pépin, par exemple si un malloc ou un fopen échoue.
Il y a peut-être des cas où exit est utile, recommandé voire nécessaire (je ne sais pas, je n'ai pas la connaissance absolue non plus...), mais je constate qu'il est quand même utilisé un peu trop facilement.

Moi, je suis plutôt "branché" par les programmes un minimum structurés, de manière logique et cohérente, qui gèrent les erreurs, dans le sens où je considère que chaque fonction doit avoir un rôle spécifique et retourner une valeur (si nécessaire) indiquant à la fonction appelante si cela a réussit ou non. Et si erreur il y a, la fonction appelante la gère, ou bien la fait gérer par celle qui l'a appelée, etc. Histoire de remonter, comme une pyramide, vers le sommet jusqu'à ce qu'on ait géré proprement le problème (par "géré", je ne dis pas forcément "réparé"). Je trouve que ça fais plus propre.
Enfin, ce n'est que ma vision personnelle de la programmation, qui n'engage que moi.

[Flynet]

Cela pose-t'il un probleme d'utiliser souvent exit pour gérer les erreurs ?

[Pouet_forever]

Je connais des gens qui disent que les OS modernes libèrent automatiquement la mémoire, et que tu coup les free sont useless. D'un côté il a pas tord si on programme pour win, nux, ou mac mais si on veut faire de l'embarqué je pense que ça a son importance.
Je ne connais pas trop l'embarqué, mais je pense qu'un free non fait est utiliser de la mémoire pour rien une fois le programme terminé.

@ Flynet : Je suis de l'avis de jeroman, a savoir ne pas en abuser.

[Flynet]

Abusif ? Je pense a des erreurs d'ouvertures de fichiers ou d'initialisation de bibliotheques... Mais cela-a-t'il des consequences d'utiliser souvent exit ?

[Pouet_forever]

Ca a des conséquences si tu veux que ça en aie. Ca dépend de ton programme, si un fichier n'est pas chargé est-ce que tu quittes ton programme, ou est-ce que tu affiches un message comme quoi tu n'as pas pu ouvrir le fichier et tu fais autre chose, ou tu reviens à l'étape d'avant ? (ambiguë comme phrase )

[Flynet]

Oui certes mais j'aimerai savoir quel genre de probleme cela peut provoquer ?

[souviron34]

Bonjour,
j'ai lu qu'il était fortement conseillé de libérer les variables allouées dynamiquement en utilisant la fonction free, lorsque celles-ci n'étaient plus utilisées, et ceci en particulier juste avant que le programme se termine.
Toutefois, j'aimerais savoir ce qu'il en est dans le cas où le programme se termine avec un "statut d'erreur". C'est à dire, si avant de faire un exit(EXIT_FAILURE) il était recommandé de libérer toutes les variables avec free (ce qui compliquerait beaucoup le code dans certains cas).

Merci par avance.

Abusif ? Je pense a des erreurs d'ouvertures de fichiers ou d'initialisation de bibliotheques... Mais cela-a-t'il des consequences d'utiliser souvent exit ?

Oui certes mais j'aimerai savoir quel genre de probleme cela peut provoquer ?

Laisses-tu toutes les portes ouvertes et les lumières allumées quand tu rentres et sort de quelque part ?



C'est pas "dangereux" a priori (quoique c'est se fier au bon vouloir de l'OS, ce qui n'est pas une bonne chose, en particulier si on a affaire à un OS un peu exotique, (temps réel, embarqué, etc)..

Mais c'est surtout une très mauvaise habitude pour tout un tas de raisons..

• Si un jour tu veux transformer ton programme (ou une partie du programme) en fonction, il faudra que tu repasses à travers tout le code pour savoir ce que tu avais alloué pour le libérer..
  
  
• Si un jour tu veux ajouter quelque chose , même chose.
  
  
• C'est incorrect et rend le code non professionnel.

[ironzorg]

Oui certes mais j'aimerai savoir quel genre de probleme cela peut provoquer ?

Si tu fais du sur mesure et que tu release pas ton code, je dirais que le seul probleme va etre la tete de ton code (pour reprendre l'expression "non professionnel").

Apres, quand on est bien eleve on free tout ce qu'on alloc, faut prendre les free automatiques du systeme comme une securite et pas un outil (une boucle infinie sur un malloc est tellement vite arrivee ).

Donc pour repondre a ta question: n'utilise jamais exit quand tu as le choix, ca te permet de faire une gestion d'erreurs potable (entre autres).

[salseropom]

Tout à fait, je partage votre point de vue : à chaque *alloc correspond un free, bien faire un ptr = NULL après un free, d'où ma macro

Code :

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

1
2
3
4
5
6
7
8
 
#define FREE(p)\
do\
{\
  free(p);\
  p = NULL;\
}\
while(0)

Efin, priviliégier des fonctions qui renvoient des indicateurs d'erreur et les remonter jusqu'au mail()

[gl]

Enfin, privilégier des fonctions qui renvoient des indicateurs d'erreur et les remonter jusqu'au mail()

Je suppose que tu voulais écrire main() ?

Ceci étant pourquoi privilégier ce genre de chose ? Pourquoi ne pas tout simplement traiter l'erreur là où il est pertinent de le faire (ce qui peut être dans le main() - typiquement dans le cas où l'on souhaite tout simplement arrêter le programme -, localement ou dans une fonction intermédiaire selon les cas) plutôt que de la remonter dans le main ?

[salseropom]

Je suppose que tu voulais écrire main() ?

Ceci étant pourquoi privilégier ce genre de chose ? Pourquoi ne pas tout simplement traiter l'erreur là où il est pertinent de le faire (ce qui peut être dans le main() - typiquement dans le cas où l'on souhaite tout simplement arrêter le programme -, localement ou dans une fonction intermédiaire selon les cas) plutôt que de la remonter dans le main ?

oui tout à fait, je voulais parler de main(). Pardon pour la faute de frappe. Sinon, je remonte l'erreur jusqu'au main() pour ensuite libérer correctement toute la mémoire. Mais bien sûr, l'erreur est traitée dans la fonction qui a renvoyé l'erreur.