Discussion :

[KrusK]

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 :

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

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 :

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
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 ) 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).

[fearyourself]

Tu initialises pas ton pointeur suivant correctement...

Déjà,

Code :

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

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

devrait être

Code :

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

 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 :

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

Jc

[KrusK]

Merci pour ton aide Jc

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

Code :

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

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

[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

[KrusK]

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
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 :

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

sizeof(carnet)

à transformer en

Code :

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

sizeof(temp->pers)

et d'ajouter

Code :

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

suivant->suivant = NULL;