如何使用 unique_ptr 作为需要基类作为参数的函数的参数

Question

我正在学习使用托管指针，特别是 unique_ptr。

我已经实现了这些简单的类，但是函数 makeAnimalNoise() 不起作用，即使我认为我传递了正确的参数。

函数 makeAnimalNoise() 有什么问题？这个函数的正确参数应该是什么？还是定义本身有误？

class Animal{
public:
  Animal(){
    cout<<"An animal is born"<<endl;
  }
  virtual ~Animal(){
    cout<<"An animal dies"<<endl;
  }
  virtual void doAnimalStuff()=0;
};

class Dog: public Animal{
public:
  Dog(){
    cout<<"A dog is born"<<endl;
  }
  ~Dog() {
    cout<<"A dog dies"<<endl;
  }
  void doAnimalStuff() override {
    cout<<"I'm a dog"<<endl;
  }
};

class Cat: public Animal{
public:
 Cat(){
   cout<<"A cat is born"<<endl;
 }
~Cat() {
   cout<<"A cat dies"<<endl;
 }
void doAnimalStuff() override {
  cout<<"I'm a cat"<<endl;
 }
};

void makeAnimalNoise(unique_ptr<Animal>an){
  an->doAnimalStuff();
}

int main(){
  unique_ptr<Animal>Dog=unique_ptr<Dog>(new Dog());
  unique_ptr<Animal>Cat=unique_ptr<Cat>(new Cat());
  makeAnimalNoise(Dog); //doesn't work here
  makeAnimalNoise(Cat); //doesn't work here
}

Answer 1

makeAnimalNoise的参数声明为传值；但是std::unique_ptr不能被复制；你可以像这样移动它

unique_ptr<Animal> dog(new Dog);
unique_ptr<Animal> cat(new Cat);
makeAnimalNoise(std::move(dog));
makeAnimalNoise(std::move(cat));

Answer 2

您不想将所有权转移到您的 makeAnimalNoise 函数中。智能指针只能由责任的所有者使用；你绝对应该不用智能指针替换all你的代码。

你的函数应该只是一个参考：

void makeAnimalNoise(Animal& an) {
  an.doAnimalStuff();
}

在通话现场：

makeAnimalNoise(*Dog);
makeAnimalNoise(*Cat);

重复一遍：只有当指针的所有权是 API 的主体时，智能指针才应该出现在 API 中。当只对被指针的值感兴趣时，使用引用或普通指针参数。

换一种说法：智能指针是一种管理职责的工具，而不是用于传递值或实现引用语义的工具。