工厂方法创建shared_ptr对象

Question

当使用工厂创建对象时，例如下面的示例，在某些情况下，由shared_ptr包装的对象显然会在返回过程中被删除（在调试过程中，该对象已创建，但是将其分配给this->xs引发异常）。 当我更改工厂方法以返回原始指针，并且Link::xs成员成为unique_ptr ，代码运行良好。 shared_ptr到底是什么原因导致它表现出这种行为？ 它与shared_ptr<CrossSection>包装一个Circular对象有关吗？ 测试已使用MS Visual C ++ 2012完成。

class Link
{
private:
    std::shared_ptr<xs::CrossSection> xs;
public:
    void parseXsection(const std::vector<std::string>& parts);
    std::shared_ptr<xs::CrossSection> getXs() { return this->xs; }
};
void Link::parseXsection(const std::vector<std::string>& parts)
{
    this->xs = xs::Factory::create(parts[1]);
}

namespace xs
{
    class CrossSection
    {
    };
    class Circular : public CrossSection
    {
    };
    class Dummy : public CrossSection
    {
    };
    class Factory
    {
    public:
        static std::shared_ptr<CrossSection> create(const std::string& type);
    };
    std::shared_ptr<CrossSection> Factory::create(const std::string& type)
    {
        if (geom == "circular")
        {
            return std::shared_ptr<CrossSection>(new Circular());
        }
        else
        {
            return std::shared_ptr<CrossSection>(new Dummy());
        }
    }
}

Answer 1

因此，马丁有一个解决析构函数问题的选项。 您可以添加虚拟析构函数。

但是，由于您使用的是std::shared_ptr ，它使用了一些类型的擦除，因此您可以进行较小的修复：

std::shared_ptr<CrossSection> Factory::create(const std::string& type)
{
    if (geom == "circular")
        return std::shared_ptr<Circular>(new Circular());
    else
        return std::shared_ptr<Dummy>(new Dummy());
}

或者，甚至更好：

std::shared_ptr<CrossSection> Factory::create(const std::string& type)
{
    if (geom == "circular")
        return std::make_shared<Circular>();
    else
        return std::make_shared<Dummy>();
}

Answer 2

为了使用多态性，您绝对需要在CrossSection基类中定义一个虚拟析构函数，即，为了声明派生类并使用它们来代替父类（因此通常几乎每次您都想使用派生类...）

class CrossSection {
  public:
    virtual ~CrossSection() { /* Nothing to do here ... */ }
};

参见例如何时使用虚拟析构函数？ 还是每个类都应该有一个虚拟的析构函数？ 有关更多说明。

PS：我现在不能说，如果这是导致您shared_ptr问题的原因，但是它看起来很像是您忘记虚拟析构函数时可能遇到的问题...