Как вернуть ссылку на unique_ptr, содержащим nullptr

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 фу nctions:

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>, и вы затем возвращаете ссылку на него, следовательно, предупреждение. Простое решение состоит в том, чтобы определить staticunique_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, если объект не найден.