Question

我是新来使用delete操作符，我很困惑，为什么我在我的程序总是崩溃，当我尝试从一个指针数组删除元素一点点。它崩溃上都删除，我注释掉另一个用于循环来测试此。

int main()
{
    char ** str_array = nullptr;
    char * str = nullptr;
    char delimiter = ' ';
    int arraySize = 0;

    str = new char [51];

    cout << "Enter a string" << endl;
    cin.clear();
    cin.ignore( cin.rdbuf() -> in_avail() );
    cin.getline(str, 50);
    cin.clear();

    arraySize = StrToArray(str_array, str, delimiter);

    for (int i = 0; i < arraySize; i++)
    {
        cout << str_array[i] << endl;
    }

    for (int i = 0; i < arraySize; ++i)
    {
        delete [] str_array[i];
    }
    delete [] str_array;

    return 0;
}

这是我用来初始化str_array的函数。

int StrToArray(char **& str_array, char * str, char delimiter)
{
    int size = 0;
    int arraySize = 0;
    int start = 0;
    for (int i = 0; i < strlen(str); i++)
    {
        if (str[i] == ' ') 
        {
            size++;
            if (str[i + 1] == '\0' || str[i + 1] == ' ')
            {
                size--;
            }
        }
    }

    str_array = new char * [size];

    for (int i = 0; i < strlen(str) + 1; i++)
    {
        if ((str[i] == ' ' || str[i] == '\0') && !(str[i + 1] == '\0' || str[i + 1] == ' '))
        {
            str_array[arraySize] = new char[i-start];
            arraySize++;
            for (int x = 0; x <= i - start; x++)
            {
                str_array[arraySize-1][x] = str[x + start];

                if (x == i-start)
                {
                    str_array[arraySize - 1][x] = '\0';
                }
            }

            i++;
            start = i;
        }

    }
    return (size + 1);
}

我的str_array初始化为一个指针数组，每个指针都指向一个cString。我想做的是删除数组的每个元素，然后删除数组本身。但是，每次我尝试执行任何一个删除操作符时，我的代码都会崩溃。

Answer 1

您的str_array和str_array[i]都是数组，您应该使用delete[]运算符来释放它们。请参见下面的简单示例。

void f(size_t size)
{
    char** str_array = new char* [size];
    for (int i = 0; i < size; ++i) {
        str_array[i] = new char[i];
    }

    for (int j = 0; j < size; ++j) {
        delete[] str_array[j];
    }

    delete[] str_array;
}

Answer 2

我可以看到两个错误，都“一个人犯错”

       str_array[arraySize] = new char[i-start];

应该是

       str_array[arraySize] = new char[i-start+1];

您没有为nul终止符留出足够的空间。

和

str_array = new char * [size];

应该是

str_array = new char * [size+1];

因为空格数比单词数少一。

我使用“ hello world”进行了测试，并通过这两个更改为我的代码运行了。

哦，alijandro所做的更正也是正确的，但对我而言，并没有造成任何崩溃。

计划停止使用删除的操作员工作时

2 个答案: