Pb de détection de "fausse" fin de fichier...

**shkyo** · 03/11/2005, 09h58

Bonjour à tous,

Je m'explique, j'ai un fichier texte brut qui contient des données arrivant par un port RS232, le fichier se rempli bien, pas de soucis.

Mais parfois, suite à une erreur de transmission (je pense), j'ai un caractère "ÿ" ou FF en hexa qui s'insère dans les données.

Et comme je les traite caractère par caractère, quand je tombe sur celui-ci, c'est considéré comme une fin de fichier, (que je teste avec le test suivant : caract!=EOF) et mon prog s'arrête !!!

Quelqu'un a-t-il une solution ???
Merci d'avance.

**Médinoc** · 03/11/2005, 11h14

Bon, je suppose que tu lis les données RS232 avec les fonctions d'IO standard (stdio.h):

Quelque part, tu utilises un type char pour storkcer le retour d'une fonction qui renvoit justement un int pour éviter ce genre de problèmes...

Trouve ce char et corrige le problème pour tester le retour de la fonction AVANT de convertir le résultat en char.

**gege2061** · 03/11/2005, 11h28

Bonjour,

Envoyé par shkyo

Et comme je les traite caractère par caractère, quand je tombe sur celui-ci, c'est considéré comme une fin de fichier, (que je teste avec le test suivant : caract!=EOF) et mon prog s'arrête !!!

feof te confirme cela?
c'est plutot bizare puisque EOF est un caractère invalide, généralement de code -1.

Code :

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

printf ("%x\n", EOF);

affiche FF ?

**shkyo** · 03/11/2005, 11h30

C'est pas exactement ça, je m'explique mieux, la réception des données dans un fichier texte se fait par l'hyperterminal de Windows et non pas par mon prog en C.

En fait, j'ouvre le fichier, ma variable 'caract' étant de type char, je lis avec getc de la façon suivante : caract=getc(head_file_base) et après je fais les tests sur 'caract'.

Mais si cela vient du type char, en quoi je le mets ? en byte ?

**Médinoc** · 03/11/2005, 11h41

getc()/fgetc()/compagnie renvoient un int...

donc en théorie, en ce moment, tu as un truc du genre:

Code :

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

1
2
3
4
5
char carac;
carac=getc(head_file_base);
if(carac==EOF) {
// ...
}

à remplacer par ceci:

Code :

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

1
2
3
4
5
6
7
int nCarac;
char carac;
nCarac=getc(head_file_base);
carac = (char)nCarac; //le cast explicite évite des warnings
if(nCarac==EOF) {
// ...
}

[/code]

**shkyo** · 03/11/2005, 11h42

Je crois que le plus simple c'est de mettre mon code car en fait, il n'est pas si long, alors s'il vous plait ne tapez pas car je ne suis pas un roi du C...

Voici le truc, et merci d'avance pour toute suggestion :

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
  FILE       *head_file_base,*head_file_fin;
  char       name_file[50],caract,ligne[40],date[9],type_prod[3];
  char       name_file2[60],ligne2[43];
  int        i,j,normale,taille;
 
  system("CLS");
 
  /* on ouvre le fichier à traiter */
  printf("Donnez le chemin et le nom de fichier à ouvrir :\n");
  scanf("%s",name_file);
  taille=strlen(name_file);
  head_file_base=fopen(name_file,"r+");
  /* test si le fichier s'est bien ouvert */
  if (head_file_base==NULL)
  {
     printf("\nIMPOSSIBLE D\'OUVRIR CE FICHIER-CI !!!\n");
     system("PAUSE");
     return 0;
     }
 
  /* on charge le 1er caractère et on le met dans le tableau */
  caract=getc(head_file_base);
  ligne[0]=caract;
 
  /* on finit de charger la 1ère ligne */
  for (i=1;i<40;i++)
      ligne[i]=getc(head_file_base);
 
  /* est-ce une ligne 'normale' ?*/
  if (ligne[15]=='N')
      normale=1;
  else
      normale=0;   
 
  /* récupération de la date et du type de prod */
  if (normale==1)
  {
     for (i=0;i<8;i++)
     {
         if (ligne[i]!='/')
             date[i]=ligne[i];
         else
             date[i]='_';
            }
     date[8]='\0';     // pour que la chaine se termine correctement !
 
     type_prod[0]=ligne[26];
     type_prod[1]=ligne[27];
     type_prod[2]='\0';     // idem
     }
  /*else
      return;*/
 
  /* on cré le fichier de réception en utilisant le nom de base+date+prod */
  /* création du nom */
  taille=taille-4;
  for (i=0;i<taille;i++)
      name_file2[i]=name_file[i];
 
  name_file2[i]='\0';      // toujours même raison
 
  strncat(name_file2,"_",1);
  strncat(name_file2,date,8);
  strncat(name_file2,"_",1);
  strncat(name_file2,type_prod,2);
  strncat(name_file2,".txt",4);
 
  /* création (ou ajout s'il existe déjà) du fichier */
  head_file_fin=fopen(name_file2,"a+");
  /* test si le fichier s'est bien ouvert */
  if (head_file_fin==NULL)
  {
     printf("\nIMPOSSIBLE DE CREER LE FICHIER !!!\n");
     system("PAUSE");
     return 0;
     }
 
  fclose(head_file_base);
  head_file_base=fopen(name_file,"r+");
  /* test si le fichier s'est bien ouvert */
  if (head_file_base==NULL)
  {
     printf("\nIMPOSSIBLE DE REOUVRIR CE FICHIER !!!\n");
     system("PAUSE");
     return 0;
     }
 
  /* et on lit et traite jusqu'à la fin du fichier */
  do
  {
        //caract=getc(head_file_base);
        for(i=0;i<40;i++)
        {
            caract=getc(head_file_base);
            while(caract=='ÿ')
               caract=getc(head_file_base);
            if (caract!=EOF)
            {
               ligne[i]=caract;
               if (caract=='\n')
               {
                  //printf("FIN DE LIGNE INATTENDU !!!\n");
                  break;
               //ligne[i]=caract;
                  }
               }
            else
            {
                printf("\nFIN DE FICHIER ATTEINTE\n");
                system("PAUSE");
                return 0;
                }
            }
           //printf("%s\n\n",ligne);            
           if (ligne[15]=='N')   
           {
           j=0;  // initialise l'indice de calage
           for(i=0;i<40;i++)
           {
               if (ligne[i]!=' ' && ligne[i]!=',' && ligne[i]!='k')
                  ligne2[j++]=ligne[i];
 
               if (ligne[i]==' ' && i!=26)
               {
                  ligne[i]='Z';
                  ligne2[j++]=ligne[i];
                  }
 
               if (ligne[i]==' ' && i==26)
                  ligne2[j++]=ligne[i];
 
               if (ligne[i]==',' && i==28)
               {
                  ligne2[j++]='Z';
                  ligne2[j++]=ligne[i];
                  ligne2[j++]='Z';
                  }
 
               if (ligne[i]=='k' && i==32)
               {
                  ligne2[j++]='Z';
                  ligne2[j++]=ligne[i];
                  }
               }
           //printf("%s\n",ligne2);
           for (i=0;i<42;i++)
               putc(ligne2[i],head_file_fin);     // on transfert la ligne
           }
           else
           {
               while(caract!='\n')
                    caract=getc(head_file_base);
 
               for (i=0;i<42;i++)
                   ligne2[i]='0';  // on remet à zéro !!
               for (i=0;i<39;i++)
                   ligne[i]='0';  // on remet à zéro !!
 
               printf("LIGNE ANORMALE !\n");
 
               /* pour le calage de la suite */
               //caract=getc(head_file_base);
               }
               for (i=0;i<42;i++)
                   ligne2[i]='0';  // on remet à zéro !!
               for (i=0;i<39;i++)
                   ligne[i]='0';  // on remet à zéro !!
 
           //caract=getc(head_file_base);
           } while (caract!=EOF);
 
  /* on a fini le traitement, on ferme donc tous les fichiers */
  fclose(head_file_base);
  fclose(head_file_fin);
 
  system("PAUSE");	
  return 0;
}

Et voici un petit extrait de fichier texte que je traite, sachant que cette saleté de caractère 'ÿ' se met la plupart du temps au début.

26/04/58 13h49 Np: 179087 25,160kg
26/04/58 13h49 Np: 179088 25,720kg
26/04/58 13h49 Np: 179089 25,260kg
26/04/58 13h49 Np: 179090 25,540kg
26/04/58 13h50 ANNULATION
---------------------------------------
26/04/58 13h50 Nb doses: 7553 HT: 62
Total: 91619,400kg Moy.: 12,130kg
Cadence: 9s/dose

26/04/58 13h50 ANNULATION
---------------------------------------
26/04/58 13h50 Nb doses: 0000 HT: 00
Total: 0,000kg Moy.: 12,130kg
Cadence: 0s/dose

26/04/58 13h51 Np: 179091 25,480kg
26/04/58 13h51 Np: 179092 25,260kg
26/04/58 13h51 Np: 179093 26,040kg HT+
26/04/58 13h51 ACQ. HT
26/04/58 13h51 Np: 179094 26,040kg
26/04/58 13h51 Np: 179095 25,700kg
26/04/58 13h51 Np: 179096 25,700kg

**Emmanuel Delahaye** · 03/11/2005, 14h19

Envoyé par shkyo

Et comme je les traite caractère par caractère, quand je tombe sur celui-ci, c'est considéré comme une fin de fichier, (que je teste avec le test suivant : caract!=EOF) et mon prog s'arrête !!!

Ouvrir son livre de C et constater que le type retourné par fgetc() est int. Definir 'caract' en int et non en char. Ce type permet de contenir EOF et de le différencier de 0xFF (ou ÿ en ASCII).

**shkyo** · 03/11/2005, 15h14

Envoyé par Emmanuel Delahaye

Envoyé par shkyo

Et comme je les traite caractère par caractère, quand je tombe sur celui-ci, c'est considéré comme une fin de fichier, (que je teste avec le test suivant : caract!=EOF) et mon prog s'arrête !!!

Ouvrir son livre de C et constater que le type retourné par fgetc() est int. Definir 'caract' en int et non en char. Ce type permet de contenir EOF et de le différencier de 0xFF (ou ÿ en ASCII).

OKOK je le note...

Mais comme je l'ai dis plus haut, je bricole en C, cela doit se voir dans mon code, je pense...

Merci beaucoup en tout cas, je vais tester tout ça de ce pas

**gege2061** · 03/11/2005, 15h30

Envoyé par Emmanuel Delahaye

Envoyé par shkyo

Et comme je les traite caractère par caractère, quand je tombe sur celui-ci, c'est considéré comme une fin de fichier, (que je teste avec le test suivant : caract!=EOF) et mon prog s'arrête !!!

Ouvrir son livre de C et constater que le type retourné par fgetc() est int. Definir 'caract' en int et non en char. Ce type permet de contenir EOF et de le différencier de 0xFF (ou ÿ en ASCII).

Je ne sais pas qui à dit que les cordonnier les plus mal chaussés mais il avait raison : Fonctions retournant un caractère (*getc*)

**David.Schris** · 03/11/2005, 23h22

Envoyé par gege2061

Je ne sais pas qui à dit que les cordonnier les plus mal chaussés mais il avait raison : Fonctions retournant un caractère (*getc*)

Mouais, enfin on peut quand même y lire "Ces fonctions retournent un int". Ca semble assez clair.
Le titre peut prêter à confusion, mais n'est pas faux : ces fonctions renvoient (sauf en cas d'EOF) bien un caractère (mais pas un "char"...nuance...).

Envoyé par man fgetc

fgetc(), getc() et getchar() renvoient un caractère, lu comme un unsigned char, et transformé en int, ou EOF à la fin du fichier, ou en cas d'erreur.

**shkyo** · 04/11/2005, 09h45

Envoyé par Médinoc

getc()/fgetc()/compagnie renvoient un int...

donc en théorie, en ce moment, tu as un truc du genre:

Code :

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

1
2
3
4
5
char carac;
carac=getc(head_file_base);
if(carac==EOF) {
// ...
}

à remplacer par ceci:

Code :

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

1
2
3
4
5
6
7
int nCarac;
char carac;
nCarac=getc(head_file_base);
carac = (char)nCarac; //le cast explicite évite des warnings
if(nCarac==EOF) {
// ...
}

[/code]

YES

ça marche !!! J'avais déjà entendu parlé des histoires de 'cast' sans trop comprendre l'intéret du truc, et bien maintenant je le sais...

Merci à tous pour votre aide !

**Emmanuel Delahaye** · 04/11/2005, 10h23

Envoyé par Médinoc

à remplacer par ceci:

Code :

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

1
2
3
4
5
6
7
int nCarac;
char carac;
nCarac=getc(head_file_base);
carac = (char)nCarac; //le cast explicite évite des warnings
if(nCarac==EOF) {
// ...
}

C'est bien compliqué. Pourquoi pas

Code :

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

1
2
3
 
int carac=getc(head_file_base);
if(carac==EOF) {

**shkyo** · 04/11/2005, 11h07

Envoyé par Emmanuel Delahaye

Envoyé par Médinoc

à remplacer par ceci:

Code :

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

1
2
3
4
5
6
7
int nCarac;
char carac;
nCarac=getc(head_file_base);
carac = (char)nCarac; //le cast explicite évite des warnings
if(nCarac==EOF) {
// ...
}

C'est bien compliqué. Pourquoi pas

Code :

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

1
2
3
 
int carac=getc(head_file_base);
if(carac==EOF) {

Pourquoi pas, je ne suis pas sectaire, mais qu'est-ce que cela change

: à part le nombre de ligne bien sur...

**Emmanuel Delahaye** · 04/11/2005, 11h58

Envoyé par shkyo

Envoyé par Emmanuel Delahaye

C'est bien compliqué. Pourquoi pas

Code :

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

1
2
3
 
int carac=getc(head_file_base);
if(carac==EOF) {

Pourquoi pas, je ne suis pas sectaire, mais qu'est-ce que cela change

: à part le nombre de ligne bien sur...

Plus simple, plus clair, une variable en moins...

Pb de détection de "fausse" fin de fichier...

C

Partager

Partager