2010-04-21 15 views
5
#include <vector> 
#include <memory> 

using namespace std; 

class A { 
public: 
    A(): i(new int) {} 
    A(A const& a) = delete; 
    A(A &&a): i(move(a.i)) {} 

    unique_ptr<int> i; 
}; 

class AGroup { 
public: 
    void     AddA(A &&a) { a_.emplace_back(move(a)); } 

    vector<A> a_; 
}; 

int main() { 
    AGroup ag; 
    ag.AddA(A()); 
    return 0; 
} 

ne compile pas ... (dit que ce constructeur de unique_ptr copie est supprimé)Comment puis-je obtenir ce code impliquant unique_ptr à compiler?

J'essayé de remplacer avec mouvement vers l'avant. Je ne sais pas si je l'ai bien fait, mais cela n'a pas fonctionné pour moi.


[~/nn/src] g++ a.cc -o a -std=c++0x 
/opt/local/include/gcc44/c++/bits/unique_ptr.h: In member function 'A& A::operator=(const A&)': 
a.cc:6: 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> >, _Args&& ...) [with _Args = A, _Tp = A, _Alloc = std::allocator<A>]' 
/opt/local/include/gcc44/c++/bits/vector.tcc:100: instantiated from 'void std::vector<_Tp, _Alloc>::emplace_back(_Args&& ...) [with _Args = A, _Tp = A, _Alloc = std::allocator<A>]' 
a.cc:17: instantiated from here 
/opt/local/include/gcc44/c++/bits/unique_ptr.h:219: error: deleted function 'std::unique_ptr<_Tp, _Tp_Deleter>& std::unique_ptr<_Tp, _Tp_Deleter>::operator=(const std::unique_ptr<_Tp, _Tp_Deleter>&) [with _Tp = int, _Tp_Deleter = std::default_delete<int>]' 
a.cc:6: error: used here 
In file included from /opt/local/include/gcc44/c++/vector:69, 
       from a.cc:1: 
/opt/local/include/gcc44/c++/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> >, _Args&& ...) [with _Args = A, _Tp = A, _Alloc = std::allocator<A>]': 
/opt/local/include/gcc44/c++/bits/vector.tcc:100: instantiated from 'void std::vector<_Tp, _Alloc>::emplace_back(_Args&& ...) [with _Args = A, _Tp = A, _Alloc = std::allocator<A>]' 
a.cc:17: instantiated from here 
/opt/local/include/gcc44/c++/bits/vector.tcc:314: note: synthesized method 'A& A::operator=(const A&)' first required here 
+0

Veuillez inclure l'erreur. –

+0

fait. (gcc 4.4.0) –

Répondre

3

probablement votre bibliothèque standard n'a pas (encore) définissent unique_ptr<T>::unique_ptr(unique_ptr &&). J'ai vérifié mes en-têtes dans 4.5 et c'est là, alors peut-être essayer de mettre à jour.

Quand il ne trouve pas le constructeur de mouvement, il rechercherait le constructeur de copie et trouver supprimé.

je reçois d'autres erreurs quand je compile, cependant.

EDIT: eu à travailler. Je ne comprends pas pourquoi vous devez move un objet qui est déjà une référence rvalue, mais vous. Le seul problème était un opérateur d'assignation manquant.

#include <vector> 
#include <memory> 

using namespace std; 

class A { 
public: 
    A(): i(new int) {} 
    A(A const& a) = delete; 
    A &operator=(A const &) = delete; 
    A(A &&a): i(move(a.i)) {} 
    A &operator=(A &&a) { i = move(a.i); } 

    unique_ptr<int> i; 
}; 

class AGroup { 
public: 
    void     AddA(A &&a) { a_.emplace_back(move(a)); } 

    vector<A> a_; 
}; 

int main() { 
    AGroup ag; 
    ag.AddA(A()); 
    return 0; 
} 
+0

Wow! Je suis tellement reconnaissant pour votre solution. Cela me frustrait infiniment. –

+0

Testé votre solution et cela fonctionne. Je souhaite vraiment que l'opérateur = par défaut à l'aide du constructeur de copie correspondant. Peut-être avec des «concepts d'auto», ils le feront? –

+1

Vous en savez probablement plus à ce sujet que moi; J'ai résolu ceci en étant un jockey d'erreur-message. – Potatoswatter