副本构造器（源自小甲鱼的C++快速入门）

清风袋袋

在小甲鱼的C++快速入门当中，[36]副本构造器那一课。为了避免“逐个复制”的问题，使用重构“=”赋值还有自己定义副本构造器的方法。仿造小甲鱼代码如下：

1. #include <iostream>
   
2. 
   
3. class MyClass
   
4. {
   
5. public:
   
6.         MyClass(int *p);
   
7.         MyClass(const MyClass &rhs);
   
8.         ~MyClass();
   
9. 
   
10.         MyClass & operator = (const MyClass &rhs);
    
11.         void print();
    
12. private:
    
13.         int *ptr;
    
14. };
    
15. 
    
16. MyClass::MyClass(int *p)
    
17. {
    
18.         std::cout << "进入主构造器\n";
    
19.         ptr = p;
    
20.         std::cout << "离开主构造器\n";
    
21. }
    
22. 
    
23. MyClass::MyClass(const MyClass &rhs)
    
24. {
    
25.         std::cout << "进入副本构造器\n";
    
26.         *this = rhs;
    
27.         std::cout << "离开副本构造器\n";
    
28. }
    
29. MyClass::~MyClass()
    
30. {
    
31.         delete ptr;
    
32. }
    
33. 
    
34. 
    
35. MyClass & MyClass::operator = (const MyClass &rhs)
    
36. {
    
37.         std::cout << "进入赋值语句重载\n";
    
38.         if(this != &rhs)
    
39.         {
    
40.                 delete ptr;                //必须先删除之前的内存
    
41. 
    
42.                 ptr = new int;
    
43.                 *ptr = *(rhs.ptr);
    
44.         }
    
45. 
    
46.         std::cout << "离开赋值语句重载\n";
    
47.         return *this;
    
48. }
    
49. 
    
50. void MyClass::print()
    
51. {
    
52.         std::cout << *ptr <<"\n";
    
53. }
    
54. 
    
55. int main()
    
56. {
    
57.         MyClass obj3(new int(3));
    
58.         MyClass obj4 = obj3;
    
59.         obj3.print();
    
60.         obj4.print();
    
61. 
    
62.         std::cin.get();
    
63.         return 0;
    
64. }

复制代码

应该是没有问题的，但是运行总是出错，出错语句:

1. MyClass & MyClass::operator = (const MyClass &rhs)
   
2. {
   
3. std::cout << "进入赋值语句重载\n";
   
4. if(this != &rhs)
   
5. {
   
6. delete ptr;   //总是在此处出错
   
7. 
   
8. ptr = new int;
   
9. *ptr = *(rhs.ptr);
   
10. }
    
11. 
    
12. std::cout << "离开赋值语句重载\n";
    
13. return *this;
    
14. }

复制代码

各位请帮帮忙啊~

Xiao_肚oO

你的 MyClass obj4 = obj3;这句话说明obj4里面的int*ptr没有申请过内存，那么当你delete ptr的时候肯定就要报错啰。

仰望天上的光

MyClass obj3(new int(3));
这句使得obj3.ptr指向动态分配出来的空间（为方便描述，称之为空间A）
MyClass obj4 = obj3;
这句使得obj4.ptr也指向空间A

obj4对象消亡的时候会释放空间A。
obj3对象消亡的时候企图再次释放空间A，所以报错

本质原因是副本构造函数写得有问题，应该使用深拷贝而非浅拷贝（若看不懂“深拷贝”和“浅拷贝”请自行baidu）

清风袋袋

仰望天上的光 发表于 2015-1-3 13:10
MyClass obj3(new int(3));
这句使得obj3.ptr指向动态分配出来的空间（为方便描述，称之为空间A）
MyClas ...

但是我已经对“=”赋值语句进行了重载，

1. MyClass & MyClass::operator = (const MyClass &rhs)
   
2. {
   
3.         std::cout << "进入赋值语句重载\n";
   
4.         if(this != &rhs)
   
5.         {
   
6.                 delete ptr;                //必须先删除之前的内存
   
7. 
   
8.                 ptr = new int;
   
9.                 *ptr = *(rhs.ptr);
   
10.         }
    
11. 
    
12.         std::cout << "离开赋值语句重载\n";
    
13.         return *this;
    
14. }

复制代码

这里指针已经不是指向同一个内存了，属于深拷贝。
但是我发现，将“delete ptr;”删掉就没有问题了。那么是不是可以解释成：在拷贝前，并没有生成obj4这个内存，但是却对其进行了删除，导致出错。我这样的解释对吗？

清风袋袋

Xiao_肚oO 发表于 2015-1-2 23:32
你的 MyClass obj4 = obj3;这句话说明obj4里面的int*ptr没有申请过内存，那么当你delete ptr的时候肯定就要 ...

恩恩，的确是这样的，看来源码就是有点问题，十分谢谢您。

仰望天上的光

清风袋袋 发表于 2015-1-3 21:12
但是我已经对“=”赋值语句进行了重载，
这里指针已经不是指向同一个内存了，属于深拷贝。
但是我发现 ...

你的程序根本没有使用到赋值运算符

仰望天上的光

MyClass obj4 = obj3;
这里调用的是拷贝构造函数而不是赋值运算符。

清风袋袋

仰望天上的光 发表于 2015-1-4 10:58
MyClass obj4 = obj3;
这里调用的是拷贝构造函数而不是赋值运算符。

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

复制代码

副本构造器里面有调用吧？
MyClass obj4 = obj3;首先进入副本构造器，然后调用重载的“=”运算符，不是这样吗？

GRRRARD

清风袋袋 发表于 2015-1-9 11:52
副本构造器里面有调用吧？
MyClass obj4 = obj3;首先进入副本构造器，然后调用重载的“=”运算符，不 ...

我运行了一下，没错啊

清风袋袋

GRRRARD 发表于 2015-1-9 15:18
我运行了一下，没错啊

小甲鱼也没错。。。。就是在我这有错，哎，算了，这个问题。但其实理论上是有错的，需要改的~