c++ - 重载运算符 + c++ eclipse

Question

我正在尝试重载运算符 +。 重要的是，operator + 准确地获取了我想要它获取的数据，但是一旦我将它发送给 operator=，其中的所有指针由于某种原因都被删除了。

Movie& Movie:: operator+ (const Movie& other) {

Movie toReturn,toCheck;
Worker* toAdd;
std::list<Worker*>::iterator checkSecond;
length > other.getLength() ?  toReturn=*this, toCheck=other : toReturn =  
other , toCheck = *this;

//I WANT TO TAKE THE LONGER MOVIE, AND ADD THE WORKERS FROM THE SHORTER 
  MOVIE TO THAT MOVIE

if(toCheck.getNumOfWorkers() > 0 ) //LOOK FOR WORKERS IN SHORT MOVIE

{
for (checkSecond=toCheck.getWorkersInMovie().begin(); checkSecond != 
toCheck.getWorkersInMovie().end(); ++checkSecond)
           {
                toAdd= (*checkSecond);
                toReturn.addWorker(toAdd);
                toAdd = NULL;


            } //END FOR
    }
}
}

*this=toReturn;
cout << "this is what my new object has after +" <<endl; 
this->printMovie(); //PRINTS EXACTLY WHAT I EXPECT, WITH NEW WORKERS
return *this;

}

但是当我将它发送给 operator = 时，它并没有按预期被复制，比如说 worker 的数量是 0。 MOVIE BOTH = MOVIEFIRST + MOVIETOADD

Movie& Movie:: operator= (const Movie& other) {

WHEN IT GETS HERE, MOVIE OTHER GETS DELETED..
return *this;

}

复制构造函数在所有情况下都按预期工作:

Movie::Movie(const Movie& toCopy) {

  cout << "inside copy" << endl;
 *this=toCopy;
}

帮助...？

Answer 1

运算符 + 为 A + B 之类的表达式调用。从语义上讲，这个表达式不应该改变 A 的状态，而是返回一个表示总和的新对象。

因此，重载 + 运算符的正确签名应该如下所示:

Movie Movie::operator+ (const Movie& other) const { }

如果你想避免创建这个临时对象，那么你不应该重载 + 运算符，而是重载 += 如下:

Movie& Movie::operator+= (const Movie& other) { }

由于 += 具有赋值语义，因此您可以修改对象。

遵循此规则将帮助您解决您在代码中遇到的问题。