Ошибка c2804 бинарный operator имеет слишком много параметров

Here is my class:

#ifndef CLOCK_H
#define CLOCK_H
using namespace std;

class Clock
{
    //Member Variables

    private: int hours, minutes;

    void fixTime( );

    public:
        //Getter & settor methods.
        void setHours(int hrs); 
        int getHours() const;
        void setMinutes(int mins); 
        int getMinutes() const; 

        //Constructors
        Clock(); 
        Clock(int);
        Clock(int, int);
        //Copy Constructor
        Clock(const Clock &obj);
        //Overloaded operator functions
        void operator+(const Clock &hours);
        void operator+(int mins);
        void operator-(const Clock &hours);
        void operator-(int minutes1);
        ostream &operator<<(ostream &out, Clock &clockObj); //This however is my problem where i get the error C2804. Saying that it has to many parameters 
};
#endif

All this function is supposed to do is out the values of a clock at different times.

Trying to implement operator overloading using the following code:

class Number
{
    T value;
public:
    Number(T v);
    Number();
    Number<T> operator+ (Number<T>&, const Number<T> &);
    T getValue() { return value; };
};


template <typename T>
Number<T>::Number(T val):value(val) { }

template <typename T> 
Number<T> Number<T>::operator+ (Number<T>& lhs, const Number<T> & rhs)  {    
     return lhs.value + rhs.value;
}

Trying to emulate similar examples found online, but this attempt generates several compiler errors

• ‘{‘ missing function header (old-style format list?)

• binary ‘operator +’ has too many parameters

• class template «Number» has no member «operator+»

  Number<T> Number<T>::operator+ (Number<T>& lhs, const Number<T> & rhs)

With all the decisions: whether or not to include «<T>«; whether or not to use references for sends and returns; whether or not to use «const» and/or «friend»; and whether or not to use «this», «new» and/or «->»; it’s confusing enough to search for outside help :).

Any idea what (many things) I’m doing wrong?

Thanks for your consideration

asked Jan 24, 2022 at 4:15

Remove this line:

Number<T> operator+ (Number<T>&, const Number<T> &);

and define operator+ out of the class body this way:

template<typename T>
Number<T> operator+(Number<T> const& lhs, Number<T> const& rhs) {
    auto ret{ lhs };
    ret.value += rhs.value;
    return ret;
}

answered Jan 24, 2022 at 4:56

Trying to implement operator overloading using the following code:

class Number
{
    T value;
public:
    Number(T v);
    Number();
    Number<T> operator+ (Number<T>&, const Number<T> &);
    T getValue() { return value; };
};


template <typename T>
Number<T>::Number(T val):value(val) { }

template <typename T> 
Number<T> Number<T>::operator+ (Number<T>& lhs, const Number<T> & rhs)  {    
     return lhs.value + rhs.value;
}

Trying to emulate similar examples found online, but this attempt generates several compiler errors

• ‘{‘ missing function header (old-style format list?)

• binary ‘operator +’ has too many parameters

• class template «Number» has no member «operator+»

  Number<T> Number<T>::operator+ (Number<T>& lhs, const Number<T> & rhs)

With all the decisions: whether or not to include «<T>«; whether or not to use references for sends and returns; whether or not to use «const» and/or «friend»; and whether or not to use «this», «new» and/or «->»; it’s confusing enough to search for outside help :).

Any idea what (many things) I’m doing wrong?

Thanks for your consideration

asked Jan 24, 2022 at 4:15

Remove this line:

Number<T> operator+ (Number<T>&, const Number<T> &);

and define operator+ out of the class body this way:

template<typename T>
Number<T> operator+(Number<T> const& lhs, Number<T> const& rhs) {
    auto ret{ lhs };
    ret.value += rhs.value;
    return ret;
}

answered Jan 24, 2022 at 4:56