删除传递给类的指针向量

Question

我有一个抽象的module 类，特定模块派生自该类。在运行时，我会解析一个配置文件并确定配置文件中每个模块的具体模块类型：

std::vector<module*> modules;
module *temp;

//Let nm be the result of parsing as a std::string

if(nm == "type1")
{
    temp = new module_1(...);
}
else if(nm == "type2")
{
    temp = new module_2(...);
}
else if(nm == "type3")
{
    temp = new module_3(...);
}
else
{
    //Syntax error
    return -1;
}

modules.push_back(temp);
partition p;
p.modules = modules;

将向量 modules 传递给 partition 类：

class partition
{
    public:
    //Member functions

    private:
    //...Other variables
    std::vector<module*> modules;
};

用完这些模块指针后，为这些模块指针释放内存的正确方法是什么？我试图在 partition 类的析构函数中删除它们，如下所示，但最终出现了分段错误：

partition::~partition()
{
    for(unsigned i=0; i<modules.size(); i++)
    {
         delete modules[i];
    }
}

Answer 1

这取决于您希望如何处理所有权以及分区是否将具有值语义。想想如果你复制一个分区会发生什么：

分区的副本是否会与原始分区共享模块。模块中的更改是否要在分区之间共享？

如果是，您应该为您的分区使用std::shared_ptr。所有的痛苦都消失了。

如果没有，请为您的分区实现一个复制构造函数和赋值运算符，以执行模块列表的深拷贝。实现一个析构函数，删除列表中的每个模块。这是安全的，因为每个分区都有自己的模块对象。

总的来说，我赞成第二种方法。如果您不想实现深度复制，只需将partition 设为不可复制或仅移动并使用std::unique_ptr 处理删除。