为什么这个拷贝构造函数不起作用?

时间:2015-12-26 18:10:12

标签: c++ class oop

g ++说调用(Order)(Order&)时没有匹配,当它涉及主程序中调用order的复制构造函数的行时。

Order.h-它包含班级订单

class Order
{

private:

int orderNo;



public:

Order()
{}

//Copy Constructor
Order(Order &o)
{
    orderNo = o.putOrderNo();
}

Order(int i)
{
    orderNo = i;
}

};

错误代码在主文件中 -

bool setCurrentOrder(Order o)
{
     CurrentOrder(o);     //currentOrder is a global var defined elsewhere.
                          //after this, there are some comparisons to determine 
                          //whether the object was copied, then true or false is returned.
}     

编译器拒绝接受“CurrentOrder(o)”行,它将o的详细信息复制到名为CurrentOrder的Order类的另一个对象中。 我是否必须重载=运算符,还是有其他方法可以克服这个问题?

修改

这段代码已经过时了,我已经修复了这个问题,我也重写了这个和其他代码,所以我现在有一些新的,我会尽快问一下!

2 个答案:

答案 0 :(得分:9)

副本构造函数构造一个新对象。由于CurrentOrder已经存在,再次构建它没有意义:CurrentOrder(o)没有意义。要指定 oCurrentOrder的值,请使用作业:CurrentOrder = o;

答案 1 :(得分:0)

好吧,我的问题只是通过重载=运算符来解决。

基本上,我想使用复制构造函数来定义如何复制对象,但是,在这种情况下,运算符重载可能是最好的选择,因为我的目的是将旧对象中的值分配给新对象。 Pete的另一个答案对此进行了解释。

这是如何完成的 -

void operator=(Order &o)
{
 orderNo = o.orderNo;
}

感谢您的建设性批评。在提出更多问题时我会记住这一点。