JavaRush/Помощь по Java задачам/не правильно реализован метод equals, help

15 уровень

23.12.2024
193просмотров
11комментариев

не правильно реализован метод equals, help

Вопрос по задаче

Решен

здравствуйте, помогите пожалуйста, заранее спасибт. не пускает второй пункт мой код. Уже кучу ифов использовал, тесты проходит. может не всё тесты делаю

В методе main создаются два айфона с одинаковыми параметрами.
В консоли выводится результат их сравнения. Разберись, почему сейчас результат отрицательный и сделай так, чтобы он был положительным. Для этого тебе нужно переопределить метод equals(Iphone), который будет учитывать все параметры.
У двух равных объектов значения полей должны быть одинаковыми. Метод main не участвует в проверке.

Требования:

В классе Iphone должен быть переопределен метод public boolean equals(Object).
Метод equals должен возвращать true для двух равных объектов типа Iphone и false — для разных.
Метод equals должен возвращать false, если в него передали null.

package com.javarush.task.pro.task10.task1010; import java.util.Objects; /* Два айфона */ public class Iphone { private String model; private String color; private int price; public Iphone(String model, String color, int price) { this.model = model; this.color = color; this.price = price; } //напишите тут ваш код public String getModel(){ return model; } public String getColor(){ return color; } public int getPrice(){ return price; } @Override public boolean equals(Object obj) { if(this == obj) { return true; } if (obj == null){ return false; } if(!(obj instanceof Iphone)){ return false; } Iphone iphone = (Iphone) obj; //--------------------------- if(this.getModel() == null || iphone.getModel() == null) { if(this.getModel() == null && iphone.getModel() == null) { if(this.getColor() == null || iphone.getColor() == null) { if(this.getColor() == null && iphone.getColor() == null) { if(this.getPrice() == 0 || iphone.getPrice() == 0) { if(this.getPrice() == 0 && iphone.getPrice() == 0) { return true; } else { return false; } } else { if (this.getPrice() == iphone.getPrice()){ return true; } else { return false; } } } else { return false; } } else { if(this.getColor().equals(iphone.getColor())) { if(this.getPrice() == 0 || iphone.getPrice() == 0) { if(this.getPrice() == 0 && iphone.getPrice() == 0) { return true; } else { return false; } } else { if (this.getPrice() == iphone.getPrice()) { return true; } else { return false; } } } else { return false; } } } else { return false; } } if(this.getColor() == null || iphone.getColor() == null) { if(this.getColor() == null && iphone.getColor() == null) { if(this.getPrice() == 0 || iphone.getPrice() == 0) { if(this.getPrice() == 0 && iphone.getPrice() == 0) { return true; } else { return false; } } else { if (this.getPrice() == iphone.getPrice()){ return true; } else { return false; } } } else { return false; } } else { if(this.getColor().equals(iphone.getColor())) { if(this.getPrice() == 0 || iphone.getPrice() == 0) { if(this.getPrice() == 0 && iphone.getPrice() == 0) { return true; } else { return false; } } else { if (this.getPrice() == iphone.getPrice()) { return true; } else { return false; } } } else { return false; } } //--------------------------- } public static void main(String[] args) { Iphone iphone1 = new Iphone("X", "Black", 999); Iphone iphone2 = new Iphone("X", "Black", 999); System.out.println(iphone1.equals(iphone2)); } }

Комментарии (11)

популярные
новые
старые

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

hidden #3303140

Уровень 2

23 декабря 2024, 07:10

полезный

1. Нафиг использовать геттеры внутри класса? Обращайся сразу к полям. 2. Что не так с price = 0? Ты всё проверки на null делаешь только из-за того, что у null нельзя вызвать метод equals и сравнить значения. А примитивы просто сравниваются по значению через ==. 3. То, что ты написал кучу ифов, это не достижение, а проблема. Если в классе будет 5-10-15 полей, то написание одного такого метода будет занимать три дня, а поиск ошибки в нём три недели. Нафиг оно надо? Не стоит сравнивать все поля одновременно. Чтобы вернуть true, достаточно отсеять все варианты, когда объекты разные

if (this.color == null) {
    if (other.color != null) {
        return false;
    }
} else if (!this.color.equals(other.color)) {
    return false;
}
// здесь color одинаковые

Всё, сравнили цвета и забыли, больше к ним не возвращаемся. Не надо вкладывать одно в другое. Так по очереди сравниваем поле за полем, пока не сравним все. Примитивы сразу сравниваем через ==.

Soup🍲

Уровень 15

23 декабря 2024, 07:34

сделал так ,не пущает if(this == obj) { return true; } if (obj == null){ return false; } if(!(obj instanceof Iphone)){ return false; } Iphone iphone = (Iphone) obj; if (this.model == null) { if(iphone.model != null) { return false; } else if (!this.model.equals(iphone.model)){ return false; } } if (this.color == null) { if(iphone.color != null) { return false; } else if (!this.color.equals(iphone.color)){ return false; } } if (this.price == 0) { if(iphone.price != 0) { return false; } else if (!(this.price == iphone.price)){ return false; } } return this.model.equals(iphone.model) && this.color.equals(iphone.color) && this.price == iphone.price; }

hidden #3303140

Уровень 2

23 декабря 2024, 08:24

полезный

Ты читал, что я тебе написал выше? Про сравнение примитивов в частности. Если да, то объясни что это и зачем?

if (this.price == 0) {
	if(iphone.price != 0) {
		return false;
	} else if (!(this.price == iphone.price)){
		return false;
	}
}

Это первое. Второе, ты даже не смог правильно переписать блок кода, который я тебе привёл. Сравни эти два блока:

// у меня
if (this.color == null) {
	if (iphone.color != null) {
		return false;
	}
} else if (!this.color.equals(iphone.color)) {
	return false;
}

// у тебя
if (this.color == null) {
	if(iphone.color != null) {
		return false;
	} else if (!this.color.equals(iphone.color)){
		return false;
	}
}

Видишь разницу? Она чертовски принципиальная. Ты можешь вызывать у переменной метод equals, если переменная не содержит null. А что происходит в твоём коде? И, наконец, последнее: ты всё ещё не понимаешь, что ты делаешь и зачем. Я тебе объясняю, что после этого сравнения цветов, значение полей color уже одинаковое (иначе бы произошёл выход из метода). Зачем ты их сравниваешь второй раз? Ты забываешь, что "одинаковое" может означать и ту ситуацию, когда оба значения одинаково равны null. А ты снова пишешь return this.color.equals... Ты же при этом снова исключение поймаешь, если в поле будет null. Твой метод equals должен представлять из себя такую структуру

if (цвета разные) return false;
if (модели разные) return false;
if (цены разные) return false;
// если мы дошли до этой строки, значит модели, цвета и цены одинаковые
return true;

Вот это вот "цвета одинаковые" означает либо, что оба цвета одновременно равны null, либо что они содержат одно и то же значение. Поэтому в блоке кода, который я привёл я провожу два сравнения.

hidden #3303140

Уровень 2

23 декабря 2024, 08:29

полезный

А в случае с примитивами их нужно только сравнить по значениям, и всё. Поэтому вот эти строки

if (цены разные) return false;
return true;

можно упростить и свести вот к этому сравнению

if (цвета разные) return false;
if (модели разные) return false;
// цвета и модели одинаковые
return this.price == iphone.price;

Ты сравнил цвета, ты сравнил модели. Если сравнение продолжается, значит они одинаковые. Осталось сравнить цены, и при сравнении цен, если цены равны, то и объекты айфонов будут равны, и наоборотм. Поэтому мы просто возвращаем результат сравнения цен в конце. Поскольку цена - примитив (int), то сравниваем просто через "==".

Soup🍲

Уровень 15

23 декабря 2024, 09:10

Сделал по точному твоему шаблону ,валидатор принял . Понял,что нахрена сравнивать с нулем прайс, когда их надо сравнивать обьектами . далее , про разницу реализации,так и не нашел, ничего не понял,что хотел сказать, не догоняю. А так же, не понял, если сравниваются два обьекта ,у одного из полей null, то как сравнение происходит с null и не выкидывает exception ? вот код,который делал по твоему решению public boolean equals (Object obj) { if(this == obj) { return true; } if (obj == null){ return false; } if(!(obj instanceof Iphone)){ return false; } Iphone iphone = (Iphone) obj; if (this.color != iphone.color) { return false; } if (this.model != iphone.model) { return false; } return this.price == iphone.price; }

hidden #3303140

Уровень 2

23 декабря 2024, 10:32

решение

Это тоже неправильный ответ, потому что здесь ты сравниваешь строки только через "==". Это сработает только если строки кэшируются в стринг пуле, а это не всегда так. Кроме того, поля же не всегда только String и int, они могут быть любого типа данных. А ссылочные типы данных надо сравнивать через equals. Поэтому правильно должно быть так:

@Override
public boolean equals(Object obj) {
	if (this == obj) return true;
	if (!(obj instanceof Iphone)) return false;

	Iphone iphone = (Iphone) obj;

	if (this.color == null) {
		if (iphone.color != null) {
			return false;
		}
	} else if (!this.color.equals(iphone.color)) {
		return false;
	}

	if (this.model == null) {
		if (iphone.model != null) {
			return false;
		}
	} else if (!this.model.equals(iphone.model)) {
		return false;
	}

	return this.price == iphone.price;
}

Здесь ты не полагаешься на String pool и сравниваешь ссылочные переменные через equals. Как работает код:

if (this.color == null) {
	if (iphone.color != null) {
		return false;
	}
} else if (!this.color.equals(iphone.color)) {
	return false;
}

У тебя возможны 4 варианта:

| № | this.color | iphone.color |
|---|------------|--------------|
| 1 |    null    |     null     |
| 2 |    null    |    не null   |
| 3 |   не null  |     null     |
| 4 |   не null  |    не null   |

В первых двух случаях сработает условие if (this.color == null). Тут всё просто, если iphone.color == null, то поля одинаковые, и мы продолжаем проверку остальных, а если поля разные, то и айфоны разные, возвращаем false. В третьем и четвёртом случае проверка if (this.color == null) вернёт false, значит будет выполняться проверка else: else if (!this.color.equals(iphone.color)) Продолжение ниже.

hidden #3303140

Уровень 2

23 декабря 2024, 10:36

решение

Итак, поскольку в this.color не содержится null, у него можно вызвать метод equals. Если в iphone.color содержится null, то this.color.equals(null) вернёт false, следовательно и айфоны будут разные. А если в iphone.color не содержится null, то equals вернёт результат сравнения. И если строки будут разные, то мы вернём false, а если нет, то продолжим сравнивать другие поля. Если применить правила логических вычислений, то весь блок

if (this.color == null) {
	if (iphone.color != null) {
		return false;
	}
} else if (!this.color.equals(iphone.color)) {
	return false;
}

можно объединить в одну общую проверку:

if (this.color != null && !this.color.equals(iphone.color) || iphone.color != null) {
	return false;
}

Здесь последняя часть iphone.color != null будет проверяться только в том случае, если this.color == null. Кроме того, чтобы каждый раз не писать эти простыни текста для сравнения полей с учётом null, в стандартной библиотеке существует утилитный метод, который объединяет в себе и проверку на null, и сравнение полей. Это метод Objects.equals:

public static boolean equals(Object a, Object b) {
	return (a == b) || (a != null && a.equals(b));
}

@Override
public final boolean equals(Object o) {
	if (!(o instanceof Iphone iphone)) return false;

    return price == iphone.price
            && Objects.equals(color, iphone.color)
            && Objects.equals(model, iphone.model);
}

Ты можешь использовать этот метод в реализации equals для любого класса. Но прежде чем объяснять тебе этот "лёгкий" путь, требовалось, чтобы ты разобрался, как нужно самому применять проверки на null, потому что это будет пригождаться постоянно.

Soup🍲

Уровень 15

23 декабря 2024, 11:14

Спасибо, надеюсь смогу использовать полученные знания.

Nikolay

Уровень 24

23 декабря 2024, 20:47

Привет! Подскажи, пожалуйста, почему Идеа ошибку мне выдает на сравнениях this? Даже если верное решение подставить, то ошибка не уходит.

hidden #3303140

Уровень 2

23 декабря 2024, 21:14

Потому что параметр метода equals должен быть типа Object, а не Objects. Objects - это другой класс, содержащий набор разных утилитных методов для работы с объектами. Идея тебе подсказывает, что он не может быть равен объекту типа Iphone и не может быть приведён к этому типу. Ты даже не можешь создать инстанс типа Objects, строго говоря.

Nikolay

Уровень 24

24 декабря 2024, 08:05

Спасибо большое, проглядел лишнюю букву)