Przydział do inicjalizacji w C++

Question

Plik memset jest czasami używany do inicjowania danych w konstruktorze, jak w przykładzie poniżej. Czy to działa ogólnie? Czy to ogólnie dobry pomysł?

class A { 
public: 
    A(); 
private: 
    int a; 
    float f; 
    char str[35]; 
    long *lp; 
}; 

A::A() 
{ 
    memset(this, 0, sizeof(*this)); 
} 

Answer 1

Nie używaj memset. Jest to zatrzymanie z C i nie będzie działać na urządzeniach innych niż POD. W szczególności użycie go w klasie pochodnej, która zawiera jakiekolwiek funkcje wirtualne - lub dowolna klasa zawierająca nie-wbudowaną - spowoduje katastrofę.

C++ zapewnia specyficzną składnię dla inicjalizacji:

class A { 
public: 
    A(); 
private: 
    int a; 
    float f; 
    char str[35]; 
    long *lp; 
}; 

A::A() 
    : a(0), f(0), str(), lp(NULL) 
{ 
} 

Szczerze mówiąc, nie jestem pewien, ale memset może również być zły pomysł na pływających punktów od ich format jest nieokreślona.

Answer 2

To okropny pomysł. Po prostu miażdżysz dane, nie zwracając uwagi na sposób inicjowania obiektów. Jeśli twoja klasa jest wirtualna, prawdopodobnie zlikwidujesz również wskaźnik vtable.

memset działa na surowych danych, ale w C++ nie chodzi o surowe dane. C++ tworzy abstrakcje, więc jeśli chcesz być bezpieczny, korzystasz z tych abstrakcji. Użyj listy inicjalizatora, aby zainicjować członków.

Ty może zrobić typów POD:

struct nothing_fancy_here 
{ 
    bool b; 
    int i; 
    void* p; 
}; 

nothing_fancy_here x; 
memset(&x, 0, sizeof(x)); 

Ale jeśli robisz to na this, co oznacza, że ​​jesteś w konstruktora zdefiniowanego przez użytkownika i nie kwalifikują się jako typ POD . (Chociaż jeśli wszyscy twoi członkowie to POD, może to zadziałać, o ile żadna z nich nie zawiera wartości pułapki 0, jestem pewien, że nie ma tu zastosowania żadne inne źródło niezdefiniowanych zachowań.)