Discussion :

[NicolasJolet]

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 :

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

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

[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.

[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 ?

[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...

[Matthieu Brucher]

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.

[Invité]

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 :

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

1
2
3
MainWindow::MainWindow() {
    btnEssai(*this) ;// Pour connaître la fenêtre parente
}

[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...

[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

[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ù.