Problem with operator ==
-
13-09-2019 - |
Question
I am facing some problem with use of operator == in the following c++ program.
#include < iostream>
using namespace std;
class A
{
public:
A(char *b)
{
a = b;
}
A(A &c)
{
a = c.a;
}
bool operator ==(A &other)
{
return strcmp(a, other.a);
}
private:
char *a;
};
int main()
{
A obj("test");
A obj1("test1");
if(obj1 == A("test1"))
{
cout<<"This is true"<<endl;
}
}
What's wrong with if(obj1 == A("test1"))
line ?? Any help is appreciated.
Solution
bool operator ==( const A &other)
Use const reference, so a temporary object that is constructed in if statement can be used as parameter for operator==.
OTHER TIPS
strcmp
returns 0 when the strings are equal, so you want:
return strcmp(a, other.a) == 0;
You should also use a const
reference like Cătălin Pitiș says in his answer, because then you can use temporary objects with the operator, and you should also make the method itself const
(since it doesn't modify the object) as Andreas Brinck says in the comments below. So your method should be:
bool operator ==(const A &other) const
{
return strcmp(a, other.a) == 0;
}
It looks to me like you want this in your operator:
strcmp(a, other.a) == 0
strcmp
returns 0 when strings match, and a number indicating whether the comparison is greater or less than otherwise.
Your error is that you create an instant value and pass it as reference to the operator==
method. But your error is in your operator definition that should be:
bool operator==(const A& other) const
the body being the same.