chargement d'une liste chainée à partir d'un fichier binaire qui contient des structures

Version imprimable

Bonjour,
j'ai un petit problème au niveau de la récupération des structures (Etudiant) enregistrées dans un fichier binaire ,pour les charger ensuite dans une liste chainée..

voici mon bout de code(en bas) si quelqu'un a une idée pourquoi ça ne marche pas. je travaille avec le langage C sous Dev C++, à l'exécution j'ai le le debogueur juste-à-temps de visual studio.

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
 
 
//Fonction de récupération et de chargement des données a partir du fichier
void chargerListe()
{
 
         flecture=fopen("etudiants.bin","rb" );    
 
         while(!feof(flecture))
         {
              Etudiant *l=(Etudiant *) malloc(sizeof(Etudiant)) ;
              fread(l, sizeof(*l), 1, flecture);
              printf("\n identifiant = %d , nom : %s , moyenne : %f , appreciation : %s ",l->identifiant,l->nom,l->moyenne,l->appreciation);
 
 
         };
 
         fclose(flecture);
}
 
//Fonction d'enregistrement dans le fichier
void enregistrerListe(Etudiant *et)
{
    Etudiant *e;
    e=et;
    fecriture=fopen("etudiants.bin","wb" );
    while(e != NULL)
    {
     fwrite(&e, sizeof(e), 1, fecriture);
     e=e->suivant;
 
    }
    fclose(fecriture);
}

Merci d'avance!

23/06/2011, 02h02
orfix
L'erreur liée directement à ton problème se situe ici:

Code:

fwrite(&e, sizeof(e), 1, fecriture);

fwrite attend en premier argument un pointeur sur l'objet à écrire, e est déjà un pointeur sur Etudiant en rajoutant le & tu lui envois un pointeur sur un pointeur sur Etudiant (Etudiant**):

Code:

fwrite(e, sizeof(*e), 1, fecriture);

Sinon j'ai plusieurs remarques:
- Évite d'utiliser des variables globales, sauf si c'est justifié.
- Une ouverture de fichier peut échouer, c'est pourquoi il faut tester le retour de fopen().
- feof() ne fait pas ce que tu penses, pour plus de détail: A quoi sert la "fonction" feof ?.
- Inutile de caster le retour de malloc().
- d'autres remarques concernant la portabilité et la sérialisation mais je ne veux pas t'embrouiller pour l'instant.
Bon courage.

Bonjour,
Merci pour votre réponse.
J'ai pris en compte vos remarques et j'ai modifié mon code:

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
 
//Enregistrement dans le fichier
void enregistrerListe(Etudiant *et)
{
    Etudiant *e;
    e=et;
    fecriture=fopen("etudiants.bin","wb" );
    if(fecriture == NULL) printf("L'Ouverture du fichier a echoué...!");
    while(e != NULL)
    {
     fwrite(&e, sizeof(Etudiant), 1, fecriture);
     e=e->suivant;
 
    }
    fclose(fecriture);
}
 
//Chargerliste
void chargerListe()
{
 
         Etudiant *l=malloc(sizeof(Etudiant)) ;
 
         flecture=fopen("etudiants.bin","rb" );
         if(flecture == NULL) printf("L'Ouverture du fichier a echoué...!");
         while(fread(l, sizeof(Etudiant), 1, flecture)!= 0)
         {
 
                    printf("\n identifiant = %d , moyenne : %f ",l->identifiant,l->moyenne);
 
 
         };
 
         fclose(flecture);
}

ça me charge mes enregistrements qui se trouvent au niveau du fichier mais avec des valeurs bizarres:
identifiant = 4793560 , moyenne : 0.000000
identifiant = 4793608 , moyenne : 0.000000
identifiant = 4793656 , moyenne : 0.000000

Merci encore une fois, et je reste toujours ouverte aux autres remarques!:)

24/06/2011, 15h09
matafan

Comme indiqué plus haut c'est fwrite(e), pas fwrite(&e), puisque e est déjà un Etudiant *.
24/06/2011, 15h12
miss_poopoucy

Oui c'est bien ça :ccool:, merci beaucoup:mrgreen:
24/06/2011, 15h22
orfix

Code:

if(fecriture == NULL) printf("L'Ouverture du fichier a echoué...!");

Si l'ouverture du fichier échoue il ne faut pas tenter une lecture/écriture dessus, je te conseille donc d'englober le reste du code après le if dans un else.
24/06/2011, 15h33
miss_poopoucy

Oui tu as raison, c'est fait;).

rep

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
 
 //Enregistrement dans le fichier
char nom_fich[30]="etudiants.bin"
void enregistrerListe(Etudiant *et)
{
    Etudiant *e;
    e=et;
    fecriture=fopen(nom_fich,"wb" );
    if(fecriture == NULL) printf("L'Ouverture du fichier a echoué...!"); exit(-1);
    while(e != NULL)
    {
     fwrite(e, sizeof(Etudiant), 1, fecriture);
     e=e->suivant;
    }
    fclose(fecriture);
}
 
//Chargerliste
Etudiant * chargerListe()
{
Etudiant *ma_list=NULL;
Etudiant *l ;
flecture=fopen(nom_fich,"rb" );
if(flecture == NULL) printf("L'Ouverture du fichier a echoué...!");
do{
         l=malloc(sizeof(Etudiant)) ;
         fread(l,sizeof(Etudiant), 1, flecture);
l-suivant=ma_list;
ma_list=l;
   printf("\n identifiant = %d , moyenne : %f ",l->identifiant,l->moyenne);
         }while(!feof(flecture));
 
         fclose(flecture);
return ma_list;}

j ai repris le code puis j ai reboucle dans la lecture de fichier
il y a un seul probleme c'est que on enregistre meme me champ suivant de notre structure et ce qui me pose probleme personnellement

28/06/2011, 08h45
sam1507

Pour éviter d'enregistrer le champs suivant dans ton fichier, tu peux modifier ta liste chainée pour stocker les données utiles dans une autre structure et sauvegarder uniquement cette structure.
28/06/2011, 17h19
youssef_the_king

re

oui ça sera un solution tres utile sa changera beaucoup de chose dans l organigramme du programme et aussi le propritaire juge que le probleme et resolut ce qu on espere bien