Discussion :

[Torx26]

Bonjour

Je fais un petit programme qui prend en parametre un fichier texte contenant les données a traiter.

Pour la lecture des donnes, j'ouvre le fichier (open) lis les données et ferme le fichier (close), ceci par trois fonctions distinctes qui verifient qu'il n'y a pas d'erreur (ouverture, fermeture, etc) auquel cas je fais un EXIT_FAILURE.

Mon probleme est le suivant, existe t'il un moyen pour que, par exemple dans le cas du erreur a l'ouverture, on quitte le programme et non simplement la fonction, a partir de la fonction ayant "trouve" cette erreur ?

merci

[valefor]

je fais un EXIT_FAILURE

Vous faites un "return EXIT_FAILURE" ?

Essayez avec un "exit(EXIT_FAILURE)"

[Torx26]

Bonjour

Merci pour votre attention.
En cas de succes je retourne EXIT_SUCCES et en cas d'echec EXIT_FAILURE. Le type de retour de ma fonction est int hors exit(EXIT_FAILURE) est vu de type void.
msg du compilateur: void value not ignored as it ought to be.

[Kirilenko]

Il nous faudrait le code.
Tu es sûr que tu n'as pas un prototype de la fonction qui est différent ?

[valefor]

Le type de retour de ma fonction est int hors exit(EXIT_FAILURE) est vu de type void.
msg du compilateur: void value not ignored as it ought to be.

N'auriez vous pas écrit

Code :

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

return exit(EXIT_FAILURE)

par hasard ?

Il faut écrire

Code :

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

exit((EXIT_FAILURE)

tout court. Pas de return.

Et concernant l'avis de souviron34, je ne suis pas aussi catégorique... A mon avis (ce n'est donc pas à prendre comme une généralité), ne pas utiliser exit est une peur qui pourrait venir du monde windows. Par exemple il me semble que quand on fait un exit sous windows, toutes les ressources ne sont pas bien "nettoyées". Encore récemment, j'ai vu une socket qui ne se fermait pas bien ce qui avait un impact sur le code qui tournait à l'autre bout...

Sous Linux, les fichiers sont fermés, les ressources mémoire libérées... bref, je ne vois pas ce que l'on aurait besoin de faire de plus, manuellement...

[souviron34]

Et concernant l'avis de souviron34, je ne suis pas aussi catégorique... A mon avis (ce n'est donc pas à prendre comme une généralité), ne pas utiliser exit est une peur qui pourrait venir du monde windows. Par exemple il me semble que quand on fait un exit sous windows, toutes les ressources ne sont pas bien "nettoyées". Encore récemment, j'ai vu une socket qui ne se fermait pas bien ce qui avait un impact sur le code qui tournait à l'autre bout...

Sous Linux, les fichiers sont fermés, les ressources mémoire libérées... bref, je ne vois pas ce que l'on aurait besoin de faire de plus, manuellement...

Je n'ai jamais (quasi) programmé sous Windows

Ce n'est pas une peur du tout, c'est une bonne pratique, au même titre que proscrre, sauf cas très très particulier, le goto, car cela amène à du code "spagetti", où le (futur) lecteur ne peut pas suivre le flux du code sans s'arracher les cheveux...

Lorsqu'un logiciel fait plusieurs milliers, voire dizaines ou centaines de milliers de lignes, ne pas avoir de "point central" de sortie est une abomination...

Cela n'a rien à voir avec Windows ou pas.. C'est du développement logiciel..

je ne vois pas ce que l'on aurait besoin de faire de plus, manuellement...

ça peut être fermer des ressources non systèmes (une base de données par exemple), enlever des locks, libérer ou détruire des fichiers temporaires, envoyer un message, stocker un drapeau dans une BD, fermer des sockets...

Etant donné que tout développeur dont c'est le métier va finir un jour ou l'autre par travailler sur des sources complexes, et de plus de 100 lignes, acquérir les bons réflexes le plus tôt possible est le plus sain..

Même une erreur d'allocation mémoire n'est pas fatale.. Un logiciel peut tès bien continuer, et retenter un peu plus tard.. Avec un exit() c'est foutu...

D'autant plus que quand même beaucoup de logiciels doivent être 24/24, ne jamais s'arrêter...

[Thierry Chappuis]

Je suis de ceux qui sont d'accord avec souviron34 sur le fait de remonter l'erreur . C'est à la fonction appelante de décider quoi faire en cas d'erreur, et non à la fonction où c'est produite l'erreur. Si la fonction appelante n'est pas main(), on remonte plus haut jusqu'à ce que quelqu'un traîte l'erreur. En dernier ressort, main() peut éventuellement quitter l'application. En général, je n'autorise pas d'autre fonction que main() à terminer l'exécution du programme.

En renvoyant un code d'erreur à la fonction appelante plutôt qu'en quittant brusquement le programme, pour permettez à la fonction appelante de faire quelque chose d'intelligent avec cet erreur: réessayer plus tard, libérer des ressources après sauvegardes dans un fichier et réessayer, ou en désespoir de cause sauver ce qui peut l'être et remonter l'erreur jusqu'à main().

On voit beaucoup de code pour débutant qui, p.ex., quitte le programme en cas d'échec de la fonction malloc(). Il y a pourtant souvent des alternatives qui pourront être mises en oeuvre par la fonction appelante.

Avec mes meilleures salutations

Thierry

[souviron34]

existe t'il un moyen pour que, par exemple dans le cas du erreur a l'ouverture, on quitte le programme et non simplement la fonction, a partir de la fonction ayant "trouve" cette erreur ?

Bonjour

outre le fait que ce n'est pas une bonne pratique - la règle générale est de remonter l'erreur niveau par niveau jusqu'au premier - l'utilisation d'exit() un peu partout dans des fonctions est très fortement déconseillé.

Une solution peut consister à utiliser la fonctionalité atexit pour enregistrer des fonctions à appeler lors de la sortie, et éventuellement dans ce cas faire à l'intérieur un exit(), ce qui concentre en un seul point le code..

Enfin, ton erreur vient, comme on l'a signalé, d'une erreur dans la déclaration de main / ou d'oubli d'inclusion de stdlib.h

Code C :

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

1
2
3
int main ( int argc, char **argv )
[
}

ou

Code C :

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

1
2
3
int main ( void )
[
}