Помогите пожалуйста найти ошибку.

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

Решен

В чем ошибка не могу понять...

В методе 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 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.model == null) { if (!(iphone.model == null)) { return false; } } if (!(this.model.equals(iphone.model))) { return false; } if (this.color == null) { if (!(iphone.color == null)) { return false; } } if (!(this.color.equals(iphone.color))) { return false; } return 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)); } }

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

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

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

Денис Enterprise Java Developer

11 февраля 2022, 00:25

полезный

Твоя ошибка в проверке цвета на 44 строке и модели на 36. Если у обоих объектов в полях нулл, твоя заглушка не сработает и ты успешно попытаешься вызвать у null'a метод equals, что приведет тебя к Null pointer exception. Это кстати наглядная демонстрация слишком мудреной логики. По стилю твоего кода: его очень сложно читать. Многие сравнения можно упростить, чем например вот это !(iphone.model == null) лучше этого? iphone.model != null Ну и все это можно было схлопнуть в что-то вроде этого:

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

Lonlost

Уровень 51

12 февраля 2022, 20:36

Денис, благодарю за наводку и замечания! Приглядевшись повнимательнее тоже заметил свою ошибку с Null pointer exception. Насчет стиля Вы верно заметили, я поэтому то и запутался) А вот насчет предложенного Вами решения я не совсем согласен. С одной стороны да, можно было бы так написать, и код был бы максимально читабельный. Но я преследовал цель - написание программы на основе данных предложенных в лекциях курса Java Syntax Pro. В этих лекциях не упоминалось про команды и методы типа: @Override и Object.equals(String, String) и, как в правильном решении, getClass(). А ещё после просмотра правильного решения, было интересно, пропустит ли компилятор JavaRush решение без @Override и getClass(). В итоге я своих целей достиг.)

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 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 (model != null || iphone.model != null) {
            if (model != null) {
                if (!model.equals(iphone.model)) return false;
            }
            if (model == null && iphone.model != null) return false;
        }
        //if (!model.equals(iphone.model)) return false;
        if (color != null ^ iphone.color != null) {
            if (color != null) {
                if (!color.equals(iphone.color)) return false;
            }
            if (iphone.color != null) return false;
        }
        return (price == iphone.price) && (color.equals(iphone.color));
    }

Интересно, для чего @Override вообще нужен, если компилятор и без него все компилирует.

Lonlost

Уровень 51

12 февраля 2022, 20:40

Денис, как знающий специалист, подскажите пожалуйста, для лучшей читаемости какой вариант кода лучше реализовать? Вариант 1:

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

Вариант 2:

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

Денис Enterprise Java Developer

12 февраля 2022, 21:57

Про Override странно что не упоминали, учитывая, что просят переопределить наследуемый метод equals в задаче. Вообще здесь не про все говорят, и я крайне рекомендую гуглить все непонятное :) Если вкратце, это то, что называется Аннотацией, в Java их много на все случаи жизни. Конкретно Override указывает на переопредление какого-либо унаследованного метода. Как вы знаете, все классы наследуются от Object, вот equals, равно как и hashCode это его родные методы, равно как и toString, именно по этому вы можете привести к строке решительно любой класс. Вообще, методы класса Object очень популярный вопрос на собеседованиях, особенно любят спрашивать про контракт equals и hashCode, обязательно почитайте. Строго говоря, тот код что предложил я не мой, вы можете сами получить что-то подобное в IDEA, если нажмете alt + insert и выберете пункт equals + hashCode, там же можно создавать конструкторы, getter и setter методы, очень удобно - рекомендую. То что вы не проходили класс Objects, это кстати не удивительно, он здесь по моему вообще не рассматривается, но я бы на вашем месте почитал про него тоже. В частности Objects.equals(Object o1, Object o2) это nullsafe метод сравнения двух объектов, крайне удобно.

Денис Enterprise Java Developer

12 февраля 2022, 22:25

Любопытный подход с XOR, за всех судить не берусь, но могу сказать что мне его читать было сложнее :) Я редко (никогда) видел его в реальном коде. Ваш вопрос натолкнул меня на мысль что вы не совсем понимаете разницу между OR и XOR, последний это исключающее или. XOR возвращает правду только если один из аргументов Истина. В случае с обычным OR, он может выполняться и в случае если обе части выражения есть истина. Т.е. проверив левую часть выражения и получив там true код во вторую даже не заходит - это ему уже не важно. В булевой логике это называется коротким замыканием. Однако вернемся к вашему коду.

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

После первого if мы получим однозначно следующую ситуацию, или model или iphone.model не null. Второй if проверяет не является ли model null. Если это условие выполняется, мы можем однозначно сказать, что iphone.model это null, какие-либо еще проверки нам не нужны. В сущности, если выполняется условие model != null ^ iphone.model != null можно автоматически возвращать false, потому что оно выполнится тогда и только тогда, когда модели точно не идентичны. Я немного переработал ваше условие для краткости записи:

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

Однако, легко видеть, что для проверки идентичности часть с xor вообще не нужна, достаточно следующего за ним блока if.

Денис Enterprise Java Developer

12 февраля 2022, 22:29

Можно сказать что здесь я тоже поигрался с "коротким замыканием" но реализованным через отдельные проверки if. Так например если одно из полей типа String не пройдет проверку, проверять остальное смысла не имеет, значит остальные проверки должны проводиться только если предыдущие выполнились. Именно по этому все три вложены одна в одну, а если хоть одна из них не выполнится будет возвращено false; Таким образом метод equals у нас имеет вполне аккуратный и читаемый вид:

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

Lonlost

Уровень 51

15 февраля 2022, 23:12

Спасибо, Денис, за вашу отзывчивость! Как работает XOR я правильно понял) Подчерпнул много полезного у вас. Спасибо)

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

Денис Enterprise Java Developer

15 февраля 2022, 23:14

Кстати да, про два нулла я провтычил )