VPF::Определенный оператор сравнения

IValdemar

Дата 20.3.2012, 22:14 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 43
Регистрация: 10.1.2012

Репутация: нет
Всего: нет

Собственно вот в чем проблема:
у меня в программе есть такая строчка

Код


POINT_R2 delta,LastDirection;
//..Тут строчек 200 кода
if (delta==LastDirection) //...Вот тут выкидывает ошибку

В ошибке говорится, что он не может понять какой оператор == использовать
candidates are: bool POINT_R2::operator==(POINT_R2&)
BOOL operator==(const GUID&, const GUID&)

Подскажите как можно объяснить компилятору что надо использовать первый вариант smile

класс POINT_R2 описан в отдельном header'e там оператор сравнения определен таким образом

Код


bool operator==(POINT_R2& second)
    {
        return this->x==second.x&&this->y==second.y;
    }

Я так понимаю тут что-то с namespace надо намудрить, так как если я делаю в header'e такую функцию

Код


bool cmp(POINT_R2 first, POINT_R2 second)
{
    return first==second;
}

и сравниваю ей, то все работает.

mes

Дата 20.3.2012, 22:23 (ссылка)

(нет голосов)

Загрузка ...

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 144
Всего: 250

IValdemar, покажите все описание POINT_R2

--------------------

http://opendots.net

IValdemar

Дата 20.3.2012, 22:41 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 43
Регистрация: 10.1.2012

Репутация: нет
Всего: нет

Вот полное описание smile

Код


class POINT_R2
{
    public:
    POINT_R2(int X=0,int Y=0):x(X),y(Y){};
    POINT_R2(COORD P):x(P.X),y(P.Y){}
    ~POINT_R2(){};
    friend COORD C_(POINT_R2 P);
    POINT_R2& operator=(POINT_R2 Other)
    {
        this->x=Other.x;
        this->y=Other.y;
        return *this;
    }
    POINT_R2& operator=(COORD Other)
    {
        this->x=Other.X;
        this->y=Other.Y;
        return *this;
    }
    bool operator==(POINT_R2& second)
    {
        return this->x==second.x&&this->y==second.y;
    }
    bool operator!=(POINT_R2& second)
    {
        return !((*this)==second);
    }
    int x,y;
};

volatile

Дата 20.3.2012, 23:17 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2107
Регистрация: 7.1.2011

Репутация: 37
Всего: 85

Цитата(IValdemar @ 20.3.2012, 22:41

)

  bool operator==(POINT_R2& second)
    {
        return this->x==second.x&&this->y==second.y;
    }

добавьте здесь конст. Полюбому он здесь нужен, скорей всего и проблема решится

Код


bool operator==(const POINT_R2& second) const
{
    return this->x==second.x&&this->y==second.y;
}

Это сообщение отредактировал(а) volatile - 20.3.2012, 23:19

IValdemar

Дата 20.3.2012, 23:27 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 43
Регистрация: 10.1.2012

Репутация: нет
Всего: нет

volatile
Большое спасибо, проблема исчезла. smile

Но мне все таки не понятно, почему const решил все проблемы? В общем можно это по подробнее освятить, дабы я понял зачем он там добавляется.
Я знаю что const добавляют там где элемент не должен меняться, это является хорошим тоном, то есть по идее тут он как раз к месту, но почему без него вылетает ошибка? Это ведь не более чем хороший тон программирования.

Это сообщение отредактировал(а) IValdemar - 20.3.2012, 23:28

volatile

Дата 21.3.2012, 00:01 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2107
Регистрация: 7.1.2011

Репутация: 37
Всего: 85

Цитата(IValdemar @ 20.3.2012, 23:27

)

почему без него вылетает ошибка? Это ведь не более чем хороший тон программирования.

В принципе в вашем коде, верней в том виде как вы его привели, ошибки быть не должно.
Телепатически я просто подумал, что где-то константность появляется.
может быть где-то здесь:

Цитата(IValdemar @ 20.3.2012, 22:14

)

//..Тут строчек 200 кода

Чисто асбстрактно, если все писать плохим стилем, (и не использовать библиотеки написанные хорошим стилем), то без константности можно обойтись.
Хотя этого делать категорически не рекомендуется!
Мой вам совет ставьте константость, везде где объект не меняется, не пожалеете.
Кстати, в вашем коде это не единственное место, где не помешало бы добавить константность.

mes

Дата 21.3.2012, 00:08 (ссылка)

(нет голосов)

Загрузка ...

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 144
Всего: 250

Цитата(IValdemar @ 20.3.2012, 22:27

)

но почему без него вылетает ошибка

Вы уверены что привели ошибочную строчку верно ? может там сравнивается возвращаемое из функции ?

Добавлено через 4 минуты и 49 секунд
полагаю Ваша ошибка неподолеку от этого примера :
http://liveworkspace.org/code/3e95956bea39...9273c3bb0fdff6c

Добавлено через 6 минут и 17 секунд
и в след. раз желательно сообщение об ошибке и саму строчку передавать точно, а не своими словами..
а еще лучше сделать минимальный пример , демонстрирующий ошибку...

--------------------

http://opendots.net

IValdemar

Дата 21.3.2012, 15:11 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 43
Регистрация: 10.1.2012

Репутация: нет
Всего: нет

Такс, хотел сделать чистый пример и показать вам ошибку. НО! Как я ни изощрялся не получилось у меня сделать там ошибку smile

Даже без const все работает, так что видимо действительно

Цитата(volatile @ 21.3.2012, 00:01

)

может быть где-то здесь:

Цитата(IValdemar @ 20.3.2012, 22:14

)

//..Тут строчек 200 кода

Что-то я там напортачил smile

Всем спасибо за участие и помощь. Тему закрываю.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »