Discussion :

[Despak]

Bonjour,

J'ai écris un simple petit programme qui recopie le contenu d'un fichier dans un autre.
Il prend donc en paramètre le nom du fichier à copier.

Je me demandais où il fallait placer le feof().
Dans les FAQ il est mit après la boucle while qui parcourt le fichier.
Moi je l'ai mis dedans.

Est-ce que celà a-t-il un sens ?

Moi j'estime qu'il faut tester que tout ailles bien avant chaque lecture d'une nouvelle ligne.

Merci.

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
26
27
28
29
30
31
32
33
34
 
  1 #include <stdio.h>
  2 #include <stdlib.h>
  3 #include <errno.h>
  4 
  5 #define LEN 200
  6 
  7 int main(int argc, char** argv ){
  8 
  9         FILE* fi_in ;
 10         FILE* fi_out ;
 11         char* ligne ;
 12         
 13         ligne = malloc( sizeof(char)*LEN );
 14         if( ( fi_in = fopen(argv[1], "r" ) ) == NULL ){
 15                 fprintf(stderr,"Erreur fopen de fi_in %d\n", errno);
 16                 exit (1) ;
 17         }
 18         if( (fi_out = fopen("fichierOut.txt","w+" )) == NULL){
 19                 fprintf(stderr,"Erreur fopen de fi_out %d\n", errno);
 20                 exit(2);
 21         }
 22 
 23         while( ( fgets( ligne, LEN, fi_in ) != NULL ) ){
 24                 if( feof(fi_in) ){
 25                         exit (3) ;
 26                         fprintf(stderr, "Erreur de fin de fichier %d\n", errno);
 27                 }
 28                 fputs(ligne, fi_out);
 29         }
 30         fclose(fi_in);
 31         fclose(fi_out);
 32 
 33 }

[Invité(e)]

Bonsoir,

Moi j'estime qu'il faut tester que tout ailles bien avant chaque lecture d'une nouvelle ligne.

Et non, feof et ferror sont deux fonctions à appeler pour savoir quelle est la cause de l'échec d'une fonction de lecture comme fgetc, fgets...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
FILE *f = fopen(...)
if(NULL != f) {
    while(NULL != fgets(..., f)) {
        /* faire quelque chose des données lues... */
    }
    if(feof(f)) {
        printf("lecture du fichier terminée\n");
    }
    if(ferror(f)) {
        perror("erreur pendant la lecture\n");
    }
    fclose(f);
}

Dans ton cas, on écrirait plutôt :

Code :

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

1
2
3
4
5
6
7
8
9
while( ( fgets( ligne, LEN, fi_in ) != NULL ) ){
    fputs(ligne, fi_out);
}
if(feof(fi_in)) {
    fprintf(stderr, "lecture du fichier terminée\n");
}
if(ferror(fi_in)) {
    perror("erreur pendant la lecture");
}

[orfix]

Moi je l'ai mis dedans.

à l'intérieur de la boule le test ne sert à rien car si le corps de la boucle est exécuté cela signifie qu'aucunes erreurs n'a été rencontrée ( fgtes() retourne une valeur autre que NULL ) le code :

Code :

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

1
2
3
 
exit (3) ;
fprintf(stderr, "Erreur de fin de fichier %d\n", errno);

ne sera jamais exécuté

Moi j'estime qu'il faut tester que tout ailles bien avant chaque lecture d'une nouvelle ligne.

c'est ce que tu faits avec le test du retour de fgets, si quelque chose ne va pas on sort de la boucle et c'est à ce moment qu'il faut vérifier la nature de l'erreur ( une fin de fichier entre autres ).

[Despak]

AAAh ok !

J'ai tout compris, merci beaucoup !