Discussion :

[Drannor]

Bonjour,

J'essaye désespérément de créer un constructeur par copie :

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
Animation::Animation(const Animation &UneImage){
	short int* cadre_tmp ;
	short int* position_tmp;
	short int* rvb_tmp ;
 
	position_tmp = UneImage.RecupererPosition();
	cadre_tmp = UneImage.RecupererCadre();
	rvb_tmp = UneImage.RecupererCouleurTransparente();
 
//s'occupe de charger l'image
 
	ModifierAnimation(UneImage.RecupererNom(),UneImage.RecupererType(),UneImage.RecupererChemin(),position_tmp[0] ,position_tmp[1],cadre_tmp[0],cadre_tmp[1],cadre_tmp[2],cadre_tmp[3],rvb_tmp[0] ,rvb_tmp[1] ,rvb_tmp[2]);
 
	if (UneImage.RecupererFps() != FPS) temps.ModifierFps(UneImage.RecupererFps()) ; 
};

Seulement une fois à la compilation, je tombe sur les erreurs de compilation suivante et je comprend pas du tout comment les résoudre :

../Animation_Propre/Animation.cpp: In copy constructor ‘Animation::Animation(const Animation&)’:
../Animation_Propre/Animation.cpp:221: erreur: passing ‘const Animation’ as ‘this’ argument of ‘short int* Animation::RecupererPosition()’ discards qualifiers
../Animation_Propre/Animation.cpp:222: erreur: passing ‘const Animation’ as ‘this’ argument of ‘short int* Animation::RecupererCadre()’ discards qualifiers
../Animation_Propre/Animation.cpp:223: erreur: passing ‘const Animation’ as ‘this’ argument of ‘short int* Animation::RecupererCouleurTransparente()’ discards qualifiers
../Animation_Propre/Animation.cpp:227: erreur: passing ‘const Animation’ as ‘this’ argument of ‘char* Animation::RecupererNom()’ discards qualifiers
../Animation_Propre/Animation.cpp:227: erreur: passing ‘const Animation’ as ‘this’ argument of ‘typeAnimation Animation::RecupererType()’ discards qualifiers
../Animation_Propre/Animation.cpp:227: erreur: passing ‘const Animation’ as ‘this’ argument of ‘char* Animation::RecupererChemin()’ discards qualifiers

Si quelqu'un pouvait m'expliquer l'erreur et comment la resoudre je lui en serais grès.

merci

[Médinoc]

Toutes ces méthodes doivent être déclarées en tant que méthodes const puisqu'elles ne modifient pas l'objet auquel elles s'appliquent.

[Drannor]

Merci beaucoup de cette réponse mais le problème c'est que je suis confronté a un nouveau problème.

Car nombre de mes accesseurs renvoient des char* ou des short int*.
Hors pour une fonction définit comme const

Code :

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

1
2
3
4
 
 char*  Image::RecupererNom() const {
	if (VerificationInitialisation()) return nom_fichier ; else return NULL ;
};

Le compilateur me dit :

Image.cpp: In member function ‘char* Image::RecupererNom() const’:
Image.cpp:174: erreur: invalid conversion from ‘const char*’ to ‘char*’

Il faut que je cast chaque objet que je return avec un const_cast ?
Car je ne peux pas définir nom_fichier comme constant vu qu'il est modifié dans certaines fonctions.

Y'a t'il une astuce ?

Merci

[Médinoc]

Tu peux faire deux définitions de la fonction, l'une const et l'autre non :

Code :

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

1
2
3
4
5
6
char*  Image::RecupererNom() {
	if (VerificationInitialisation()) return nom_fichier ; else return NULL ;
};
char const*  Image::RecupererNom() const {
	if (VerificationInitialisation()) return nom_fichier ; else return NULL ;
};

PS: Ne devrais-tu pas plutôt utiliser des std::string que des char* ?

[Drannor]

Cette solution m'entraine des erreurs à la compilation

Image.h:131: erreur: ‘const char* Image::RecupererNom() const’ cannot be overloaded
Image.h:130: erreur: with ‘char* Image::RecupererNom() const’
Image.h:131: erreur: ‘const char* Image::RecupererNom() const’ cannot be overloaded
Image.h:130: erreur: with ‘char* Image::RecupererNom() const’

Pour le string oui je pourrais effectivement les utiliser à la place des char* même si je suis beaucoup moins familiariser avec ce type. Mais comment remplacer les short int* ?

Merci

[Médinoc]

Tu as laissé les deux fonctions en const, seule celle qui retourne const char * doit l'être.

Pour tes short int *, s'il s'agit de tableaux, tu peux utiliser des std::vector<>...