创建对象数组然后有效删除它们的更好方法

Question

我有一个名为“Menu”的类，该类有一个指向“MenuItem”类对象的指针数组。 Menu 类的前三个相关行如下所示：

class Menu
{
    MenuItem *items[5];

在 Menu 类的构造函数中，我将 items 创建为：

for(int i=0; i<nItems; i++)
{
    items[i] = new MenuItem(titles[i],...); //with all necessary parameters

在 Menu 类的析构函数中，我将 items 删除为：

~Menu()
{
for(int i=0; i<nItems; i++)
    delete items[i];

问题：我的问题是当我调用这个析构函数时，我得到了一个错误Debug Assertion Failed!。除了这个析构函数，一切都很好。如果我不调用此析构函数，则没有错误。我想知道如何解决这个问题？我想删除这个数组，让我的记忆变得干净。

Answer 1

如果没有更多上下文，很难说你的问题是什么（尽管我敢打赌复制构造函数和/或赋值运算符的问题，或者nItems 的值错误）。

管理这些原始指针的更好方法是一开始就没有它们。

考虑：

智能指针数组：

std::unique_ptr<MenuItem> items[5];

或者一个std::array 的智能指针：

std::array<std::unique_ptr<MenuItem>, 5> items;

或者一个vector<> 的智能指针：

std::vector<std::unique_ptr<MenuItem>> items;

Answer 2

我认为您发布的代码没有任何问题。

虽然断言发生在对象被销毁时，但明显的错误发生在其他地方；在对象生命周期的某个时间点，有些东西被乱写了。

这种调试断言并不意味着“错误现在发生了！”。它的意思是“这个错误发生在前一段时间，我才注意到它！”，基本上。

在代码的其他地方寻找错误。有许多调试工具可以检测代码并尝试识别未定义的行为，例如踩踏未初始化或未分配的内存。