Discussion :

[salseropom]

Bonjour, je suis en train de reprendre un code. La fonction qui lit le fichier texte contenant les inputs est comme suit :

Code :

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

1
2
3
4
5
 
while(feof(file))
{
  fgets(blabla);
}

Ce n'est pas très joli...

J'ai lu http://c.developpez.com/faq/?page=fi...RS_fin_fichier.

Pouvez-vous s'il vous plait me donner un exemple ou le code précédent ne fait pas ce que l'on croit : feof() détecte une fin de fichier alors que ce n'est pas vrai.

Merci d'avance

[3DArchi]

C'est pas tant ça le problème mais plutôt de continuer dans une boucle de lecture alors que tu peux avoir une erreur. D'où, l'idéé de la faq: lire tant que pas d'erreur. Puis, traiter soit fin de fichier, soit erreur.

[salseropom]

OK, je comprends mieux la nuance

[Obsidian]

Bonjour, je suis en train de reprendre un code. La fonction qui lit le fichier texte contenant les inputs est comme suit :

Code :

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

1
2
3
4
5
 
while(feof(file))
{
  fgets(blabla);
}

Ce n'est pas très joli...

Pouvez-vous s'il vous plait me donner un exemple ou le code précédent ne fait pas ce que l'on croit : feof() détecte une fin de fichier alors que ce n'est pas vrai.

Je me suis effectivement fait reprendre comme un débutant sur ce piège-là. Mais déjà, ce code est erroné : il faudrait un point d'exclamation dans ta condition while avant le feof().

Comme indiqué dans la page que tu nous donnes - et contrairement à d'autres langages - feof() n'indique pas directement si un fichier a atteint sa fin, mais si la dernière erreur est due à une fin de fichier en particulier ou pas. À y réfléchir, c'est vrai que ça se tient car quand le flux de caractères vient en fait de la console, du port série ou même d'une connexion TCP, par exemples, on ne peut pas savoir à l'avance si l'on a atteint la fin du fichier ou pas. Il faut attendre le signal. Un while(feof()) réentrerait alors dans la boucle, et le signal de fin de fichier tomberait au moment de la lecture du caractère suivant.

Cependant, il est bien possible que ton code fonctionne quand même (modulo le point d'exclamation), car tu fais un fgets(), fonction conçue pour lire itérativement les caractères, jusqu'à tomber sur un retour à la ligne. Or, si le caractère reçu est un marqueur EOF, il est clair que fgets() ne va pas l'intégrer à la chaîne ainsi composée, contrairement à un fgetc() qui va renvoyer le résultat tel quel. Néanmoins, la fin de fichier ayant quand même été atteinte, et le code EOF renvoyé, feof() va renvoyer un signal positif.

Ça n'en fait pas pour autant un programme valide car le problème exposé plus haut demeure. Le dernier caractère envoyé peut être une fin de ligne. Cela ferait sortir fgets(), faire faire un tour de plus à while(feof()) qui retomberait sur un nouveau fgets(), laquelle fonction serait alors obligée de renvoyer une ligne vide si le signal de fin de fichier arrive à ce moment-là. Et cette ligne vide est fantôme dans le sens où elle ne proviendrait pas du fichier original.

Si tu fais, par exemple, un programme qui compte le nombre de lignes, ce compte serait faussé.

[salseropom]

OK, merci de tes explications. Je comprends mieux maintenant pourquoi le code

Code :

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

1
2
3
4
5
 
while(!feof(file))
{
  fgets(blabla);
}

est foireux.

[Obsidian]

OK, merci de tes explications. Je comprends mieux maintenant pourquoi le code [...] est foireux.

Pour être tout-à-fait complet, je m'aperçois que si ce que je dis est vrai, il n'y a pas moyen pour autant de recevoir un code EOF avec fgets(). Donc, un petit tour dans la man-page s'impose :

Code :

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

1
2
3
4
5
6
7
8
       char *fgets(char *s, int size, FILE *stream);

RETURN VALUE
       ...

       gets() and fgets() return s on success, and NULL on
       error or when end of file occurs while no characters
       have been read.

On peut donc écrire de façon sûre :

Code :

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

1
2
3
4
5
6
7
8
9
10
char buffer [LENGTH];
FILE * f = stdin;

while (fgets(buffer,LENGTH,f)==buffer)
{
    traitement ...
}

if (ferrof(f)) { erreur ... }

[gl]

On peut donc écrire de façon sûre :

Code :

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

1
2
3
4
5
6
7
8
9
10
char buffer [LENGTH];
FILE * f = stdin;

while (fgets(buffer,LENGTH,f)==buffer)
{
    traitement ...
}

if (ferrof(f)) { erreur ... }

Généralement, on utilise plutôt:

Code :

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

while(fgets(buffer,sizeof buffer,f) != NULL)

[Emmanuel Delahaye]

On peut donc écrire de façon sûre :

Code :

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

1
2
3
4
5
6
7
8
9
10
char buffer [LENGTH];
FILE * f = stdin;

while (fgets(buffer,LENGTH,f)==buffer)
{
    traitement ...
}

if (ferrof(f)) { erreur ... }

Oui, enfin, la forme canonique est

Code :

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

1
2
3
4
5
6
7
8
 
char buffer [LENGTH];
FILE * f = stdin; /* ou fopen(..., "r"); */
 
while (fgets (buffer, sizeof buffer, f) != NULL)
{
    /* traitement ...*/
}

Ne pas trop chercher à s'en éloigner...

[nicolas.sitbon]

Oui, enfin, la forme canonique est

Code :

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

1
2
3
4
5
6
7
8
 
char buffer [LENGTH];
FILE * f = stdin; /* ou fopen(..., "r"); */
 
while (fgets (buffer, sizeof buffer, f) != NULL)
{
    /* traitement ...*/
}

Ne pas trop chercher à s'en éloigner...

plutôt

Code :

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

1
2
3
4
5
6
7
8
 
char buffer [LENGTH];
FILE * f = stdin; /* ou fopen(..., "r"); */
 
while (fgets (buffer, LENGTH, f) != NULL)
{
    /* traitement ...*/
}

[Obsidian]

Oui, enfin, la forme canonique est

Code :

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

1
2
3
4
5
 
while (fgets (buffer, sizeof buffer, f) != NULL)
{
    /* traitement ...*/
}

Ne pas trop chercher à s'en éloigner...

J'évite volontairement « != NULL » dans ce cas précis car la man page précise que la fonction renvoie le premier argument en cas de succès ou NULL en cas d'échec, à l'exclusion, donc, de toute autre valeur. Si, pour une raison quelquonque, une autre valeur est retournée (par exemple, si la bibliothèque est mise à jour), ce code de retour sera considéré comme un succès implicite au lieu d'être vu comme une anomalie et provoquer l'interruption du programme.