当条件太高时循环使用类函数错误

Question

上次我在使用 for 循环时经常出错。

最简单的展示方式是：

#include <iostream>
class Class
{
private:
    int number;
public:
    Class(int a);
    void get();
};

Class::Class(int a)
{
    number=a;
}

void Class::get()
{
    std::cout<<number<<std::endl;
}

int main()
{
    int amount=4;
    Class* p[amount];
    for(int i=0;i<amount;i++)
    {
        delete p[i];
        p[i]=new Class(i);
    }
    for(int i=0;i<amount;i++)
    {
        p[i]->get();
    }
    return 0;
}

程序返回 255，在调试时我收到 SIGSEGV 信号 - 分段错误。

我发现一件很奇怪的事情：当amount=3 或更少的程序正常工作时。我打算将它用于 700 多个元素，所以我发现这非常令人失望。

我正在使用带有 GNU GCC 编译器的 Windows 7 和 Code::Blocks 13.12。

我应该在此程序中进行哪些更改以使其正常运行？我很确定这只是我对 c++ 或类似东西的一点了解。

提前致谢。

Answer 1

Class* p[amount];
for(int i=0;i<amount;i++)
{
    delete p[i];
    p[i]=new Class(i);
}

您应该删除delete p[i]; 行，因为数组p 尚未初始化。

Answer 2

您一直在对不指向动态分配对象的指针调用delete：

Class* p[amount]; // array of uninitialized pointers

delete p[i]; // oops

Answer 3

这是怎么回事（见 cmets）：

// This allocates `amount` spots for pointers
Class* p[amount];
// Your pointers are not initialized yet
for(int i=0;i<amount;i++)
{
    // You are trying to delete what they point to
    delete p[i]; // <<== This is undefined behavior
    p[i]=new Class(i);
}

在分配要删除的对象之前，您不应调用delete。否则，这是未定义的行为，即您的程序可能会崩溃。

最后，您需要在完成后删除指针以避免内存泄漏。您可以为此使用智能指针向量（推荐），或者简单地将另一个循环添加到您的程序中，如下所示：

int amount=4;
Class* p[amount];
for(int i=0;i<amount;i++)
{
    p[i]=new Class(i);
}
for(int i=0;i<amount;i++)
{
    p[i]->get();
}
for(int i=0;i<amount;i++)
{
    delete p[i];
}

Answer 4

初始化 p[i] 和声明。 类* p[3] = {0}; delete 内置了 null 检查，因此您不会收到任何错误

即使数量为 3 或小于 3，您也应该得到错误