Discussion :

[damiengif]

Bonjour,

J'ai un petit souci avec la recopie d'un objet fstream depuis le passage de VC6 à VC8:

Une donnée membre _file était affectée de la manière suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class A
{
public:
    void SetFile(fstream *file);
 
protected:
    fstream _file;
};
 
void A::SetFile(fstream * file)
{
    _file = *file;
}

Sauf qu'apparemment l’opérateur = est devenu privé donc ça ne compile plus.

Le projet en question est assez complexe, donc je ne souhaite pas pour le moment reprendre la conception, mais plutôt remplacer la ligne

Code :

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

_file = *file;

par quelque chose qui ferait le même job.

J'ai pas trouvé mieux que ça :

Code :

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

memcpy(&_file,file,sizeof(fstream));

Avez-vous une meilleure idée?
Merci

[r0d]

Bonsoir,

solution très moche, mais je ne vois pas faire autrement: passer ta variable membre _file de fstream à fstream &, et ta fonction sera:

Code :

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

_file = dynamic_cast<fstream&>( *file );

Le fait d'avoir file_ en référence ne devrait pas changer le reste de ton code; sauf peut-être le constructeur.

Pour faire quelque chose de propre, tu pourras ensuite modifier ta fonction ainsi:

Code :

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

1
2
3
4
void A::SetFile(fstream & file)
{
    _file = file;
}

[Niark13]

En C++ standard, les streams ne sont PAS copiables. Il ne faut surtout pas le faire. Visual C++ 6 est un compilateur qui est sorti en 1997 (avant la normalisation du langage). Il permet donc de faire des choses qui ne sont pas censées marcher en C++ normalisé.

Je te conseille de revoir ta conception en conséquence (tout dépend de ce que tu veux faire), par exemple en stockant un pointeur ou un shared_ptr plutôt qu'une valeur.

Quand à la "solution" memcpy, ça a 99,9 chances sur 100 de ne pas marcher. Mieux vaut ne pas l'utiliser.

[LinuxUser]

Il me semblait que les fstream était non-copiable, nan?

[Bousk]

Bonjour,

fstream n'est pas copiable effectivement.
La solution de r0d ne marchera pas non plus, une référence doit être initialisée à sa déclaration.
Tu peux par contre stocker le pointeur en interne, et le déférencer quand nécessaire.

[r0d]

La solution de r0d ne marchera pas non plus, une référence doit être initialisée à sa déclaration.

Ca le forcera à faire du RAII, ce qui ne peut être que positif

[damiengif]

Je te conseille de revoir ta conception en conséquence (tout dépend de ce que tu veux faire)

C'est justement ce que je voulais éviter. Un peu par flemme, mais surtout parce que c'est un projet (très) complexe que j'ai récupéré, que je ne maîtrise pas encore et je n'ai pas envie de "tout casser" pour le portage sous VC8.
Mais si le code n'était pas bon à la base, et que vous ne voyez pas d'autre solution, je m'y résignerai.
Je doit pouvoir changer la donnée membre en fstream& et modifier le constructeur sans poser trop de problèmes aux classes dérivées.

edit:
C'est fait. Après avoir modifié une bonne dizaines de classes pour pour propager le file dans les constructeurs, changé le type de _file en fstream& et ajouté une donnée membre fstream _fileDefault pour initialiser _file à qqch dans le constructeur sans file, ça a l'air de marcher.

Merci à tous