Ananimous(1) C++找bug

class CArray
{
public:
        CArray() { pArr = NULL; nNum = 0; }
        ~CArray() { delete []pArr; }

        void Print()
        {
                if (nNum <= 0 || pArr == NULL) return;
                for (int i = 0; i < nNum; i++)
                        cout<<pArr<<" ";
                cout<<endl;
        }

        CArray& operator = (const CArray& arr)
        {
                if(arr.nNum <= 0) return *this;
               
                delete []pArr;
                pArr = new int[arr.nNum];
                nNum = arr.nNum;

                for (int i = 0; i < nNum; i++)
                        pArr[i] = arr.pArr[i];
                return *this;
        }

        void Init(int n, int* a = NULL)
        {
                assert(n>0);

                delete[] pArr;
                pArr = new int[n];
                nNum = n;
                memset(pArr, 0, sizeof(nNum*sizeof(int)));

                if (a != NULL)
                {
                        for (int i = 0; i < n; i++)
                                pArr[i] = a[i];
                }
        }

private:
        int* pArr;
        int  nNum;
};

void main()
{
    ...XX....
    ...OO....

    CArray arr2 = arr1;
    arr2.Print();

    ...XXOO.....
}

前段时间发现的一个bug,自己的疏忽。
类写的有一处可能引起使用时程序的崩溃，请指出。[/i][/i][/i][/i]

       CArray operator = (const CArray& arr)
        {
                if(arr.nNum <= 0) return *this;  //为0的时候也应该做的？
               
                delete []pArr;
                pArr = new int[arr.nNum];
                nNum = arr.nNum;

                for (int i = 0; i < nNum; i++)
                        pArr = arr.pArr;    //[i]P.S. 论坛自动把【i】去掉了？[/i]
[i]

                return *this;
        }


[/i]

没人会吗？？？？

No checking for assignment to self

And this
CArray arr2 = arr1
calls copy ctor not operator=

And this
CArray arr2 = arr1
calls copy ctor not operator=
vvnesaa 发表于 2011-4-19 10:21

    Absolutely right!!

其实我想问，这个类如果我在主程序里定义了个instance，但是没有做任何操作，程序运行完成后自动释放内存的时候是不是会挂掉？ destructor里面的delete 似乎会尝试去释放NULL。。。

其实我想问，这个类如果我在主程序里定义了个instance，但是没有做任何操作，程序运行完成后自动释放内存的 ...
danielgao 发表于 2011-4-19 12:04

delete NULL 不会出错

回复 6# wwwyhx

能讲一下为什么会出错？还有怎样写才对啊？

回复 9# kidd_yq@163.com

当你的类里面有指针的时候，通常你不希望多个实例里面的指针指向同一个东西，不仅仅因为修改一个实例会影响到其他实例，还因为一个实例被销毁的时候会导致其他实例里面的指针变成dangling pointer

所以
当你的类里面有指针的时候，至少需要写这些东西：
析构函数，用来delete指针
复制构造函数，用来在复制的时候new一个新指针然后把指针指向的东西copy过来
operator=，同上，通常operator=和复制构造函数写一个另一个一定要写的，否则这个类的行为会不consistent。同时灰常重要的，operator=一定要有self assignment检查，否则很可能在碰到a = a这种赋值的时候挂掉。