传递 boost::shared_ptr 引用的安全性

Question

我的问题是，以下代码片段是我的资源管理器的一部分，它的故障安全性如何：

 bool Load(std::string name, boost::shared_ptr<Asset::Model>& newModel)
 {
  std::map<std::string, boost::scoped_ptr<Asset::Model> >::const_iterator seeker;

  seeker = models.find(name);
  if (seeker == models.end())
   return false;

  newModel = seeker->second->Copy(); // Copy returns a boost::shared_ptr<Asset::Model>
  return true;
 }

private:
 std::map< std::string, boost::scoped_ptr<Asset::Model> >  models;

因为通过引用传递boost的shared_ptr实际上不是shared_ptr概念的一部分，如果我只在这个范围内使用它，我会遇到麻烦吗？

Answer 1

这种用法是安全的，因为无论通过引用传入的 shared_ptr<> 都会减少其引用计数（假设从 返回的 shared_ptr<> seer->second->Copy() 不是同一对象的 shared_ptr），因此它指向的对象可能会被删除。

具体来说，您不会从原始指针创建第二个 shared_ptr<> （这是一个错误，因为它将创建第二个不相关的 shared_ptr<> ）单独的引用计数，因此是对象的第二个所有者）。

您的函数是否提供您想要的行为取决于您想要什么。

Answer 2

我猜你想做这样的事情：

boost::shared_ptr<Asset::Model> ptr;
if(Load("stuff", ptr))
{
     doSomething with ptr
}

在这种情况下你应该没问题。

但是您不需要在此处使用参考。如果没有找到元素，只需返回一个 shared_ptr 并将其设置为 0：

 boost::shared_ptr<Asset::Model> Load(std::string name)
 {
  std::map<std::string, boost::scoped_ptr<Asset::Model> >::const_iterator seeker;
  boost::shared_ptr<Asset::Model> retPtr;

  seeker = models.find(name);
  if (seeker == models.end())
   return retPtr;

  retPtr = seeker->second->Copy(); // Copy returns a boost::shared_ptr<Asset::Model>
  return retPtr;
 }

但是我注意到您在标准容器中使用 scoped_ptr - AFAIK 这是不可能的。