这是一个有效的Copy Ctor吗？

Question

我想知道下面的复制构造函数是否有问题？

class A
{
   private:
      int m;
   public:
      A(A a){m=a.m}
}

Answer 1

两件事情:

• 复制构造函数必须将引用作为参数,否则它们是无限递归的(实际上语言不允许您声明这样的构造函数)

• 它没有做任何默认副本ctor没有做的事情,但做得很糟糕 - 你应该尽可能在复制文件中使用初始化列表.如果默认的复制ctor做你想要的,不要试图自己写一个版本 - 你可能只会弄错,你需要维护它.

Answer 2

有3个问题.

首先,你忘了";" 在m = am的末尾,所以你的代码不会编译.

其次,在您传递大小超过平台上寄存器大小的大多数情况下,首选通过引用.

第三,既然你不打算更改源对象,最好将它与const一起使用.所以,最后这就是我们所拥有的:

A(const A&a):m(am){}

Answer 3

问题是为值传递的参数调用了复制构造函数.因此,如果不需要非终止递归,则必须通过引用(通常是const引用)传递复制构造函数的参数.一个小问题是您没有使用初始化列表,这比初始化成员更可取.解决这两个问题:

A(A const& a)
  : m(a.m)
{}