Discussion :

[Sparky95]

Bonjour,
Je suis occupé d'écrire une classe

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
 
Class abcd{
	private:
		char* nom;
		int r, v, b;
	public:
		...// les getter setter,...
		void Save(std::ofstream&) const;
 
}

et j'essayes de faire une methode save

Code C++ :

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

1
2
3
4
5
 
void abcd::Save(ofstream &fichier) const{
	fichier.write(getNom(), sizeof(getNom()));
	fichier.write((char*) &getRouge(), sizeof(getRouge())); // cette ligne pause probleme	
}

j'ai également tenté

Code C++ :

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

fichier.write((char*) &(getRouge()), sizeof(getRouge()));

mais cela ne fonctionnes pas je suis obligé de décrarer une variable tmp et puis d'en récupérer l'adresse

Code C++ :

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

1
2
int tmp = getRouge();
	fichier.write((char*) &tmp, sizeof(getRouge()));

Pourquoi?
merci d'avance

[foetus]

Pourquoi?

À vue de nez, cela semble logique ... mais je peux me tromper.

Lorsque tu appelles ton accesseur getRouge(), le retour va être stocké dans une variable temporaire (le RVO ne fonctionne pas parce que tu ne stockes pas ce retour)
Et tu prends l'adresse de cette variable temporaire Quand cette variable va-t-elle être détruite ? juste après l'appel de ton accesseur getRouge() ? après l'appel de la fonction write ?

Avec une vraie variable, on sait qu'elle sera détruite à la fin du bloc

[Sparky95]

Je ne sais pas j'aurais dis apres la fin de l'execution de la ligne donc getRouge retourn la valeur qui est disponible jusqu'a la fin de l'execution de la ligne donc jusqu'a la fin du write dans ce cas si

[Bousk]

getRouge devrait fonctionner, effectivement il y a une variable temporaire qui sera détruite après l'appel au write. Vu que c'est un int, ça passe jusque là. Reste ensuite le cast brutal en char*, lui peut ne pas passer et faire des choses bizarres, faut tester.
Par contre fichier.write(getNom(), sizeof(getNom())); ne va certainement pas écrire le contenu de nom ne serait-ce que parce que, à moins d'avoir un nom composer de 1 caractère à la limite, sizeof(getNom()) ne retourne certainement pas ce qui est espéré..

[Sparky95]

Le truc est que la méthode demande un char* =>ostream& write (const char* s, streamsize n);.

Donc sauf erreur de ma part je suis obligé de faire un casting.
Quand au fichier.write((char*)&getRouge(), sizeof(getRouge())); il me mets cette erreur lorsque j'essayes de compiler.



idem si je rajoute les parenthèses &(getRouge())

[Bktero]

Pourquoi faire des char* en C++ alors que tu as std::string ?

Pourquoi utiliser des getters à l'intérieur d'une fonction membre quand tu pourrais directement accéder à la donnée membre ?

Pourquoi ne pas nous montrer le corps de getRouge() ? Je présume qu'elle renvoie r, qui aurait peut-être dû s'appeler rouge du coup ?

Enfin, pourquoi ne pas utiliser l'opérateur << de ton ofstream pour écrire dans le fichier et ainsi te simplifier grandement la vie ? Voir http://www.cplusplus.com/doc/tutorial/files/