Discussion :

[laTouille]

Bonjour,
j'ai un problème avec le code suivant (je n'ai mis que ce qui me semblait utile) :

Code :

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

1
2
3
4
5
6
7
8
 
FileName="fichierBidon"; 
	ofstream data(FileName.c_str(), ios::out | ios::trunc);
 
	for (int i=0; i<nombreDePoints; i++) 
	{   		
		data << i << endl;		
	}

Le fichier est bien créé, mais rien ne s'écrit dedans.
Quelqu'un aurait une idée sur la raison du problème ? (j'ai bien inclus fstream)
Merci d'avance !

[oodini]

Le code suivant fonctionne :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#include <fstream>
#include <ios>
 
int main()
{
	std::string fileName = "fichierBidon.txt"; 
	std::ofstream data(fileName.c_str(), std::ios::out | std::ios::trunc);
 
	for (int i=0; i<5; i++) 
		data << i << std::endl;		
}

[therwald]

Salut, je m'excuse si j'enfonce une porte ouverte, mais ne faudrait-il pas appeler close() après écriture pour s'assurer que les données sont bien flushées avant la fin du processus?

[Bousk]

Bonjour,

close est automatiquement appelé à la destruction de l'objet, aucun risque à ce niveau.

Le code fonctionne, si tenté que nombreDePoints soit > 0.

[therwald]

My mistake, je avais zappé que bien sûr RAII oblige la destruction ferme le fichier...
Si effectivement nombreDePoints est positif, il pourrait y avoir une erreur hors de contrôle du processus qui ne se voit pas faute de contrôle des signaux d'erreur...
Ceci dit, une valeur aberrante de nombreDePoints semble le plus probable (le fichier étant créé...)

[ptyxs]

Un détail : si je ne m'abuse dans :

Code :

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

	std::ofstream data(fileName.c_str(), std::ios::out | std::ios::trunc);

std::ios::out est inutile (c'est le mode d'ouverture par défaut d'un ofstream, non ?) et l'on pourrait écrire tout simplement :

Code :

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

	std::ofstream data(fileName.c_str(), std::ios::trunc);

Est-ce que je me trompe ?

[oodini]

Le trunc n'a pas ailleurs pas d'effet, car c'est le mode par défaut (son complément étant append, qui lui, doit être impérativement précisé).

[Flob90]

En effet le fichier sera forcément fermé, mais si tu fais juste confiance au RAII alors tu peux oublier l'idée d'être certain que les données ont bien été flushées.

[ptyxs]

En effet le fichier sera forcément fermé, mais si tu fais juste confiance au RAII alors tu peux oublier l'idée d'être certain que les données ont bien été flushées.

Tu es bien sûr de ce que tu avances ?
J'avais cru comprendre que le destructeur d'un ofstream appelait close() qui elle-même vide le tampon... et que c'était là un des principaux intérêts des ofstream...
Je me trompe ?

Le trunc n'a pas ailleurs pas d'effet, car c'est le mode par défaut (son complément étant append, qui lui, doit être impérativement précisé).

Finalement on peut donc remplacer le :

Code :

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

	std::ofstream data(fileName.c_str(), std::ios::out | std::ios::trunc);

par un simple :

Code :

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

	std::ofstream data(fileName.c_str());

[Flob90]

close retourne une valeur ou lance une exception qui correpsond (plus ou moins) à la réussite d'avoir flush les données. Ce que close assure c'est de fermer le fichier, pas de flusher les données (il tente néanmoins de la faire). Or si close est appelé à travers le destructeur (je ne rentre pas dans le détail que quel "close" est appelé), le retour ou l'excpetion est ignoré : le fichier est bien fermé mais il n'y a aucune assurance que les données soient flushées.

[ptyxs]

Une lecture bien faite fait alors usage du manipulateur flush, qui règle ce problème.

[Flob90]

Pas exactement, si tu veux tout les détails, prends 1h et regardes la norme, tu verras que c'est pas si évident que ca que close ne puisse pas lancer après un << std::flush (c'est lié à overflow).

D'autre part les exceptions ne rendent pas nécessairement compte de la validité de l'état du buffer (pour ca il faut utiliser rdstate), alors que close va forcément dire si quelque chose à rater.

Pour fermer et pouvoir traiter correctement en cas de problèmes, à part un close avec un if autour, il n'y aura rien de plus clair, AMA. Après la question c'est de savoir si tu peux faire quelque chose en cas de problèmes, sinon ça sert à rien.

PS: Je ne distingue pas lecture/écriture ici.

[ptyxs]

En tout cas merci pour ces indications et ces pistes... cela donne utilement à réfléchir.