Discussion :

[da_latifa]

SVP, Dites moi si vous voyez mon erreur, car celà m'afficher toutes les valeurs
que j'ai dans mon fichier de lecture et il repète la dernière valeur
alors qu'il est déjà rendu à la fin du fichier

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
int entier;
FILE *fichier;
fichier = fopen(argv[1], "r");
if (fichier != NULL)
   {
	while ( !feof(fichier) ) { //On lit le fichier tant que c'est pas la fin
 
		fscanf(fichier, "%d", &entier);
		printf("%d\n", entier);
 
	}
   }
fclose(fichier);

voilà le contenu de mon fichier
3 4 5 6 2 2 3 4 5 1 2 5

voilà l'affichage
3
4
5
6
2
2
3
4
5
1
2
5
5



Merci à tous

[Invité]

Bonjour,

Pourquoi ne pas tester directement la valeur de retour de fscanf ?

A mon sens, le problème vient justement de là.
Tester via feof() va t'indiquer à un instant T si le fichier que tu lis est EOF. Le soucis avec cette manière de faire, c'est que si il reste une crasse dans le fichier, après ton dernier nombre, un \n par exemple, le fichier ne sera pas considéré comme EOF. Du coup tu repasses dans le fscanf(), qui lui fera bien gentiment son boulot. Il ne va pas lire la valeur restante car ce n'est pas un nombre et arrivera sur EOF, laissera donc inchangé le contenu de la variable entier, et retournera la valeur d'erreur EOF. Comme tu ne vérifies pas ce retour, il affiche sans encombre le printf().

Pour reprendre ton exemple à toi...

Code :

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

1
2
3
4
while ( !feof(fichier) ) { //On lit le fichier tant que c'est pas la fin
            if(fscanf(fichier, "%d", &entier) != EOF)
                printf("%d\n", entier);
        }

Affiche bien en sortie

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
~/Bureau$ ./popo
3
4
5
6
2
2
3
4
5
1
2
5

Qui peut avantageusement être remplacé par ceci, qui est à mes yeux plus propres encore.

Code :

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

1
2
while ( fscanf(fichier, "%d", &entier) != EOF )
    printf("%d\n", entier);

[Sve@r]

Bonjour

A mon sens, le problème vient justement de là.
Tester via feof() va t'indiquer à un instant T si le fichier que tu lis est EOF. Le soucis avec cette manière de faire, c'est que si il reste une crasse dans le fichier, après ton dernier nombre, un \n par exemple, le fichier ne sera pas considéré comme EOF. Du coup tu repasses dans le fscanf(), qui lui fera bien gentiment son boulot. Il ne va pas lire la valeur restante car ce n'est pas un nombre et arrivera sur EOF, laissera donc inchangé le contenu de la variable entier, et retournera la valeur d'erreur EOF. Comme tu ne vérifies pas ce retour, il affiche sans encombre le printf().Qui peut avantageusement être remplacé par ceci, qui est à mes yeux plus propres encore.

Code :

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

1
2
while ( fscanf(fichier, "%d", &entier) != EOF )
    printf("%d\n", entier);

C'est même pas une question de crasse ou autre. Même avec un fichier super correct ça ne peut pas fonctionner parce que feof() indique si le fichier est eof non pas au dernier enregistrement mais une fois qu'on a tenté de lire au-delà de ce dernier (quand on lit un livre on ne sait pas qu'on est à la dernière page tant qu'on n'a pas essayé de regarder la suivante). En effet, feof() ne sert pas à détecter une fin de fichier. Il sert à indiquer, une fois que la lecture s'est arrêtée, si l'arrêt est dû à une fin de fichier ou pas (donc autre chose comme par exemple une erreur disque ou autre).

Il s'ensuit que si tu fais un

Code c :

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

1
2
3
4
5
while (feof(...))
{
    lecture_fichier(...)
   ... traitement...
}

Au moment où tu liras le dernier élément de ton fichier, cet élément ayant été correctement lu (et traité), feof() ne renverra aucun soucis. Tu feras alors une itération de plus ce qui amènera une lecture de plus ; lecture se faisant au delà du fichier donc te renvoyant un code d'état spécial mais que tu ne verras pas et donc tu feras un traitement de trop par rapport au nombre d'éléments de ton fichier.

Une première solution sera alors de vérifier l'abscence de ce code spécial pour faire ou ne pas faire le traitement

Code c :

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

1
2
3
4
5
while (feof(...))
{
    if (lecture_fichier(...) != code_special)
       ... traitement...
}

Mais de là, on peut se dire que si la fonction de lecture renvoie un code spécial quand elle lit "trop loin", on peut alors directement l'utiliser pour vérifier justement qu'on n'est pas allé "trop loin" et ne plus se servir de feof(), ce qui donne

Code c :

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

1
2
3
4
while (lecture_fichier(...) != code_special)
{
    ... traitement...
}

Ce qui est exactement ta solution avec ton while ( fscanf(fichier, "%d", &entier) != EOF )...

[Invité]

Comme quoi, on apprends toujours plus de ses maîtres...

[da_latifa]

Merci beaucoup à vous deux pour l'explication...