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.

[youssef_the_king]

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

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

[youssef_the_king]

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