structure recursive et fichier

Version imprimable

Bonjour,

J'enregistre dans un fichier une liste chainee d'elements.
J'écris dans le fichier élément par élément, cela se passe bien.

Voici de quoi se compose ma liste :
Code:

1 2 3 4 5 6 7 8 struct car { personne pers; //personne est une structure struct car *suivant; }; typedef struct car carnet; typedef carnet *liste;
Le code pour charger une liste a partir de mon fichier :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 liste car; liste temp, suivant; FILE *pf; pf=fopen("carnet.dat", "r"); if(pf != NULL) { car = (liste) malloc(sizeof(carnet)); fread(&car->pers, sizeof(carnet), 1, pf); temp=car; while(!feof(pf)) { suivant = (liste) malloc(sizeof(carnet)); fread(&suivant->pers, sizeof(carnet), 1, pf); temp->suivant = suivant; temp = suivant; } } fclose(pf);
Voila mon probleme : Ca marche bien (bon ce n'est pas ca mon probleme :roll: ) sauf que j'ai toujours un maillon de trop, par exemple si j'ai 5 personnes dans ma liste, j'ai bien mes 5 personnes avec le chainage qui va bien, mais j'ai toujours une personne en plus en fin de liste qui a des valeurs vides (valeurs de la structure personne).

Tu initialises pas ton pointeur suivant correctement...

Déjà,

Code:

fread(&suivant->pers, sizeof(carnet), 1, pf);

devrait être

Code:

fread(&suivant->pers, sizeof(suivant->pers), 1, pf);

Le problème, c'est que tu ne mets pas systèmatiquement le pointeur à jour lorsque tu le lis ton fichier, une meilleure solution est de faire:

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
42
43
44
45
46
47
48
49
 
liste lireficher()
{
liste car=NULL;
liste temp, suivant;
FILE *pf;
 
  pf=fopen("carnet.dat", "r");
 
  if(pf != NULL)
    {
      //On lit le premier élément, on suppose que le fichier a au moins un élément
      car = (liste) malloc(sizeof(carnet));
      //On test toujours après un malloc
      if(car == NULL)
          return NULL;
 
      //C'est toujours mieux de tester le retour des fonctions...
      if(fread(&car->pers, sizeof(car->pers), 1, pf) != sizeof(car->pers))
            {
            free(car);
            return NULL;
            }
 
      car->suivant = NULL;
      temp=car;
 
      while(!feof(pf))
     {
      //A titre d'exercice, je laisse le test du malloc et le retour du fread... il faudra
      //libérer la liste crée s'il y a un problème
      suivant = (liste) malloc(sizeof(carnet));
 
     //on récupére l'info sur la personne
     fread(&suivant->pers, sizeof(suivant->pers), 1, pf);
 
     //on met le pointeur à null
     suivant->suivant  = NULL;
 
     //on fait le lien
     temp->suivant = suivant;
     temp = suivant;
     }
    }
 
  fclose(pf);
 
return car;
}

04/01/2006, 11h43
KrusK

Merci pour ton aide Jc

J'ai cependant 2 problemes :
-Je rentre dans le test

Code:

if(fread(&car->pers, sizeof(car->pers), 1, pf) != sizeof(carnet))

-Je n'arrive a voir que les valeurs de la premiere personne
-Et j'ai toujours une personne en plus :roll:
04/01/2006, 11h46
fearyourself

Code corrigé, le test était faux... Désolé...

Hmm, bizarre, montre le code de création de la liste et le code qui créé le fichier...

Jc

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 void Enregistrer(liste car) { FILE *pf; liste temp = car; pf=fopen("carnet.dat", "w"); if(pf != NULL) { while(temp != NULL) { fwrite(&temp->pers, sizeof(temp->pers), 1, pf); temp = temp->suivant; } } fclose(pf); }
Pour initialiser, je fais juste des scanf pour le moment pour tester, par exemple scanf("%s", temp->pers.valeur1);

Après avoir retiré le test, rebranché mon cerveau, le gros du changement venait de

Code:

sizeof(carnet)

à transformer en

Code:

sizeof(temp->pers)

et d'ajouter

Code:

suivant->suivant = NULL;