xiaobei 发表于 2015-4-9 20:35:31

C++副本构造器的问题

同样的一段代码在code：：blocks运行没问题，但是在VC6.0中就出现问题；代码如下：


int main()
{
        MyClass obj1( new int(1) );
        MyClass obj2( new int(2) );
        obj2 = obj1;
        obj1.print();
        obj2.print();

        std::cout << "----------------------\n";

       MyClass obj3( new int(3) );
        MyClass obj4 = obj3;
       
        obj3.print();
        obj4.print();


        std::cout << "-----------------------\n";

        MyClass obj5( new int(5) );
        obj5 = obj5;
        obj5.print();

        return 0;
}


仰望天上的光 发表于 2015-4-9 21:55:00

我猜：这个和副本构造器（拷贝构造函数没关系），是你没有重载operator=这个赋值运算符；这个赋值运算符的默认实现是浅拷贝，也就是说
obj2 = obj1;后，obj1和obj2中的int*指针指向了同一个new出来的对象（有没发现这里obj2new出来的new int(2)你已经无法访问了，这回导致内存泄漏），接着当main函数结束，obj2析构的时候delete一次该int*对象；obj2析构的时候，企图再次delete该int*所以报错。
code：：blocks运行没问题不等于说它每次都运行没问题，每次都运行没问题也不等于说你的程序没有严重的问题。

最后，希望你不要总是把别人看成神仙。。。不把 MyClass的实现贴出来就问问题。。。

Victory_6226 发表于 2015-4-10 08:22:41

对，把myClass贴出来看看啊

xiaobei 发表于 2015-4-10 15:17:13

#include <iostream>
#include <string>

class MyClass
{
public:
        MyClass(int *p);
        MyClass(const MyClass &rhs);
        ~MyClass();

        MyClass &operator=(const MyClass &rhs);
        void print();
private:
        int *ptr;
};

MyClass::MyClass(int *p)
{
        std::cout << "进入主构造器\n";
        ptr = p;
        std::cout << "离开主构造器\n";
}

MyClass::MyClass(const MyClass &rhs)
{
        std::cout << "进入副本构造器\n";
        *this = rhs;
        std::cout << "离开副本构造器\n";
}

MyClass::~MyClass()
{
        std::cout << "进入析构器\n";
        delete ptr;
        std::cout << "离开析构器\n";
}
//a = a;a = b;

//重载赋值操作符；
MyClass &MyClass::operator=(const MyClass &rhs)
{
        std::cout << "进入赋值语句重载\n";
        if( this != &rhs )
        {
                delete ptr;

                ptr = new int;
                *ptr = *rhs.ptr;
        }
        else
        {
                std::cout << "赋值号两边为同个对象，不作处理！\n";
        }

        std::cout << "离开赋值语句重载\n";

        return *this;
}

void MyClass::print()
{
        std::cout << *ptr << std::endl;
}

xiaobei 发表于 2015-4-10 15:19:40

仰望天上的光 发表于 2015-4-9 21:55
我猜：这个和副本构造器（拷贝构造函数没关系），是你没有重载operator=这个赋值运算符；这个赋值运算符的 ...

我的错，没说清楚。在类里面重载了赋值运算符，就是main函数里，两个实例，如果去掉obj3和obj4，或者改成和obj1，obj2一样的赋值形式就没有问题。

仰望天上的光 发表于 2015-4-10 15:48:45

xiaobei 发表于 2015-4-10 15:19
我的错，没说清楚。在类里面重载了赋值运算符，就是main函数里，两个实例，如果去掉obj3和obj4，或者改成 ...

引发问题的语句是：
MyClass obj4 = obj3;
这里调用了拷贝构造函数，然而你的拷贝构造函数直接调用了重载的赋值运算符。
重载的赋值运算符里做了：
if( this != &rhs )
      {
                delete ptr;

                ptr = new int;
                *ptr = *rhs.ptr;
      }
这里delete ptr;会被执行。然而，你现在在构造函数中还没有为ptr new出一块空间，所以报错。

恍恍惚惚 发表于 2017-11-27 12:40:56

仰望天上的光 发表于 2015-4-10 15:48
引发问题的语句是：
MyClass obj4 = obj3;
这里调用了拷贝构造函数，然而你的拷贝构造函数直接调用了重 ...

但是那个代码的目的是为了测试直接定义个新MyClass变量，用boj3直接赋值给他的啊，目的不变的情况下怎么办。不然就直接MyClass obj4（new int （2））；就达不到之前的目的了。

Zy沉下心 发表于 2018-2-1 22:35:55

代码有问题

兮离 发表于 2018-4-11 15:49:27

仰望天上的光 发表于 2015-4-10 15:48
引发问题的语句是：
MyClass obj4 = obj3;
这里调用了拷贝构造函数，然而你的拷贝构造函数直接调用了重 ...

此处如果注释掉delete ptr；应该会导致内存泄漏吧，那可不可以在前面加一句让*ptr=0；然后再删除呢，这样其实也起到了初始化指针指向内存的问题。
页: [1]
查看完整版本: C++副本构造器的问题