C++

écrire dans un fichier

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : écrire dans un fichier

kason

Ab uno disce omnes

Je souhaite écrire dans un fichier en mode append (nom du fichier : filename)

ofstream Sortie( filename, ios::app );

Contexte :
j'ai une classe out
je souhaite :
- ouvrir le fichier dans le constructeur.
- faire des écritures dans les fonctions membres (sans avoir à ouvrir le fichier et le refermer, car déjà ouvert)
- fermer le fichier dans le destructeur

Est-ce possible ?

J'ai bien essayé de créer un objet ofstream Sortie comme donnée membre mais cela ne marche pas... :cry:

Tout ça, c'est pour éviter d'ouvrir et de fermer le fichier à chaque fois qu'on y écrit qq chose.

Question : comment dois-je m'y prendre ?

kason

:hello:

---------------
Seul le calme paisible d'un lac peut endiguer la force furieuse d'un torrent.

Publicité

Harkonnen

Modérateur
Un modo pour les bannir tous

Pourquoi le fait de déclarer Sortie en variable membre ne fonctionne pas :??:

---------------
J'ai un string dans l'array (Paris Hilton)

kason

Ab uno disce omnes

En fait, dans la classe out, je déclare un objet ofstream comme donnée membre (protected)

std::ofstream Sortie;

autre donnée membre :
char *filename;

Puis, dans le constructeur, j'écris :

Code :

Sortie( filename, ios::app ); // refusé à la compilation
// erreur : no match for call '(std::ofstream) (char*&, const std::_Ios_Openmode&)'

:??:

Code :

ofstream Sortie( filename, ios::app ); // fonctionne mais à l'exécution rien n'est écrit (instanciation locale au constructeur)

NB. impossible d'écrire std::ofstream Sortie( filename, ios::app ); dans la classe -> erreur sur filename

kason

---------------
Seul le calme paisible d'un lac peut endiguer la force furieuse d'un torrent.

Harkonnen

Modérateur
Un modo pour les bannir tous

le premier paramètre du constructeur de ofstream est un const char* et toi tu lui fournis un char *... normal !

il faut lire les messages du compilateur :sarcastic:

---------------
J'ai un string dans l'array (Paris Hilton)

kason

Ab uno disce omnes

Je me demande si cette méthode est possible (ouvrir le fichier dans un constructeur, écrire via les méthodes de la classe, et fermer le fichier dans le destructeur) :??:

J'ai essayé en créant une donnée membre :
ofstream Sortie;

Marche pas :fou:

+ d'infos sur l'implémentation :

Code :

class out
{
private:
char *filename;
ofstream Sortie;
public:
out(char *name);
~out();
void f1();
void f2();
};
out::out(char *name)
{
filename = strdup(name); // comme strcpy avec malloc implicite
Sortie( filename, ios::app); // initialisation
// refusé à la compilation...
// ofstream Sortie( filename, ios::app); est compilé mais
// à l'exécution, fichier vide
// -> pb de portabilité de variable : conflit avec la donnée membre
}
out::~out()
{
free(filename); // libère la mémoire (allocation dynamique)
Sortie.close(); // fermeture du fichier
}
out::f1()
{
string Line;
Line = "titi\n";
Sortie << Line << flush;
}
out::f2()
{
string Line;
Line = "toto\n";
Sortie << Line << flush;
}

Des suggestions, des remarques ou une autre idée d'implémentation ?

kason

:hello:

---------------
Seul le calme paisible d'un lac peut endiguer la force furieuse d'un torrent.

Harkonnen

Modérateur
Un modo pour les bannir tous

déjà, inutile de créer une variable membre filename, ça ne sert à rien vu que tu ne l'utilises que dans le constructeur.

ensuite, regarde la surcharge de l'opérateur << dans la classe ofstream ! il n'utilise pas de string !

enfin, à la place de flush, tu peux utiliser endl qui ajoute une ligne et qui flushe ensuite.

ce qui nous donne le code suivant :

Code :

#include <fstream>
#include <iostream>
using namespace std;
class out
{
private:
ofstream *Sortie;
public:
out(const char *name);
~out();
void f1();
void f2();
};
out::out(const char *name)
{
Sortie = new ofstream(name, ios::app); // initialisation
}
out::~out()
{
Sortie->close(); // fermeture du fichier
delete Sortie;
}
void out::f1()
{
*Sortie << "titi" << endl;
}
void out::f2()
{
*Sortie << "toto" << endl;
}

en théorie, ça doit fonctionner. j'attends maintenant la réaction de Taz ou Musaran qui ne vont pas manquer de me dire que mon code est dégueulasse et/ou pas standard

Message édité par Harkonnen le 05-02-2003 à 22:31:28

---------------
J'ai un string dans l'array (Paris Hilton)

Taz

bisounours-codeur

non c'est bon mais quel interet d'avoir un pointeur?

Harkonnen

Modérateur
Un modo pour les bannir tous

++Taz a écrit :

non c'est bon mais quel interet d'avoir un pointeur?

aucun dans ce cas précis effectivement !
c'est juste une sale habitude que j'ai de déclarer des pointeurs partout

---------------
J'ai un string dans l'array (Paris Hilton)

Taz

bisounours-codeur

ben alors enleve le

Harkonnen

Modérateur
Un modo pour les bannir tous

++Taz a écrit :

ben alors enleve le

à vos ordres !
version sans pointeur :

Code :

#include <fstream>
#include <iostream>
using namespace std;
class out
{
private:
ofstream Sortie;
public:
out(const char *name);
~out();
void f1();
void f2();
};
out::out(const char *name):Sortie(name, ios::app)
{
}
out::~out()
{
Sortie.close(); // fermeture du fichier
}
void out::f1()
{
Sortie << "titi" << endl;
}
void out::f2()
{
Sortie << "toto" << endl;
}

---------------
J'ai un string dans l'array (Paris Hilton)

Publicité

Taz

bisounours-codeur

SETI pas bo comme ço? :jap:

petite remarque: beaucoup ne réalise pas que endl == '\n' + flush

et le flush est la plus part du temps inutile (surtout pour un fichier avec un buffer de ligne, genre terminal). endl n'est pas un '\n' plus joli à la mode C++, mais bien quelque chose de différent. il n'est pas sale de mettre des '\n' dans ses chaines de caractères

Harkonnen

Modérateur
Un modo pour les bannir tous

mon code est Taz-compliant !!!! [:youpi]

kason > au passage, en relisant ton source, j'ai pu constater cette magnifique initialisation dans ton constructeur :

Sortie( filename, ios::app); // initialisation

pas étonnant que ça compilait pas... :sarcastic:

Message édité par Harkonnen le 05-02-2003 à 22:57:26

---------------
J'ai un string dans l'array (Paris Hilton)

Taz

bisounours-codeur

Harkonnen a écrit :

mon code est Taz-compliant !!!! [:youpi]

comme quoi j'en demande pas trop... :jap:

remarque: il faut utiliser les allocateurs C++ et non malloc/free qui se contente de gérer des zone mémoire non-initaliser

new = constructeur+malloc
delete = destructeur+free

deplus ça balance des exceptions, alors que dans le cas de malloc il faut tester le pointeur.

bref, il faut abandonner toutes ses vieilles habitudes de C

et se débarasser des char* au profit des string (je veux plus voir de strdup ou d'autre truc comme ça)

Musaran

Cerveaulté

Citation :

Code :

out::~out()
{
Sortie.close(); // fermeture du fichier
}

Ce destructeur est inutile.
Les membres sont automatiquement détruits avec leur 'porteurs'.

Harkonnen a écrit :

le premier paramètre du constructeur de ofstream est un const char* et toi tu lui fournis un char *... normal !

Ça gène rien (c'est l'inverse qui n'est pas possible).

Code :

char a;
char* pa= &a;
const char* cpa= pa; //ou &a

Quand un pointeur/référence se dit "sur const", ça veut dire qu'il s'engage à ne pas modifier son référent, pas que celui-ci ne peut pas changer par d'autres intermédiaires.

Conclusion: C++ n'a pas de spécification de constance pure pour une variable.

---------------
Bricocheap: Montage de ventilo sur paté de mastic silicone

Taz

bisounours-codeur

Musaran a écrit :

Citation :

Code :

out::~out()
{
Sortie.close(); // fermeture du fichier
}

Ce destructeur est inutile.
Les membres sont automatiquement détruits avec leur 'porteurs'.

tout en effet

Harkonnen

Modérateur
Un modo pour les bannir tous

Musaran a écrit :

Ce destructeur est inutile.
Les membres sont automatiquement détruits avec leur 'porteurs'.

Encore une habitude que j'ai de toujours fermer/détruire ce que j'ouvre/construit.

Musaran a écrit :

Ça gène rien (c'est l'inverse qui n'est pas possible)

Oui, disons que j'étais en train de bosser quand j'ai lu son 1er post, et j'ai un peu lu ce 1er post en biais...

---------------
J'ai un string dans l'array (Paris Hilton)

kason

Ab uno disce omnes

En suivant les tuyaux que vous m'avez donné, cela marche :wahoo:

Merci à tous.

:hello:

kason

---------------
Seul le calme paisible d'un lac peut endiguer la force furieuse d'un torrent.

Harkonnen

Modérateur
Un modo pour les bannir tous

kason a écrit :

En suivant les tuyaux que vous m'avez donné, cela marche :wahoo:

euh, c'est plus que des tuyaux qu'on t'a filé :whistle:

---------------
J'ai un string dans l'array (Paris Hilton)

kason

Ab uno disce omnes

oui, c'est vrai :lol:

kason :whistle:

---------------
Seul le calme paisible d'un lac peut endiguer la force furieuse d'un torrent.

FORUM HardWare.fr

Programmation

C++

écrire dans un fichier

Sujets relatifs
Ecrire un fichier texte sur plusieurs lignes en vb6	écrire dans un fichier texte a un endroit précis
Ecrire un prog qui permet de downloader un fichier	[BCB6] Ecrire un fichier au format Access ?
[PHP] Ecrire un fichier sur le disque dur du client...	[VB] écrire dans un fichier
[ColdFusion] Ecrire un fichier sur la machine locale et pas sur server	écrire dans un fichier
[C++] fstream : comment ouvrir, lire, écrire dans un fichier texte !?!	[WIN32]Ecrire fichier a partir dialogbox
Plus de sujets relatifs à : écrire dans un fichier

Page générée en 0.110 secondes