带有 unique_ptr-s 的向量

Question

我有这样的代码：

#include <memory>
#include <vector>

namespace daq
{

class Animal
{
public:
    Animal(){};
};

class Pig : public Animal
{
public:
    Pig() : Animal () {};
};

class Farm
{
public:
    void addAnimal(Animal& animal)
    {
        mAnimals.push_back(std::unique_ptr<Animal>(animal)); // error
    }

private:
    std::vector<std::unique_ptr<Animal>> mAnimals;
};
} /* namespace daq */

但我在方法 Farm::addAnimal: 中遇到错误：

没有匹配函数调用'std::unique_ptr::unique_ptr(daq::Animal&)'

我应该将什么传递给 push_back 方法？

Answer 1

unique_ptr 接受 pointer 作为构造函数参数，但您传递的是 reference。这基本上是编译器告诉你的：你不能从 daq::Animal& 构造 std::unique_ptr。

您可以将原始指针传递给使用new 分配的Animal 类型的对象，或者（最好）您应该传递以这种方式构造的unique_ptr 并在提供时从它移开它作为push_back()的参数：

void addAnimal(std::unique_ptr<Animal>&& animal)
//                                    ^^
//                                    This is to make it absolutely clear that
//                                    your intention is to bind to an object
//                                    the client wants to move from. It is not
//                                    especially needed here (unique_ptr is not
//                                    copyable), but it makes your interface
//                                    more explicit about it. [Credits to sehe]
{
    mAnimals.push_back(std::move(animal)); // OK
}

int main()
{
    daq::Farm farm;
    std::unique_ptr<daq::Animal> pig(new daq::Pig());
    farm.addAnimal(std::move(pig)); // OK
}

您必须在此处使用std::move()，因为unique_ptrs 是不可复制的，因此您有效地将pig 的所有权从调用它的例程转移到包含它的向量（addAnimal() 是中间人）。