这是一个有效的Copy Ctor吗？

Question

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

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

Answer 1

两件事：

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

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

Answer 2

有3个问题。

首先，你忘了“;”在m = a.m的末尾，所以你的代码不会编译。

其次，在传递大小的东西的大多数情况下，首选通过引用 大于平台上的寄存器大小。

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

  

A（const A＆amp; a）：m（a.m）{}

Answer 3

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

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