Discussion :

[miss_poopoucy]

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 :

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
 
 
//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!

[orfix]

L'erreur liée directement à ton problème se situe ici:

Code :

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

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 :

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

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.

[miss_poopoucy]

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

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
 
//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!

[matafan]

Comme indiqué plus haut c'est fwrite(e), pas fwrite(&e), puisque e est déjà un Etudiant *.

[miss_poopoucy]

Oui c'est bien ça , merci beaucoup

[orfix]

Code :

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

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.

[miss_poopoucy]

Oui tu as raison, c'est fait.