Как сделать перегрузку «==»?

Question

[totorialman]

Есть класс Planet с перегрузкой friend bool operator==(Planet&, char*); (первый аргумент это переменная из этого же класса, второй переменная извне)
Инициализирую эту перегрузку уже вне класса
bool operator==(Planet& first, char* second) {
if (strlen(first.name) != strlen(second)) return false;
for (int t = 0; t < strlen(first.name); t++) {
if (first.name[t] != second[t]) {
return false;
}
}
return true;
}
(second это char*)
Проблема в том, что перегрузка не вызывается. Но если friend bool operator==(Planet&, Planet&); (и если инициализацию переписать) так, то все работает, подскажите в чем проблема. Если неправильно использую терминологию прошу поправить

Comments

[Dmitrii]

Выложите свой класс и тестируемый код на пастебин, сейчас не очень понятно что происходит. Выглядит так, что один операнд приводится к другому и вызывается оператор сравнения двух двух ссылок Planet.

[totorialman]

Dmitrii,
https://pastebin.com/VA68bdjP оставил только то, что касается перегрузки

[Wataru]

totorialman,

if (planets[i].name == name_d) { // здесь использую перегрузку

Нет. Вы тут не используете перегрузку. Вы сравниваете поле Plant (которое char *) со сторокой. Удалите там .name и тогда будет вызываться перегруженный оператор.

[Dmitrii]

totorialman, все выполняется так как написано:
вы сравниваете поле name (тип которого char*) с char*, что, кстати, никогда не будет true.

[totorialman]

Wataru, О боже хахахахах, спасибо, я этого не замечал, все не понимал почему перегрузка не вызывается

[totorialman]

Dmitrii, Спасибо, а почему никогда не будет true?

[Dmitrii]

totorialman, в исправленном варианте возможно true. В исходном вы сравнивали две переменные-указателя: true вернулся бы только если бы эти указетели указывали на один и тот же адрес.

[Wataru]

totorialman, Это сравнение двух char* не работает, потому что сравниваются указатели - адреса строк. strcmp или какого-то посмивольного сравнения не происходит.

Answer 1

[Dmitrii]

Замените строку
if (planets[i].name == name_d)
на
if (planets[i] == name_d)
Если хотите, чтобы вызывался operator==
Но у вас там кроме этого ещё очень много странностей.
Зачем сравниваете строки напрямую? Есть strcmp
Зачем вообще работаете с сырыми строками, если даже не следите за очисткой памяти?
Лучше замените все на std::string.