c++ - 从函数返回 shared_ptr

Question

我有一个返回 std::shared_ptr aka Product_SPTR 的类:

Product_SPTR Mill::Production(sf::Time time)
{
    if(m_isProducing)
    {
        if(elapsedTime.getElapsedTime()>m_manufacturingTime)
        {
            elapsedTime.restart();
            Flour_SPTR a(new Flour(5,1,ProductType::CONSTRUCTION),deleter<Flour>);
            return  a  ;

        }
    }

}

然后我有typedef std::vector<Product_SPTR> VectorProduct_SPTR

当我尝试将新的 Product_SPTR 添加到 vector 时，我有 segmentation fault

这里:

products.push_back(a->Production(gameTime.getElapsedTime()));

但是当我做这样的事情时:

products.push_back(new Flour(5,1,ProductType::CONSTRUCTION),deleter<Flour>);

问题没有发生....

我刚开始使用智能指针，所以可能我不知道如何使用它..

Answer 1

当两个 if 中的条件都计算为 false 时，您错过了 return 语句。它编译，可能会给你编译器警告。除非您了解警告及其含义，否则您应该始终在最高警告级别上工作并逐一消除所有警告。

另外，不用写

Flour_SPTR a(new Flour(5, 1, ProductType::CONSTRUCTION),deleter<Flour>);
return  a;

你可能应该写

return Flour_SPTR(new Flour(5, 1, ProductType::CONSTRUCTION), deleter<Flour>);

现代编译器在优化冗余变量方面不应该有任何问题，但帮助编译器完成它的工作总是好的。如果你可以跳过删除器，你也可以这样写:

return std::make_shared<Flour>(5, 1, ProductType::CONSTRUCTION);