Warning lors du this passé dans un constructeur

Version imprimable

10/08/2006, 08h45
NicolasJolet

Warning lors du this passé dans un constructeur
Bonjour à tous,
Je suis en train de créer des classes pour encapsuler des APIs Windows.

Pour simplifier, j'ai une classe MainWindow qui hérite de Window et une classe Button.

Voici le code :
Code:

1 2 3 4 5 6 7 8 9 class MainWindow : public Window { private: Button btnEssai; }; MainWindow::MainWindow() : btnEssai(*this) // Pour connaître la fenêtre parente {}
Mais j'ai un warning lors du passage de this dans le constructeur de btnEssai.
Dans mon implémentation actuelle, le bouton est créé sur la fenêtre parente dans son constructeur et je n'ai absolument aucun problème mais ce warning m'embete.

Avez-vous une idée pour éviter cela?

Merci d'avance,
Nicolas
10/08/2006, 09h09
Matthieu Brucher

This n'existe qu'une fois dans le constructeur. Dans la liste d'initialisation, il n'existe pas... d'où warning et plantage à venir.
10/08/2006, 09h15
Invité

pourquoi tu essaye de construire un objet a partir de lui meme ?

apparemment, c'est pour que ton bouton connaisse son parent (d'apres ton commentaire). donc l'initialization list est hors jeu pour toi dans ce cas. as tu essayé de déplacer l'initialisation de ton bouton dans le corps du constructeur ?
sinon, ton bouton n'aurais pas par hasard une methode create() qui fait la meme chose, mais apres la construction ?
10/08/2006, 09h20
NicolasJolet

Miles > il semblerait que le compilateur de visual studio définisse déja le this avant la liste d'initialisation donc pour le moment pas de problème, enfin du moins avec des objets non alloués dynamiquement...

toxcct > Si je déplace l'initialisation de mon bouton dans le corps du constructeur je serais obligé de jouer avec des pointeurs sans compter les couples new/delete, ce que je trouve un peu lourd...
Sinon une solution serait effectivement d'implémenter une méthode Create ou Init mais dans ce cas je devrais, pour chaque objet créé, appeller cette méthode. Ce que je trouvais un peu lourd aussi...
10/08/2006, 09h25
Matthieu Brucher

Citation:

Envoyé par NicolasJolet

Miles > il semblerait que le compilateur de visual studio définisse déja le this avant la liste d'initialisation donc pour le moment pas de problème, enfin du moins avec des objets non alloués dynamiquement...

toxcct > Si je déplace l'initialisation de mon bouton dans le corps du constructeur je serais obligé de jouer avec des pointeurs sans compter les couples new/delete, ce que je trouve un peu lourd...
Sinon une solution serait effectivement d'implémenter une méthode Create ou Init mais dans ce cas je devrais, pour chaque objet créé, appeller cette méthode. Ce que je trouvais un peu lourd aussi...

Ce n'est pas un comportement standard, c'est un truc à éviter au maximum...
Garde-le dans la liste des initialisations - sinon, c'est le constructeur par défaut qui est appelé -, puis dans le constructeur parent, appelle une fonction du bouton pour placer le père.
10/08/2006, 09h25
Invité
Citation:

Envoyé par NicolasJolet

Si je déplace l'initialisation de mon bouton dans le corps du constructeur je serais obligé de jouer avec des pointeurs sans compter les couples new/delete, ce que je trouve un peu lourd...

hein ???
pourquoi pas juste ca :
Code:

1 2 3 MainWindow::MainWindow() { btnEssai(*this) ;// Pour connaître la fenêtre parente }
10/08/2006, 09h26
Matthieu Brucher

Parce que ça ne marche pas ? Initialisation = liste d'initialisation. Dans le constructeur parent, l'objet est déjà construit, alors )à moins qu'il ait une fonction operator() qui prenne un parent en paramètre...
10/08/2006, 09h29
NicolasJolet

miles > oui je pense que je ne vais pas avoir le choix, c'est un peu du "double travail" (dans le sens où il y a appel de 2 méthodes, le constructeur et la méthode Init()) mais au moins je n'aurais plus de problèmes...

Enfin merci à tous les 2 :)
10/08/2006, 09h38
Luc Hermitte

C'est un problème de VC. Qui se justifie parfaitement : il te prévient que tu cours au devant de dangers. Tant que tu ne fais que donner la référence à *this pour construire un lien qui ne sera exploité que plutard par le membre construit tout va bien. Mais VC te fera tout de même un warning.

J'avais avais parlé plus précisément ici. Mais tu as un point d'une FAQ ou d'une série de tips qui en parle quelque part. Je ne sais juste plus où.
10/08/2006, 09h43
NicolasJolet

Il est possible que ça soit ça => ICI

EDIT : reflexion faite, ça semble plutot parler du this dans le constructeur et non dans la liste d'initialisation, si l'on fait bien la distinction entre les deux...