保持从工厂 Create 方法返回的 shared_ptr 的最佳方法

Question

我有一个作用域保护对象工厂，它返回一个 shared_ptr 给我想在作用域期间保留的保护。

工厂的Create方法声明为：

virtual boost::shared_ptr<GenericGuard> Create() const = 0;

我的问题是——写起来更好吗：

boost::shared_ptr<SelectorGenericGuard> guard = factory->Create();

或：

const boost::shared_ptr<SelectorGenericGuard>& guard = factory->Create();

第一个版本可以/可能/将创建shared_ptr 的副本，因此可以/可能/将导致额外的引用计数器增量 - 第二个版本显然可读性较差，我无法确定我的“默认”应该是 - 所有被认为可读性的东西通常是我最喜欢的，我想听听其他人的想法。

或者我应该在这里使用auto_ptr（将来是unique_ptr）并免费使用第一个更易读的表格吗？ （这里唯一的缺点是使用auto_ptr ;)）

Answer 1

很可能两者都会产生完全相同的代码：

1. 这会将返回值绑定到常量引用。
   没有优化的理论开销：一个引用更多的内存，访问的额外间接。
   实际上：在 as-if 规则下优化掉开销。

   const boost::shared_ptr<SelectorGenericGuard>& guard = factory->Create();
   

2. 这通过初始化一个变量来保存返回值。
   没有优化的理论开销：一个动作和一个 dtor
   实际上：根据复制省略规则（as-if 规则除外）优化掉开销。
   增加的危险：只有在类型完全匹配时才能进行优化。

   boost::shared_ptr<SelectorGenericGuard> guard = factory->Create();
   

3. 与 2 相同，但不可能出现类型不匹配且更短（我的首选）：

   auto guard = factory->Create();
   

关于auto_ptr:
总是喜欢unique_ptr。
如果不能，请注意陷阱： 将语义移到写入副本的位置，从而导致标准算法和容器失败。考虑定义自己的不可复制版本，并在需要的任何地方进行明确的移动。将其保留在 ABI 之外，以便以后替换：函数参数（和异常类型）。

关于unique_ptr：如果可以的话，unique_ptr 比shared_ptr 更可取，因为可以减少（说：否）开销。