fichier texte et Liste Chainée

**alexglvr** · 31/10/2008, 12h06

Bonjour à tous,

je cherche à lire un fichier texte structuré (5 colonnes de nombres entiers).
Je cherche à mémoriser toutes ces valeurs dans une sorte de matrice.Or comme je ne connais pas le nombre de lignes du fichier texte, je suis parti sur une liste chainée... Voici mon code

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
#include <stdio.h>
#include <stdlib.h>

int main(int n) //, char *nomfile[])
{
FILE *Out;
int Nb_Combi=0;

struct Liste {
       int Indice;
       int Combi[5];
       int N4;
       int N3;
       int N2;
       int N1;
       
       struct Liste * psuiv;
       };
       
struct Liste *pNouveau;
struct Liste *pTete;
struct Liste *pCourant;

pTete = NULL;
pNouveau = NULL;
pCourant = NULL;

Out=fopen("resultats_num_treated.txt", "rb");

// Premier maillon = Premiere Ligne
Nb_Combi=1;
pNouveau = malloc(sizeof(struct Liste));
pTete = pNouveau;
pCourant = pTete;
pCourant->psuiv = NULL;
pCourant->Indice = Nb_Combi;
pCourant->N4=0;
pCourant->N3=0;
pCourant->N2=0;
pCourant->N1=0;

fscanf(Out, " %d %d %d %d %d\n ", pCourant->Combi[1], pCourant->Combi[2], pCourant->Combi[3], pCourant->Combi[4], pCourant->Combi[5]);
////////////////////////////////////

while (!feof(Out))) {

Nb_Combi = Nb_Combi+1;

pNouveau = malloc(sizeof(struct Liste));
pCourant->psuiv = pNouveau;
pCourant = pNouveau;
pCourant->psuiv = NULL;
pCourant->Indice = Nb_Combi;
pCourant->N4=0;
pCourant->N3=0;
pCourant->N2=0;
pCourant->N1=0;
fscanf(Out, " %d %d %d %d %d\n ", pCourant->Combi[1], pCourant->Combi[2], pCourant->Combi[3], pCourant->Combi[4], pCourant->Combi[5]);

}

 fclose(Out);

J'ai un problème sur les fscanf...

LE reste semble t il correct?

Merci de votre aide,

Alex

**fregolo52** · 31/10/2008, 13h26

salut

je n'ai jamais aussez utiliser fscanf, mais j'ai l'impression que tu as oublié le "&" devant les paramètre : pCourant->Combi[1] n'est pas une adresse.

**souviron34** · 31/10/2008, 13h31

M'enfin, où avez-vous appris le C ???

Code :

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

1
2
 
while (!feof(Out))) {

feof ne fait pas ce que tu crois .....

**alexglvr** · 31/10/2008, 13h31

effectivement, j'avais mis le & avant, mais le même problème...

Apparemment, c'est le -> qui pose pb...

Alex

**somallg** · 31/10/2008, 22h13

Code :

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

fscanf(Out, " %d %d %d %d %d\n ", pCourant->Combi[1], pCourant->Combi[2], pCourant->Combi[3], pCourant->Combi[4], pCourant->Combi[5]);

Bonjour
Je crois que le problem est dans ton fscanf : parceque Combi[5] n'exite pas ( le 1ie indince d'un tableau est toujours 0 , pas 1 comme ce que tu as fait)

**alexglvr** · 01/11/2008, 09h59

M'enfin, où avez-vous appris le C ???

les bases il y a une petite dizaine d'année à mon école d'ingé...
Depuis très peu pratiqué en info. Beaucoup en elec pour la programmation de micro contrôleur... Et oui, je suis électronicien... Mais ce n'est pas la même chose...

Malheureusement, pour un certain niveau de technicité, il faut faire ses outils soit même...

En tout cas merci de ton encouragement...

**alexglvr** · 01/11/2008, 10h34

je viens de regarder pour feof...

Voila ce que j'ai trouvé

Code :

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

1
2
3
4
5
 
int c;
file *in;
while ((c=fgetc(in))!=EOF) {
}

j'ai testé. La boucle s'arrête à condition que c soit un int. A l'intérieur de ma boucle, je teste le caractère et le recopie dans un deuxième fichier si il correspond à ce que je cherche...
Mais avec un int pour c, la recopie ne marche pas...
J'ai essayé de caster... Pareil...

Alex

**Emmanuel Delahaye** · 01/11/2008, 10h37

Envoyé par alexglvr

A l'intérieur de ma boucle, je teste le caractère et le recopie dans un deuxième fichier si il correspond à ce que je cherche...
Mais avec un int pour c, la recopie ne marche pas...
J'ai essayé de caster... Pareil...

Ca doit fonctionner. Montre ton code.

(je suis aussi électronicien...)

**alexglvr** · 01/11/2008, 10h51

Merci de ta reponse!

voila le code:

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
 
int car;
file *In, *Out;
 
In=fopen("resultats_num.txt", "rb");
Out=fopen("resultats_num_treated.txt", "wb");
 
 
printf("Treating File Num...\n\n");
while ((car=fgetc(In))!=EOF) {
 
  if ((car!='[')&&(car!=']')&&(car!=','))
     {
     putc(car,Out);
     }     
     car = fgetc(In);
}
fclose(In);
fclose(Out);

la forme du fichier d'entrée est:

[5, 200, 20, 10]
[7, 58, 29, 15]

etc...

**Emmanuel Delahaye** · 01/11/2008, 11h37

Envoyé par alexglvr

Merci de ta reponse!

voila le code:
<...>

Tu fais 2 lectures dans la boucle. C'est une de trop. Ceci devrait fonctionner :

Code :

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

1
2
3
4
5
6
7
8
 
while ((car = fgetc (In)) != EOF) 
{
   if ((car != '[') && (car != ']') && (car != ','))
   {
      putc (car, Out);
   }     
}

**alexglvr** · 03/11/2008, 13h36

bonjour à tous,

la lecture et le traitement du fichier se passe bien, par contre, je n'arrive toujours pas à mettre les valeurs dans une liste chainée...

Ça plante toujours sur le fscanf :

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
#include <stdio.h>
#include <stdlib.h>
 
int main(int n) //, char *nomfile[])
{
FILE *In, *Out, *Out2;
int car, i;
int Nb_Combi=0;
 
struct Liste {
       int Indice;
       int Combi[5];
       int N4;
       int N3;
       int N2;
       int N1;
 
       struct Liste * psuiv;
       };
 
struct Liste *pNouveau;
struct Liste *pTete;
struct Liste *pCourant;
 
pTete = NULL;
pNouveau = NULL;
pCourant = NULL;
 
 
In=fopen("resultats_num.txt", "rb");
Out=fopen("resultats_num_treated.txt", "wb");
 
printf("Treating File Num...\n\n");
 
while ((car=fgetc(In))!=EOF) {
 
  if ((car!='[')&&(car!=']')&&(car!=','))
     {
     putc(car,Out);
     }     
  else {
       if (car=='[') {Nb_Combi++;}
       }
}
fclose(In);
fclose(Out);
 
Out=fopen("resultats_num_treated.txt", "rb");
Out2=fopen("analyse_res_num.txt", "wb");
 
printf("Creation liste Chainee...\n\n");
 
// Premier maillon = Premiere Ligne
 
pNouveau = malloc(sizeof(struct Liste));
pTete = pNouveau;
pCourant = pTete;
pCourant->psuiv = NULL;
pCourant->Indice = 1;
pCourant->N4=0;
pCourant->N3=0;
pCourant->N2=0;
pCourant->N1=0;
fscanf(Out, " %d %d %d %d %d\n ", pCourant->Combi[0], pCourant->Combi[1], pCourant->Combi[2], pCourant->Combi[3], pCourant->Combi[4]);
////////////////////////////////////
 
for (i=2;i<Nb_Combi+1;i++) {
 
pNouveau = malloc(sizeof(struct Liste));
pCourant->psuiv = pNouveau;
pCourant = pNouveau;
pCourant->psuiv = NULL;
pCourant->Indice = i;
pCourant->N4=0;
pCourant->N3=0;
pCourant->N2=0;
pCourant->N1=0;
fscanf(Out, " %d %d %d %d %d\n ", pCourant->Combi[0], pCourant->Combi[1], pCourant->Combi[2], pCourant->Combi[3], pCourant->Combi[4]);
 
}
 
 fclose(Out);
 fclose(Out2);
 
 
printf("Done !!!\n\n");
}

merci de votre aide,

Alex

**souviron34** · 03/11/2008, 14h04

normal..

fscanf est pour un fichier formatté, read est pour un fichier binaire.

Vu que tu ouvres ton fichier en binaire (pourquoi ???) il te faut utiliser read...

Le plus simple serait de l'ouvrir en texte (open (fichier, "r")) et de faire un fscanf...

Et d'ouvrir tous tes fichiers en formatté...

**Emmanuel Delahaye** · 03/11/2008, 14h07

Envoyé par alexglvr

la lecture et le traitement du fichier se passe bien, par contre, je n'arrive toujours pas à mettre les valeurs dans une liste chainée...

Tu ne devrais pas poster du code bourré de fautes de codage.

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
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c:5: warning: 'main' takes only zero or two arguments
C:\dev\hello\main.c: In function `main':
C:\dev\hello\main.c:70: warning: format argument is not a pointer (arg 3)
C:\dev\hello\main.c:70: warning: format argument is not a pointer (arg 4)
C:\dev\hello\main.c:70: warning: format argument is not a pointer (arg 5)
C:\dev\hello\main.c:70: warning: format argument is not a pointer (arg 6)
C:\dev\hello\main.c:70: warning: format argument is not a pointer (arg 7)
C:\dev\hello\main.c:87: warning: format argument is not a pointer (arg 3)
C:\dev\hello\main.c:87: warning: format argument is not a pointer (arg 4)
C:\dev\hello\main.c:87: warning: format argument is not a pointer (arg 5)
C:\dev\hello\main.c:87: warning: format argument is not a pointer (arg 6)
C:\dev\hello\main.c:87: warning: format argument is not a pointer (arg 7)
C:\dev\hello\main.c: At top level:
C:\dev\hello\main.c:4: warning: unused parameter 'n'
C:\dev\hello\main.c: In function `main':
C:\dev\hello\main.c:95: warning: control reaches end of non-void function
Output size is 20.90 KB
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 13 warnings

J'ai pas déjà indiqué comment configurer son compilateur correctement ?

http://emmanuel-delahaye.developpez....tm#cfg_compilo

Ton code est confus. Je conseille la réduction de la portée des variables

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
 
#include <stdio.h>
#include <stdlib.h>
struct node
{
   int Indice;
   int Combi[5];
   int N4;
   int N3;
   int N2;
   int N1;
 
   struct node *psuiv;
 
};
 
struct liste
{
   struct node *pTete;
};
 
int main (void)
{
   int Nb_Combi = 0;
   struct liste liste = { 0 };
 
   {
      FILE *In = fopen ("resultats_num.txt", "rb");
      if (In != NULL)
      {
         FILE *Out = fopen ("resultats_num_treated.txt", "wb");
         if (Out != NULL)
         {
            int car;
            printf ("Treating File Num...\n\n");
 
            while ((car = fgetc (In)) != EOF)
            {
 
               if ((car != '[') && (car != ']') && (car != ','))
               {
                  putc (car, Out);
               }
               else
               {
                  if (car == '[')
                  {
                     Nb_Combi++;
                  }
               }
            }
            fclose (Out);
         }
         fclose (In);
      }
   }
   {
      FILE *Out = fopen ("resultats_num_treated.txt", "rb");
      if (Out != NULL)
      {
         FILE *Out2 = fopen ("analyse_res_num.txt", "wb");
         if (Out2 != NULL)
         {
            printf ("Creation liste Chainee...\n\n");
 
/* Premier maillon = Premiere Ligne */
            {
               struct node *pCourant = NULL;
               struct node *pNouveau = malloc (sizeof *pNouveau);
               liste.pTete = pNouveau;
               {
                  pCourant = liste.pTete;
                  pCourant->psuiv = NULL;
                  pCourant->Indice = 1;
                  pCourant->N4 = 0;
                  pCourant->N3 = 0;
                  pCourant->N2 = 0;
                  pCourant->N1 = 0;
                  fscanf (Out, " %d %d %d %d %d\n ", &pCourant->Combi[0],
                          &pCourant->Combi[1], &pCourant->Combi[2],
                          &pCourant->Combi[3], &pCourant->Combi[4]);
               }
/* ////////////////////////////////// */
               {
                  int i;
                  for (i = 2; i < Nb_Combi + 1; i++)
                  {
 
                     struct node *pNouveau = malloc (sizeof *pNouveau);
                     pCourant->psuiv = pNouveau;
                     pCourant = pNouveau;
                     pCourant->psuiv = NULL;
                     pCourant->Indice = i;
                     pCourant->N4 = 0;
                     pCourant->N3 = 0;
                     pCourant->N2 = 0;
                     pCourant->N1 = 0;
                     fscanf (Out, " %d %d %d %d %d\n ", &pCourant->Combi[0],
                             &pCourant->Combi[1], &pCourant->Combi[2],
                             &pCourant->Combi[3], &pCourant->Combi[4]);
 
                  }
               }
            }
 
            fclose (Out2);
         }
         fclose (Out);
      }
   }
   printf ("Done !!!\n\n");
 
   return 0;
}

et le découpage en fonctions qui en découle... (j'ai simplifié la structure de donnée, pour l'exemple)

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
 
#include <stdio.h>
#include <stdlib.h>
struct data
{
   int Indice;
   int N4;
   int N3;
   int N2;
   int N1;
 
};
 
struct node
{
   struct data data;
   struct node *psuiv;
 
};
 
struct liste
{
   struct node *pTete;
   struct node *pQueue;
};
 
static void Filtrage (void)
{
   FILE *In = fopen ("resultats_num.txt", "r");
   if (In != NULL)
   {
      FILE *Out = fopen ("resultats_num_treated.txt", "w");
      if (Out != NULL)
      {
         int car;
         printf ("Treating File Num...\n\n");
 
         while ((car = fgetc (In)) != EOF)
         {
 
            if ((car != '[') && (car != ']') && (car != ','))
            {
               putc (car, Out);
            }
 
         }
         fclose (Out);
      }
      fclose (In);
   }
}
 
static void add (struct liste *p_liste, struct data const *p_data)
{
   struct node *pNouveau = malloc (sizeof *pNouveau);
   if (pNouveau != NULL)
   {
      pNouveau->data = *p_data;
      pNouveau->psuiv = NULL;
 
      if (p_liste->pTete == NULL)
      {
/* Premier maillon = Premiere Ligne */
         p_liste->pTete = pNouveau;
      }
      else
      {
         p_liste->pQueue->psuiv = pNouveau;
      }
      p_liste->pQueue = pNouveau;
   }
}
 
static void LoadCombi (struct liste *p_liste)
{
   FILE *Out = fopen ("resultats_num_treated.txt", "r");
   if (Out != NULL)
   {
      struct data data;
 
      printf ("Creation liste Chainee...\n\n");
      {
         int i = 0;
         while (fscanf (Out, " %d %d %d %d\n ", &data.N1,
                        &data.N2, &data.N3,
                        &data.N4) == 4)
         {
            data.Indice = i;
            i++;
            add (p_liste, &data);
         }
      }
      fclose (Out);
   }
}
 
static void aff (struct liste *p_liste)
{
   struct node const *p = p_liste->pTete;
 
   while (p != NULL)
   {
      struct data const *p_data = &p->data;
      printf ("[%d] %d %d %d %d\n ", p_data->Indice, p_data->N1,
              p_data->N2, p_data->N3,
              p_data->N4);
      p = p->psuiv;
   }
}
 
int main (void)
{
   struct liste liste = { NULL, NULL };
   Filtrage ();
   LoadCombi (&liste);
   printf ("Done !!!\n\n");
   aff (&liste);
   return 0;
}

Avec ce fichier

Code :

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

1
2
3
 
5 200 20 10
7 58 29 15

ça donne :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Treating File Num...
 
Creation liste Chainee...
 
Done !!!
 
[0] 5 200 20 10
[1] 7 58 29 15
 
Process returned 0 (0x0)   execution time : 0.025 s
Press any key to continue.

**alexglvr** · 04/11/2008, 12h15

merci pour ton code...

Une question peut être stupide, mais pourquoi utiliser 'const' dans le passage des paramètres de la fonction add et dans la déclaration de *p de la fonction aff?

**Médinoc** · 04/11/2008, 12h17

Pourquoi pas?
La bonne procédure pour const, c'est "on le met sauf si on a une raison de ne pas le mettre".

Donne-moi donc tes raisons de ne pas le mettre...

PS: Personnellement, si je n'ai pas de problème de performance, je pousse la const-correctness plus loin, en interdisant l'accès direct au pointeur suivant d'un chaînon: Seules des fonctions const-correctes peuvent alors y accéder, et évitent qu'un utilisateur puisse, à partir d'un pointeur const sur un élément, obtenir un pointeur non-const sur l'élément suivant.

**Emmanuel Delahaye** · 04/11/2008, 12h43

Envoyé par alexglvr

pourquoi utiliser 'const' dans le passage des paramètres de la fonction add et dans la déclaration de *p de la fonction aff?

Parce que c'est (enfin, ça devrait être) la règle générale (programmation 'défensive' : on protège les données).

A priori, on a pas à modifier les données. On accorde le droit de modification (pas de 'const') que si il est parfaitement justifié. C'est alors indiqué dans le document de conception.

**alexglvr** · 04/11/2008, 19h07

j'ai rajouté deux fonctions...

Encore une fois, ça plante... Mais je comprends pas pourquoi. J'ai un warning aussi à la compilation...

Pourriez vous m'expliquer SVP?

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
#include <stdio.h>
#include <stdlib.h>
struct data
{
   int Indice;
   int Combi[5];
   int N[4];
 
};
 
struct node
{
   struct data data;
   struct node *psuiv;
 
};
 
struct liste
{
   struct node *pTete;
   struct node *pQueue;
};
 
static void Filtrage (void)
{
   FILE *In = fopen ("resultats_num.txt", "r");
   if (In != NULL)
   {
      FILE *Out = fopen ("resultats_num_treated.txt", "w");
      if (Out != NULL)
      {
         int car;
         printf ("Treating File Num...\n\n");
 
         while ((car = fgetc (In)) != EOF)
         {
 
            if ((car != '[') && (car != ']') && (car != ','))
            {
               putc (car, Out);
            }
 
         }
         fclose (Out);
      }
      fclose (In);
   }
}
 
static void add (struct liste *p_liste, struct data const *p_data)
{
   struct node *pNouveau = malloc (sizeof *pNouveau);
   if (pNouveau != NULL)
   {
      pNouveau->data = *p_data;
      pNouveau->psuiv = NULL;
 
      if (p_liste->pTete == NULL)
      {
/* Premier maillon = Premiere Ligne */
         p_liste->pTete = pNouveau;
      }
      else
      {
         p_liste->pQueue->psuiv = pNouveau;
      }
      p_liste->pQueue = pNouveau;
   }
}
 
static void LoadCombi (struct liste *p_liste)
{
   FILE *Out = fopen ("resultats_num_treated.txt", "r");
   if (Out != NULL)
   {
      struct data data;
 
      printf ("Creation liste Chainee...\n\n");
      {
         int i = 0;
         while (fscanf (Out, " %d %d %d %d %d\n ", &data.Combi[0],
                        &data.Combi[1], &data.Combi[2],
                        &data.Combi[3], &data.Combi[4]) == 5)
         {
            data.Indice = i;
            int j = 0;
            for (j=0; j<4; j++) {
            data.N[j]=0;
            }
            i++;
            add (p_liste, &data);
         }
      }
      fclose (Out);
   }
}
 
/*static void aff (struct liste *p_liste)
{
   struct node const *p = p_liste->pTete;
 
   while (p != NULL)
   {
      struct data const *p_data = &p->data;
      printf ("[%d] %d %d %d %d %d\n ", p_data->Indice, p_data->Combi[0],
              p_data->Combi[1], p_data->Combi[2], p_data->Combi[3],
              p_data->Combi[4]);
      p = p->psuiv;
   }
}*/
 
 
static void scan_N (struct liste *p_liste, struct data *p_data)
{
    struct node const *p = p_liste->pTete;
    int i=0;
 
    while (p != NULL) 
    {
      int Compteur=0;
      for (i=0; i<5; i++) 
      {
      if (p->data.Combi[i]==p_data->Combi[i]) {Compteur++;} 
      }
      printf("%d\n", Compteur);
    if ( ( Compteur > 0 ) && ( Compteur < 5 ) ) 
      {p_data->N[Compteur]=p_data->N[Compteur]+1;
      }
      p = p->psuiv; 
    } 
}
 
static void analyse (struct liste *p_liste)
{
    struct node const *p = p_liste->pTete;
 
       while (p != NULL)
         {
          struct data *p_data = &p->data;
          scan_N(p_liste, p_data);
          p = p->psuiv;
          }
}
 
int main (void)
{
   struct liste liste = { NULL, NULL };
   Filtrage ();
   LoadCombi (&liste);
   printf ("Done !!!\n\n");
   /*aff (&liste);*/
   analyse (&liste);
   return 0;
}

**Emmanuel Delahaye** · 04/11/2008, 19h33

Envoyé par alexglvr

j'ai rajouté deux fonctions...

Encore une fois, ça plante... Mais je comprends pas pourquoi. J'ai un warning aussi à la compilation...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c: In function `LoadCombi':
C:\dev\hello\main.c:86: warning: ISO C90 forbids mixed declarations and code
C:\dev\hello\main.c: In function `analyse':
C:\dev\hello\main.c:139: warning: initialization discards qualifiers from pointer target type
Output size is 22.20 KB
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 2 warnings

Il faut définir j en début de bloc

p est const. Il faut donc que p_data soit aussi const. Evidemment, ça se répercute sur scan_N()... et l'erreur finit par se révéler :

Code :

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

1
2
3
4
5
6
7
8
9
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c: In function `scan_N':
C:\dev\hello\main.c:132: error: assignment of read-only location
Process terminated with status 1 (0 minutes, 0 seconds)
1 errors, 0 warnings

Si tu cherches à modifier ce qui est en lecture seule, c'est qu'il y a un bug quelque part.

- Soit tu ne devrais modifier cet objet
- Soit l'objet doit être modifiable et dans ce cas, il faut retirer les 'const' et agir en conséquences, notamment sur le type de la variable de départ.

**alexglvr** · 05/11/2008, 09h23

j'ai modifié les const... plus de pb de compilation.

Par contre, le programme plante toujours. Vu que je n'arrive pas a faire fonctionner le debugger sur DEVC++, avec des printf, j'ai réussi à trouver ou était l'erreur... Mais je vois pas la solution...

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
#include <stdio.h>
#include <stdlib.h>
struct data
{
   int Indice;
   int Combi[5];
   int N[4];
 
};
 
struct node
{
   struct data data;
   struct node *psuiv;
 
};
 
struct liste
{
   struct node *pTete;
   struct node *pQueue;
};
 
static void Filtrage (void)
{
   FILE *In = fopen ("resultats_num.txt", "r");
   if (In != NULL)
   {
      FILE *Out = fopen ("resultats_num_treated.txt", "w");
      if (Out != NULL)
      {
         int car;
         printf ("Treating File Num...\n\n");
 
         while ((car = fgetc (In)) != EOF)
         {
 
            if ((car != '[') && (car != ']') && (car != ','))
            {
               putc (car, Out);
            }
 
         }
         fclose (Out);
      }
      fclose (In);
   }
}
 
static void add (struct liste *p_liste, struct data const *p_data)
{
   struct node *pNouveau = malloc (sizeof *pNouveau);
   if (pNouveau != NULL)
   {
      pNouveau->data = *p_data;
      pNouveau->psuiv = NULL;
 
      if (p_liste->pTete == NULL)
      {
/* Premier maillon = Premiere Ligne */
         p_liste->pTete = pNouveau;
      }
      else
      {
         p_liste->pQueue->psuiv = pNouveau;
      }
      p_liste->pQueue = pNouveau;
   }
}
 
static void LoadCombi (struct liste *p_liste)
{
   FILE *Out = fopen ("resultats_num_treated.txt", "r");
   if (Out != NULL)
   {
      struct data data;
 
      printf ("Creation liste Chainee...\n\n");
      {
         int i = 0;
         int j = 0;
         while (fscanf (Out, " %d %d %d %d %d\n ", &data.Combi[0],
                        &data.Combi[1], &data.Combi[2],
                        &data.Combi[3], &data.Combi[4]) == 5)
         {
            data.Indice = i;
            for (j=0; j<4; j++) {
            data.N[j]=0;
            }
            i++;
            add (p_liste, &data);
         }
      }
      fclose (Out);
   }
}
 
/*static void aff (struct liste *p_liste)
{
   struct node const *p = p_liste->pTete;
 
   while (p != NULL)
   {
      struct data const *p_data = &p->data;
      printf ("[%d] %d %d %d %d %d\n ", p_data->Indice, p_data->Combi[0],
              p_data->Combi[1], p_data->Combi[2], p_data->Combi[3],
              p_data->Combi[4]);
      p = p->psuiv;
   }
}*/
 
 
static void scan_N (struct liste *p_liste, struct data *p_data)
{
    struct node *p = p_liste->pTete;
    int i=0;
    while (p != NULL) 
    {
      int Compteur=0;
      for (i=0; i<5; i++) 
      {
      if (p->data.Combi[i]==p_data->Combi[i]) {Compteur++;} 
      }
      printf("%d\n", Compteur);
    if ( ( Compteur > 0 ) && ( Compteur < 5 ) ) 
      {p_data->N[Compteur]=p_data->N[Compteur]+1;
      }
      p = p->psuiv; 
    } 
}
 
static void analyse (struct liste *p_liste)
{
    struct node *p = p_liste->pTete;
 
       while (p != NULL)
         {
          struct data *p_data = &p->data;
          scan_N(p_liste, p_data);
          p = p->psuiv;
          }
}
 
int main (void)
{
   struct liste liste = { NULL, NULL };
   Filtrage ();
   LoadCombi (&liste);
   printf ("Done !!!\n\n");
   /*aff (&liste);*/
   analyse (&liste);
   return 0;
}

Le bug se situe apparemment sur p = p->psuiv de analyse...

**Emmanuel Delahaye** · 05/11/2008, 09h58

Envoyé par alexglvr

j'ai modifié les const... plus de pb de compilation.

Par contre, le programme plante toujours. Vu que je n'arrive pas a faire fonctionner le debugger sur DEVC++, avec des printf, j'ai réussi à trouver ou était l'erreur... Mais je vois pas la solution...

avec ce fichier :

Code :

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

1
2
3
 
[5, 200, 20, 10]
[7, 58, 29, 15]

et ton code, j'obtiens :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
Treating File Num...
 
Creation liste Chainee...
 
Done !!!
 
5
 
Process returned 0 (0x0)   execution time : 0.016 s
Press any key to continue.

et en réactivant l'affichage de la liste :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Treating File Num...
 
Creation liste Chainee...
 
Done !!!
 
[0] 5 200 20 10 7
 5
 
Process returned 0 (0x0)   execution time : 0.032 s
Press any key to continue.

fichier texte et Liste Chainée

C

Discussions similaires

Partager

Partager