Question

我正在尝试制作一个包含'item'对象数组的程序。当程序运行足够的时间然后将其从数组中删除时，它应该执行某些操作。但是程序会在

处触发断点

    if ((newItemList = (item*)realloc(itemList, newItemCount * sizeof(item))) == NULL)

我不明白为什么？

这是我的代码

#include "stdafx.h"
#include <iostream>
using namespace std;
struct item
{
    const char *name;
    unsigned long time;
    bool complete = 0;
};
item *itemList;
unsigned int itemCount = 1;
unsigned long currentTime = 0;
unsigned long lastCheckTime = 0;
int main()
{
    itemList = (item *)malloc(itemCount);
    if (itemList == NULL)
        exit(1);
    itemList[0].name = "Item";
    itemList[0].time = 15;
    itemList[0].complete = 0;
    while (1)
    {
        currentTime += 1;
        if (lastCheckTime != currentTime)
        {
            lastCheckTime = currentTime;
            if (itemList == NULL)
            {
                exit(2);
            }
            else
            {
                unsigned int newItemCount = 0;
                for (unsigned int itemCheck = 0; itemCheck < itemCount; itemCheck++)
                {
                    if (itemList[itemCheck].time <= currentTime)
                    {
                        cout << "Start item " << itemList[itemCheck].name << " scheduled for " << itemList[itemCheck].time << " seconds @ " << currentTime << " seconds" << endl;
                        itemList[itemCheck].complete = 1;
                    }
                    else
                    {
                        cout << "Item " << itemList[itemCheck].name << " scheduled for " << itemList[itemCheck].time << " seconds" << endl;
                    }
                }
                item *newItemList = NULL;
                for (unsigned int itemCheck = 0; itemCheck < itemCount; itemCheck++)
                {
                    if (!itemList[itemCheck].complete)
                    {
                        newItemCount++;
                        if ((newItemList = (item*)realloc(itemList, newItemCount * sizeof(item))) == NULL)
                        {
                            exit(3);
                        }
                        else
                        {
                            item newItem = itemList[itemCheck];
                            itemList = newItemList;
                            itemList[newItemCount - 1] = newItem;
                        }
                    }
                    else
                    {
                    //  cout << "removed item " << itemList[itemCheck].name << " scheduled for " << itemList[itemCheck].time << " seconds" << endl;
                    }
                    free(newItemList);
                }
                itemCount = newItemCount;
            }
        }
    }
    return 0;
}

输出：

Critical error detected c0000374
event.exe has triggered a breakpoint.

The program '[7460] event.exe' has exited with code 0 (0x0).

为什么会这样？我做错了什么？

Answer 1

该代码是C，而不是C ++。除非这是学习理解旧式内存管理的某种教育方式，否则请将其抛弃并使用C ++中的现代容器类正确地完成它。避免使用C样式指针，避免在堆上分配对象，如果必须使用带有智能指针的new。

话虽如此，我在阅读您的代码时发现了两个明显的问题：

itemList = (item *)malloc(itemCount);

这里只分配1个字节。

item newItem = itemList[itemCheck];

在将itemList传递给realloc之后，您不能访问它。

realloc在程序中触发断点

1 个答案: