正常块后检测到堆损坏

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"; 
} 

EDIT：（固定代码）

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); 
} 

在主

（），我还添加

delete a; 

调用返回1之前;

Answer 1

您需要分配字符串+1的长度来解决null。你是正确的memsetting。

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

Answer 2

（发布拉斐尔的答案被接受后，理所应当的。）

缓冲区溢出肯定这种崩溃的根本原因，但可避免通过简化的实施而崩溃调整为遵守Rule of Three。换句话说，既然你实现了一个析构函数（为了处理mp_str），你还应该实现一个拷贝构造函数和一个赋值操作符。但是，坚持使用TRoT的另一种方式是避免需要实施任何这些东西。在这种情况下，使用std::string代替char *会解决了这两个碰撞并符合小跑：

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)"; 
    } 
};