从函数返回shared_ptr

Question

我有一个class，它返回std:：shared_ptr，也就是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添加到向量时，我有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>);

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

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