Вроде все вычисляет но задача решена не верно(

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

Решен

Уже бьюсь над этой задачей второй день(( Что я делаю не так?

В методе 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(Iphone iphone2) { if (this == iphone2) return true; if (iphone2 == null) return false; //if (!(iphone2 instanceof Iphone)) // return false; //Iphone iphone1 = (Iphone) iphone2; return this.model.equals(iphone2.model) && this.color.equals(iphone2.color) && this.price == iphone2.price; } public static void main(String[] args) { var iphone1 = new Iphone("X", "Black", 999); var iphone2 = new Iphone("X", "Black", 99); System.out.println(iphone1.equals(iphone2)); } }

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

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

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

qpuhuk

Уровень 27

23 декабря 2020, 18:12

У меня вот такое решение прошло. @Override public boolean equals(Object iphone1) { if (this == iphone1) { return true; } if (iphone1 == null || !(iphone1 instanceof Iphone)) { return false; } Iphone iphone = (Iphone) iphone1; return this.price == iphone.price && this.model.equals(iphone.model) && this.color.equals(iphone.color);

Nikita SerkovBackend Developer в Kameleoon

1 декабря 2020, 13:25

Вот так будет работать: if (this == a) // если равны ссылки return true; if (a == null) // если второй объект равен null return false; if (!(a instanceof Iphone)) // проверяет объект а на принадлежность к классу Iphone return false; Iphone one = (Iphone) a;//создали объект one класса Iphone и присвоили ему значение объекта а if (this.model == null) // отсюда начинается проверка всех поле твоего объекта на значение null if (this.color == null) return (one.model == null && one.color == null && this.price == one.price); else return (one.model == null && this.color.equals(one.color) && this.price == one.price); else if (this.color == null) return (this.color.equals(one.model) && one.color == null && this.price == one.price); else return (this.model.equals(one.model) && this.color.equals(one.color) && this.price == one.price);

ushackov

Уровень 30

2 декабря 2020, 09:45

Все равно не валидирует валидатор((

Ksenia VolkovaJava Developer в DXC

Master

2 декабря 2020, 10:20

Если model и color будет null, у тебя будет ошибка.

Nikita SerkovBackend Developer в Kameleoon

2 декабря 2020, 16:58

Ты не совсем точно переписал у меня условие (this.model == null) находится внутри условия (this.color == null), а у тебя 2 условия отдельно.

Ksenia VolkovaJava Developer в DXC

Master

1 декабря 2020, 11:20

решение

Если model или color у 1-го айфона будут null, программа упадет.

ushackov

Уровень 30

1 декабря 2020, 11:42

Добавил если модель или цвет null то false. Но все равно не проходит.

Ksenia VolkovaJava Developer в DXC

Master

1 декабря 2020, 11:44

Ну если у обоих null, то по идее должно быть true

ushackov

Уровень 30

1 декабря 2020, 11:59

Ладно отложу пока. Не выходит каменный цветок. При проверке вообще говорит что метода equals нет в коде.

Ksenia VolkovaJava Developer в DXC

Master

1 декабря 2020, 12:56

В условии фигурирует equals(Iphone), а в требованиях - equals(Object). Попробуй на Object переделать.