ce programme , pourriez vous le vérifier ?

**dnmfaw** · 08/09/2007, 18h10

salut ,

j'ai écrit un prog en C , la compilation n'affiche aucune erreur , par contre l'exécution donne Erreur de segmentation .. si qq pourra le vérifier pour savoir où est la faute ... je vous remercie d'avance :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "structure.h"
 
 
void Lecturefichier ( FILE *f , Module t[]);
void Ecriturefichier ( Module t[]);
 
 
 
 
int main () 
{ 
  Module exemple[LenM]; FILE *f;
  f = fopen ("donnes.csv ", "r");
  Lecturefichier (f,exemple);
  fclose (f);
  Ecriturefichier(exemple);
  return 1;
}
 
 
 
void Lecturefichier ( FILE *f , Module t[])
{
  char *pointeur;
  char *buffer;
  char ch[80];
  char chaine[1000];
  char *separateur = {";"};
  int i = 1;
  int valeur ; 
  int j = 1;
  t[1].Mod ="AP11";
 
  while ((fgets (chaine,1000,f)) != NULL )
    {
      fgets (ch,80,f);
      buffer = strdup (ch);
      pointeur = strtok(buffer,separateur);
 
      if ( (strcmp (pointeur,t[i].Mod ))== 0 )
	{
	  t[i].Mod = pointeur ; 
	  pointeur = strtok (NULL , separateur);
	  t[i].Sem = pointeur ; 
	  pointeur = strtok (NULL , separateur);
	  t[i].Prom = pointeur ; 
	  pointeur = strtok (NULL , separateur);
	  t[i].Cord = pointeur ;
	  pointeur = strtok (NULL , separateur);
	  t[i].act[j].Nom = pointeur ; 
	  pointeur = strtok (NULL , separateur);
	  sscanf(pointeur,"%d",&valeur);
	  t[i].act[j].Ng = valeur ; 
	  pointeur = strtok (NULL , separateur);
	  sscanf (pointeur ,"%d", &valeur);
	  t[i].act[j].Neg = valeur;
	  pointeur = strtok (NULL , separateur);
	  sscanf (pointeur,"%d", &valeur);
	  t[i].act[j].Nhg = valeur;
	  j++;
	}
      if ((strcmp (pointeur, t[i].Mod)) != 0 )
	{
	  i++;
          t[i].Mod = pointeur ; 
	  pointeur = strtok (NULL , separateur);
	  t[i].Sem = pointeur ; 
	  pointeur = strtok (NULL , separateur);
	  t[i].Prom = pointeur ; 
	  pointeur = strtok (NULL , separateur);
	  t[i].Cord = pointeur ;
	  pointeur = strtok (NULL , separateur);
	  t[i].act[j].Nom = pointeur ; 
	  pointeur = strtok (NULL , separateur);
	  sscanf(pointeur,"%d",&valeur);
	  t[i].act[j].Ng = valeur ; 
	  pointeur = strtok (NULL , separateur);
	  sscanf (pointeur ,"%d", &valeur);
	  t[i].act[j].Neg = valeur;
	  pointeur = strtok (NULL , separateur);
	  sscanf (pointeur,"%d", &valeur);
	  t[i].act[j].Nhg = valeur;
	}
    }
}
 
 
 
 
 
void Ecriturefichier ( Module t[] )
{
  char nomfich[21];
  int i,j;
  FILE *sortie;
 
  printf ( "nom du fichier à créer : ");
  scanf ( "%20s" , nomfich);
  sortie = fopen (nomfich , "w" );
 
  for ( i=1;i<LenM;i++)
    {
      for ( j=1;j<LenA;j++)
	{
	  fputs (t[i].Mod , sortie ); fputc (';',sortie);
	  fputs (t[i].Sem , sortie ); fputc (';',sortie);
	  fputs (t[i].Prom, sortie ); fputc (';',sortie);
	  fputs (t[i].Cord, sortie ); fputc (';',sortie);
	  fputs (t[i].act[j].Nom, sortie ); fputc (';',sortie);
	   putw (t[i].act[j].Ng , sortie ); fputc (';',sortie);
	   putw (t[i].act[j].Neg, sortie ); fputc (';',sortie);
	   putw (t[i].act[j].Nhg, sortie ); fputc (';',sortie);
	}
    }
  fclose (sortie);
}

pour structure.h voilà 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
 
# define LenA 10
 
# define LenM 50
 
FILE *f;
 
typedef struct { char *Nom;
                 int Ng, Neg, Nhg; } Activite ;
 
typedef struct { char *Mod, *Sem, *Prom, *Cord;
  Activite act[LenA]; } Module ; 
 
 
void Lecturefichier ( FILE *f , Module t[]);
 
void Ecrirefichier ( Module t[]);

donnees.csv est un fichier csv de cette forme :
"qsds";"sfsdf";"hjj";"hsdfh";1;5;5;
"gdfhjg";"sd";"dkfns";"dksk";2;5;5
...ainsi de suite

merci

**aoyou** · 08/09/2007, 20h02

Une première chose.
Tu dis que ton fichier s'appelle donnees.csv et dans le code il s'appelle donnes.csv. f est alors nul et au premier fgets ça plante. Vérifie le retour de fopen.

**dnmfaw** · 08/09/2007, 20h07

je vais donc corriger .. merci

**aoyou** · 08/09/2007, 20h08

Deuxième chose.
Est-il normal que la première ligne du fichier se termine par un point-virgule et pas la deuxième ?

**Skyounet** · 08/09/2007, 20h14

Toujours vérifier le retour de fopen, si ce dernier renvoi NULL alors tu as une erreur c'est pas plus compliqué. Ensuite un petit perror pour voir l'erreur et tu seras de suite informé.

Et puis la variable globale tu pourrais t'en passer.

**dnmfaw** · 08/09/2007, 20h16

Non , au fait , c'est une erreur de frappe , pas de ; à la fin des lignes .
y a-t-il d'autres remarques ? merci d'avance

**aoyou** · 08/09/2007, 20h18

Ensuite, tu vas avoir des problèmes avec le remplissage de tes structures.

Code :

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

1
2
 
t[i].Sem = pointeur ;

Il faut que tu alloues l'espace mémoire pour t[i].Sem et que tu y copies la chaîne de caractères pointée par pointeur.

**dnmfaw** · 08/09/2007, 20h20

allouer l'espace mémoire et copier la chaine ? peux tu m'expliquer comment ? cad le code ? merci

**Skyounet** · 08/09/2007, 20h21

Envoyé par aoyou

Ensuite, tu vas avoir des problèmes avec le remplissage de tes structures.

Code :

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

1
2
 
t[i].Sem = pointeur ;

Il faut que tu alloues l'espace mémoire pour t[i].Sem et que tu y copies la chaîne de caractères pointée par pointeur.

Pourquoi ? t[i].Sem pointera vers pointeur.

Apres je sais pas si il veut faire exactement ça.

Si tu veux que le contenu soit copié alors utilise strdup.
Et pas seulement pour Sem, mais pour tous les autres pointeur aussi.

**aoyou** · 08/09/2007, 20h22

Enfin, tu as un strtok de trop.

Apres je sais pas si il veut faire exactement ça.

A priori, je pense qu'il veut stocker toutes ses données dans un tableau, avant de les utiliser par autre part.

**valefor** · 08/09/2007, 20h26

Tu devrais profiter de ce zoli segfault pour t'initier à l'utilisation d'un débugger. C'est très simple :

1) tu compiles ton programme avec l'option '-g'
2) tu lances la commande 'gdb ./monprogramme'
3) tu tapes run
4) tu tapes where, list, help, et tout ce que tu veux pour approfondir.

Voila, j'imagine que ta curiosité te pousseras à en savoir plus, mais pour un début y'a là de quoi te dépaner tout seul (c'est grisant comme outils).

**Vincent Rogier** · 08/09/2007, 20h29

Toujours s'assurer de la validité des arguments passés à une fonction de la librairie standard C ainsi que des valeurs de retour...

Sinon, les probalilités de bug sont de 100%...

**dnmfaw** · 08/09/2007, 21h29

oui , en fait c'est ça , je veux stocker les infos du fichier dans un tableau avant de les utiliser une deuxième fois .
donc pour faire , j'utilise strdup comme ça ?

Code :

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

1
2
3
 
pointeur = strtok ( NULL , separateur);
t[i].Sem = strdup (pointeur);

cela permettera d'avoir le contenu ?
c'est où le strtok de trop ?
merci

**aoyou** · 08/09/2007, 22h38

Oui, tu utilises strdup pour toutes les chaînes de caractères que tu veux stocker.

Pour le strtok, en fait je ne suis pas ce que tu veux faire. Tu as 8 strtok pour stocker une ligne qui ne comporte que 7 éléments.

Tu as deux fgets à la suite, c'est normal ? Tel que c'est écrit, tu sautes une ligne sur deux.

Code :

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

1
2
3
4
5
6
7
8
9
 
      if ( (strcmp (pointeur,t[i].Mod ))== 0 )
	{
          ...
	}
      if ((strcmp (pointeur, t[i].Mod)) != 0 )
	{
	  ...
	}

s'écrit

Code :

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

1
2
3
4
5
6
7
8
9
 
      if ( (strcmp (pointeur,t[i].Mod ))== 0 )
	{
          ...
	}
      else
	{
	  ...
	}

**Emmanuel Delahaye** · 08/09/2007, 22h52

Envoyé par dnmfaw

Non , au fait , c'est une erreur de frappe , pas de ; à la fin des lignes .
y a-t-il d'autres remarques ? merci d'avance

Oui. Apprends à te relire, car on est pas là pour corriger tes fautes de frappes...

**Emmanuel Delahaye** · 08/09/2007, 22h54

Envoyé par dnmfaw

oui , en fait c'est ça , je veux stocker les infos du fichier dans un tableau avant de les utiliser une deuxième fois .
donc pour faire , j'utilise strdup comme ça ?

Code :

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

1
2
3
 
pointeur = strtok ( NULL , separateur);
t[i].Sem = strdup (pointeur);

Et n'oublie pas que strdup() a utilisé malloc(), donc le bloc alloué (la chaine) doit être libéré après usage avec free().

Rappel : strdup() n'est pas une fonction standard du C, mais elle est POSIX.1, donc très portable.

**dnmfaw** · 08/09/2007, 23h36

salut ,

alors j'ai fait comme vous avez induiqué mais l'exécution donne tjs le message d'erreur : erreur de segmentation .
Vous pensez que c'est du à quoi ?

**aoyou** · 09/09/2007, 00h38

Oui enfin là on ne peut pas être devin.

Soit tu t'inities au debugger comme l'a dit valefor. On ne sera pas toujours là pour t'aider.

Soit tu repostes le code mais essaies d'abord gdb. Il te dira quelle ligne est en cause.

Je te conseille de lire le topic suivant http://www.developpez.net/forums/showthread.php?t=443.

**dnmfaw** · 09/09/2007, 11h31

Salut ,

tout d'abord je m'excuse du message peu courtois que j'ai posté hier , mais j'étais un peu énervé à cause du programme.

pour le programme ,
j'ai corrigé le code suivant les remarques données , en fait l'erreur s'est produite au niveau de la lecture du fichier : donnees.csv ne s'ouvre pas ! qq peut voir la raison ?! comment corriger cette erreur

merci