0

私はクラス、マトリックスを持っています

class Matrix
{
private:
    int cols;
    int rows;
    int **matrix;
public:
    //constructors and methods
    friend Matrix addandupdate(Matrix a, Matrix b); 
}

関数addandupdateを外部で次のように定義しています。

Matrix addandupdate(Matrix a, Matrix b)
{
    int rsize = a.rows;
    int csize = a.cols;

    Matrix c(csize, rsize);

    for(int i=0; i<rsize; i++)
    {
        for(int j=0; j<csize; j++)
        {
            c.set(i,j,a.matrix[i][j]+b.matrix[i][j]);
        }
    }
    return c;
}

それから主に、

Matrix x(c, r);
//filling values in x
//printing x
Matrix newx = addandupdate(x, x);    //something is not right here
//printing newx

ランタイムエラーをスローする原因となっている上記の行を除いて、すべて問題ありません。

*** glibc detected *** ./a.out: munmap_chunk(): invalid pointer: 0x085c6020 ***
======= Backtrace: =========
/lib/i386-linux-gnu/libc.so.6(+0x73e42)[0x183e42]
/lib/i386-linux-gnu/libc.so.6(+0x74525)[0x184525]
/usr/lib/i386-linux-gnu/libstdc++.so.6(_ZdlPv+0x1f)[0x84d51f]
./a.out[0x8048c69]
./a.out[0x8048b05]
/lib/i386-linux-gnu/libc.so.6(__libc_start_main+0xf3)[0x1294d3]
./a.out[0x8048751]

久しぶりにC++コードを書いています。ばかげたことなら失礼します。

4

1 に答える 1

1

Matrix newx = addandupdate(x, x);まだ実装していないコピーコンストラクターを呼び出します。この場合、浅いコピーでは不十分なので必要です。

代入演算子とデストラクタも必要です。

または、Markが指摘したように、に置き換えint **matrix;ますstd::vector<std::vector<int> >。

于 2012-08-02T18:05:09.120 に答える