Что не так?

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

Решен

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

В методе 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; } @Override public boolean equals(Object object) { if (object == null) { return false; } else if (this == object) { return true; } else if (! (object instanceof Iphone)) { return false; } Iphone iphone = (Iphone)object; boolean isModelNullOk; boolean isColorNullOk; if(this.model == null || iphone.model == null) { isModelNullOk = Objects.equals(this.model, iphone.model); if(this.color == null || iphone.color == null) { isColorNullOk = Objects.equals(this.color, iphone.color); return isModelNullOk && isColorNullOk && this.price == iphone.price; } return isModelNullOk && this.color.equals(iphone.color) && this.price == iphone.price; } else { return this.model.equals(iphone.model) && this.color.equals(iphone.color) && this.price == iphone.price; } } //напишите тут ваш код 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)); } }

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

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

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

hidden #598481

Уровень 27

17 февраля 2022, 07:02

решение

Что-то ты намудрила. Если начинать использовать Objects.equals, то можно убрать все ифы (имею в виду вторую половину метода equals), и использовать только Objects.equals, там внутри проверка на null есть. Если есть интерес всё же разобраться с тем, чтобы написать самостоятельно последовательное сравнение полей, то лучше без использования готового метода, писать последовательно проверки на null и затем сравнивать содержимое полей через equals. А твой код не будет работать для такой пары:

Iphone iphone1 = new Iphone("X", null, 999);
Iphone iphone2 = new Iphone("X", "Black", 999);

Поймаешь NPE Ну и с точки зрения структуры написания методов, вот частный пример, как не надо. Зачем в трех разных местах писать && this.price == iphone.price? А если в одном месте забудешь? Можно же перед этим один раз написать, что если они не равны, то return false, и дальше забыть про это поле. Избегай самоповторов.

Уровень 39

17 февраля 2022, 07:50

Ой! А я изначально хотела сравнить с помощью ==, IDEA предложила заменить на Objects.equals. Вышло иронично, потому что всю эту мороку я придумала с расчетом на то, чтобы упростить код :с Спасибо за наводку, получилось решить. Остался только один вопрос. Часть после приведения типа я переписала так. И если строка с ошибкой в таком виде, то IDEA пишет, что мой метод не возвращает значения. Если я удаляю tlse и пишу с новой строчки "if", то все ок. Не подскажешь, почему так? Вроде иерархия не нарушена

if (this.model == null || iphone.model == null) {
        if (this.model == iphone.model) {
            if (this.color == null || iphone.color == null) {
                if (this.color == iphone.color) {
                    return this.price == iphone.price;
                }
            } else {
                return this.color.equals(iphone.color) && this.price == iphone.price;
            }
        } else {
            return false;
        }
    } else if (this.color == null || iphone.color == null) { // строка с ошибкой
        if (this.color == iphone.color) {
            return this.model.equals(iphone.model) && this.price == iphone.price;
        } else {
            return false;
        }
    } else {
        return this.model.equals(iphone.model) && this.color.equals(iphone.color) && this.price == iphone.price;
    }

}

hidden #598481

Уровень 27

17 февраля 2022, 09:03

Во-первых, ты почему-то теперь стала сравнивать строки через ==, а не через equals. Во-вторых, чисто по структуре твоего ветвления у тебя не хватает else вот к этому ифу:

if (this.model == null || iphone.model == null) {
        if (this.model == iphone.model) {
            if (this.color == null || iphone.color == null) {
                if (this.color == iphone.color) { // а если не равно, что возвращаем?
                    return this.price == iphone.price;
                }

Ну и в-третьих это все просто монструозно и чересчур сложно, вот ты и путаешься. Смотри, вот стандартная реализация метода equals от самой идеи:

@Override
public boolean equals(Object o) {
    if (this == o) {
        return true;
    }
    if (o == null || getClass() != o.getClass()) {
        return false;
    }

    Iphone iphone = (Iphone) o;

    if (price != iphone.price) {
        return false;
    }
    if (model != null ? !model.equals(iphone.model) : iphone.model != null) {
        return false;
    }
    if (color != null ? !color.equals(iphone.color) : iphone.color != null) {
        return false;
    }

    return true;
}

Тернарные операторы тождестенны вызову Objects.equals для этих полей. Если заменить оба оператора на вложенные ифы, и немного упростить, то получится следующая картина:

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

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

if (color == null) {
    return iphone.color == null;
}
return color.equals(iphone.color);

Ты не пытаешься проверять всё разом, а проверяешь наоборот последовательно, и к каждой следующей проверке ты подходишь, только если предыдущие поля одинаковые.

Уровень 39

17 февраля 2022, 09:08

Ой, теперь понимаю как переборщила. Спасибо!

hidden #598481

Уровень 27

17 февраля 2022, 09:12

Ну а на практике ты просто будешь использовать сравнение через Objects.equals и не ломать голову по поводу возможных null:

@Override
public boolean equals(Object o) {
    if (this == o) {
        return true;
    }
    if (o == null || getClass() != o.getClass()) {
        return false;
    }
    Iphone iphone = (Iphone) o;
    return price == iphone.price
            && Objects.equals(model, iphone.model)
            && Objects.equals(color, iphone.color);
}

Для генерации кода можешь использовать Alt+Ins и выбрать пункт equals & hashCode.