Overaload operator of a struct

[luisSimoes]

Hey!

I'm wondering how to overload an operator in a struct.

struct TeamPoints{
string team_name;
int points;
};

What I wanted was to overload "<" and ">" to compare only points and not team_name.

Anyone can help?

[kbw]

bool operator<(const TeamPoints &a, const TeamPoints &b)
{
    return a.team_name < b.team_name;
}

[luisSimoes]

Hey! thnks for helping..

I'm getting this error:

ld: duplicate symbol operator<(TeamPoints const&, TeamPoints const&)in ./Tournament.o and ./Season.o
collect2: ld returned 1 exit status

[tummychow]

There should be a section in the tutorial on this website about operator overload.
Anyway you have seen the basics so just change the operator provided by kbw to:

bool operator<(const TeamPoints &a, const TeamPoints &b)
{
    return a.points < b.points;
}

Chances are the errors you are referring to are a result of the code that calls this operator. Could you post it?

[helios]

Don't put the function definition in the header.

[Galik]

You could always inline it:

inline
bool operator<(const TeamPoints &a, const TeamPoints &b)
{
    return a.team_name < b.team_name;
}

[tummychow]

When did headers get into this?

[helios]

Duplicate function definitions are a common symptom of putting definitions in headers. Since OP appear to be a newbie, I inferred he did that.

[luisSimoes]

it woeked with the inline

[Galik]

inline is probably right for this kind of trivial function, but normally you wouldn't put your function definitions in the header. You would put them in a separate file.

[jsmith]

@Galik:

disagree. the header is the right place for the inline function. you can't put it in a .cpp
file and expect the function to be inlined in any translation unit other than that .cpp file
then.

[helios]

I think what he meant is that most functions should not be inlined.

[Galik]

@helios

Absolutely right. I guess I was a little ambiguous there.

[jsmith]

oh, ok. Comment withdrawn then.