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

Question

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

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;

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

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+ 应该返回一个三位数（即不是指向一的指针）。对于您的操作员来说，隐式新（或 malloc）数据是非常糟糕的做法，因为用户不会期望这样做。 RVO 意味着归还副本并不是什么大问题。