2010-04-15 5 views
7

Considérez le code suivant:problèmes de compilation avec le vecteur <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; 
} 

Lorsque vous essayez de le compiler, je reçois l'erreur suivante du compilateur obscur 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 

Il me semble qu'il y est une sorte de problème avec consts ici. Cela signifie-t-il que auto_ptr ne peut pas être utilisé dans vector s?

+11

N'utilisez jamais 'auto_ptr' dans un conteneur. C'est une chose malheureuse. :( – GManNickG

+2

Voir cette question pour savoir pourquoi 'auto_ptr' ne peut pas être utilisé avec des conteneurs STL: http://stackoverflow.com/questions/111478/why-is-it-wrong-to-use-stdauto-ptr-with-stl -containers – Naveen

+0

Pourquoi n'utilisez-vous pas la recherche? Existe d'autres questions avec ces idées –

Répondre

18

Correct, std::auto_ptr ne peut pas être utilisé dans std::vector.

Ce que le compilateur se plaint, c'est que l'opérateur d'affectation pour auto_ptr modifie l'objet assigné, et ne peut donc pas être const.

Vous voulez utiliser soit boost::ptr_vector ou un vecteur de boost::shared_ptr s

+0

Merci, alors je vais pour boost :: shared_ptr, je l'ai déjà utilisé – petersohn

+1

En fait, si vous y avez accès, 'unique_ptr' est bien meilleur si vous n'avez pas besoin d'une propriété partagée, mais cela nécessite une sémantique de déplacement C++ 0x. –

5

auto_ptr a un constructeur de copie avec un paramètre non-const, de sorte que le compilateur ne peut pas l'appeler à partir vector::push_back() puisque celui-ci a paramètre const.

La raison en est lorsque vous initialisez un auto_ptr exemple d'une autre la nouvelle instance déconnecte l'objet de l'autre instance et il se connecte à l'auto afin d'éviter une situation de pointeur ballants quand une instance delete de l'objet et l'autre détient encore un pointeur à lui.

Questions connexes