c++ - 如何删除指向动态分配对象的指针数组

Question

这是我尝试使用的代码，它编译但在 Turbo C++ 中给出了意想不到的结果，但程序在 dev C++ 中崩溃，经过反复试验，我发现关键字 delete 导致了问题，但我无法找出如何更正错误。有人可以找出程序中的错误并向我解释。另外，有人可以告诉我如何使用智能指针编写相同的代码吗？

代码如下:

#include<iostream.h>
#include<process.h>

class list1
{
  public:
    virtual void getdata() 
    { }
    virtual ~list1() 
    { 
        cout<<endl<<"Destructor list1 called " ;
    }
    virtual void display() 
    { }
};
class list2: public list1
{
  public:
    int data;
    void getdata()
    {
        cout<<endl<<"Enter the data " ;
        cin>>data ;
    }
    void display()
    {
        cout<<endl<<data ;
    }
    ~list2()
    {
        cout<<endl<<"Destructor list2 called" ;
    }
};

int main()
{
    list1* ptr[3]; //array of pointers
    char ch = 'y';
    int n = 0;
    while(ch=='y')
    {
        ptr[n] = new list2 ;
        ptr[n]->getdata() ;
        n++ ;
        cout<<endl<<"Do you want to enter more of data " ;
        cin>>ch ;
        cout<<endl<<"The no of items currently added are: "<<n ;
    }
    ch='y';
   while(ch=='y')
    {
        delete ptr[n];
        n--;
        cout<<endl<<"Item deleted " ;
        cout<<endl<<"Do you want to delete more elements " ;
        cin>>ch ;
        cout<<endl<<"The no of items currently in the list are "<<n ;
    }
    int i = 0;
    while(i < n)
    {
        ptr[i]->display() ;
        i++ ;
    }
    cout<<endl ;
    system("pause") ;
    return 0;
}

Answer 1

您没有进行边界检查。

list1* ptr[3];

ptr最多有3个元素，放多了就有踩到其他东西的可能。 (谁知道你的数组后面是什么？)

while(ch=='y')
{
    delete ptr[n];

这个也一样。如果按 y 的次数比按 delete 的次数多于按 create 的次数，那么您将删除数组开始之前的内容，谁知道那里有什么？它可能是无效指针，这将导致您的运行时错误。

编辑

我说的是问题，但 Nikos C. 有正确答案。

delete ptr[n] 

将访问未初始化的内存。删除前需要自减

因此，如果你按 y 分配一次，n 将为 0 并放入 ptr[0]，但当你按 y 释放时，n 将为 1，ptr[1] 将被删除，这是未分配的。