Comment déclarer un vecteur de unique_ptr en tant que membre de données de classe?

Question

Je voudrais avoir un vecteur de unique_ptr en tant que membre d'une classe que je fais.

class Foo { 
    [...] 

private: 
    vector<unique_ptr<Bar>> barList; 
} 

Mais je commence à recevoir des messages d'erreur énigmatiques du compilateur VS2010:

error C2248: 'std::unique_ptr<_Ty>::operator =' : cannot access private member declared in class 'std::unique_ptr<_Ty>' 

Avec une poignée de lignes d'erreur ci-dessous ce qui plongent dans la mise en œuvre de Microsoft std::_Copy_impl<> ...

J'ai changé la déclaration de membre en

vector<unique_ptr<Bar>>* barList; 

Et il compile. Mais je ne peux pas m'empêcher de me demander pourquoi je ne peux pas le faire comme je le voulais initialement? Pour des grimaces, j'ai essayé cela et il fonctionne très bien:

vector<Bar> barList; 

Mais maintenant, je perds la commodité de unique_ptr. Je veux mon gâteau et je veux le manger aussi!

Answer 1

Le problème ici est que quelque part, votre code tente d'appeler l'opérateur "copy-assignment" de Foo.

Cela provoque le compilateur pour tenter de générer un opérateur d'affectation de copie qui appelle les opérateurs d'affectation de copie de tous les sous-objets de Foo. Finalement, cela conduit à une tentative de copier un unique_ptr, une opération qui n'est pas possible.

Answer 2

Vous ne pouvez pas utiliser unique_ptr dans un vecteur car l'implémentation vectorielle repose fortement sur les valeurs assigner l'opérateur, qui est privé dans unique_ptr. Utilisez shared_ptr à partir de boost ou d'une autre implémentation smart ptr à partir de C++ 11.

Answer 3

unique_ptr n'a pas de sémantique de copie, donc vous ne pouvez pas utiliser de méthodes qui copieraient l'objet contenu. Vous pouvez faire cela avec des références rvalue en utilisant std::move dans les endroit (s) il essaie de faire une copie. Sans voir votre code je ne peux pas dire où cela serait.

S'il compile dans la deuxième forme, vous n'avez pas utilisé le même code ou il y a un bogue de compilateur. Les deux devraient échouer de la même manière. Votre troisième exemple, le stockage par valeur est le moyen le plus simple à moins que vos objets ne soient grands et coûteux à stocker/copier par valeur.

Answer 4

Il manque souvent un std::move(iUniquePtr) quelque part (par exemple lorsque vous utilisez push_back).

Answer 5

extraits Un de www.cplusplus.com

std::unique_ptr::operator= 

cession unique_ptr L'objet acquiert la propriété du contenu de x, y compris à la fois le pointeur stocké et le Deleter stocké (ainsi que la responsabilité de supprimer l'objet à un moment donné). Tout objet appartenant à l'objet unique_ptr avant l'appel est supprimé (comme si le destructeur de unique_ptr était appelé).

Mais il y a un avertissement aussi:

Cette page décrit une fonctionnalité introduite par la dernière révision de la norme C++ (2011). Les compilateurs plus anciens peuvent ne pas le supporter.

MSVC 2010 définit comme operator= privée (non copiable), mais prend en charge la méthode swap.