求解C++析构函数释放内存错误原因,头疼。

拜托看一下我这个矩阵类实现里的析构函数哪里错了,编译没问题,运行出错。
删掉析构函数就可以运行了,为什么呢?

  • #include
  • using namespace std;
  • class CMatrix
  • {
  • public:
  • CMatrix(int, int); //构造函数
  • void output(); //输出矩阵各元素
  • CMatrix operator +(CMatrix &M2); //重载矩阵加运算
  • CMatrix operator -(CMatrix &M2); //重载矩阵减运算
  • friend istream & operator>>(istream &, CMatrix &);
  • friend ostream & operator<<(ostream &, CMatrix &);
  • ~CMatrix()
  • {
  • delete[] m_pData;
  • }
  • private:
  • int m_row; //矩阵行数
  • int m_col; //矩阵列数
  • double *m_pData; //使用指针指向矩阵各元素
  • };
  • //构造函数
  • CMatrix::CMatrix(int r, int c)
  • {
  • m_row = r;
  • m_col = c;
  • m_pData = new double[r*c];
  • for (int i = 0; i<m_row*m_col; i++)
  • m_pData[i] = 0;
  • }
  • //矩阵输出函数
  • void CMatrix::output()
  • {
  • int j = 0;
  • for (int i = 0; i<m_row*m_col; i++)
  • {
  • cout << m_pData[i] << " ";
  • if ((i + 1) % m_col == 0)
  • cout << endl;
  • }
  • }
  • //重载矩阵相加运算
  • CMatrix CMatrix::operator +(CMatrix &M2)
  • {
  • CMatrix M(m_row,m_col);
  • for (int i = 0; i<m_row*m_col; i++)
  • M.m_pData[i] = m_pData[i] + M2.m_pData[i];
  • return M;
  • }
  • //重载矩阵相减运算
  • CMatrix CMatrix::operator -(CMatrix &M2)
  • {
  • CMatrix M(this->m_row,this->m_col);
  • for (int i = 0; i<m_row*m_col; i++)
  • M.m_pData[i] = m_pData[i] - M2.m_pData[i];
  • return M;
  • }
  • istream & operator>>(istream &input, CMatrix &c)
  • {
  • for (int i = 0; i < c.m_row*c.m_col; i++)
  • input >> c.m_pData[i];
  • return input;
  • }
  • ostream & operator<<(ostream &output, CMatrix &c)
  • {
  • for (int i = 0; i < c.m_row*c.m_col; i++)
  • {
  • output << c.m_pData[i];
  • if ((i + 1) % c.m_col == 0)
  • output << endl;
  • }
  • return output;
  • }
  • int main()
  • {
  • CMatrix M1(2, 2), M2(2, 2), M3(2, 2);
  • cin >> M1;
  • cout << M1;
  • cin >> M2;
  • cout << M2;
  • M3 = M1 + M2;
  • cout << M3;
  • M3 = M1 - M2;
  • cout << M3;
  • return 0;
  • }

4个回答

haizhiyutwb
随毅 大佬,这个帖子的问题好像是语法错误,我这个和他不一样。
接近 3 年之前 回复

具体报的什么错误呢

haizhiyutwb
随毅 Debug时就停在析构函数那里了,显示的是Unhandled exception at 0x0FF1125C (ucrtbased.dll) in Chapter4.exe: 0xC0000005: Access violation reading location 0xBF0E3FD8.
接近 3 年之前 回复
 ~CMatrix()
{
delete[] m_pData;   //这里可以为空,不能释放,因为没使用一次就释放了,下次在使用时指针为空。。
}

如果真想释放内存,建议写一个函数释放,在变量彻底不使用时释放!!!

haizhiyutwb
随毅 嗯,好的嘞,就是在CMatrix中添加一个函数专门用来释放的对吗?
接近 3 年之前 回复

CMatrix& CMatrix::operator +(CMatrix &M2)
{
CMatrix M(m_row,m_col);
for (int i = 0; i<m_row*m_col; i++)
M.m_pData[i] = m_pData[i] + M2.m_pData[i];
return M;

}

CMatrix::CMatrix(CMatrix& a)
{
this->m_row = a.m_row;
this->m_col = a.m_col;
this->m_pData = new double[a.m_col * a.m_row];
memcpy(this->m_pData, a.m_pData, a.m_col * a.m_row);
}
加一个引用就可以了,或者加一个拷贝构造函数就可以了

haizhiyutwb
随毅 回复美行楼中楼:嗯,原来是这个原因,谢谢啦。
接近 3 年之前 回复
tennyear
美行楼中楼 回复随毅: 你还得重载一个 = 号,因为你有 = 的操作,加完拷贝构造函数和 重载=号之后就不会有问题了。 析构应该不是C3的吧,那个得生命周期结束才会调用析构函数的
接近 3 年之前 回复
haizhiyutwb
随毅 添加了引用,测试了一下析构函数,发现在C3=C1+C2之后就紧接着执行C3的析构了,为啥呢?
接近 3 年之前 回复
Csdn user default icon
上传中...
上传图片
插入图片
抄袭、复制答案,以达到刷声望分或其他目的的行为,在CSDN问答是严格禁止的,一经发现立刻封号。是时候展现真正的技术了!
立即提问
相关内容推荐