Discussion :

[nabodix]

bonjour

Je suis débutant en C; j'ai un petit travail à remettre, et je me demandais comment l'écrire le plus proprement possible...


Voici une version (très) épuré/simplifié de mon code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
int main(int argc, char *argv[]){
    ...
    flux(argv[1],argv[2]);
    ...
}
 
/*
 *méthode qui écrira "nbr" caractères au hasard dans le fichier "fichier"
 */
int flux(char *nbr,char *fichier){
    ...
}

mon problème est le suivant: où vérifier:
1) le nombre d'arguments de main (je suppose comme première instruction du main, mais vaut mieux un exit(-1) ou un return -1 ?)
2) que argv[1] est convertissable en entier (je peux bien entendu changer la signature de flux)
3) que argv[2] a pu être ouvert (dans la méthode flux je suppose, mais vaut mieux faire un exit(-1) ou un return -1 pour gérer l'erreur dans le main?)

Vu que j'ai plusieurs type d'erreurs possibles, vaut mieux avoir des codes différents pour chaque erreurs?

Merci beaucoup pour votre attention/conseils !

[Médinoc]

1. Tester au début du main, et retourner EXIT_FAILURE (pas -1) avec un bon vieux return.
2. Si c'est un prérequis pour ton programme, tu testes ça dans le main aussi. D'ailleurs, je te conseille de passer directement un entier à la fonction flux(), plutôt qu'une chaîne.
3. Utiliser un return autant que possible. N'utilise exit() que si tu n'arrives absolument pas à faire autrement.

En effet, utiliser plusieurs codes d'erreurs différents peut être intéressant. Attention toutefois, bien que ce soit accepté par DOS, Windows et les unixoïdes, ça n'est pas strictement standard (le standard n'accepte que 0, EXIT_SUCCESS, et EXIT_FAILURE).


PS: Le second paramètre de flux() devrait être un char const * si tu ne trafiques pas le nom de fichier.

[nabodix]

Merci bcp Médinoc!

Donc il vaut mieux que je mette un "return EXIT_FAILURE;" (si il n'arrive pas à ouvrir le fichier par exemple) dans ma méthode flux() et dans un autre "return EXIT_FAILURE;" dans mon main si la méthode flux() ne s'est pas bien déroulé?

[Médinoc]

En fait, je serais du genre à limiter le return EXIT_FAILURE au main(), et retourner -1 dans les sous-fonctions.

Ce qui donnerait un truc du genre:

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
19
20
21
22
23
24
25
int main(int argc, char *argv[]){
    ...
    if(argc < 3) {
        ...
        return EXIT_FAILURE;
    }
 
    int nbr = ...
    if(flux(nbr, argv[2]) < 0) {
        return EXIT_FAILURE;
    }
    ...
    return EXIT_SUCCESS;
}
 
/*
 *méthode qui écrira "nbr" caractères au hasard dans le fichier "fichier"
 */
int flux(int nbr, char const *fichier){
    ...
    if( erreur ) {
        return -1;
    }
    return 0;
}

[nabodix]

Merci, je m'en vais suivre tes conseils