Delegat Konstruktor C++

Question

Czy robię to dobrze? Próbuję delegować konstruktor klasy C++, ponieważ jest to zasadniczo ten sam kod powtarzający się 3 razy. Czytam w C++ x11 i czytam, że g ++ 4.7.2 pozwala na to, ale nie jestem pewien, czy robię to po prawej:

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

czy trzeba zrobić:

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

trzeba zrobić drugi. Delegowanie konstruktorów działa tylko na liście inicjalizacyjnej konstruktora, w przeciwnym razie po prostu utworzymy tymczasowy lub wykonamy inne błędy, o których wspomniałeś.

Answer 2

Poprawna składnia jest

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

    char _x; 
    int _y; 
}; 

Twój pierwszy przykład tworzy tymczasowy, który jest niszczony od razu.

Answer 3

Drugi przykład z użyciem listy inicjalizacyjnej jest prawidłowy. Pierwszy przykład kończy się tworzeniem tymczasowego obiektu.