Errore durante la dichiarazione di una classe con std :: vector di struct contenenti std :: unique_ptr

Question

Sebbene abbia lavorato per diversi anni con C#, fare qualcosa in C++ è ancora difficile per me. Approvo pienamente l'uso di puntatori intelligenti, ma ora mi trovo di fronte al seguente puzzle

Ho una struttura Foo, ad es.

struct Foo 
{ 
    Foo(std::unique_ptr<Bar> bar) : m_myBar(std::move(bar)) {} 

private: 
    std::unique_ptr<Bar> m_myBar;   
}; 

in una classe diversa, voglio avere un vettore che contiene le istanze di Foo, ma la seguente riga

std::vector<Foo> m_Foos; 

rendimenti errori di compilazione dicendo che il costruttore di copia viene eliminata. Nel thread SO "Why can I not push_back a unique_ptr into a vector?" viene fornita una spiegazione e un rimedio. Tuttavia, la domanda riguarda un vettore di puntatori univoci, mentre io ho un vettore di strutture contenente un puntatore univoco. La soluzione suggerita è usare la semantica del movimento, ma come si applica nella mia situazione? O dovrei fare qualcos'altro?

Answer 1

Come dici tu, m_Foos è in realtà un membro di dati di un'altra classe (lo chiamerò FooHolder). Se non hai fornito un costruttore di copia per FooHolder, il compilatore ne genererà uno automaticamente. Il costruttore di copia chiamerà i costruttori di copia di tutti i membri di dati di FooHolder, incluso m_Foos. Naturalmente, il costruttore di copie di std::vector<Foo> contiene un errore di compilazione, dal momento che Foo non è copiabile. Questo è probabilmente il motivo per cui stai ricevendo l'errore.

È necessario fornire un costruttore di copia appropriato per FooHolder, se è possibile e desidera che tale classe sia copiabile. In caso contrario, si può semplicemente dichiarare un costruttore di movimento (possibilmente in default), che renderà il costruttore di copia eliminata:

struct FooHolder 
{ 
    FooHolder(FooHolder&&) = default; 

private: 
    std::vector<Foo> m_Foos; 
}; 

Answer 2

Non è possibile copiare puntatori univoci. Si può solo spostare loro:

Foo(std::unique_ptr<Bar> bar) : m_myBar(std::move(bar)) {} 
//          ^^^^^^^^^^^^^^