1

我正在尝试为 Matrix 程序重载“+”运算符。这是我的代码，对我来说看起来不错。但是当我在我的主函数中添加两个矩阵时，什么也没有发生。有人可以帮忙吗？谢谢 ：）

顺便提一句：

- 程序编译并运行得很好，直到它应该添加到矩阵中。

- 我认为在我的 operator+() 函数的实现中存在问题，因为我已将代码复制到 add(Mtrx,Mtrx) 函数中进行测试，但它也不起作用。

//Mtrx.h 

#ifndef MTRX_H_
#define MTRX_H_
#include <iostream>
#include <string>

using namespace std;
using std::ostream;

class Mtrx {
    int lines,cols;
    float **p;
public:

    Mtrx();
    Mtrx(int,int);
    int getLines();
    int getCols();
    float getElement(int,int);
    void setLines(int);
    void setCols(int);
    void setElement(int,int,float);

    Mtrx operator+(Mtrx&);

        ~Mtrx();
};

ostream& operator<<(ostream& os, Mtrx& m);

#endif /* MTRX_H_ */

//mtrx.cpp

//...
//...
Mtrx::~Mtrx(){
delete p;
p = NULL;
}

Mtrx Mtrx::operator+(Mtrx& m){
if(this->getLines() == m.getLines() && this->getCols() == m.getCols()){
    Mtrx res(getLines(),getCols());

    for (int i = 1; i <= this->getLines(); i++){
        for(int j = 1; j <= this->getCols(); j++){
            res.setElement(i,j,(this->getElement(i,j)+m.getElement(i,j)));
        }
    }

    return res;
}
4

2 回答 2

3

您有一个析构函数，但缺少一个复制构造函数和一个赋值运算符。根据经验，如果您拥有其中任何一个，则应该拥有所有这些。

Mtrx(const Mtrx&);
Mtrx& operator=(const Mtrx&);
~Mtrx();

如果没有显式的复制构造函数，编译器将为您生成一个。但是，它并不聪明，因此它不知道p在复制矩阵时分配新内存。它只是复制指针，导致原始矩阵和副本都引用相同的内存。当他们的析构函数运行时，他们两个都会调用delete p，这对第二个人来说是个坏消息。

这正是operator+返回和res复制时发生的情况。

于 2013-02-26T02:10:13.600 回答
1

检查你的牙套。您要么缺少一个，要么您的 if(false) 控制路径没有返回。

Mtrx Mtrx::operator+(Mtrx& m){
if(this->getLines() == m.getLines() && this->getCols() == m.getCols()){
    Mtrx res(getLines(),getCols());

    for (int i = 1; i <= this->getLines(); i++){
        for(int j = 1; j <= this->getCols(); j++){
            res.setElement(i,j,(this->getElement(i,j)+m.getElement(i,j)));
        }
    }

    return res;
}
于 2013-02-26T02:14:05.097 回答