使用delete[]和重载+=的c++堆损坏

Question

我必须为家庭作业创建一个重载String类。我在测试一些操作符时遇到了一个问题:

int main() {
    MyString i;
    cin >> i;
    cin.ignore(100, 'n');
    MyString temp = i;
    while(!(i > temp)) {
        temp += i;
        cin >> i;
        cin.ignore(100, 'n');
    }
    cout << endl << temp;
    return 0;
}
MyString operator+= (const MyString& op1) {
    _len += (op1._len);
    char* temp = new char[_len+1];
    strcpy(temp, _str);
    strcat(temp, op1._str);
    if(_str) {
        delete [] _str;
        _str = NULL;
    }
    _str = new char(_len+1);
    strcpy(_str, temp);
    return *this;
}
istream& operator>> (istream& inStream, MyString& in) {
    char temp[TSIZE];
    inStream >> temp;
    in._len = strlen(temp);
    if(in._str) {
        delete [] in._str;
        in._str = NULL;
    }
    in._str = new char[in._len+1];
    strcpy(in._str, temp);
    return inStream;
}
MyString(const MyString& from) {
        _len = from._len;
        if(from._str) {
            _str = new char[_len+1];
            strcpy(_str, from._str);
        } else _str = NULL;
    }
explicit MyString(const char* from) {
    if(from) {
        _len = strlen(from);
    _str = new char[_len+1];
        strcpy(_str, from);
    } else {
        _len = 0;
        _str = NULL;
    }
}

我对这个仍然很陌生，但显然问题发生在第二次调用+=操作符时，而不是第一次。我很抱歉，如果我没有给所有需要的信息，我不想包括更多的需要。谢谢您的帮助

Answer 1

_str = new char(_len+1);

通过使用圆括号而不是方括号，你分配了一个字符，并用一个奇怪的值初始化它。我很确定你的意思是分配一个数组。

_str = new char[_len+1];

但是既然你已经分配了temp，为什么不直接使用它呢?

_str = temp;
// strcpy(_str, temp); // delete this line

这也修复了内存泄漏。您没有释放为temp分配的内存，但是使用此方法，您不必这样做。