A propos de fread..

**TheDoci** · 26/09/2009, 18h20

Bonjour,

j'ouvre des fichiers en mode wb+ (donc logiquement en lecture/écriture binaire) mais j'ai un soucis : je commence par écrire, avec fwrite, puis je fais un fflush et un rewind (pour revenir au début du fichier), et enfin lis ce que je viens d'écrire avec fread.

Le problème c'est que fread ne me lit rien (l'appel à perror(NULL) me renvoie : "Protocol error").
Il faut que je ferme le fichier et le réouvre en mode rb pour que je puisse lire.

J'ai bien ouvert le fichier en mode ab+, et ça marche, mais j'ai vraiment besoin d'utiliser wb+, car plus tard j'utiliserai la fonction tmpfile, qui ouvre les fichiers dans ce mode.

Avez-vous une idée pour régler ce problème ?

**ram-0000** · 26/09/2009, 19h53

Un petit bout de code pour nous aider à t'aider, j'ai oublié ma boule de cristal au boulot

Merci

**TheDoci** · 26/09/2009, 20h18

Il n'y a rien de spécial dans le code, c'est pour ça que je ne l'ai pas mis.

Déjà, est-ce que le fait de faire des fwrite puis des fread en mode wb+ marche chez quelqu'un ?

Voici le magnifique bout de code, qui n'a rien de spécial (les fread et fwrite marchent, c'est sûr, mais pas en wb+) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
ftmp[0] = fopen("tetmp3", "wb+");
 
for (p=0 ; p < nl ; p++)
	fwrite(rouge[p], np, 1, ftmp[0]);
 
fflush(ftmp[0]);
rewind(ftmp[0]);
 
for (j=0 ; j < nl ; j++)
	fread(rouge[j], np, 1, ftmp[0]);
 
fclose(ftmp[0]);

**ram-0000** · 26/09/2009, 20h57

Je ne vois rien de particulier dans ton code si ce n'est qu'il n'y a aucun contrôle sur les valeurs de retour.

Il faut ajouter des tests après fopen(), fread(), fwrite(), fflush() et fclose(), cela permettra de diagnostiquer un éventuel problème ici.

Au fait, quelle est la valeur de nl et np lorsque tu rentres dans ta fonction (par la même occasion, peut être qu'un nom plus parlant serait le bienvenue aussi).

Le problème c'est que fread ne me lit rien (l'appel à perror(NULL) me renvoie : "Protocol error").

fread lit quelque chose ou retourne une erreur. Après, peut être qu'il le range au mauvais endroit ...

**TheDoci** · 26/09/2009, 21h25

C'est ici une version épurée du code, donc je n'ai pas fait apparaître tous les contrôles sur les valeurs de retour.

fread ne lit rien (donc retourne 0), mais perror indique Protocol error
Ca ne peut donc pas être rangé au mauvais endroit, puisqu'aucun octet n'est lu

Quant aux noms des variables nl et np je suis du même avis que toi mais ce sont des variables pré-existantes

**ram-0000** · 26/09/2009, 23h26

Envoyé par ram-0000

Au fait, quelle est la valeur de nl et np lorsque tu rentres dans ta fonction

**TheDoci** · 27/09/2009, 09h16

nl vaut 600 et np vaut 443

Mais j'ai essayé avec nl = np = 10 et ça ne change rien, j'ai toujours des Protocol error

Edit : j'ai regardé si le rewind marche correctement, et c'est le cas (j'appelle ftell() pour savoir où je suis dans le fichier)

Edit 2 : ce que je trouve bizarre aussi, c'est que si je supprime le fflush et le rewind, fread se comporte exactement de la même façon

**souviron34** · 27/09/2009, 12h43

d'une part rouge[j] ne devrait-il pas être une adresse dans le fread ?
d'autre part, le "w+" indique, si mes souvenirs sont bons, une écriture en mode append..

Pas une lecture..

**diogene** · 27/09/2009, 12h58

souviron34 :

d'une part rouge[j] ne devrait-il pas être une adresse dans le fread ?

ce doit être le cas, sinon il aurait déjà un problème avec le fwrite (à moins que les données écrites soient fausses, sans que le programme plante ouvertement ?).

d'autre part, le "w+" indique, si mes souvenirs sont bons, une écriture en mode append..

append est identifié par a

w+ truncate to zero length or create text file for update

**souviron34** · 27/09/2009, 13h07

Envoyé par diogene

ce doit être le cas, sinon il aurait déjà un problème avec le fwrite

je sais pas.. A voir l'écriture, j'ai comme un doute...

Maintenant, pour le flag, c'est donc bien un flag d'écriture...

**diogene** · 27/09/2009, 13h13

Envoyé par souviron34

je sais pas.. A voir l'écriture, j'ai comme un doute...

Les arguments de fwrite et de fread sont du même type.

Maintenant, pour le flag, c'est donc bien un flag d'écriture...

Le mode update permet la lecture et l'écriture.

**TheDoci** · 27/09/2009, 13h19

rouge est un tableau à 2 dimensions donc no problem

Et en plus je vous ais dis que ça marche avec le mode ab+, donc il faut bien chercher du côté de wb+ à mon avis

**diogene** · 27/09/2009, 13h31

Et en rb+, ça marche ?

**TheDoci** · 27/09/2009, 13h59

oui, mais ça ne crée pas le fichier s'il n'existe pas

gl · 27/09/2009, 14h52

Lorsque tu exécutes le programme avec le mode "wb+", le fichier est-il bien créer (penses à le supprimer avant l'exécution afin d'être certain) ? Est-il correctement rempli ?
Sous quel environnement travailles-tu ?
Peux-tu fournir un code minimal mais compilable permettant de reproduire ce problème ?

**TheDoci** · 27/09/2009, 16h17

oui le fichier se crée bien et est bien rempli
Je travaille sous Ubuntu.

Voici un code simplifié qui compile, n'a aucune erreur chez moi, mais dont le fread ne marche pas avec wb+ mais qui marche avec ab+ :

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
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/wait.h>
 
int main(int argc, char **argv)
{
	int i;
	int tab[10] = {1, 3, 5, 7, 9, 2, 4, 6, 8, 10}, tab2[10] = {0};
	FILE *ftmp;
 
	ftmp = fopen("tetmp1", "wb+");
	if (!ftmp)
		exit(-1);
 
	switch (fork())
	{
		case -1:
			puts("Erreur lors du fork\n");
			exit(-1);
		case 0:
			printf("Fils de pid %d\n", getpid());
 
			fwrite(tab, sizeof(int), 10, ftmp);
			exit(0);
		default:
			break;
	}
 
	wait(NULL);
 
	fflush(ftmp);
	rewind(ftmp);
 
	fread(tab2, sizeof(int), 10, ftmp);
 
	for (i=0 ; i < 10 ; i++)
		printf("T2 %d : %d\n", i, tab2[i]);
 
	fclose(ftmp);
 
	return 0;
}

gl · 27/09/2009, 17h12

Tu avais omis de préciser que tu travaillais avec deux processus différents, ce qui a sont importance.

Pour simplifier à l'extrême, fork() crée un processus en dupliquant le contexte mémoire. De ce fait je doute fortement qu'un appel à fflush() dans le père ait un quelconque impact sur le buffer d'écriture du fils.

Juste deux petits conseils en passant :

Dans de nombreux cas, ouvrir un fichier en lecture/écriture est une mauvaise idée. Lorsque l'on en arrive à le faire, il faut se poser un instant et bien peser toutes les conséquences.
Pour faire communiquer deux processus entre eux, j'aurais plus tendance à utiliser un pipe qu'un fichier classique.

**souviron34** · 27/09/2009, 18h06

disons que ça peut se faire avec 2 processus différents, mais comme tu le dis il faut bien faire attention..

Et c'est nettement plus simple avec 2 processus vraiment différents qu'un père et son fils, qui partagent un certain nombre de variables...

Un processus qui écrit (et qui a ouvert en w) et un qui lit (qui a ouvert en r)

ça ça marche bien..

Si les process sont vraiment différents, les pointeurs le sont aussi.

Sinon, c'est la pagaille... L'un écrit et déplace le pointeur, l'autre fait rewind, l'autre veut réécrire et écrase...

Disons que dans ton cas, le fopen serait mieux dans le case, avec un open différent suivant le père ou le fils..

gl · 27/09/2009, 18h23

Envoyé par souviron34

Un processus qui écrit (et qui a ouvert en w) et un qui lit (qui a ouvert en r)

ça ça marche bien..

Oui ça peut fonctionner si c'est correctement fait, mais est-ce vraiment adapté, sachant qu'il existe différents autres mécanismes qui font ça très bien et dont c'est le rôle (en tout cas davantage le rôle qu'un simple fichier) : pipe, socket, message queue, mailbox, etc. ?

**TheDoci** · 27/09/2009, 18h32

Exact, désolé j'ai oublié cette info

Pour les pipe, je ne peux pas les utiliser, car mon prof (c'est pour un TP) veut qu'on utilise le fork, et que le fils écrive dans un fichier, que le père lit après

Après, étant donné que je devrai utiliser tmpfile, je ne peux pas fermer le fichier pour le rouvrir qu'en lecture, c'est pour ça que je simule le tmpfile avec le wb+ (pour le débugage donc).

J'ai déplacé le fflush et le rewind dans le fils, ça devrait marcher normalement non ? si j'ai bien compris.
Mais ça ne marche toujours pas

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
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/wait.h>
 
int main(int argc, char **argv)
{
	int i;
	int tab[10] = {1, 3, 5, 7, 9, 2, 4, 6, 8, 10}, tab2[10] = {0};
	FILE *ftmp;
 
	ftmp = fopen("tetmp1", "wb+");
	if (!ftmp)
		exit(-1);
 
	switch (fork())
	{
		case -1:
			puts("Erreur lors du fork\n");
			exit(-1);
		case 0:
			printf("Fils de pid %d\n", getpid());
 
			fwrite(tab, sizeof(int), 10, ftmp);
			fflush(ftmp);
			rewind(ftmp);
			exit(0);
		default:
			break;
	}
 
	wait(NULL);
	//fflush(ftmp);
	//rewind(ftmp);	
 
	fread(tab2, sizeof(int), 10, ftmp);
 
	for (i=0 ; i < 10 ; i++)
		printf("T2 %d : %d\n", i, tab2[i]);
 
	fclose(ftmp);
 
	return 0;
}

A propos de fread..

C

Discussions similaires

Partager

Partager