0

2 次元行列でいくつかの操作を実行しようとしています。計算を行うために (+ 、 - 、および *) をオーバーロードしました。私は（私が信じている）メモリ管理に関して問題があります。次のコードを見てください。

Mtx M1(rows1,cols1,1.0); //call the constructor 
Mtx M2(rows2,cols2,2.0); //call the constructor 
Mtx M3(rows3,cols3,0.0); //call the constructor 

M3 = M1 + M2;
cout << M3 << endl;

Mtx Mtx::operator+(const Mtx &rhs)
{

double **ETS;
ETS = new double*[nrows];
for (int i = 0; i < rhs.nrows; i++) {
    ETS[i] = new double[rhs.ncols];
}
if (ETS == NULL) {
    cout << "Error Allocation on the Heap" << endl;
    exit(1);
}

for (int i = 0; i < rhs.nrows; i++) {
    for (int j = 0; j < rhs.ncols; j++) {
        ETS[i][j] = 0.0;
    }
}

for (int i = 0; i < rhs.nrows; i++) {
    for (int j = 0; j < rhs.ncols; j++) {
        ETS[i][j] = ets[i][j];
    }
}


for (int i = 0; i < rhs.nrows; i++) {
    for (int j = 0; j < rhs.ncols; j++) {
        ETS[i][j] = ETS[i][j] + rhs.ets[i][j];
    }
}

Mtx S(nrows, ncols, ETS);
delete [] ETS;
return S;
}

私の問題はここにあると思います：

Mtx S(nrows, ncols, ETS); 
delete [] ETS;
return S;

これは正しい帰り方ETSですか？それとも、問題はコンストラクターにあると思いますか? 上記のリターンを行ったときに出力が得られませんでした!

これはコンストラクタですMtx S(nrows, ncols, ETS);

Mtx::Mtx(int rows, int cols, double **ETS)
{
ets = new double*[nrows];
for (int i = 0; i < nrows; i++) {
    ets[i] = new double[ncols];
}
for (int i = 0; i < nrows; i++) {
    for (int j = 0; j < ncols; j++) {
        ets[i][j] = ETS[i][j];
    }
  }
}

私のコピーコンストラクタ：

Mtx::Mtx(const Mtx& rhs)
:nrows(rhs.nrows), ncols(rhs.ncols)
    {
ets = new double*[nrows];
for (int i = 0; i < nrows; i++) {
    ets[i] = new double[ncols];
}

for (int i = 0; i < rhs.nrows; i++) {
    for (int j = 0; j < rhs.ncols; j++) {
        ets[i][j] = rhs.ets[i][j];
    }
  }
}

過負荷<<で印刷しM3ました。印刷M1とをテストしたので、問題なく動作しM2ます。

私も次のことを行いましたが、まだ機能していません。

Mtx S(nrows, ncols, ETS);
for (int i = 0; i < rhs.nrows; i++) {
    delete [] ETS[i];
}
delete [] ETS;
return S;
}
4

3 に答える 3

3

代わりに使用できます

std::vector< std::vector<double> >

それ以外の

double **

境界に関する限り、それはより安全になります。そして、関数を使用するだけです

std::vector::size()

と

std::vector::clear()

デストラクタで。:)

于 2012-07-22T05:07:57.683 に答える
2

はい、ご指摘の点に問題があります。ETS のポインタで指されているすべてのメモリを削除する必要があります。したがって、次のようになります。

for (int i = 0; i < rhs.nrows; i++) {
    delete [] ETS[i];
}
delete [] ETS;

new を呼び出すたびに、delete を呼び出さなければならないというのが良いルールです。nrows+1new の呼び出しで配列を割り当てるため、同じ番号で削除する必要があります。それは難しいことではありませんが、何かがうまくいかないときに手がかりになります.

于 2012-07-22T03:43:06.117 に答える
2

二項算術演算子を実装する一般的な方法はoperator+=、クラス メンバー関数およびoperator+独立関数として定義することです。+演算子は演算子の観点から実装されており+=、その左側のオペランドを copyで取得することに注意してください。コピー コンストラクターと代入演算子が正しい限り、これは機能するはずです。

// member function
Mtx& Mtx::operator+=(const Mtx &rhs)
{
    for (int i = 0; i < nrows; ++i) {
        for (int j = 0; j < ncols; ++i) {
            ets[i][j] += rhs.ets[i][j];
        }
    }
    return *this;
}

// non-member function
Mtx operator+(Mtx lhs, const Mtx &rhs)
{
    lhs += rhs;
    return lhs;
}
于 2012-07-22T05:05:07.073 に答える