memset para la inicialización en C++

Question

memset se usa a veces para inicializar datos en un constructor como el ejemplo a continuación. ¿Funciona en general? ¿Es una buena idea en general?

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

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

Answer 1

No utilizar memset. Es un holdover de C y no funcionará en los no POD. Específicamente, usarlo en una clase derivada que contenga funciones virtuales, o cualquier clase que contenga un componente no incorporado, provocará un desastre.

C++ proporciona una sintaxis específica para la inicialización:

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

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

Para ser honesto, no estoy seguro, pero memset también podría ser una mala idea de puntos flotantes, ya que su formato no está especificado.

Answer 2

Es una idea terrible. Simplemente está analizando los datos, sin prestar atención a cómo deberían inicializarse los objetos. Si tu clase es virtual, es probable que elimines el puntero vtable también.

memset funciona en datos sin procesar, pero C++ no se trata de datos sin procesar. C++ crea abstracciones, así que si quieres estar seguro usas esas abstracciones. Use la lista de inicializadores para inicializar miembros.

Usted puede hacerlo a tipos de POD:

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

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

Pero si lo estás haciendo en this, que significa que está en un constructor definido por el usuario y ya no califica como un tipo POD . (Aunque si todos sus miembros son POD, podría funcionar, siempre y cuando ninguno contenga 0 como valor de trampa. Estoy seguro de que no estoy seguro si entran en juego otras fuentes de comportamiento indefinido.)