¿Es malo usar auto_ptr con el nuevo char [n]

Question

Si declaro un búfer de caracteres borrado automático temporal utilizando

std::auto_ptr<char> buffer(new char[n]); 

continuación, el búfer se elimina automáticamente cuando el buffer se sale del ámbito. Supongo que el buffer se borra usando delete.

Sin embargo, el búfer se creó utilizando new [], y por lo tanto estrictamente hablando, el búfer debe eliminarse utilizando delete [].

¿Qué posibilidad hay de que esta falta de coincidencia pueda causar una pérdida de memoria?

Answer 1

El comportamiento de invocar eliminar en un puntero asignado con new [] es undefined. Como suponía, auto_ptr does call delete cuando el puntero inteligente se sale del alcance. No se trata solo de pérdidas de memoria de las que tiene que preocuparse: son posibles los bloqueos y otros comportamientos extraños.

Si no necesita transferir la propiedad del puntero, la clase scoped_array de Boost puede ser lo que está buscando.

Answer 2

Eso arroja un comportamiento indefinido (podría ser peor que la pérdida de memoria, por ejemplo, la corrupción del montón) intente boost's scoped_array or shared_array en su lugar.

Answer 3

La llamada a eliminar en los datos asignados con la nueva [] no está definida. Esto significa que el compilador puede generar código que puede hacer cualquier cosa. Sin embargo, en este caso, probablemente funcione, ya que no es necesario destruir los caracteres individuales de la matriz, solo la matriz misma.

Aún como este comportamiento no está definido, recomendaría utilizar std::vector<char> o boost::scoped_array<char>/boost::shared_array<char>. Todas son opciones perfectamente viables y superiores para usar std::auto_ptr<> en este caso. Si usa std::vector, también tiene la posibilidad de aumentar dinámicamente el búfer si es necesario.

Answer 4

Usaría un vector de char como buffer.

std::vector<char> buffer(size); 

read(input,&buffer[0],size); 

Básicamente, no desea llamar a nuevo si no es necesario.
Un vector proporciona un búfer de tiempo de ejecución que puede usar como una matriz (búfer).

La mejor parte es que el vector se limpia después de sí mismo y el estándar garantiza que todos los elementos en el vector estarán en contigious storage. Perfecto para un buffer.

O más formaly la garantía es:

(&buffer[0]) + size == (&buffer[size]) 

Answer 5

¿Hay una buena razón para no usar std :: string? std :: vector, como otros han sugerido? Lo que estás haciendo está mal, pero sin saber lo que estás tratando de hacer es más difícil recomendar otra cosa.

Answer 6

Esto parece tremendamente complejo para una solución muy simple. ¿Qué le pasa usando

char *c=new char[n] 

aquí y luego eliminándolo? O, si necesita una solución un poco más dinámica,

vector<char> c 

navaja de Occam, hombre.:-)

Answer 7

Sí, está mal. Envuelva con una envoltura trivial.

typedef< typename T_ > 
struct auto_vec{ 
    T_* t_; 
    auto_vec(T_* t): t_(t) {} 
    ~auto_vec() { delete[] t_; } 
    T_* get() const { return t_; } 
    T_* operator->() const { return get(); } 
    T_& operator*() const { return *get(); } 
    /* you should also define operator=, reset and release, if you plan to use them */ 
} 

auto_vec<char> buffer(new char[n]); 

Answer 8

Han pasado unos años desde que se formuló la pregunta.

Pero llegué a esta página de una búsqueda, por lo que pensé que también podría señalar: std :: unique_ptr, la sustitución de C++ 11 para auto_ptr, puede manejar la eliminación de objetos creados con el nuevo [].

hay dos versiones de std :: unique_ptr: 1) Gestiona la vida útil de un único objeto (por ejemplo, asignado con nuevo) 2) Gestiona el tiempo de vida de un matriz asignada dinámicamente de objetos (por ejemplo, asignado con la nueva [])

cppreference unique_ptr