C++一元 - 运算符重载无法编译

Question

我正在尝试创建一个重载的一元 - 运算符，但无法编译代码。代码的精简版本如下：-

class frag
{
    public:

        frag myfunc  (frag oper1,
                      frag oper2);
        frag myfunc2  (frag oper1,
                      frag oper2);

        friend frag operator + (frag &oper1,
                                frag &oper2);

        frag operator - ()
        {
            frag f;
            f.element = -element;
            return f;
        }

    private:

        int element;

};

frag myfunc (frag oper1, frag oper2)
{
    return oper1 + -oper2;
}

frag myfunc2 (frag oper1, frag oper2)
{
    return oper1 + oper2;
}

frag operator+ (frag &oper1, frag &oper2)
{
    frag innerfrag;
    innerfrag.element = oper1.element + oper2.element;
    return innerfrag;
}

编译器报告...

/home/brian/Desktop/frag.hpp: In function ‘frag myfunc(frag, frag)’:
/home/brian/Desktop/frag.hpp:41: error: no match for ‘operator+’ in ‘oper1 + oper2.frag::operator-()’
/home/brian/Desktop/frag.hpp:16: note: candidates are: frag operator+(frag&, frag&)

有人可以建议我在这里需要做什么吗？

Answer 1

常量正确性

这必须是

 frag operator+ (const frag &oper1, const frag &oper2);

，否则操作数不能是临时的，例如 operator- 的返回值，

而一元减号应该是：

frag operator - () const;

因为它应该' t 修改操作数。

Answer 2

您没有可以对临时对象进行操作的 operator+。临时对象不能作为非常量引用传递。

将 operator+ 的签名更改为：

frag operator + (const frag &oper1, const frag &oper2);

Answer 3

尽管您的问题已经得到了相当好的回答，但我认为值得提及关于您的代码的另一点。现在，您有以下声明：

class frag
{
    public:

        frag myfunc  (frag oper1,
                      frag oper2);
        frag myfunc2  (frag oper1,
                      frag oper2);

...并且您有以下函数：

frag myfunc (frag oper1, frag oper2)
{
    return oper1 + -oper2;
}

frag myfunc2 (frag oper1, frag oper2)
{
    return oper1 + oper2;
}

我猜您打算用这两个函数来实现您在 frag 中声明的成员函数 - 但它们没有't。相反，您有两个由 never Defined 声明的成员函数，并且这两个函数是碰巧具有相似名称的全局函数。为了使它们成为您声明的成员函数，您需要将声明更改为以下内容：

frag frag::myfunc(frag oper1, frag oper2) { 
    return oper1 + -oper2;
}

frag frag::myfunc2(frag oper1, frag oper2) { 
    return oper1 + oper2;
}

另一方面，这些方式也没有任何意义 - 特别是作为成员函数，它们通常会被调用为：a.myfunc(b,c); 不过，它们实际上都是像全局函数一样编写的——作为成员函数，它们通常只需要一个参数，并使用 this 作为第一个参数：

frag frag::myfunc1(frag oper) { 
    return *this + -oper;
}
frag frag::myfunc2(frag oper) { 
    return *this + oper;
}

当然，这可能只是尝试将原始代码减少到发布所需的最低限度而产生的意外副作用。如果是这样，请随意忽略整个“答案”......

Answer 4

答案已经给出（const 参数），但我想提一下，Visual C++ 9 (VS-2008) 确实会在没有警告的情况下编译上述内容。