fichier et liste chainée

Version imprimable

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

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

Citation:
Envoyé par seifdev
voila j'ai un probléme avec cette fonction qui crée a partire d'un fichier une liste chainée
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 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() ?

22/04/2007, 23h34
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
23/04/2007, 02h07
seifdev

ça y est j'ai trouver l'erreur ma fonction ne contient pas d'erreur l'erreur vient du fichier que j'utilise
23/04/2007, 02h08
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.