使用vector的push_back()方法，编译错误

wuzen007

vector<Diameter> return_msgs;
Diameter msg;
return_msgs.push_back(msg);
主要程序如上所示，Diameter类为自己创建的类，程序编译时出现错误如下：
"/opt/aCC/include_std/vector.cc", line 121: error #2349: no operator "="
          matches these operands
            operand types are: Diameter = const Diameter
          *__position = __x;
                      ^
          detected during instantiation of "void std::vector<_TypeT,
                    _Allocator>::_C_insert_aux(_Allocator::pointer,
                    _Allocator::const_reference) [with _TypeT=Diameter,
                    _Allocator=std::allocator<Diameter>]" at line 566 of
                    "/opt/aCC/include_std/vector"

求大神解释是什么原因导致的？

sxcong

错误提示是Diameter里面没有合适地处理=
有两种方法解决，一是vector<Diameter*>,用指针。另一种就是自己实现=的重载。
因为push的过程，执行了一次赋值

wuzen007

push_back()不是调用构造函数吗？回复 2# sxcong


   

myworkstation

回复 3# wuzen007

Diameter这个类重载操作符operator "="了？看不到这个类的定义很难断定问题在哪儿，貌似是因为重载操作符不正常导致无法进行匹配。这个push_back内部实现应该是有个临时对象，其内部有如下过程iameter __x = value;(这儿调用的是copy构造函数，然后是*__position = __x;此时调用的是=操作符。
   

wuzen007

这样啊，是有重载了“=”，如下所示：
Diameter& Diameter:perator=(Diameter& msg)
{
    if(this == &msg)
        return *this;
    string raw;
    msg.dumpto(raw);
    parser(raw.data(), raw.length());
    return *this;
}
回复 4# myworkstation


   

myworkstation

回复 5# wuzen007


    需要有这样的原型：Diameter& Diameterperator=(const Diameter& msg), const是关健，你再定义一个函数或者改一下原有的这个。

wuzen007

嗯，但换成构造函数不加const行吗？还有一个问题想请教就是你说的push方法为什么第一次会调构造函数，第二次变成调=操作符了
回复 6# myworkstation


   

myworkstation

回复 7# wuzen007

Diameter __x = value;这种方式是典型的拷贝初始化(copy initialize)，它调用的是copy构造函数，在变量定义时初始就是这样，而定义变量后再进行单独赋值则是正常的Assignment操作了。
重载操作符=时，肯定应该是const引用的参数。你进行赋值肯定不应该修改参数所代表的原对象，你修改只是自身对象，如果在这个操作符中能修改原对象，从设计上就是不合理的，其次由于函数返回（返回值非常引用时）或者显示的临时对象都是rvalue只能赋值给const引用，那么在这种情况下如果参数不是const那么同样无法进行匹配，那样的操作符重载太有局限性了，所以参数总应该是const引用，这样设计最合理，适配性最强。
   

wuzen007

大致懂了，感谢大神！
回复 8# myworkstation