Problème avec fgets

Version imprimable

Problème avec fgets

salut à tous, j'ai une erreur très étrange, j'ai deux fonctions :
TraitementSMS et DetermineTypeBoitier

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
void TraitementSMS() 
{
char *LigneLue;   // Contient la ligne lue du fichier texte
char TypeBoitier; // Contient la lettre qui détermine le type de boitier
 
    LigneLue = (char*)malloc(sizeof(char));
 
    //---Déclarations et initialisations des données---//
    TypeBoitier = DetermineTypeBoitier();
 
    //---Ouverture du fichier de données---//
    FILE *f = NULL;
    f = fopen("CompteurP.txt","r");   
    if(f == NULL)
    {
      perror("fopen");
      exit(EXIT_FAILURE);
    } 
 
    fgets(LigneLue,100,f);
    printf("%s",LigneLue);
fclose(f);

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
char DetermineTypeBoitier()
{
 //---Déclarations et initialisations des variables---//
    char *pointeur;
    char LigneLue[100];
 
    int NbLigne = 0;
 
    FILE *fp;
 
    //---Ouverture du fichier de données---//
    fp = fopen("CompteurP.txt","r");
    if(fp == NULL)
    {
     perror("fopen");
     exit(EXIT_FAILURE);
    }
 
    //---On lit les 2 premières lignes du fichier---//
    while(NbLigne <2)
    {          
        fgets(LigneLue,100,fp);
        NbLigne++;
    }
fclose(fp);
}

Le problème est que pour la fonction DetermineTypeBoitier, la fonction fgets fonctionne très bien, le résultat obtenu est correct. Ensuite je réouvre le même fichier, je refais fgets et là l'exécution du programme plante. De plus, à l'origine un autre programme utilisait ces fonctions qui marchaient très bien, donc je trouve cela bien étrange...

14/05/2007, 17h41
Freed0

Salut ;)

Ta fonction TraitementSMS est incomplète ;) Donc il manque déjà le fclose ;)
14/05/2007, 17h45
Skyounet

Dans ta boucle while dans la 2ème fonction, la condition de sortie ne change jamais.

D'où vient LigneLue ???

Variable globale ? :vomi:

Pis y'a plein de variable pas déclaré c'est horrible ?

Tu fais du multi-thread ???
14/05/2007, 17h50
fabpeden

Désolé pour les variables, je les ai enlevé de mon post pour ne pas rendre l'ensemble illisible. De même pour la boucle, j'ai essayé de réduire le code au minimum, mais je vais arranger cela.
14/05/2007, 17h55
Skyounet

Quant au fclose il faut l'englober dans ton if(fp != NULL)

[EDIT] J'ai rien dis tu as un if(fp == NULL) désolé.

Code:

1 2 3 4 5 6 7 void TraitementSMS() { char *LigneLue; // Contient la ligne lue du fichier texte ... LigneLue = (char*)malloc(sizeof(char)); ... fgets(LigneLue,100,f);
Il y a la place pour stocker UN caractère, soit le zéro terminal de la chaîne. A la première lecture , on déborde du tableau réservé -> le programme est détruit.

Code:

LigneLue = malloc(100);

C'est mieux dans
Code:

1 2 3 4 5 6 char DetermineTypeBoitier() { ... char LigneLue[100]; ... fgets(LigneLue,100,fp);

Citation:
Envoyé par diogene
C'est mieux dans
Code:

1 2 3 4 5 6 char DetermineTypeBoitier() { ... char LigneLue[100]; ... fgets(LigneLue,100,fp);
J'ai une preference pour
Code:

1 2 3 char LigneLue[100]; fgets(LigneLue,sizeof LigneLue,fp);
qui est a est plus facile a faire evoluer

14/05/2007, 23h02
Emmanuel Delahaye
Citation:
Envoyé par fabpeden

salut à tous, j'ai une erreur très étrange, j'ai deux fonctions :
TraitementSMS et DetermineTypeBoitier

Code:

1 2 3 4 void TraitementSMS() { <...> fgets(LigneLue,100,f);

Code:

1 2 3 4 5 6 7 8 char DetermineTypeBoitier() { <...> while(NbLigne <2) { fgets(LigneLue,100,fp); NbLigne++; }
La forme canonique pour lire un fichier texte est :
Code:

1 2 3 4 5 6 7 char ligne[TAILLE]; while (fgets (ligne, sizeof ligne, fp) != NULL) { /* traiter la ligne */ }
ce que tu fais est très douteux... (en gros, aucun contrôle de la validité de la lecture...) et je ne parle pas des pointeurs sur des zones trop petites... C'est pas bien de mentir à son compilateur (genre allouer 1, autoriser 100...)
15/05/2007, 00h31
dj.motte

Salut,

De plus si une ligne du fichier excède 100 caractères, le buffer ( ligneLue ) dans fgets, sera en situation de débordement.

Voici le code de ma procédure :

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
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
//---Déclarations et initialisations des données---//
    char *LigneLue;   // Contient la ligne lue du fichier texte
    char *DebutSMS;   // Contient l'en tête du SMS
    char *pointeur;   // Contient la chaîne de caractère extraite entre chaque TAB
    char TypeBoitier; // Contient la lettre qui détermine le type de boitier
    char *NumModem;
 
    unsigned int VM_CptIn[96];     // Tableau contenant les DonnéesIn
    unsigned int VM_CptOut[96];    // Tableau contenant les DonnéesOut
 
    unsigned int nb_mot = 1;       // Contient le nombre de chaîne de caractère extraite de la ligne lue
    unsigned int i = 0;
    unsigned int NbLigne = 1;      // Permet de connaître le nombre de ligne du fichier donc l'heure de la fin
 
    DebutSMS = (char*)malloc(sizeof(char));  // allocation de la mémoire pour la variable DébutSMS
   // LigneLue = (char*)malloc(sizeof(char));
 
    //---initialisations des tableaux---//
    for(i=0;i<96;i++)
    {
         VM_CptIn[i] = 0;
         VM_CptOut[i] = 0;                         
    }                 
    i = 0;
 
    // Lecture des deux premières lignes afin de déterminer le type du compteur, P,Q,T,U
    TypeBoitier = DetermineTypeBoitier();
 
    //---Ouverture du fichier de données---//
    FILE *fp = NULL;
    fp = fopen("CompteurP.txt","r");   
 
    //------------------------------------------------------------------------------------//
    // LECTURE DU FICHIER, MISE EN FORME DE L'EN TETE ET REMPLISSAGE DU TABLEAU           //
    //------------------------------------------------------------------------------------//
    while (fgets(LigneLue,100,fp) != NULL)
     {                       
             printf("Ligne : %s",LigneLue);
             //--- On extrait la chaîne de caractère qui précède le caractère 0x09(Tabulation)---//
             pointeur = strtok( LigneLue, "\x09" );
             printf("s1 : %s\n",pointeur);
             if(NbLigne == 1)
             {
                 //---Mise en forme de l'en tête du SMS---//
                 switch (TypeBoitier)
                 {
                        case 'P' : strcat(DebutSMS,"P");
                                   break;
                        case 'T' : strcat(DebutSMS,"T");
                                   break;
                        case 'Q' : strcat(DebutSMS,"Q");
                                   break;
                        case 'U' : strcat(DebutSMS,"U");
                                   break;
                 }
 
                 strcat(DebutSMS,pointeur);
                 NumModem = pointeur;
 
                 //--- selon le type de boitier, on met en forme le début du SMS ---//
                 if ((TypeBoitier == 'P') || (TypeBoitier == 'Q'))
                 {
                     strcat(DebutSMS,"/012/0189/02/C");                 
                 }
                 else
                 {
                     strcat(DebutSMS,"/012/0189/02/I");
                 }
             } 
             printf("Ligne : %s",LigneLue);         
            // tant que l'on a pas lu tout le fichier...
            while( pointeur != NULL )
            {
                 // Cherche les autres caractères TAB '0x09' dans la ligne lue
                 pointeur = strtok( NULL, "\x09" );
                 // On a trouvé un caractère TAB
                 if ( pointeur != NULL )
                 {
                     nb_mot++; 
                     printf("s2 : %s\n",pointeur);
                     // Dans un compteur Pilot, on a 3 mots, NumModem,Date,DonnéeCumul
                     // si DonnéeCumul
                     if ((nb_mot == 3) && ((TypeBoitier == 'P') || (TypeBoitier == 'Q')))
                     {
                        VM_CptIn[i] = atoi(pointeur);
                        i++;                                   
                     }
 
                     // Dans un compteur Twin, on a 5 mots, NumModem, Date, DonnéeCumul, DonnéeIn, DonnéeOut
                     // si DonnéeIn
                     if (nb_mot == 4) 
                     {
                        VM_CptIn[i] = atoi(pointeur);
                     }
 
                     // si DonnéeOut
                     if (nb_mot == 5) 
                     {
                        VM_CptOut[i] = atoi(pointeur);
                        i++;
                     }
                 }  
            }
            nb_mot = 1;
            NbLigne++;     
     }

Il se passe quelque d'étrange, lorsque NbLigne == 1, on passe dans la condition qui va bien, et quand j'affiche LigneLue celle-ci a changé de valeur :
Par exemple, elle vaut 153TAB20070402150000000TAB1 elle se tranforme en /02/C, ce qui fausse la suite du traitement, je ne comprends pas commment cela est possible?

On a toujours les mêmes erreurs d'allocation qu'avant. Tu n'as toujours pas compris le problème!

Code:

1
2
3
4
5
6
7
8
9
    char *DebutSMS;   // 
    char *pointeur;   // Contient la chaîne de caractère extraite entre chaque TAB
...
    DebutSMS = (char*)malloc(sizeof(char));  // 1 caractère d'alloué
   // LigneLue = (char*)malloc(sizeof(char)); Pas d'allocation pour LigneLue
...
while (fgets(LigneLue,100,fp) != NULL)

strcat(DebutSMS,.....);

15/05/2007, 10h10
fabpeden
C'est en commentaire parce que j'ai testé avec plusieurs cas possibles. Voici ce que j'ai mis :
Code:

1 2 DebutSMS = (char*)malloc(sizeof(char)); // allocation de la mémoire pour la variable DébutSMS LigneLue = (char*)malloc(100*sizeof(char));
Mais comment puis-je connaître à l'avance le nombre de caractères que j'aurais besoin pour DebutSMS, je mets une valeur que je choisie?
J'ai mis 200 et je crois que cela fonctionne bien, je fais faire d'autres tests. Merci en tous cas.
15/05/2007, 10h17
diogene

Citation:

Mais comment puis-je connaître à l'avance le nombre de caractères que j'aurais besoin pour DebutSMS, je mets une valeur que je choisie?

- Je ne peux pas répondre à cette question, tu connais ton problème mieux que moi.
- Dans tous les cas, 1 caractère ne laisse de la place que pour le 0 terminant la chaîne et ça ne peut pas marcher.
- A cause des strcat que tu fais sur DebutSMS, la chaîne DebutSMS doit être initialisée par un 0 en tête avant.
15/05/2007, 10h25
fabpeden

En tout cas, merci à tous pour votre aide et vos précieux conseils!