Discussion :

[seifdev]

voila j'ai un probléme avec cette fonction qui crée a partire d'un fichier une liste chainée

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
void charge(struct liste **ob,char adr[])
{ struct liste *parc;
  struct liste *de;
 
  FILE *rd;
  rd=fopen(adr,"r");
  de=(struct liste *)malloc(sizeof(struct liste));
  parc=(struct liste *)malloc(sizeof(struct liste));
parc=de;
 /* chargement du fichier dans une liste chain‚ */
 /*----------------------------------------------*/
 fscanf(rd,"%s\n",parc->name);
 
fscanf(rd,"%s\n",parc->prenom);
printf("\n--%s",parc->name);
 getch();
fscanf(rd,"%s\n",parc->bdate);
fscanf(rd,"%s\n",parc->code);
fscanf(rd,"%c\n",parc->sexe);
fscanf(rd,"%d\n",parc->nbe);
fscanf(rd,"%s\n",parc->enfcode);
while(!feof(rd))
{
parc->suiv=(struct liste *)malloc(sizeof(struct liste));
parc=parc->suiv;
 fscanf(rd,"%s\n",parc->name);
 
 
  printf("\n--%s",parc->name);
 getch();
fscanf(rd,"%s\n",parc->prenom);
fscanf(rd,"%s\n",parc->bdate);
fscanf(rd,"%s\n",parc->code);
fscanf(rd,"%c\n",parc->sexe);
fscanf(rd,"%d\n",parc->nbe);
fscanf(rd,"%s\n",parc->enfcode);
//parc=parc->suiv;
}
parc->suiv=NULL;
*ob=de;
}

[gl]

voila j'ai un probléme avec cette fonction qui crée a partire d'un fichier une liste chainée

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
void charge(struct liste **ob,char adr[])
{ struct liste *parc;
  struct liste *de;
 
  FILE *rd;
  rd=fopen(adr,"r");
  de=(struct liste *)malloc(sizeof(struct liste));
  parc=(struct liste *)malloc(sizeof(struct liste));
parc=de;
 /* chargement du fichier dans une liste chain‚ */
 /*----------------------------------------------*/
 fscanf(rd,"%s\n",parc->name);
 
fscanf(rd,"%s\n",parc->prenom);
printf("\n--%s",parc->name);
 getch();
fscanf(rd,"%s\n",parc->bdate);
fscanf(rd,"%s\n",parc->code);
fscanf(rd,"%c\n",parc->sexe);
fscanf(rd,"%d\n",parc->nbe);
fscanf(rd,"%s\n",parc->enfcode);
while(!feof(rd))
{
parc->suiv=(struct liste *)malloc(sizeof(struct liste));
parc=parc->suiv;
 fscanf(rd,"%s\n",parc->name);
 
 
  printf("\n--%s",parc->name);
 getch();
fscanf(rd,"%s\n",parc->prenom);
fscanf(rd,"%s\n",parc->bdate);
fscanf(rd,"%s\n",parc->code);
fscanf(rd,"%c\n",parc->sexe);
fscanf(rd,"%d\n",parc->nbe);
fscanf(rd,"%s\n",parc->enfcode);
//parc=parc->suiv;
}
parc->suiv=NULL;
*ob=de;
}

Quel probleme rencontres-tu exactement ?

Sinon quelques remarques :
* Si tu veux que les gens puissent t'aider, l faut poster un code compilable, ici il manque par exemple la definition de struct liste.
*Il n'y a aucun test sur le retour des fonctions qui peuvent echouer (fopen, malloc, fscanf) et donc aucune gestion d'erreur.
* fscanf attends des adresses, ce n'est donc pas fscanf(rd,"%d\n",parc->nbe); mais fscanf(rd,"%d\n",&parc->nbe);
* A moins de controler le format du fichier et d'etre a 100% certains du contenu, il est dangeureux d'utiliser fscanf(..., "%s",...) qui n'effectue aucun controle sur la taille de la chaine. A remplacer par fgets.
* Tu alloue de la memoire pour parc, puis tu modifies l'adresse contenue dans parc en y recopiant l'adresse contenue dans de. La zone memoire allouee est donc perdue ==> fuite de memoire.
* feof ne fonctionne pas comme ca. Si une erreur de lecture est rencontree, la fonction feof() permet de determiner si il s'agit de la fin de fichier ou non.
* getch() n'est pas une fonction standard du C. Pourquoi ne pas utliser fgetc() ?

[seifdev]

mon type de probléme la liste chainée n'est pas correcte les lignes ne prend pas la place qui devrais prendre dans chaque liste
disons si un fichier contient
nom
prenom
date
la liste ne reçoit pas ce qu'elle devrais recevoir

[seifdev]

ça y est j'ai trouver l'erreur ma fonction ne contient pas d'erreur l'erreur vient du fichier que j'utilise

[souviron34]

je suis quasi-certain que dans ta structure tu as déclaré tes variables (nom par exemple) comme un pointeur.. .

Donc il faut ALLOUER de la place pour pouvoir stocker les valeurs dedans.

D'autre part, très nettement préférer fgets + sscanf que scanf.

Enfin, poste ta structure si tu veux qu'on y pige quelque chose.