Problème avec fgets

**fabpeden** · 14/05/2007, 18h39

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

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
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);

et

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
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...

**Freed0** · 14/05/2007, 18h41

Salut

Ta fonction TraitementSMS est incomplète

Donc il manque déjà le fclose

**Skyounet** · 14/05/2007, 18h45

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

D'où vient LigneLue ???

Variable globale ?

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

Tu fais du multi-thread ???

**fabpeden** · 14/05/2007, 18h50

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.

**Skyounet** · 14/05/2007, 18h55

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é.

**diogene** · 14/05/2007, 19h02

Code :

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

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 :

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

LigneLue = malloc(100);

C'est mieux dans

Code :

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

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

gl · 14/05/2007, 21h41

Envoyé par diogene

C'est mieux dans

Code :

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

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

J'ai une preference pour

Code :

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

1
2
3
 
char LigneLue[100];
fgets(LigneLue,sizeof LigneLue,fp);

qui est a est plus facile a faire evoluer

**Emmanuel Delahaye** · 15/05/2007, 00h02

Envoyé par fabpeden

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

Code :

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

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

Code :

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

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 :

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

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...)

**dj.motte** · 15/05/2007, 01h31

Salut,

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

**fabpeden** · 15/05/2007, 10h01

Voici le code de ma procédure :

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
//---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?

**diogene** · 15/05/2007, 11h01

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

Code :

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

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,.....);

**fabpeden** · 15/05/2007, 11h10

C'est en commentaire parce que j'ai testé avec plusieurs cas possibles. Voici ce que j'ai mis :

Code :

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

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.

Problème avec fgets

C

Vue hybride

Discussions similaires

Partager

Partager