C++ operator overloading error C2804 too many parameters

Asked Feb 08 '18 at 20:14

Active Feb 08 '18 at 20:39

Viewed 162 times

I am trying to create an operator overloading function to add 2 vectors. Declaration goes like this:

class VecOperations
{
public:
    std::vector<double> operator + (const std::vector<double> &, const std::vector<double> &);
};

But there is error binary operator + has too many parameters. Any idea why this is so? I want to add 2 vectors with operator overloading.

Edit: After taking a look at https://stackoverflow.com/a/4421719/9329547

class X {
  X& operator+=(const X& rhs)
  {
    // actual addition of rhs to *this
    return *this;
  }
};
inline X operator+(X lhs, const X& rhs)
{
  lhs += rhs;
  return lhs;
}

I am confused as in my question I am not passing class instance to the operator overloaded function, but passing 2 vectors. Can someone explain what is proper way to declare the function?

edited Feb 08 '18 at 20:39

asked Feb 08 '18 at 20:14

bn4365

3

The non-static member functions have `this` as one argument and need only one to be binary operators. – Mihayl Feb 08 '18 at 20:16
2

You probably want a namespace instead of class here. – Jarod42 Feb 08 '18 at 20:16
There are two proper ways, your edit shows the two alternatives. – Micha Wiedenmann Feb 09 '18 at 08:02

C++ operator overloading error C2804 too many parameters

0 Answers0