Operator overload

[JohnJH]

How come is this incorrect?

class car {
public:
    car(int);
    int HorsePower;
     int getHorsepower();
    car operator == (const car,car rhs){
        if(this->lhs.getHorsepower() < rhs.getHorsepower())
        {
            cout << "Too strong" << endl;
        }
        
        else
            cout << "too weak" << endl;
    }

};

int car::getHorsepower()
{
    return HorsePower;
}
car::car(int c)
{
    HorsePower = c;
}

[MrHutch]

A comparison operator should return a bool. Also, the syntax is a bit messed up there.

Note, the example is a bit simple but it shows how it works.
http://ideone.com/nASIce

[tipaye]

The whole operator== is wrong.

Syntax:
"lhs" is never defined
member operator== should have one parameter

operator == should return something, and that something should be a bool not a car

consider this example

    bool operator == (const car &rhs)
    {
        if(HorsePower < rhs.HorsePower)
        {
            cout << "Too strong" << endl;
            return false;
        }
        else if (HorsePower > rhs.HorsePower)
        {
        	cout << "too weak" << endl;
        	return false;
    	}
    	
    	return true;
    }

[yulingo]

I'd also like to point out that by convention, overloading operator== shouldn't do anything besides returning true or false. You should really write another function besides the operator overload to do the comparison and print the result.