Constructeur délégué C ++

Question

Suis-je en train de faire ça? J'essaie de déléguer un constructeur de classe C ++ car c'est fondamentalement le même code qui se répète 3 fois .. J'ai lu sur C ++ x11 et lu que g ++ 4.7.2 le permet mais je ne sais pas si je le fais droite:

 Bitmap::Bitmap(HBITMAP Bmp)
{
   //Construct some bitmap stuff..
}

Bitmap::Bitmap(WORD ResourceID)
{
   HBITMAP BMP = (HBITMAP)LoadImage(GetModuleHandle(NULL), MAKEINTRESOURCE(ResourceID), IMAGE_BITMAP, 0, 0, LR_SHARED);

   Bitmap(BMP);   //Delegates to the above constructor? Or does this create a temporary?
}
 

OU dois-je faire:

 Bitmap::Bitmap(HBITMAP Bmp)
{
   //Construct some bitmap stuff..
}

Bitmap::Bitmap(WORD ResourceID) : Bitmap((HBITMAP)LoadImage(GetModuleHandle(NULL), MAKEINTRESOURCE(ResourceID), IMAGE_BITMAP, 0, 0, LR_SHARED))
{
}
 

Answer 1

La syntaxe correcte est

 struct Foo {
  Foo(char x, int y) {}
  Foo(int y) : Foo('a', y) {}
};
 

Votre premier exemple crée un temporaire qui est immédiatement détruit.

Answer 2

Vous devez faire la seconde. La délégation des constructeurs ne fonctionne que dans la liste d'initialisation du constructeur, sinon vous allez simplement créer un temporaire ou faire d'autres erreurs comme vous l'avez mentionné.

Answer 3

Le deuxième exemple utilisant la liste d'initialisation est le bon. Le premier exemple va finir par créer un objet temporaire.