2010-04-15 7 views
7

Considere el siguiente código:problemas de compilación con el vector <auto_ptr<>>

#include <iostream> 
#include <memory> 
#include <vector> 

using namespace std; 

struct A 
{ 
    int a; 
    A(int a_):a(a_) {} 
}; 

int main() 
{ 
    vector<auto_ptr<A> > as; 
    for (int i = 0; i < 10; i++) 
    { 
     auto_ptr<A> a(new A(i)); 
     as.push_back(a); 
    } 
    for (vector<auto_ptr<A> >::iterator it = as.begin(); it != as.end(); ++it) 
     cout << (*it)->a << endl; 
} 

Al tratar de compilarlo, me sale el siguiente error del compilador oscura desde g ++:

g++ -O0 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/proba.d" -MT"src/proba.d" -o"src/proba.o" "../src/proba.cpp" 
/usr/include/c++/4.1.2/ext/new_allocator.h: In member function ‘void __gnu_cxx::new_allocator<_Tp>::construct(_Tp*, const _Tp&) [with _Tp = std::auto_ptr<A>]’: 
/usr/include/c++/4.1.2/bits/stl_vector.h:606: instantiated from ‘void std::vector<_Tp, _Alloc>::push_back(const _Tp&) [with _Tp = std::auto_ptr<A>, _Alloc = std::allocator<std::auto_ptr<A> >]’ 
../src/proba.cpp:19: instantiated from here 
/usr/include/c++/4.1.2/ext/new_allocator.h:104: error: passing ‘const std::auto_ptr<A>’ as ‘this’ argument of ‘std::auto_ptr<_Tp>::operator std::auto_ptr_ref<_Tp1>() [with _Tp1 = A, _Tp = A]’ discards qualifiers 
/usr/include/c++/4.1.2/bits/vector.tcc: In member function ‘void std::vector<_Tp, _Alloc>::_M_insert_aux(__gnu_cxx::__normal_iterator<typename std::_Vector_base<_Tp, _Alloc>::_Tp_alloc_type::pointer, std::vector<_Tp, _Alloc> >, const _Tp&) [with _Tp = std::auto_ptr<A>, _Alloc = std::allocator<std::auto_ptr<A> >]’: 
/usr/include/c++/4.1.2/bits/stl_vector.h:610: instantiated from ‘void std::vector<_Tp, _Alloc>::push_back(const _Tp&) [with _Tp = std::auto_ptr<A>, _Alloc = std::allocator<std::auto_ptr<A> >]’ 
../src/proba.cpp:19: instantiated from here 
/usr/include/c++/4.1.2/bits/vector.tcc:256: error: passing ‘const std::auto_ptr<A>’ as ‘this’ argument of ‘std::auto_ptr<_Tp>::operator std::auto_ptr_ref<_Tp1>() [with _Tp1 = A, _Tp = A]’ discards qualifiers 
/usr/include/c++/4.1.2/bits/stl_construct.h: In function ‘void std::_Construct(_T1*, const _T2&) [with _T1 = std::auto_ptr<A>, _T2 = std::auto_ptr<A>]’: 
/usr/include/c++/4.1.2/bits/stl_uninitialized.h:86: instantiated from ‘_ForwardIterator std::__uninitialized_copy_aux(_InputIterator, _InputIterator, _ForwardIterator, __false_type) [with _InputIterator = __gnu_cxx::__normal_iterator<std::auto_ptr<A>*, std::vector<std::auto_ptr<A>, std::allocator<std::auto_ptr<A> > > >, _ForwardIterator = __gnu_cxx::__normal_iterator<std::auto_ptr<A>*, std::vector<std::auto_ptr<A>, std::allocator<std::auto_ptr<A> > > >]’ 
/usr/include/c++/4.1.2/bits/stl_uninitialized.h:113: instantiated from ‘_ForwardIterator std::uninitialized_copy(_InputIterator, _InputIterator, _ForwardIterator) [with _InputIterator = __gnu_cxx::__normal_iterator<std::auto_ptr<A>*, std::vector<std::auto_ptr<A>, std::allocator<std::auto_ptr<A> > > >, _ForwardIterator = __gnu_cxx::__normal_iterator<std::auto_ptr<A>*, std::vector<std::auto_ptr<A>, std::allocator<std::auto_ptr<A> > > >]’ 
/usr/include/c++/4.1.2/bits/stl_uninitialized.h:254: instantiated from ‘_ForwardIterator std::__uninitialized_copy_a(_InputIterator, _InputIterator, _ForwardIterator, std::allocator<_Tp>) [with _InputIterator = __gnu_cxx::__normal_iterator<std::auto_ptr<A>*, std::vector<std::auto_ptr<A>, std::allocator<std::auto_ptr<A> > > >, _ForwardIterator = __gnu_cxx::__normal_iterator<std::auto_ptr<A>*, std::vector<std::auto_ptr<A>, std::allocator<std::auto_ptr<A> > > >, _Tp = std::auto_ptr<A>]’ 
/usr/include/c++/4.1.2/bits/vector.tcc:279: instantiated from ‘void std::vector<_Tp, _Alloc>::_M_insert_aux(__gnu_cxx::__normal_iterator<typename std::_Vector_base<_Tp, _Alloc>::_Tp_alloc_type::pointer, std::vector<_Tp, _Alloc> >, const _Tp&) [with _Tp = std::auto_ptr<A>, _Alloc = std::allocator<std::auto_ptr<A> >]’ 
/usr/include/c++/4.1.2/bits/stl_vector.h:610: instantiated from ‘void std::vector<_Tp, _Alloc>::push_back(const _Tp&) [with _Tp = std::auto_ptr<A>, _Alloc = std::allocator<std::auto_ptr<A> >]’ 
../src/proba.cpp:19: instantiated from here 
/usr/include/c++/4.1.2/bits/stl_construct.h:81: error: passing ‘const std::auto_ptr<A>’ as ‘this’ argument of ‘std::auto_ptr<_Tp>::operator std::auto_ptr_ref<_Tp1>() [with _Tp1 = A, _Tp = A]’ discards qualifiers 
make: *** [src/proba.o] Error 1 

Me parece que hay es algún tipo de problema con las constras aquí. ¿Esto significa que auto_ptr no se puede utilizar en vector s?

+11

Nunca use un 'auto_ptr' en un contenedor. Es una cosa desafortunada :( – GManNickG

+2

Consulte esta pregunta sobre por qué 'auto_ptr' no se puede usar con contenedores STL: http://stackoverflow.com/questions/111478/why-is-it-wrong-to-use-stdauto-ptr-with-stl -contenedores – Naveen

+0

¿Por qué no utiliza la búsqueda? Existen otras preguntas con estas ideas. –

Respuesta

18

Correcto, std::auto_ptr no se puede utilizar en std::vector.

Lo que el compilador se queja es que el operador de asignación para auto_ptr cambia el objeto asignado, por lo que no puede ser const.

que desea utilizar ya sea boost::ptr_vector o un vector de boost::shared_ptr s

+0

Gracias. Luego iré por boost :: shared_ptr, lo he usado antes. – petersohn

+1

En realidad, si tiene acceso, 'unique_ptr' es mucho mejor si no necesita propiedad compartida. Sin embargo, necesita una semántica de movimiento C++ 0x. –

5

auto_ptr tiene un constructor de copia con un parámetro no constante, por lo que el compilador no puede llamarlo desde vector::push_back() ya que este último tiene parámetro const.

La razón es al inicializar una auto_ptr ejemplo de otro la nueva instancia se desconecta el objeto de la otra instancia y lo conecta a uno mismo de modo de evitar una situación de referencia colgante cuando una instancia delete s el objeto y el otro todavía mantiene un puntero lo.

Cuestiones relacionadas