c++ - 安全的 std::tr1::shared_ptr 用法

Question

这种方法不安全吗？

#include <tr1/memory>

Foo * createFoo()
{
  return new Foo(5);
}

int main()
{
  std::tr1::shared_ptr<Foo> bar(create());

  return 0;
}

或者 createFoo 会更好吗？返回 shared_ptr<Foo>对象？

Answer 1

您的示例按照您编写的方式是安全的。但是，您可以通过让工厂方法 createFoo() 返回一个自动指针而不是原始指针来使其更加防漏。这样你就可以保证不会有泄漏。

那么你得到的是:

#include <memory>
#include <tr1/memory>

std::auto_ptr<Foo> createFoo()
{
  return std::auto_ptr<Foo>(new Foo(5));
}

int main()
{
  std::tr1::shared_ptr<Foo> bar(createFoo());

  return 0;
}

当然也可以让您的工厂方法返回一个 shared_ptr，但这可能被视为矫枉过正，因为返回的指针通常很快就会超出范围，因为它将在赋值或构造函数中使用。此外，使用 auto_ptr 可以更清楚地说明指针的预期用途，当不熟悉您的代码的人必须理解它时，这总是一个加号。