抽象基类无虚析构函数触发delete警告：是否需担忧及优化方案？

我用Qt写了一段代码（QList可替换为任意STL容器），最小复现示例如下：

class DataType
{

};

class DerivedDataType : public DataType
{

};

class OtherDerivedDataType : public DataType
{

};

class AbstractDataContainer
{
public :

    ~AbstractDataContainer() { for(DataType* pData : qAsConst(dataList)) { delete pData; } };
    virtual void insertData() = 0;

protected:

    AbstractDataContainer() {};
    QList<DataType*> dataList;
};

template<class T>
class DataContainer : public AbstractDataContainer
{
public:
    DataContainer() : AbstractDataContainer() {};
    ~DataContainer() {};

    void insertData() {DataType* pData = new T(); dataList.append(pData);}
};

void main(void)
{
    QList<AbstractDataContainer*> listOfAbstractDataContainers;
    listOfAbstractDataContainers.append(new DataContainer<DerivedDataType>());
    listOfAbstractDataContainers.append(new DataContainer<OtherDerivedDataType>());

    for(AbstractDataContainer* pAbstractDataContainer : qAsConst(listOfAbstractDataContainers))
    {
        pAbstractDataContainer->insertData();
    }

     /*... do stuff ... */

    /* Finish : clean up. */
    for(AbstractDataContainer* pAbstractDataContainer : qAsConst(listOfAbstractDataContainers))
    {
        delete pAbstractDataContainer;
    }
}

编译时收到警告：Delete called on base class 'AbstractDataContainer' that is abstract but has non-virtual destructor。我有几个疑问：

• 这个警告需要担忧吗？
• 当前代码是否存在错误？
• 有没有实现相同功能且不触发该警告的方法？

另外，我区分AbstractDataContainer和DataContainer的原因是：没法创建包含不同模板参数的模板类列表，同时希望以通用多态方式遍历容器元素，避免针对每种数据类型写switch分支。至少Qt的QList等容器不支持这种写法：

QList<const template<class T> DataContainer> listOfDataContainers;
listOfDataContainers.append(DataContainer<DerivedDataType>);
listOfDataContainers.append(DataContainer<OtherDerivedDataType>);

1. 是否需要担忧？必须重视 #

这个警告绝对需要重视，它指向了C++多态对象销毁时的核心风险——未定义行为。虽然当前你的DataContainer子类没有自己的成员变量，销毁时可能看起来没问题，但一旦子类后续添加了需要清理的资源（比如动态分配的内存、文件句柄、Qt对象等），直接通过基类指针删除时，子类的析构函数不会被调用，必然导致资源泄漏。

2. 当前代码是否存在错误？ #

严格来说，当前代码的行为是未定义的。C++标准规定：如果通过基类指针删除派生类对象，而基类的析构函数不是虚函数，那么行为是未定义的。哪怕现在你的子类析构是空实现，这依然是不符合标准的写法，存在潜在风险。

3. 解决方法：给基类添加虚析构函数 #

最直接且标准的修复方式，就是把AbstractDataContainer的析构函数声明为虚函数。修改后的基类代码如下：

class AbstractDataContainer
{
public :
    // 声明为虚析构函数
    virtual ~AbstractDataContainer() { for(DataType* pData : qAsConst(dataList)) { delete pData; } };
    virtual void insertData() = 0;

protected:
    AbstractDataContainer() {};
    QList<DataType*> dataList;
};

这样修改后：

• 删除基类指针时，会正确调用子类的析构函数（即使子类析构是空实现，也会保证多态销毁的正确性）
• 编译器的警告会消失
• 后续子类扩展资源时，不会出现销毁不彻底的问题

关于核心需求的补充说明 #

你通过抽象基类+模板子类的方式实现“统一管理不同类型容器”的思路是完全正确的，这是C++中处理这类场景的常规手段。因为模板实例化后是不同的类型（DataContainer<DerivedDataType>和DataContainer<OtherDerivedDataType>是两个独立类），无法直接放到同一个容器中，必须通过公共基类的指针/引用实现多态管理。

另外，如果你想进一步避免手动管理裸指针，可以考虑使用智能指针，比如Qt的QSharedPointer或者标准库的std::unique_ptr，这样能更安全地管理容器中的对象，避免忘记删除的情况。例如：

QList<QSharedPointer<AbstractDataContainer>> list;
list.append(QSharedPointer<AbstractDataContainer>(new DataContainer<DerivedDataType>()));
// 后续无需手动delete，智能指针会自动处理销毁

内容的提问来源于stack exchange，提问作者Charles