*** glibc detected *** ./a.out: double free or corruption (out): 0x084602f8 ***

welcome008

原帖由 guojkd 于 2008-5-28 23:47 发表
不要用这样的方法来掩盖你程序的错误。。。。

matrix::matrix(matrix const &m) {
        row = m.row;
        col = m.col;
        elems = new double[row * col];
        for(int i = 0; i< row  ...

多谢你的回复与仔细的看
其实我程序里是不是那样的，而是下面这样：
matrix::matrix(matrix const &m) {
        row = m.row;
        col = m.col;
        elems = new double[row * col];
        for(int j = 0; j< row * col ; j++) {
                elems[j]=m.elems[j];
        }
}

不知道为啥贴出去的时候那\[i\]会丢掉了，把i改为j就可以显示了。

[ 本帖最后由 welcome008 于 2008-5-29 11:05 编辑 ]

guojkd

最好把代码都贴出来哦，光看这个找不出来问题~~

welcome008

原帖由 guojkd 于 2008-5-29 19:14 发表
最好把代码都贴出来哦，光看这个找不出来问题~~

#include<iostream>
using namespace std;

class matrix {
        double * elems;
        short row,col;
    public:
        matrix(short vrow,short vcol) ;
        matrix(matrix const &m) ;
        ~matrix();
        int set (short vrow,short vcol,double val);
};

matrix::matrix(short vrow,short vcol) {
        row = vrow;
        col = vcol;
        elems = new double[row * col];
}
matrix::matrix(matrix const &m) {
        row = m.row;
        col = m.col;
        elems = new double[row * col];
        for(int j = 0; j< row * col ; j++) {
                elems[j]=m.elems[j];
        }
}
matrix::~matrix() {
        delete []elems;
}
int tt(int a,int b) {
        return a+b;
}
int matrix::set (short vrow,short vcol,double val) {
        if (vrow >= 0 && vrow < row && vcol >= 0 && vcol < col ) {
                elems[vrow * col + vcol] = val;
                return 0;
        } else {
                return 1;
        }
}

int main() {

        matrix m1(1,1),m2(1,1);
        m1.set (0,0,1);
        m2 = m1 ;
        return 1;
}

guojkd

int main() {

        matrix m1(1,1),m2(1,1);
        m1.set (0,0,1);
        m2 = m1 ;
        return 1;
}

加一个函数：
matrix &matrix:perator=(const matrix &m)
{
    if(this == &m) return *this;
    row = m.row;
    col = m.col;
    if(elems) {delete []elems; elems=NULL;}
    elems = new double[row*col];
    for(int j = 0; j< row * col ; j++) {
               elems[j]=m.elems[j];
        }
}
当类里面有指针的时候，用默认的=重载会有问题。

[ 本帖最后由 guojkd 于 2008-5-30 12:06 编辑 ]

welcome008

原帖由 guojkd 于 2008-5-30 11:38 发表
int main() {

        matrix m1(1,1),m2(1,1);
        m1.set (0,0,1);
        m2 = m1 ;
        return 1;
}

这部分怎么了？

能说明白些吗？

guojkd

你在m2=m1的时候会调用=函数，在类里面有指针的话，不能使用默认的=函数，需要自己重载=。

如果用拷贝构造函数，应该是这样：matrix m2 = m1; 或者matrix m2(m1);

原帖由 guojkd 于 2008-5-30 11:38 发表
int main() {

        matrix m1(1,1),m2(1,1);
        m1.set (0,0,1);
        m2 = m1 ;
        return 1;
}

加一个函数：
matrix &matrix:perator=(const matrix &m)
{
    if(this == &m ...

welcome008

原帖由 guojkd 于 2008-5-30 14:17 发表
你在m2=m1的时候会调用=函数，在类里面有指针的话，不能使用默认的=函数，需要自己重载=。

如果用拷贝构造函数，应该是这样：matrix m2 = m1; 或者matrix m2(m1);

呵呵，终于明白了，鞠躬致谢！

welcome008

不过还有一问：

matrix m2;
m2 = m1;

和
matrix m2 = m1;

他们的差别在哪里？
难道说：
matrix m2 = m1;是调用了拷贝构造函数？
而第一个没有只是使用了默认的浅拷贝？

guojkd

个人理解：
matrix m2;   //这时候已经调用构造函数了，已经生成对象了
m2 = m1;    //现在就是赋值操作了

而
matrix m2 = m1;   //使用拷贝构造函数生成对象

77h2_eleven

为null时free是不会出问题的吧？