fgets, comment déterminer une erreur de lecture

Version imprimable

19/10/2005, 11h39
kap

fgets, comment déterminer une erreur de lecture
Bonjour à tous,
Je dois récupérer des données dans un fichier. J'ouvre celui-ci en mode binaire et j'utilise ensuite fgets.

Code:

while(fgets(ligne, sizeof ligne, fIn)!=NULL)

Mais fgets ne parcourt pas toutes les lignes de mon fichier. J'ai donc testé à la fin de mon programme si la fin de fichier avait été rencontré par
Code:

1 2 if(feof(fOut)) fprintf(fOut,"fin de fichier rencontré"); else fprintf(fOut,"erreur");
et cela me renvoie "erreur". J'en ai donc déduis que fgets rencontrait une erreur mais comment puis-je déterminer cette erreur de lecture et y remédier?
19/10/2005, 11h46
gege2061

Bonjour et bienvenue sur le forum ;)
la réponse se trouve dans les pages man :

Citation:

The function fgets() may also fail and set errno for any of the errors specified for the routines
19/10/2005, 11h46
Franck.H

Heu je crois que pour lire dans un fichier binaire il faut utiliser fread ... il me semble bien que les fichiers binaires sont toujours formattés d'une manière bien spécifique !
Si tu dis que tu as testé ton fgets avec un feof à la fin de ton programme ... hum hum, normalement ca se test directement dans la boucle while, directement au début apres le fgets !
19/10/2005, 12h06
kap

je ne sais pas si je l'ai bien utilisé mais errno me renvoie 0 donc à priori il n'y a pas d'erreur :?
je ne comprends pas alors que le fgets ne s'effectue pas jusqu'au bout puisque la fin de fichier n'est pas rencontré non plus :cry:
19/10/2005, 12h08
Franck.H

On peut voir ton code ?

Code:

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
#include <stdio.h>
#include <unistd.h>
#include <errno.h>
 
int main(void)
{
 
char ligne[3000];
FILE* fIn; 
FILE* fOut;
int position,j,type_ligne;
extern int errno; 
 
if((fIn= fopen("YTOPTIC.D", "rb")) == NULL) // ouverture du fichier source
         {
         printf("erreur, mauvais fichier source rentré");      
         system("pause"); 
         return (1);
         }  
 
if((fOut = fopen("optic.tmp", "w")) == NULL) //ouverture du fichier destination
         {
         printf("erreur, mauvais fichier destination entré");      
         fclose(fIn);
         return (1);
         }
 
while(fgets(ligne, sizeof ligne, fIn)!=NULL) 
          {type_ligne=0;
 
 
          if(((int)ligne[4]==49 && (int)ligne[5]==57 )|| ((int)ligne[4]==50 && (int)ligne[4]==48))
            {
            type_ligne=1;
            }
 
           switch(type_ligne)
 
                 {case 1:                                                                                  
                  for(j=0;j<90;j++)
                  {
                  putc(ligne[position],fOut);
                  if (j==77 /*|| j==18 || j==33 || j==37*/)fprintf(fOut,";");
                  position++;
                  }
                  fprintf(fOut,";");                                                                                           fprintf(fOut,"%d",errno); 
                  fprintf(fOut,"\n");
                  break;
 
                  }
          }
 
if(feof(fOut)) fprintf(fOut,"fin de fichier rencontré");
else fprintf(fOut,"erreur");       
}

j'ai enlevé à l'intérieur du case plein de petites boucles du type

Code:

1
2
3
4
for(position=0;position<10;position++)
                  {
                  putc(ligne[position],fOut);
                  }

j'espères que c'est clair

19/10/2005, 12h17
Franck.H
C'est ce que je pensais au niveau de l'erreur de codage. Ceci:
Code:

1 2 3 4 5 6 7 if(feof(fOut)) { fprintf(fOut,"fin de fichier rencontré"); ... /* On sort de la boucle */ break; }
devrait être dans ta boucle While et meme mieux, cela doit etre ta premiere instruction dans la boucle ... endehors de la boucle, ca ne sert pas a grand chose !

Et ta boucle un peu modifiée en consequence:

Code:

while(fgets(ligne, sizeof ligne, fIn))

Voila j'ai essayé ta solution
Code:

1 2 3 4 while(fgets(ligne, sizeof ligne, fIn)!=NULL) { if(feof(fOut)) fprintf(fOut,"fin de fichier rencontré"); else fprintf(fOut,"erreur\n");
mais ça m'affiche toujours "erreur" donc la fin de fichier n'est jamais recontrée

19/10/2005, 12h33
artatum
Code:

1 2 if(((int)ligne[4]==49 && (int)ligne[5]==57 )|| ((int)ligne[4]==50 && (int)ligne[4]==48))
je suppose que le dernier, tu voulais dire ligne[5]

d'autre part, fgets lit des chaînes sur un flux, c'est une opération formatée. Je ne l'ai jamais essayé sur un flux binaire, mais je suis dubitatif. moi j'utiliserais fread.

enfin l'utilisation de la macro putc au lieu de la fonction fputc te fait certes gagner un peu de temps d'exécution, mais tu prends le risque des effets de bord.

Citation:
Envoyé par kap
Voila j'ai essayé ta solution
Code:

1 2 3 4 while(fgets(ligne, sizeof ligne, fIn)!=NULL) { if(feof(fOut)) fprintf(fOut,"fin de fichier rencontré"); else fprintf(fOut,"erreur\n");
mais ça m'affiche toujours "erreur" donc la fin de fichier n'est jamais recontrée
Deja pour lire un fichier binaire le mieux et le plus approprié est de lire avec fread car comme dit plus haut, un fichier binaire dans la plupart des cas est un fichier formatté de façon bien spécifique, donc tu doit récupérer les données de la même maniere qu'à l'écriture !

Exemple de boucle de lecture:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 while (1) { int nblocs = fread (&ta_struct_ou_varaible_etc, sizeof (ta_struct_ou_varaible_etc), fIn); /* ensuite tu peut tester la fin du fichier */ if (feof (entree)) break; /* on peut aussi tester la variable nblocs */ if (nblocs != 1) break; /* Erreur de lecture */ ... }

19/10/2005, 12h51
kap

j'utilise fgets parceque je dois traiter les lignes de mon fichier différemment et je ne dois pas récupérer toutes les données, seulement celles qui m'intéressent ou en tout cas qui ont un sens à mes yeux. J'ouvre le fichier en binaire parceque le fichier en question contient notament des prix codés en héxadécimal : 021A25 pour 21.25 euros et donc le caractère de fin de fichier apparait avant la fin de fichier. Je vois pas trop comment utiliser fread parceque si je mets le resultat dans un tableau de caractère ca va être galère pour remmettre tout ça en forme afin de l'exporter sous excel...
19/10/2005, 13h00
kap

euh bon en fait artatum avait vu juste

Citation:

Code:

if(((int)ligne[4]==49 && (int)ligne[5]==57 )|| ((int)ligne[4]==50 && (int)ligne[4]==48))

je suppose que le dernier, tu voulais dire ligne[5]

ça change un peu tout désolé de vous avoir fait perdre votre temps avec une erreur à la c.. :roll:
par contre ça m'intéresserait de savoir comment vous vous y prendriez avec fread pour traiter mon problème
19/10/2005, 13h03
Franck.H

Citation:

Envoyé par kap

euh bon en fait artatum avait vu juste

Citation:

Code:

if(((int)ligne[4]==49 && (int)ligne[5]==57 )|| ((int)ligne[4]==50 && (int)ligne[4]==48))

je suppose que le dernier, tu voulais dire ligne[5]

ça change un peu tout désolé de vous avoir fait perdre votre temps avec une erreur à la c.. :roll:
par contre ça m'intéresserait de savoir comment vous vous y prendriez avec fread pour traiter mon problème

Pour utiliser fread, il faut deja savoir comment sont enregistrer les données dans le fichier car, dans le cas où les données se trouvaient dans un structure, il faudrais au mieux que tu lise les données et les ajoutent (par le biais de la condition de la boucle) dans une structure identique et biensur dans le meme ordre !
19/10/2005, 13h41
kap

Citation:

il faut deja savoir comment sont enregistrer les données

Ca je sais pas trop car je recupère des données de clients qui étaient chez un concurrent pour les injecter dans notre logiciel.
Si tu veux encore y consacrer du temps je peux t'envoyer le fichier mais dans la mesure où mon erreur a été trouvé on peut aussi s'arréter là.

Merci à tous pour votre aide :D
19/10/2005, 13h46
gege2061
Citation:
Envoyé par CSoldier

C'est ce que je pensais au niveau de l'erreur de codage. Ceci:

Code:

1 2 3 4 5 6 7 if(feof(fOut)) { fprintf(fOut,"fin de fichier rencontré"); ... /* On sort de la boucle */ break; }

devrait être dans ta boucle While et meme mieux, cela doit etre ta premiere instruction dans la boucle ... endehors de la boucle, ca ne sert pas a grand chose !

Et ta boucle un peu modifiée en consequence:

Code:

while(fgets(ligne, sizeof ligne, fIn))
8O bah non, il était très bien où il était : on boucle tant qu'une erreur ou la fin du fichier n'est pas apparue. Il fois que fgets nous renvoi NULL, on sort de la boucle et c'est là que l'on va tester la cause de la sortie de la boucle (grâce à feof ou ferror). Si c'est une erreur, errno pour en savoir plus.

@CSoldier : avec ton code, si on a une erreur, on ne sortira jamais de la boucle.

@kap : n'oublie pas le fclose à la fin.
19/10/2005, 13h49
Franck.H

Bah moi en generale je fait tous les tests possibles dans la boucle directement, si une erreur, un ptit break; et on sort !
19/10/2005, 14h01
gege2061

désolé, si tu sors de la boucle puisque fgets renvoi NULL mais avec une boucle while, le

Code:

if (feof(...))

ne seras pas exécuté une fois que fgets aura retourné NULL (puisque la boucle sera fini) donc tu perd du temps lorsque que tu ais en phase de lecture et une fois que le test devient utile, tu ne t'en sert pas :(
19/10/2005, 14h03
Franck.H

J'avais pas vu les choses sous cet angle, en effet :oops:
19/10/2005, 14h07
Emmanuel Delahaye

Citation:

Envoyé par CSoldier

Heu je crois que pour lire dans un fichier binaire il faut utiliser fread ... il me semble bien que les fichiers binaires sont toujours formattés d'une manière bien spécifique !

Un fichier n'est ni binaire ni texte. C'est une suite d'octets, c'est tout.

Simplement, la façon de l"ouvrir (texte, binaire) influence la façon d'interpréter les fins de lignes. En mode binaire, une fin de ligne est exactement le caractère '\n' (code 10 en ASCII). En mode texte, ça dépend du système.

Unixoide : '\n'
Mac OS (sauf X) : '\r'
DOS-Windows : '\n\r'
etc.

Mais le langage C fournit l'abstraction qui fait que, vue du programmeur, une fin de ligne est toujours '\n' lorsque le fichier est ouvert en mode texte.

D'autre part, le fait que le fichier soit organisé en lignes et qu'il ne comporte que des caractères imprimables est lié à la volonté de celui qui a écrit dedans et non à son mode d'ouverture. La seule chose est que lorsqu'on écrit ' \n' dans un fichier ouvert en mode texte, la séquence produite peut changer d'un système à l'autre.

Ceci dit, tu as raison de dire que fgets() n'est certainement pas la meilleure des fonctions pour lire du binaire.

Tout ceci pour dire qu'il ne faut pas confondre 'organisation des données dans un fichier' et 'mode d'ouverture', même si il est préférable de les faire coïncider.