Problème avec fscanf, bug ?

Version imprimable

Bonjour à tous,

je me prend la tete depuis hier soir sur un pb avec la fonction "fscanf" pour parcourir un fichier de type CSV.
mon bout de code fonctionne tres bien lorsque je le compile sur un serveur debian avec gcc version "gcc (GCC) 3.3.5 (Debian 1:3.3.5-13)"

et ne fonctionne que partiellement sur un autre serveur avec gcc version "(GCC) 4.1.2 20061115 (prerelease) (Debian 4.1.1-21)"

quand je dit "partiellement" c'est parce-que 8 fois sur 10, ma variable "pattern" est NULL, alors que le fichiers CSV est bien renseigné :-(

est-ce que vous avez deja rencontré ce genre de pb
PS: je compile comme ceci : gcc -o test.cgi test.c

PS2 : j'ai plusieurs serveur dedié, et mon code fonctionne sur tout les srv Debian Sarge et sur AUCUN debian Etch !
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 #include <stdio.h> #include <stdlib.h> #define MYCSV "test.csv" int main() { /* pointer fichier csv */ FILE *csv; /* contenu du csv 0,texte1,valeur1,valeurb,valeura,valeurc 5,texte2,valeur2,valeura,valeurb,valeurc 2,texte3,valeur3,valeurb,valeurc,valeura */ int sid; char pattern[255]; char val1[10]; char val2[16]; char val3[16]; if((csv=fopen(MYCSV,"r")) == NULL) { printf("Failed to open csv !"); } else { while((fscanf(csv,"%d,%[^,],%[^,],%[^,],%s\n",&sid,pattern,val1,val2,val3)) != EOF ) { printf("- %s\n",pattern); } fclose(csv); } }

Je viens de tester sous Windows et ton code fonctionne (du moins pour ce que j'ai pu en voir)

Juste une remarque, je n'utilise jamais scanf et ses dérivés (sscanf, fscanf, ...).

Dans ton cas, j'utiliserai fgets pour lire le fichier ligne par ligne et ensuite je séparerai les mots avec strtok(). Je n'ai jamais eu de problèmes.

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
56
57
58
 
char ligne[1024];
char *sid;
char *pattern;
char *val1;
char *val2;
char *val3;
int sid_number;
 
// ouvrir le fichier
...
 
// lire le fichier
while( (ptr = fgets(ligne, sizeof(ligne), csv) != NULL)
{
 
   sid = strtok(ligne, ",");
   if(sid == NULL)
   {
      // erreur de format
      continue;
   }
   sid_number = atoi(sid);
 
   pattern = strtok(NULL, ",");
   if(pattern == NULL)
   {
      // erreur de format
      continue;
   }
 
   val1 = strtok(NULL, ",");
   if(val1 == NULL)
   {
      // erreur de format
      continue;
   }
 
   val2 = strtok(NULL, ",");
   if(val2 == NULL)
   {
      // erreur de format
      continue;
   }
 
   val3 = strtok(NULL, ",");
   if(val3 == NULL)
   {
      // erreur de format
      continue;
   }
 
   // traiter les valeurs lues
   ...
}
 
// fermer le fichier
...

Le code semble un peu plus "bourrin" (et est certainement optimisable) comme cela mais cela marche

26/03/2008, 15h33
alainlc

bonjour et merci pour ton astuce, je la mettrai certainement en pratique si je ne trouve pas de solution à mon pb
voir même si ta méthode est plus "performante"

++
26/03/2008, 18h22
alainlc
j'ai trouvé comment faire en sorte que mon programme ne deconne plus, mais etant debutant je ne comprend pas tres bien de l'impact de ma modif et surtout si c'est bien ou pas :

Avant (fonctionne partiellement en gcc 4.1) :
Code:

1 2 3 4 5 char pattern[255]; char val1[10]; char val2[16]; char val3[16];
Apres :
Code:

1 2 3 4 5 char *pattern[255]; char *val1[10]; char *val2[16]; char *val3[16];
merci pour vos lumieres

Ceci fonctionne (N.B. il ne faut pas mettre le caractère de fin de ligne \n à la fin de la chaine de format):

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
56
57
58
59
60
#include <stdio.h>
#include <stdlib.h>
 
#define STR(s) STR_(s)
#define STR_(s) #s
 
#define MYCSV   "test.csv"
 
#define PATTERN_LEN 255
#define VAL1_LEN 10
#define VAL2_LEN 16
#define VAL3_LEN 16
 
static void
fpurge(FILE *fp)
{
    if (fp != NULL)
    {
        int c;
 
        while ((c = fgetc(fp)) != '\n' && c != EOF)
        {
        }
    }
}
 
int main(void)
{
    FILE *csv = NULL;
    int sid;
    char pattern[PATTERN_LEN + 1] = "";
    char val1[VAL1_LEN + 1] = "";
    char val2[VAL2_LEN + 1] = "";
    char val3[VAL3_LEN + 1] = "";
    int ret = EXIT_SUCCESS;
 
    csv = fopen(MYCSV, "r");
    if (csv == NULL)
    {
        fprintf(stderr, "Failed to open csv!\n");
        ret = EXIT_FAILURE;
    }
    else
    {
        while (fscanf(csv,
                       "%d,"
                       "%"STR(PATTERN_LEN)"[^,],"
                       "%"STR(VAL1_LEN)"[^,],"
                       "%"STR(VAL2_LEN)"[^,],"
                       "%"STR(VAL3_LEN)"s",
                       &sid, pattern, val1, val2, val3) == 5)
        {
            fpurge(csv);
            printf("- %s\n", pattern);
        }
        fclose(csv), csv = NULL;
    }
 
    return ret;
}

Thierry

26/03/2008, 19h01
alainlc

merci Thierry, je vient de tester et effectivement ta solution fonctionne nickel :D

je vois bien les différences par rapport à mon code mais il y à quelque truc que je ne comprend pas, comme le rôle de la fonction "fpurge" par exemple, si je l'enleve sa fonctionne toujours 8O (mais j'ai l'impression qu'elle ne fait rien du tout puisque le while est vide)

et comment tu détecte la fin du fichier csv ?
encore merci !
26/03/2008, 19h25
Thierry Chappuis

Citation:

Envoyé par alainlc

merci Thierry, je vient de tester et effectivement ta solution fonctionne nickel :D

je vois bien les différences par rapport à mon code mais il y à quelque truc que je ne comprend pas, comme le rôle de la fonction "fpurge" par exemple, si je l'enleve sa fonctionne toujours 8O (mais j'ai l'impression qu'elle ne fait rien du tout puisque le while est vide)

et comment tu détecte la fin du fichier csv ?
encore merci !

Lorsque la fin de fichier est rencontrée ou qu'une erreur de lecture ou de conversion intervient, fscanf() renvoie une valeur différente de 5 et on sort de la boucle.

En ce qui concerne l'utilité de la fonction fpurge(), elle permet comme son nom l'indique de purger le tampon du flux entrant à chaque tour de boucle. En effet, le spécificateur de format %s ne lit pas les caractères d'espacement. Ainsi, il reste apres lecture au minimum un caractère de fin de ligne '\n' dans le tampon du flux entrant. La présence de ce caractère dans ton tampon ne pose pas spécialement de problème, car tu commences ta saisie par la lecture d'un entier qui a l'avantage d'ignorer les caractères d'espacement. Toutefois, il se peut que la taille de val3 ne soit pas suffisante pour accueillir tous les caractères précédant le caractère de fin de ligne. Dans ce cas, la saisie serait tronquée et les caractères surnuméraires resteraient dans le tampon du flux entrant, entraînant une erreur de lecture au prochain tour de boucle. fpurge() permet d'éviter cette situation.

Thierry
27/03/2008, 13h08
corrector

fpurge

Citation:

Envoyé par Thierry Chappuis

fpurge() permet d'éviter cette situation.

Dans le man linux :

Citation:

Envoyé par man fpurge

CONFORMING TO

These functions are nonstandard and not portable. The function fpurge() was introduced in BSD 4.4 and is not available under Linux. The function __fpurge() was introduced in Solaris, and is present in glibc 2.1.95 and later.
27/03/2008, 13h30
Thierry Chappuis

Citation:

Envoyé par corrector

Dans le man linux :

Lorsque je fais référence à fpurge(), je parle de la fonction que j'ai implantée ici: http://www.developpez.net/forums/sho...76&postcount=5. Je n'ai pas dit que cette fonction était standard. D'ailleurs, il n'existe à ma connaissance pas de fonction standard permettant de purger le tampon d'un flux entrant. La fonction proposée est parfaitement portable et elle n'a rien à voir avec la page de man que tu cites.

Thierry
27/03/2008, 16h35
corrector

fpurge : Thierry Chappuis vs. BSD 4.4

Citation:

Envoyé par Thierry Chappuis

Lorsque je fais référence à fpurge(), je parle de la fonction que j'ai implantée ici: http://www.developpez.net/forums/sho...76&postcount=5.

J'ai lu trop vite ton code (et pas très attentivement), résultat j'ai compris de travers.

Citation:

Envoyé par Thierry Chappuis

Je n'ai pas dit que cette fonction était standard.

En effet. Mais il se trouve que le nom fait penser à une fonction standard prenant un FILE*, et j'ai recherché fpurge comme fonction de la libc.

Citation:

Envoyé par Thierry Chappuis

D'ailleurs, il n'existe à ma connaissance pas de fonction standard permettant de purger le tampon d'un flux entrant. La fonction proposée est parfaitement portable et elle n'a rien à voir avec la page de man que tu cites.

Effectivement, ton fpurge n'a absolument aucun rapport avec le fpurge de BSD 4.4 décrit par cette page de man, à part le nom. Elle n'a pas du tout le même effet.

Mais pour le lecteur qui aurait entendu parler de cette fonction fpurge, la confusion que j'ai faite est possible.

Sachant qu'il existe une fonction non-standard fpurge, qui, conformément à une abominable tradition, est déclarée dans un en-tête standard (<stdio.h>), penses-tu vraiment que ce soit une bonne idée de nommer ta fonction fpurge?
27/03/2008, 16h54
Thierry Chappuis

Citation:

Envoyé par corrector

Sachant qu'il existe une fonction non-standard fpurge, qui, conformément à une abominable tradition, est déclarée dans un en-tête standard (<stdio.h>), penses-tu vraiment que ce soit une bonne idée de nommer ta fonction fpurge?

flush_input_stream(), c'est mieux? Je l'ai appelée fpurge() parce que c'est sous cet identificateur qu'on la trouve généralement publiée sur ce forum et que fflush() est une déjà une fonction standard permettant de purger le tampon d'un flux sortant. fclean() est également déjà utilisée sur ce forum pour désigner une fonction permettant d'éliminer le caractère de fin de ligne saisi par la fonction fgets() lors d'une saisie utilisateur et de purger le tampon du flux entrant en cas de saisie tronquée.

Ni fpurge, ni fclean ne sont, à ma connaissance des identificateurs réservés pour une future extension de la bibliothèque standard. Le paragraphe 7.26.9 de la norme ne stipule en effet aucune main mise sur l'espace de nom f[a-z]*.

Mais je prends bonne note de l'existence d'une fonction fpurge() au sein de la bibliothèque standard utilisée sous BSD 4.4. Toutefois, il doit bien exister un flag -ansi qu'il est possible de passer au compilo pour désactiver ce type d'extensions de la libc.

Thierry