4

考虑我有一个容器std::map<int, std::shared_ptr<MyClass>>，我想将它填充到外部函数中并避免处理它的内容。所以我有

typedef Container std::map<int, std::shared_ptr<MyClass>>

Container&& f(){
    Container bar;
    auto foo = std::shared_ptr<MyClass>(new MyClass());
    bar.insert(std::make_pair(0,foo));
    std::cout<<bar.at(1)->print_smth<<'\n'; //This works
    return std::move(bar);
}

int main(){
    Container baz(f());
    std::cout<<bar.at(1)->print_smth<<'\n'; //This doesn't
    // Container baz has element 1, but shared_ptr is invalidated, because it has 0 references.

}

如果我使用传统的复制构造函数，一切都会按预期工作。

4

2 回答 2

6

这太复杂了。为什么不直接说：

int main()
{
    Container baz { { 0, std::make_shared<MyClass>() } };

    // ...
}

如果您绝对必须使用辅助函数，则必须返回一个对象，而不是悬空引用。像这样的东西：

Container f()
{
    return Container { { 0, std::make_shared<MyClass>() } };
}

很难放纵任何比这更平庸的东西，但最后一个，永远不会在家里使用的版本：

Container f()
{
   Container bar;
   auto p = std::make_shared<MyClass>;

   bar[0] = p;                        // Method #1
   // ---- ALTERNATIVELY ---
   bar.insert(std::make_pair(0, p));  // Method #2
   // ---- ALTERNATIVELY ---
   bar.emplace(0, p);                 // Method #3

   return bar;
}
于 2012-08-28T15:27:22.217 回答
0

问题是您从f(). 右值引用仍然是一个引用，就像你永远不会从函数返回对本地的引用一样，你也不能用右值引用来做到这一点。

好消息是您不必这样做。由于返回值优化，您可以简单地返回一个Containerfrom f()，它会完全按照您的要求进行操作。

于 2012-08-28T15:43:11.250 回答