Вы можете поставить Pimpl-класс в векторе

Question

У меня есть класс реализованный с использованием Pimpl Ideom:

class FooImpl {}; 

class Foo 
{ 
    unique_ptr<FooImpl> myImpl; 
public: 
    Foo(); 
    ~Foo(); 
}; 

А теперь я хочу, чтобы поместить это в станд :: вектор

void Bar() 
{ 
    vector<Foo> testVec; 
    testVec.resize(10); 
} 

Но когда я делаю это, я получаю ошибку компилятора (VC++ 2013)

ошибка C2280: «станд :: unique_ptr> :: unique_ptr (Const станд :: unique_ptr < _Ty, станд :: default_delete < _Ty >> &)»: попытка сослаться проколотой функцию

я получаю ту же ошибку с testVec.emplace_back(); и testVec.push_back(std::move(Foo()));

(В качестве временного решения, используя vector<unique_ptr<Foo>>, кажется, работает, но я не понимаю почему код выше не работает)

Рабочий пример:. http://coliru.stacked-crooked.com/a/b274e1209e47c604

Answer 1

Поскольку std::unique_ptr не является копируемыми, класс Foo не имеет допустимого конструктора копирования.

Вы можете либо deep copy or use a move constructor:

#include <memory> 
#include <vector> 

class FooImpl {}; 

class Foo 
{ 
    std::unique_ptr<FooImpl> myImpl; 
public: 
    Foo(Foo&& f) : myImpl(std::move(f.myImpl)) {} 
    Foo(){} 
    ~Foo(){} 
}; 

int main() { 
    std::vector<Foo> testVec; 
    testVec.resize(10); 
    return 0; 
} 

Живой пример: https://ideone.com/HYtPMu

Answer 2

Так что же происходит, что vector шаблон пытается получить доступ к копии конструктор Foo класса. Вы не предоставили его, поэтому компилятор пытается создать стандартную реализацию, которая вызывает конструктор копирования для всех членов. Поскольку у std::unique_ptr нет конструктора копирования из другого std::unique_ptr (что логично, поскольку он не знает, как скопировать объект), компилятор не может сгенерировать оператор присваивания для Foo, и он терпит неудачу. Так что вы можете сделать, это предоставить конструктор копирования для класса Foo и решить, как обращаться указатель:

#include <memory> 
#include <vector> 

using namespace std; 
class FooImpl {}; 

class Foo 
{ 
    unique_ptr<FooImpl> myImpl; 
public: 
    Foo() 
    { 
    } 
    ~Foo() 
    { 
    } 
    Foo(const Foo& foo) 
    { 
     // What to do with the pointer? 
    } 
    Foo& operator= (const Foo& foo) 
    { 
     if (this != &foo) 
     { 
      // What to do with the pointer? 
     } 
     return *this; 
    } 
}; 

int main(int argc, char** argv) 
{ 
    vector<Foo> testVec; 
    testVec.resize(10); 
    return 0; 
}