Probleme avec fread ou fwrite.. binaire pas lisible ?

**titimoi** · 16/01/2009, 01h19

bonjour,
Pour réviser un peu j'ai voulu me faire un programme très simple utilisant fread et fwrite... pour prendre une structure d'un fichier, la mettre en binaire dans un autre fichier, prendre ce binaire, le mettre en texte dans un fichier et la lire, voici 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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
 
 
#include <stdio.h>
#include <stdlib.h>
#define MAXN 10
#define MAXP 10
 
struct etud_s // la structure d'un étudiant
{
	char nom[MAXN];
	char prenom[MAXP];
};
 
struct etud_s tabetud;  //la structure qui me servira pour lire et ecrire
 
void edition ( struct etud_s tab, char filename[] ) // pour écrire dans tabetud
{
	FILE *datei;
	datei=fopen ( filename,"w" );
 
	printf ( "\ndonner le nom\n" ); // on met tout simplement ce que l'on veut dans tabetud
	scanf ( "%s",&tab.nom );
	fprintf ( datei,"%s\t",tab.nom );
	printf ( "donner le prenom\n" );
	scanf ( "%s",&tab.prenom );
	fprintf ( datei,"%s",tab.prenom ); // sauvegarde dans un fichier.txt OK
	fclose ( datei );
}
 
void speicher ( char output[],char input[] ) // enregistrement en binaire
{
 
	FILE *foutput;
	FILE *finput;
	foutput=fopen ( output,"wb" ); 
	finput=fopen ( input,"rb" ); // on prends notre fichier txt
 
	fwrite ( finput, sizeof ( struct etud_s ),1,foutput ); //on crée le binaire avec simplement ma structure
}
 
void lecture ( char input[], char output[],struct etud_s etud ) // pour lire à partir du binaire.. c'est LA que ça foire
{
	FILE *finput;
	FILE *foutput;
	finput=fopen ( input,"rb" ); // j'ouvre mon binaire et je vais créer un .txt
	foutput=fopen ( output,"wb" );
	fread ( finput, sizeof ( struct etud_s ),1,foutput ); //je lis donc depuis mon binaire une structure, et je la met dans un .txt
	fscanf ( foutput, "%s",&etud.nom ); // pour afficher ensuite le .txt
	fscanf ( foutput,"%s",&etud.prenom );
	printf ( "\n le nom est : %s",etud.nom );
	printf ( "\n leprenom est : %s",etud.prenom );
}
 
int main()
{
	edition ( tabetud,"edition" ); // edition contient bien ce que j'ai mis dans tabetud
	speicher ( "speicher","edition" ); //speicher contient "quelque chose" du binaire donc !
	lecture ( "speicher","lecture",tabetud ); //lecture ne contient rien.. la lecture ne se fait donc pas bien...
 
	return EXIT_SUCCESS;
}

je précise que je n'ai pas fait de teste d'ouverture de fichier pour aller plus vite car je suis certain qu'il n'y a pas de soucis là dessus, j'utilise vraissemblablement mal fread ou fwrite... mais où j'ai remanié tout ça toute la soirée sans trouver mon erreur...

merci

Invité(e) · 16/01/2009, 07h35

Bonjour,

fread permet de lire des données depuis un fichier pour les stocker en mémoire.
fwrite permet d'écrire des données en mémoire dans un fichier.
(voir leur man)

or dans le code, on voit :

Code :

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

1
2
3
fwrite ( finput, sizeof ( struct etud_s ),1, foutput ); //on crée le binaire avec simplement ma structure
/* ... */
fread ( finput, sizeof ( struct etud_s ),1,foutput ); //je lis donc depuis mon binaire une structure, et je la met dans un .txt

on donne un fichier comme paramètre là où les fonctions attendent un pointeur.

Code :

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

1
2
3
fwrite ( &tabetud, sizeof tabetud, 1, foutput ); 
/* ... */
fread ( &tabetud, sizeof tabetud, 1, finput );

Sinon, lors des appels à scanf le derneir paramètre à l'air louche.

Envoyé par titimoi

je précise que je n'ai pas fait de teste d'ouverture de fichier pour aller plus vite car je suis certain qu'il n'y a pas de soucis là dessus

Je ne pense pas comme toi, il ne faut pas avoir peur d'aller un peu moins vite pour faire les choses mieux.

**contremaitre** · 16/01/2009, 09h24

Envoyé par titimoi

je précise que je n'ai pas fait de teste d'ouverture de fichier pour aller plus vite car je suis certain qu'il n'y a pas de soucis là dessus

Très mauvaise idée. Un jour tu perdra plus de temps a chercher ton erreur qui sera que ton fichier s'est pas ouvert (et il peut y avoir beaucoup de raison pour lesquelles l'ouverture d'un fichier écohue), que le temps gagné a ne pas écrire le test.

**titimoi** · 16/01/2009, 09h48

merci pour les réponses, je m'attendais à ce genre de réponse pour les erreurs, je n'épiloguerais pas dessus.(j'ai rajouté au cas où une erreur.. mais ce n'est pas ça...)

Concernant le pointeur pour fread et fscan c'est en effet la chose qui me posais problème, le fait est que rien ne change (dans mes problème lorsque je donne une adresse à la place du fichiers lu, si ce n'est que maintenant le fichier speicher ne contient rien... d'où un problème dans fwrite dans la fonction speicher.. voici mon code modifié :

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
void speicher ( char output[],struct etud_s tabetud )
{
 
	FILE *foutput;
	foutput=fopen ( output,"wb" );
if (foutput==0)
fprintf(stderr,"probleme lors de l'ouverture de %s\n",output);
else{
	fwrite ( &tabetud, sizeof ( struct etud_s ),1,foutput );
       }
}
 
void lecture ( char input[] /*char output[]*/,struct etud_s tabetud )
{
	FILE *finput;
	finput=fopen ( input,"rb" );
	fread ( &tabetud, sizeof ( struct etud_s ),1,finput );
	printf ( "\n le nom est : %s",tabetud.nom );
	printf ( "\n leprenom est : %s",tabetud.prenom );
}
 
int main()
{
	edition ( tabetud,"edition" );
	speicher ( "speicher",tabetud );
	lecture ( "speicher",tabetud );
 
	return EXIT_SUCCESS;
}

Invité(e) · 16/01/2009, 10h21

Après, il faut aussi vérifier que la variable tabetud est à la fois globale et passée en paramètres à speicher...

Quand tu donne cette variable en paramètre à édition, elle est passée par copie est rien n'est donc écrit dedans...

Retire cette variable globale qui ne sert à rien, remplace le prototype de tes fonctions : passe un pointeur plutôt qu'une structure :

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
void edition (struct etud_s * tab, char filename[])
{
    /* à remplir */
}

void speicher(char output[],  struct etud_s * tab)
{
    /* à remplir */
}

void lecture(char input[],  struct etud_s * tab)
{
    /* à remplir */
}

int main(void)
{
    struct etud_s tabetud;
    edition(&tabetud, "edition");
    speicher("speicher", &tabetud);
    lecture("speicher", &tabetud);

    return EXIT_SUCCESS;
}

merci pour les réponses, je m'attendais à ce genre de réponse pour les erreurs, je n'épiloguerais pas dessus.(j'ai rajouté au cas où une erreur.. mais ce n'est pas ça...)

Si tu lis bien, tu verras qu'on a pas dit que l'erreur venait de là. On dit juste qu'il n'est jamais trop tôt pour prendre de bonnes habitudes.

EDIT :
Dans les prototypes, il faut lire des '*' et pas des '&', qui n'a rien à voir puisse que c'est du c++.

**titimoi** · 16/01/2009, 10h35

Concernant la variable globale tabetud, il est écris quelque chose dedans puisque je m'en sert dans edition et dans lecture, elle est donc necessaire.

Si je mets dans les paramètres de mes fonctions des

Code :

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

struct etud_s &tab

j'ai des erreurs de compilations de type :

error: expected ')' before '&' token

que je ne comprend pas, du coup j'ai mis des *tab, et voici mon code avec le rajout de tabfinal:

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
50
51
52
53
54
55
56
57
58
59
60
61
#include <stdio.h>
#include <stdlib.h>
#define MAXN 10
#define MAXP 10
 
 
 
struct etud_s
{
	char nom[MAXN];
	char prenom[MAXP];
};
 
struct etud_s tabetud;
struct etud_s tabfinal;//ici rajout de tabfinal
 
void edition ( struct etud_s *tab, char filename[] )
{
	FILE *datei;
	datei=fopen ( filename,"w" );
 
	printf ( "\ndonner le nom\n" );
	scanf ( "%s",(*tab).nom );
	fprintf ( datei,"%s\t",(*tab).nom );
	printf ( "donner le prenom\n" );
	scanf ( "%s",(*tab).prenom );
	fprintf ( datei,"%s",(*tab).prenom );
	fclose ( datei );
}
 
void speicher ( char output[],struct etud_s *tab )
{
 
	FILE *foutput;
	foutput=fopen ( output,"wb" );
 
if (foutput==0)
fprintf(stderr,"probleme lors de l'ouverture de %s\n",output);
else
{
 
	fwrite ( tab, sizeof ( struct etud_s ),1,foutput );
}
}
void lecture ( char input[] ,struct etud_s *tab )
{
	FILE *finput;
	finput=fopen ( input,"rb" );;
	fread ( tab, sizeof ( struct etud_s ),1,finput );
	printf ( "\n le nom est : %s",(*tab).nom );
	printf ( "\n leprenom est : %s",(*tab).prenom );
}
 
int main()
{
	edition ( &tabetud,"edition" );
	speicher ( "speicher",&tabetud );
	lecture ( "speicher",&tabfinal );
 
	return EXIT_SUCCESS;
}

Comment ça se fait que dans mon fichier speicher, ce ne soit pas du binaire, mais du texte, avec [nom][prenom] d'écris ?? il doit y avoir un soucis de ce côté là

Invité(e) · 16/01/2009, 10h57

Envoyé par titimoi

Si je mets dans les paramètres de mes fonctions des

Code :

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

struct etud_s &tab

j'ai des erreurs de compilations de type :

Oui, mea culpa, il fallait lire '*' et pas '&'. (habitude du c++).

Concernant la variable globale tabetud, il est écris quelque chose dedans puisque je m'en sert dans edition et dans lecture, elle est donc necessaire.

Pas si tu la déclare dans la fonction main.

**titimoi** · 16/01/2009, 11h03

Au temps pour moi je ne l'avais pas compris comme cela, le problème reste entier.. et j'ai des choses très bizarres :

./etud

donner le nom
nom
donner le prenom
prenom

le nom est :
leprenom est : @

cat speicher
nomPprenomP�]

cat edition
nom prenom

speicher écrit n'importe quoi.. et ce n'est pas du binaire...

edit mon code actuel :

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
50
51
52
53
54
55
56
57
58
59
60
61
#include <stdio.h>
#include <stdlib.h>
#define MAXN 10
#define MAXP 10
 
 
 
struct etud_s
{
	char nom[MAXN];
	char prenom[MAXP];
};
 
void edition ( struct etud_s *tab, char filename[] )
{
	FILE *datei;
	datei=fopen ( filename,"w" );
 
	printf ( "\ndonner le nom\n" );
	scanf ( "%s", ( *tab ).nom );
	fprintf ( datei,"%s\t", ( *tab ).nom );
	printf ( "donner le prenom\n" );
	scanf ( "%s", ( *tab ).prenom );
	fprintf ( datei,"%s", ( *tab ).prenom );
	fclose ( datei );
}
 
void speicher ( char output[],struct etud_s *tab )
{
 
	FILE *foutput;
	foutput=fopen ( output,"wb" );
 
	if ( foutput==0 )
		fprintf ( stderr,"probleme lors de l'ouverture de %s\n",output );
	else
	{
 
		fwrite ( tab, sizeof ( struct etud_s ),1,foutput );
	}
}
void lecture ( char input[] ,struct etud_s *tab )
{
	FILE *finput;
	finput=fopen ( input,"rb" );;
	fread ( tab, sizeof ( struct etud_s ),1,finput );
	printf ( "\n le nom est : %s", ( *tab ).nom );
	printf ( "\n leprenom est : %s\n", ( *tab ).prenom );
}
 
int main()
{
	struct etud_s tabetud;
	struct etud_s tabfinal;
 
	edition ( &tabetud,"edition" );
	speicher ( "speicher",&tabetud );
	lecture ( "speicher",&tabfinal );
 
	return EXIT_SUCCESS;
}

**Emmanuel Delahaye** · 16/01/2009, 11h13

Envoyé par titimoi

edit mon code actuel :

Tu peux déjà corriger ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c:50: warning: function declaration isn't a prototype
C:\dev\hello\main.c: In function `main':
C:\dev\hello\main.c:54: warning: passing arg 2 of `edition' discards qualifiers from pointer target type
C:\dev\hello\main.c:55: warning: passing arg 1 of `speicher' discards qualifiers from pointer target type
C:\dev\hello\main.c:56: warning: passing arg 1 of `lecture' discards qualifiers from pointer target type
Output size is 21.50 KB
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 4 warnings

Ensuite, il manquait la fermeture des fichiers...

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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
 
#include <stdio.h>
#include <stdlib.h>
#define MAXN 10
#define MAXP 10
 
struct etud_s
{
   char nom[MAXN];
   char prenom[MAXP];
};
 
void edition (struct etud_s *tab, char const filename[])
{
   FILE *datei = fopen (filename, "w");
   if (datei != NULL)
   {
      printf ("\ndonner le nom\n");
      scanf ("%s", tab->nom);
      fprintf (datei, "%s\t", tab->nom);
      printf ("donner le prenom\n");
      scanf ("%s", tab->prenom);
      fprintf (datei, "%s", tab->prenom);
      fclose (datei);
   }
   else
   {
      perror (filename);
   }
}
 
void speicher (char const output[], struct etud_s *tab)
{
   FILE *foutput = fopen (output, "wb");
 
   if (foutput != 0)
   {
      fwrite (tab, sizeof (struct etud_s), 1, foutput);
      fclose (foutput);
   }
   else
   {
      perror (output);
   }
}
void lecture (char const input[], struct etud_s *tab)
{
   FILE *finput = fopen (input, "rb");
   if (finput != NULL)
   {
      fread (tab, sizeof (struct etud_s), 1, finput);
      printf ("\n le nom est : %s", tab->nom);
      printf ("\n leprenom est : %s\n", tab->prenom);
      fclose (finput);
   }
   else
   {
      perror (input);
   }
}
 
int main (void)
{
   struct etud_s tabetud;
   struct etud_s tabfinal;
 
   edition (&tabetud, "edition");
   speicher ("speicher", &tabetud);
   lecture ("speicher", &tabfinal);
 
   return EXIT_SUCCESS;
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
donner le nom
aaa
donner le prenom
bbb
 
 le nom est : aaa
 leprenom est : bbb
 
Process returned 0 (0x0)   execution time : 3.097 s
Press any key to continue.

**titimoi** · 16/01/2009, 11h15

ah j'utilise kdevelop et il ne me donne pas d'erreur ou de warning, je vais tenter de voir ce qui ne va pas (je ne connais pas bien le type de warning que me renvoie ton compileur)

edit: non en fait je ne vois pas le problème étant donné que mes fichiers sont bien créé, en revanche ne pouvant tester avec ton compileur, il me paraît difficile de tâtonner... pour trouver le problème que tu me fais resortir,

Invité(e) · 16/01/2009, 11h20

En fermer la fichier output, ça va mieux :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void speicher(char output[], struct etud_s *tab)
{
    FILE *foutput;
    foutput = fopen(output, "wb");
 
    if (foutput == NULL)
        perror(output);
    else {
        fwrite(tab, sizeof *tab, 1, foutput);
        fclose(foutput);
    }
}

Code :

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

1
2
3
4
5
6
7
donner le nom
foo
donner le prenom
bar
 
 le nom est : foo
 leprenom est : bar

**titimoi** · 16/01/2009, 11h22

on va finir par y arriver ! merci beaucoup ça fonctionne, (même si je suis toujours surpris par ce que contient mon fichier speicher)

merci encore pour votre aide

**titimoi** · 16/01/2009, 11h24

c'est tout à fait ça.. erreur de débutant, je ne pense pas à fermer les fichiers... ça rentrera comme ça. merci à tous les deux, c'est résolu

**Emmanuel Delahaye** · 16/01/2009, 11h29

Envoyé par titimoi

ah j'utilise kdevelop et il ne me donne pas d'erreur ou de warning, je vais tenter de voir ce qui ne va pas (je ne connais pas bien le type de warning que me renvoie ton compileur)

edit: non en fait je ne vois pas le problème étant donné que mes fichiers sont bien créé, en revanche ne pouvant tester avec ton compileur, il me paraît difficile de tâtonner... pour trouver le problème que tu me fais resortir,

Mon compilateur est le même que le tient (MinGW, version de gcc pour Windows). Simplement, il est configuré correctement :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

**Emmanuel Delahaye** · 16/01/2009, 11h30

Envoyé par titimoi

(même si je suis toujours surpris par ce que contient mon fichier speicher)

Il contient exctement l'image mémoire des données. Évidemment, ce n'est pas portable...

http://emmanuel-delahaye.developpez....m#enreg_struct

**titimoi** · 16/01/2009, 11h32

Envoyé par Emmanuel Delahaye

Il contient exctement l'image mémoire des données. Évidemment, ce n'est pas portable...

http://emmanuel-delahaye.developpez....m#enreg_struct

je m'attendais plus à des trucs incompréhensibles qu'a des bouts de mes "nom" et "prenom" en plus de trucs incompréhensibles !

**Emmanuel Delahaye** · 16/01/2009, 13h14

Envoyé par titimoi

je m'attendais plus à des trucs incompréhensibles qu'a des bouts de mes "nom" et "prenom" en plus de trucs incompréhensibles !

Essaye comme ça

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
int main (void)
{
   struct etud_s tabetud = {0};
   struct etud_s tabfinal = {0};
 
   edition (&tabetud, "edition");
   speicher ("speicher", &tabetud);
   lecture ("speicher", &tabfinal);
 
   return EXIT_SUCCESS;
}

Et ce sera encore plus clair !

Probleme avec fread ou fwrite.. binaire pas lisible ?

C

Discussions similaires

Partager

Partager