使用共享指针加载到变量 C++ 中

Question

这是

合理的做法吗？

void MyFunc(std::shared_ptr<MyClass>* p) {
  // calculate a, b, c
  p->reset(new MyClass(a, b, c));
}
int main() {
  std::shared_ptr<MyClass> ptr;
  MyFunc(&ptr);
  // do some other work with ptr
  return 0;
}

我读到不建议使用原始指针，

只建议使用智能指针，但在我的函数参数中，基本上我使用的是原始指针。这是一个不好的做法

Answer 1

更正确的方法是：

std::shared_ptr<MyClass> MyFunc()
{
  // calculate a, b, c
  return std::make_shared<MyClass>(a, b, c);
}

这更具可读性，并生成更短、更干净的代码：

int main() 
{
  auto ptr = MyFunc();
  // do some other work with ptr
  return 0;
}

如果可能的话，读者应该清楚函数的意图是什么，只是查看它的名称、参数和返回类型。在您的情况下，有太多问题：

- why do I need to pass `std::shared_ptr` to the function?
- what will happen when that ptr is equal to `nullptr`?
- will this function use object passed in `std::shared_ptr` if it points to one?

当它返回std::shared_ptr时，这些问题都不存在。"函数在内部创建对象并返回共享所有权" - 这对于每个只研究函数签名的C++程序员来说都很清楚。

Answer 2

很少有充分的理由通过指针传递智能指针。改为通过引用传递它：

void MyFunc(std::shared_ptr<MyClass>& p)
{
    // calculate a, b, c
    p.reset(new MyClass(a, b, c));
}
int main()
{
    std::shared_ptr<MyClass> ptr;
    MyFunc(ptr);
    // do some other work with ptr
    return 0;
}

或者，在此示例中，只需更改函数以在返回值而不是输出参数中传递它：

std::shared_ptr<MyClass> MyFunc()
{
    // calculate a, b, c
    return std::shared_ptr<MyClass>(new MyClass(a, b, c));
    // or:
    // return std::make_shared<MyClass>(a, b, c);
}
int main()
{
    std::shared_ptr<MyClass> ptr = MyFunc();
    // do some other work with ptr
    return 0;
}

Answer 3

如果我们假设两件事：

1. 您希望遵循 Google C++ 风格指南的建议，说明输出参数应该是指针，
2. p 是输出参数或输入/输出参数，不能作为函数的返回。

我会说这是有道理的。

建议首选智能指针而不是原始指针，以使所有权更清晰且不易出错。但是在这里，您使用原始指针代替引用，引用不是问题。因此，在我看来这似乎是合理的，特别是因为原始指针的使用仅限于"手工制作"引用的情况。