c++ - 当我尝试删除指针时我的程序崩溃了

Question

每当我尝试删除一个指针时，我都会收到“Windows 错误噪音”，然后我的程序就死机了，但从未正式崩溃。

void addIngredient(char ** & ingredients, int & numOfIng)
{
    char * str = nullptr;

    char **tempArr = new char*[numOfIng];
    numOfIng++;

    //init tempArr to nullptr
    for (int i = 0; i < numOfIng; i++)
    {
        tempArr[i] = nullptr;
    }

    //set the new array to the old array
    for (int i = 0; i < numOfIng - 1; i++)
    {
        tempArr[i] = new char;
        tempArr[i] = ingredients[i];
    }

    delete [] ingredients;

    //point the old array to the new one 
    ingredients = tempArr;

    //add the new element to the end of the old array
    cout << "What new ingredient would you like to add? ";
    str = new char[25];
    cin >> str;
    ingredients[numOfIng - 1] = str;
    delete str;

    //method tought to us in class on how to clear array and what is being pointers within the array
    for (int i = 0; i < numOfIng; ++i)
    {
        delete [] tempArr[i]; //Freezes here
    }
    delete [] tempArr;
}

我希望删除数组的元素，然后删除指向该数组的指针，但是当我运行它时，我得到了标准的 Windows 错误噪音并且我的程序卡住，直到我 ctrl+c 控制台窗口。编码新手，所以请不要对我太苛刻。不确定这是否重要，但我正在使用 Visual Studio 2017 并在 x86 中进行调试。

Answer 1

您正在分配一个对象 (char)，然后忘记了新对象:

tempArr[i] = new char;
tempArr[i] = ingredients[i];

您要做的是设置数据:

tempArr[i] = new char;
*(tempArr[i]) = *(ingredients[i]);

这样新角色就不会丢失。

您还有另一个问题，当您执行 delete [] ingredients; 时，您并没有删除底层指针。然后你稍后删除临时子数组(delete [] tempArr[i])，所以你应该做的是:

for (int i = 0; i < numOfIng; ++i)
{
    delete ingredients[i]; // Note that I remove the [], as you have only new char, not new char[1]
}

之后没有删除，因为新的ingredients正在使用这些“旧的”tempArr。

还可以考虑针对您的情况使用 vector 或唯一指针。