1

我不明白为什么会出现错误delete[] *iopszString;，您能帮我解决吗？

尝试输入：1 3 aaa

如果我省略最后一个 delete[] ，它一切正常，但它没有意义，因为为了交换指针，我需要删除前一点。 编码 

// Notepad.cpp

#include <iostream>
#include <fstream>
#include <string>

using namespace std;

// Method definition
void addText(char** iopszString);

void main()
{

    // Const definition
    int const ADD    = 1;
    int const UPDATE = 2;
    int const DELETE = 3;
    int const SAVE   = 4;
    int const EXIT   = 5;

    // Variable definition
    int nUserCode;

    // Code section

    // Gets the user code
    cout << "Enter the code: " << endl;
    cin >> nUserCode;

    // + "\0" so 1 minimum!!!
    char* iopszString = new char[1];
    iopszString = "";

    // Runs until the exit code
    while (nUserCode != EXIT)
    {
        // Checks the exit code
        switch (nUserCode)
        {
            case ADD:
            {
                addText(&iopszString);
                cout << iopszString << endl;
                break;
            }
            case UPDATE:
            {

                break;
            }
            case DELETE:
            {

                break;
            }
            case SAVE:
            {

                break;
            }
            default:
            {
                cout << "Wrong code, try again" << endl;

                break;
            }
        }

        // Gets the user code
        cout << "Enter the code: " << endl;
        cin >> nUserCode;
    }

    // Delete the string cuz heap
    delete[] iopszString;
}

void addText(char** iopszString)
{
    // Variables definition
    int  nAddLength;

    // Code section

    // Gets the new length
    cout << "Enter the length of the added string: " << endl;
    cin >> nAddLength;

    // Always remember - the length you want+1!!
    char* szNewString = new char[nAddLength+1];

    // Gets the new string
    cout << "Enter the new string which you want to add: " << endl;
    cin >> szNewString;

    // Creating a new string (result)
    char* szResult = new char[nAddLength+1+strlen(*iopszString)];

    // Copies the old string to the new
    strcpy(szResult, *iopszString);
    strcat(szResult, szNewString);

    // Deletes the new string cuz we already copied
    delete[] szNewString;

    // Exchange pointers 
    //strcpy(*iopszString, szResult); <--- never

    // The problem!
    delete[] *iopszString;

    // Exchange pointer 
    *iopszString = szResult;
}
4

1 回答 1

6

错误在这两行：

char* iopszString = new char[1];
iopszString = "";

您正在分配新内存new并将其位置存储在您的指针iopszString中。然后，您将字符串文字的位置分配""给该指针，因此指针本身的值会发生变化。它现在指向其他地方，指向您尚未分配且不属于您的内存位置new。因此，您丢失了分配的内存的指针（内存泄漏），当您调用delete[]指向.""delete[]new

你可能打算写：

char* iopszString = new char[1];
iopszString[0] = '\0';

这只会设置char您分配的第一个值，'\0'因此将其转换为有效的、空的、以零结尾的字符串。

于 2013-10-22T18:14:25.960 回答