Question

这是我的代码：

typedef map<string, unique_ptr<MyClass> > DB;

const unique_ptr<MyClass>>& find_it(const string &key, const DB &db)
{
  auto itr = db.find(key);
  if (itr == db.end()) {
    return nullptr;
  }
  return itr->second;
}

返回语句会导致编译器警告：returning reference to local temporary object [-Wreturn-stack-address].

是的，我可以理解返回对本地临时变量的引用是不好的，但我想知道这里最简单的解决方法是什么：

1. Do not expose the map to the callers of find_it (I use typedef here is just for asking this question, in real code it is wrapped inside an interface).
2. Using an iterator kind of thing to indicate the position of the searched item is a hassle to me, I like to avoid it.

鉴于这些，我能提出的最好的方法是将find_it（）分解为2个函数：

bool exists(const string &key, const DB &db)
{
  auto itr = db.find(key);
  return itr != db.end();
}

const unique_ptr<MyClass>>& find_it(const string &key, const DB &db)
{
  auto itr = db.find(key);
  assert(itr != db.end());
  return itr->second;
}

有什么建议吗？

Answer 1

return nullptr语句隐式构建unique_ptr<MyClass>实例，然后您返回对它的引用，因此警告。一个简单的解决方法是定义一个static unique_ptr<MyClass>，其中包含nullptr并返回对该引用的引用。

const unique_ptr<MyClass>& find_it(const string &key, const DB &db)
{
  static unique_ptr<MyClass> not_found;

  auto itr = db.find(key);
  if (itr == db.end()) {
    return not_found;
  }
  return itr->second;
}

更好的解决方法可能是使用boost::optional。将返回类型更改为boost::optional<std::unique_ptr<MyClass>> const&，如果找不到该对象，则返回boost::none。

如何返回对包含nullptr的unique_ptr的引用

1 个答案: