注意:这个问题遵循a previous one,我希望仍然可以将其作为一个新问题提出来。
我正在尝试为树类实现“三个半大规则”(copy-and-swap idiom),如下所示:
class Tree
{
friend void swap(Tree &first, Tree &second); // Swap function
public:
Tree(const double &a, const double &b, int depth); // Public constructor (derived from the default (private) constructor)
Tree(const Tree &other); // Copy constructor
~Tree(); // Destructor
Tree & operator=(Tree other); // Copy-assignement operator
private:
Tree(double *a, double *b, int depth, int maxDepth); // Default (private) constructor
double *a, *b;
int depth, maxDepth;
Tree *leftChild, *rightChild;
};
我一直在尝试关注this guideline。这是我的复制赋值运算符的样子:
Tree & Tree::operator=(Tree other)
{
swap(*this, other);
return *this;
}
我很难让我的公共构造函数工作。有人建议我这样做:
Tree::Tree(const double &a, const double &b, int depth)
{
double aTemp(a), bTemp(b);
swap(*this, Tree(&aTemp, &bTemp, depth, depth));
}
我不确定这个想法是否奏效。无论如何,我从编译器中得到以下错误:
invalid initialization of non-const reference of type 'Tree&' from an rvalue of type 'Tree'
in passing argument 2 of 'void swap(Tree&, Tree&)'
我尝试了以下想法,我认为这会起作用:
Tree::Tree(const double &a, const double &b, int depth)
{
double aTemp(a), bTemp(b);
*this = Tree(&aTemp, &bTemp, depth, depth);
}
但它似乎也没有起作用。我认为问题在于,当我调用复制赋值运算符(*this = Tree(&aTemp, &bTemp, depth, depth)
)时,应该调用复制构造函数(因为复制 - 分配运算符的参数是按值传递的),但似乎这是没有发生。我不明白为什么。
提前感谢您的帮助!
答案 0 :(得分:1)
“Tree&”类型的非const引用的无效初始化来自'Tree'类型的右值 传递'void swap(Tree&,Tree&)'的参数2
C ++不允许通过非const
引用传递匿名对象。目的是防止调用者意外丢弃写入引用参数的函数的结果。
你可以做:
Tree::Tree(const double &a, const double &b, int depth)
{
double aTemp(a), bTemp(b);
Tree temp(&aTemp, &bTemp, depth, depth);
swap(*this, temp);
}
但它似乎也没有起作用。我认为问题是这样的 当我调用复制赋值运算符时(* this = Tree(& aTemp,& bTemp, 深度,深度)),应该调用复制构造函数(因为 copy-assignement运算符的参数由value)传递,但它 似乎没有发生这种情况。我不明白为什么。
你是如何确定它不起作用的?编译器可能会删除远离副本以避免执行不必要的工作。 (That is why your copy-assignment operator takes the argument by value.)
顺便说一句,如果你的编译器支持C ++ 11,你可以改用delegating constructors。