需要左值作为运算符重载的赋值的左操作数

Question

我正在尝试进行运算符重载，但当我从 .cpp 运行此代码时不断收到错误

头文件原型是这样的，

const Velocity & operator = (const Velocity & rhs);

.cpp 成员函数看起来像这样。

const Velocity & Velocity :: operator = (const Velocity & rhs)
{
   this->getDx() = rhs.getDx();
   this->getDy() = rhs.getDy();
   this->getX() = rhs.getX();
   this->getX() = rhs.getY();
   return *this;
}

在 main 中使用运算符的地方是这样的。

Velocity v1;
Velocity v2;
Velocity v3;

promptVelocity("Enter values for Velocity 1", v1);
promptVelocity("Enter values for Velocity 2", v2);

// Test operator =
v3 = v1;
cout << "Test operator = : " << v3 << endl;

我得到了这个->行的错误

lvalue required as left operand of assignment

请帮忙！

Answer 1

将您的功能更改为：

Velocity& Velocity::operator=(const Velocity& rhs) {
  dx_ = rhs.getDx();
  dy_  = rhs.getDy();
  x_ = rhs.getX();
  y_ = rhs.getY();
  return *this;
}

...假设“dx_”、“dy_”等是实际变量的名称。赋值的左侧必须是一个“左值”（简单地定义它 - 一个带有地址的命名变量），而在您使用“右值”之前（作为结果返回的临时变量相应的“get”函数）。为了更详细地介绍这一点，很可能您的各种“get”函数返回值类型的对象（例如“int”）或 const 引用（例如“const int&”），但是要分配给某些东西，您确实需要一个可变引用（例如“int&”）。分配给返回简单值类型的“get”函数的输出是没有意义的，因为 get 函数所做的是在返回之前创建底层数据的 副本。虽然这个副本可能有一个与之关联的内存地址（例如，如果它存储在堆栈中），但分配给它不会产生可观察到的效果，因为除非分配它，否则这个副本不会做任何事情或在任何地方看到到某个变量。但是，当您在左侧使用成员变量的名称时，赋值实际上是有意义的，因为接收赋值的变量仍然可以在其他地方引用。

我还应该注意，在我的更改中，我删除了一堆多余的this->es。只有两种情况this-> 是绝对必要的：

1. 有一个同名符号“遮盖”this 的成员（例如，比类范围更窄的参数或其他局部变量），在这种情况下，有必要使用this-> 选择成员而不是本地成员。

2. 非常不精确：涉及template，因此有必要显式使用this-> 以使C++ 在此类而不是命名空间范围内（请参阅dependent name lookup 以获得更精确的解释)。

在没有这两种情况的情况下，符号查找将正确找到没有this-> 的成员变量，因此它是多余的，因此应该避免（在我看来）。

此外，我将赋值的返回类型从“const Velocity&”更改为“Velocity&”，因为赋值运算符通常返回所分配对象的可变实例，否则可能会令人惊讶；例如，使函数返回一个 const 中断：

Velocity velocity = // ...
Velocity default_vaue = // ...
// ... many lines later ...
if (/* some condition /*) {
  ReassignIfInvalid(velocity = ComputeSomething(), default_value);
}

... 其中函数“ReassignIfInvalid”要求参数是可变的，以便它可以有条件地覆盖该值。虽然上面的代码可能不是最好的模拟代码，但如果赋值的输出是不可变的，这将证明是令人惊讶的。

Answer 2

在 .cpp 函数中，您是在告诉 getter 函数应该从另一个函数获取值，因此您实际上并没有做某事。如果有，请使用 setter 函数。尽管你最好公开你的价值观，而不是使用 getter 和 setter。