类型的无效操作数 - C++

Question

我在 c++ 代码中有一个名为 ThreeDigits 的类。我这样重载了 + 操作数：

ThreeDigits* ThreeDigits::operator+(const ThreeDigits &number) const

{
   double result= getNumber()+number.getNumber();
   ThreeDigits* new_result=new ThreeDigits(result);
   return new_result;
}

但是当我写在main函数上时：

    ThreeDigits* first=new ThreeDigits(2.55998);
    ThreeDigits* second=new ThreeDigits(5.666542);
    ThreeDigits* result=first+second;

我收到以下编译错误： ThreeDigits* 和 ThreeDigits* 类型的无效操作数到二元运算符+

你能告诉我是什么问题吗？谢谢

Answer 1

您正在尝试汇总 指向对象的指针，而不是对象本身。要调用重载运算符，您必须在对象上调用它，从而取消对指针的引用。

顺便说一句，使用new 创建所有这些对象是一种糟糕的 C++ 方式；在 C++ 中，与 Java/C# 不同，您应该仅在必要时使用new，并将所有其余部分分配到堆栈上。让operator+ 返回指向新创建对象的指针是可憎的。

编写代码的 C++ 方式是：

ThreeDigits ThreeDigits::operator+(const ThreeDigits &number) const
{
   return ThreeDigits(getNumber()+number.getNumber()); // construct a temporary and return it
}

// ...

ThreeDigits first(2.55998);
ThreeDigits second(5.666542);
ThreeDigits result=first+second;

顺便说一下，重载算术运算符的常用方法是首先重载分配版本（+=、-=、...），然后在它们之上构建“正常”版本。有关运算符重载的更多信息，请参阅operator overloading FAQ。

Answer 2

要按原样使用您的运算符，您需要编写：ThreeDigits* result=*first+*second; 以取消引用指针。

但是，您的运算符至少有两个问题：第一，它违反了最不意外的原则，因为典型的 operator+ 按值返回。其次，它返回一个指向新内存的指针，除非小心，否则很可能会在各种情况下泄漏。

看起来您可能来自 Java 背景，所有内容都被引用计数。相反，惯用的 C++ 实现看起来像这样：

ThreeDigits ThreeDigits::operator+(const ThreeDigits &number) const
{
   double result= getNumber()+number.getNumber();
   return ThreeDigits(result);
}

并使用：

ThreeDigits first(2.55998);
ThreeDigits second(5.666542);
ThreeDigits result = first+second;

Answer 3

您没有取消引用指针。试试这个：

ThreeDigits* result = *first + *second;

Answer 4

您的操作员接受两个 ThreeDigits 引用并返回一个 ThreeDigits 指针。

要按照书面形式使用您的运算符，您必须取消引用 first 和 second：

ThreeDigits* result = *first + *second;

这个操作符有一个不寻常的签名。 operator+ 应该返回一个 ThreeDigits（即不是指向一个的指针）。对于您的操作员来说，隐式地新建（或 malloc）数据是非常糟糕的做法，因为用户不会想到这一点。 RVO 意味着无论如何归还副本并不是什么大不了的事。