initialisation reference membre dans constructeur

Bonjour,

Parmis les variable membre d'un objet se trouve une reference sur un autre objet. Pour initialiser celle ci, je passe en parametre de mon constructeur une reference sur une instance de meme type pour faire une simple affectation de ma reference membre.... et la "'CCImg::img' : doit être initialisé(e) dans la liste des initialiseurs base/membre du constructeur
"

je vous donne kelke ligne pour eclaircir mes betises:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

class CCImg { ... public: IplImage &img; ... }; CCImg::CCImg (IplImage &img, ...) { this->img = img; ... }

---

ke dois je faire pour reparer mon erreur... et surtout PKOI?:roll:

Merci

Tu as fait un MULTI POST, j'ai contacté Hdd34 pour qu'il arrange ça... fait plus attention à l'avenir s'il te plaît ;)

pour ton problème, il te faut à tout prix utiliser la chaîne d'initialisation du constructeur (c'est d'ailleurs ce qui devrait êtr utiliser tout le temps ou presque : http://c.developpez.com/faq/cpp/?pag...EUR_liste_init )

remplace :

Code:

---

1 2 3 4 5

CCImg::CCImg (IplImage &img, ...) { this->img = img; ... }

---

par

Code:

---

1 2 3 4

CCImg::CCImg (IplImage &Img, ...) : img(Img) { ... }

---

d'ailleurs je te conseillerais d'utiliser des conventions de codage qui te permettront de ne jamais avoir un paramètre et une donnée membre qui ait le même nom... perso je fais toujours commencer les nom des données membres par un _ ça permet de les identifier sans ambigüité tout au long du code (même si j'utilise tout le temps this-> également on repère vachement mieux, puis ça aère le code ^^ ;) )

le problème vient du fait que les références _DOIVENT_ être assignées avec leur constructeur, si c'est l'opérateur = qui est utilisé, alors c'est une copie qui est faite, et comme tu n'as pas assigné la référence en question, elle ne pointe sur aucune variables => GROS PROBLEME ;)

Pas mieux ;)
Multipost réglé.

Merci pour la reponse, j avais ete voire du cote des listes d initialisation sur la faq sans tout saisir (me suis pas non plus trop attardé), j y retourne.:D

Pour le multi post ki n'en etait pas un (les posts etaient un poil differents), j'ai just edité plusieurs fois mon post pour rajouter les balises de codes (comme je l ai deja fait dans le passé), je ne comprend pas pkoi ce resultat...:?

Merci

OK ;) Merci Miles (et désolé Hdd34 si tu lis ça ^^)

Les listes d'initialisations permettent d'appeller directement le constructeur des données membres (et de la classe mère en cas d'héritage)

ça évite d'appeller le constructeur de copie PUIS l'opérateur d'affectation, exemple :

Code:

---

CLASS::CLASS(const Obj& O) : _O(O) { }

---

est équivalent à

Code:

---

1 2 3

Obj _O(O); // ou (les deux sont strictement équivalent Obj _O = O;

---

tandis que :

Code:

---

CLASS::CLASS(const Obj& O) { this->_O = O; }

---

est équivalent à

Code:

---

1 2	Obj _O; _O = O;

---

si Obj est une "grosse" classe, ça va entrainer de (très) nombreuses opérations supplémentaires...
puisqu'il faudra
1°) Appeller le constructeur par défaut
2°) Faire une copie de O dans _O

alors qu'avec la liste d'initialisation, seul le constructeur par copie est appelé...

de plus, c'est le seul moyen de définir les constantes et de faire pointer les références ;) (comme tu t'en es rendu compte ^^)