sort() function for user defined type

[alice]

When I'm trying to compile the below program on the GPP compiler, it is
giving me errors like
" no match for operator < ..."

Can anybody please help me figure out the error?
#include <iostream>
#include <string>
#include <algorithm>

using namespace std;

typedef struct node
{
int i;
int j;
bool operator<(struct node& n1)
{
return (*this.i < n1.i);
}
}node;

int main(void)
{
node data[10];
sort(data,data+10);
return 0;
}

[Victor Bazarov]

alice wrote:

When I'm trying to compile the below program on the GPP compiler, it
is giving me errors like
" no match for operator < ..."

Take a habit not to use '...' in your posts unless absolutely necessary.

>
Can anybody please help me figure out the error?
#include <iostream>
#include <string>
#include <algorithm>

using namespace std;

typedef struct node

Drop this C habit. Should just be

struct node

{
int i;
int j;
bool operator<(struct node& n1)

bool operator<(node const& n1) const

{
return (*this.i < n1.i);
}
}node;

And remove it here. Should just be

};

>
int main(void)
{
node data[10];
sort(data,data+10);
return 0;
}

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[Thomas Tutone]

Victor Bazarov wrote:

You missed one.

#include <iostream>
#include <string>
#include <algorithm>

using namespace std;

typedef struct node

Drop this C habit. Should just be

struct node

{
int i;
int j;
bool operator<(struct node& n1)

bool operator<(node const& n1) const

{
return (*this.i < n1.i);

The above line should be:

return (i < n1.i);

or if you insist on keeping the "this" (Why?), then it should be:

return (this->i < n1.i);

Best regards,

Tom