内存释放出错 编译环境vs2013 求大神指点

#include<iostream>
using namespace std;

class CMyString
{
public:
    CMyString(char *pData=NULL);
    CMyString(CMyString &str);
    CMyString& operator=(CMyString &str);
    ~CMyString(void);
    void Print(){ cout << m_pData; }
private:
    char* m_pData;

};

CMyString::CMyString(char *pData)
{
    if (NULL == pData)
    {
        m_pData = NULL;
        return;
    }
    else
    {
        unsigned int len = strlen(pData)+1;
        m_pData = new char[len];
        for (int i = 0; i < len; i++)
            m_pData[i] = pData[i];
        m_pData[len] = '\0';
    }
}

CMyString::CMyString(CMyString &str)
{
    unsigned int len = strlen(str.m_pData) + 1;
    m_pData = new char[len];
    for (int i = 0; i < len; i++)
        m_pData[i] = str.m_pData[i];
    m_pData[len] = '\0';
}

CMyString & CMyString::operator=(CMyString &str)
{
    if (this == &str)
        return *this;
    else
    {
        unsigned int len = strlen(str.m_pData) + 1;
        char *pTemp = new char[len];
        for (int i = 0; i < len; i++)
            pTemp[i] = str.m_pData[i];
        pTemp[len] = '\0';

        delete[]m_pData;//出错！！！不解

        m_pData = pTemp;

    }
    return *this;
}

CMyString::~CMyString()
{
    delete [] m_pData;
}

int main()
{
    char c1[10] = "majunfu";
    CMyString s1(c1);
    CMyString s2 = s1;
    s1 = s2;
    s1.Print();
    s2.Print();
    return 0;
}

释放错误，你写的析构函数作用不就是释放吗。

No,不是的，你说的不对哦，你再看看程序。

m_pData 这个指针是否分配正确空间了。你可以调试查看一下就知道了

delete[]m_pData;//出错！！！不解

因为你的 m_pData 是未初始化的值，因为你没写默认的无参构造函数，所以它的值不确定， 你delete了一个未初始化的m_pData。

而 CMyString s2 = s1; 这一句调用了 operator= ， 你把s1 赋值给 s2，但是s2的成员m_pData你没初始化。

for (int i = 0; i < len; i++)
pTemp[i] = str.m_pData[i];
pTemp[len] = '\0';

问题在这里。数组长度是len,你引用了pTemp[len]？

delete[]m_pData;//出错！！！不解

m_pData = pTemp;
释放内存之后就不在可以用来存储数据，要使m_pData = pTemp;可以运行就不要释放内存。删除就行，因为你在析构函数里已经有释放了，系统在释放对象时会自动调用析构函数