Discussion :

[harsh]

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 :

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

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?

Merci

[Swoög]

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 :

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

1
2
3
4
5
CCImg::CCImg (IplImage &img, ...)
{
this->img = img;
...
}

par

Code :

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

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

[Matthieu Brucher]

Pas mieux
Multipost réglé.

[harsh]

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.

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

[Swoög]

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 :

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

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

est équivalent à

Code :

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

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

tandis que :

Code :

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

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

est équivalent à

Code :

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

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 ^^)