正常阻塞后检测到堆损坏

Question

我有以下代码，但不确定为什么在遇到 Myclass 的析构函数时会收到堆损坏检测错误。我相信我正在正确地释放内存？？

#include <iostream>
#include <vector>
using namespace std;

class MyClass{
private:
    char* mp_str;
public:
    MyClass():mp_str(NULL){}
    ~MyClass(){
        delete [] mp_str;
    }

    void setString(const char* str);
    void printString();
};

int main(){
    MyClass* a = new MyClass();
    std::vector<MyClass> myVector;

    myVector.push_back(*a);

    a->setString("Hello World");
    myVector[0].setString("Goodbye world");

    a->printString();
    myVector[0].printString();

    return 1;
}

void MyClass::setString(const char* const str){
    if(!str)
        return;

    size_t len = strlen(str);

    if(!this->mp_str){
        this->mp_str = new char[len];
        memset(mp_str, 0, len+1);
    }
    strncpy(mp_str, str, len);
}

void MyClass::printString(){
    if(this->mp_str)
        cout << mp_str;
    else
        cout << "No string found";
}

编辑：（固定代码）

void MyClass::setString(const char* const str){
    if(!str)
        return;

    size_t len = strlen(str);

    if(!this->mp_str){
        this->mp_str = new char[len+1];
        memset(mp_str, 0, len+1);
    }
    strncpy(mp_str, str, len);
}

在main()中，我也加了

delete a;

调用return 1之前；

Answer 1

您需要分配字符串的长度+1，以考虑空值。你做对了。

if(!this->mp_str){
    this->mp_str = new char[len+1];
    memset(mp_str, 0, len+1);
}

Answer 2

（在 Rafael 的回答被接受后发布，应该如此。）

缓冲区溢出绝对是这次特定崩溃的根本原因，但可以通过简化实现同时调整以遵守Rule of Three 来避免崩溃。也就是说，既然你实现了一个析构函数（处理mp_str），你也应该实现一个复制构造函数和一个赋值运算符。

但是，遵守 TRoT 的另一种方法是完全避免需要实施任何这些事情。在这种情况下，使用 std::string 而不是 char * 既可以解决崩溃问题，也可以符合 TRoT：

class MyClass{
private:
    std::string mp_str;
public:
    void setString(const char* str) { mp_str = str ? str : ""; }
    void printString() {
        if (mp_str.size()) std::cout << mp_str;
        else std::cout << "(mp_str is empty)";
    }
};