Множество set из пользовательских объектов

[FasterHarder]

Всем хай!
Сразу к делу.
Есть такая структура, описывающая точку на плоскости:

struct Tpoint
{
    int x;
    int y;
    Tpoint(int px, int py)
    {
        x = px;
        y = py;
    }
    bool Tpoint::operator<(Tpoint p)
    {
        if(p.x + p.y < x + y)
            return true;
        else
            return false;
    }
// еще перегрузка стоит на <=, >, >=, != , ==
};

И вот создаю в main множество:

    set<Tpoint> s;
    Tpoint p1(3, 4);
    s.insert(p1);

Компиляция не проходит, ругается на s.insert(p1);
Чего тут нужно еще дописывать, чтобы он смог добавить элемент-точку в множество?

Добавлено 29.01.17, 11:28
P.S. просьба удалить тему-дубликат (что-то при создании проглючило)

M

Готово

Сообщение отредактировано: JoeUser - 01.02.17, 03:44

[shm]

bool operator<(const Tpoint& p) const
{
    return (p.x + p.y < x + y);
}

Добавлено 29.01.17, 11:40

Цитата FasterHarder @ 29.01.17, 11:27

чтобы он смог добавить элемент-точку в множество?

insert добавляет элемент по константой ссылке, а у тебя оператор неконстантен, еще и объявлен неправильно.

Добавлено 29.01.17, 11:52
Ну и да, чисто математически сравнивать x + y неверно. Это даст одинаковый результат для, например, для [1, 0] и [0, 1].

Сообщение отредактировано: shm - 29.01.17, 11:52

[Олег М]

Цитата shm @ 29.01.17, 11:39

Ну и да, чисто математически сравнивать x + y неверно. Это даст одинаковый результат для, например, для [1, 0] и [0, 1].

Тут надо определиться по какому критерию нужно упорядочивать точки. Что вообще x + y означает математически?
Может там надо что-то типа (x == p.x? y < p.y: x < p.x)?

[FasterHarder]

В сравнении на МЕНЬШЕ абсолютно ПО БАРАБАНУ, что стоит там сейчас, т к проблема была в insert(), а не с логикой реализации "<"!

[Олег М]

Согласен, похоже, надо добавить конструктор копирования

[shm]

Цитата Олег М @ 29.01.17, 17:12

надо добавить конструктор копирования

Не нужен он тут.

[Cfon]

Цитата FasterHarder @ 29.01.17, 15:37

проблема была в insert(), а не с логикой реализации "<"!

точнее в const оператора operator<? как сказал shm