C ++从向量中删除指针会导致堆错误

Question

我有一个指向我的基类对象的指针向量，然后将派生类中的对象存储在其中。 我添加指针的方式非常明显

std::vector<Element*> mvGates;
...
mvGates.push_back(new Node(x, y));

当我的程序即将关闭时，这些指针将在“ for”循环中释放

 for (int i=0; i<mvGates.size(); ++i)
 {
     delete mvGates[i];
 }

在这一行，我的调试器（我正在使用Qt Creator）显示了如下几行：

Heap block at 14E50F50 modified at 14E50F84 past requested size of 2c
Invalid address specified to RtlFreeHeap( 00030000, 14E50F58 )

原因可能是什么，我做错了什么？ 我知道我没有显示完整的代码，因为它很长，但是也许足以告诉我我的错误。 可能您会推荐我使用智能指针。 我想我会做的，但是并不能回答我的问题-此代码可能有什么问题。

编辑：这是一个非常简单的错误，您在这里可能没有注意到。 在Node类中，我声明了some_type array[0];

我一直在使用它作为2个元素的数组。 我不知道为什么它没有导致SIGSEGV，但是是导致堆错误的原因。

Answer 1

您的std :: vector包含指向Element类实例的指针，但是您正在使用Node类实例填充它。 由于编译器没有抱怨，我们可以安全地假定Node扩展了Element。

问题在于，当删除这些元素时，会将它们称为元素。 除非正确覆盖了元素析构函数，否则删除操作将调用Element ::〜Element（）而不是Node ::〜Node（）

在类定义中，确保析构函数是虚拟的：

class Element{
   virtual ~Element(){
      //Element cleanup
   }

   //....
};

Answer 2

在您的代码中，您正在通过使用索引访问vector中的每个对象，并且正在删除它们。 但是，除了“ Pierluigi”所说的（虚拟〜问题）之外，您是否不应该使用迭代器而不是直接使用索引访问每个对象并删除它们？

Answer 3

您在增量部分中弄错了。 它的i ++不需要++ i。

如果使用++ i，它将在循环到for循环之前增加，因此在迭代器编号处，它引用的是mvGates [i]，而实际上这是不存在的。 仅矢量范围从0到length-1。

for (int i=0; i<mvGates.size(); i++)
{
    delete mvGates[i];
}