C++: ¿Debo inicializar los miembros del puntero asignados en el cuerpo del constructor a NULL?

Question

Supongamos que tengo:

// MyClass.h 
class MyClass 
{ 
    public: 
    MyClass(); 

    private: 
    Something *something_; 
} 

// MyClass.cpp 
MyClass::MyClass() 
{ 
    something_ = new Something(); 
} 

¿Debo inicializar something_ a NULL (o 0) en la lista de inicialización del constructor del constructor MiClase? ¿O no es necesario porque lo estoy asignando en el cuerpo del constructor? ¿Cuál es la práctica recomendada?

Answer 1

Generalmente sólo se asigne una vez, en la lista de inicialización o el cuerpo, a menos que la inicialización del cuerpo puede o no puede ocurrir, o tiene código de condición:

MyClass::MyClass() 
{ 
    //this code must happen first 
    // _now_ max is known 
    something_ = new Something(max); 
} 

MyClass::MyClass() 
{ 
    if (max) 
     something_ = new Something(max); 
    else 
     something_ = NULL; 
} 

MyClass::MyClass() 
    : something_(new Something()) 
//as pointed out in the comments, use smart pointers 
{ 
} 

Answer 2

En general, no. Pero si en algún lugar de su puntero constructor se utiliza antes de que se haya inicializado, entonces obtendrá un comportamiento indefinido. Sin embargo, el mayor problema que tienes es este: si se lanza una excepción en el constructor, no se llama a su destructor. Imagine que tiene dos punteros a objetos y la asignación del primer objeto tiene éxito mientras que la segunda asignación falla, en ese caso termina con una pérdida de recursos. Esto se puede resolver usando punteros "inteligentes". Pero en ese caso se inicializarán en la lista de inicialización, y si no desea asignarles valor dos veces, es mejor que asignen memoria allí que en el cuerpo del constructor.

Answer 3

No es necesario, especialmente si se inicializa inmediatamente el cuerpo constructor; sin embargo, si la inicialización no es tan obvia, ¿por qué no NULL para evitar el acceso accidental a la variable no inicializada?

• mínima sobrecarga de rendimiento
• ahorra mucho tiempo/solución de problemas con la depuración de la caza de errores loca