Помогите, разобраться почему не проходит 3 пункт. На null все проверки сделал.Если из-за стилистики, то намекните,как лучше все записать.

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

Java Multithreading, 1 уровень, 5 лекция

Решен

<package com.javarush.task.task21.task2104; import java.util.HashSet; import java.util.Objects; import java.util.Set; /* Equals and HashCode */ public class Solution { private final String first, last; public Solution(String first, String last) { this.first = first; this.last = last; } public boolean equals(Solution n) { if (this == n){ return true; } if ( n instanceof Solution){ return true; } return n.first!=null && n.last!=null && this.first.equals(n.first)&& this.last.equals(n.last); } public int hashCode() { int result = first != null ? first.hashCode() : 0; result = 31 * result + (last != null ? last.hashCode() : 0); return result; } public static void main(String[] args) { Set<Solution> s = new HashSet<>(); s.add(new Solution("Donald", "Duck")); System.out.println(s.contains(new Solution("Donald", "Duck"))); } } >

В классе Solution исправить пару методов equals/hashCode в соответствии с правилами реализации этих методов(детали уточни у своего любимого поисковика).
Обе строки first и last должны принимать участие в сравнении с помощью метода equals и вычислении hashcode.
Метод main не участвует в тестировании.

Требования:

Хешкоды одинаковых объектов должны быть равны.
Метод equals должен проверять равен ли переданный объект текущему(сравнение через ==).
Метод equals должен проверять является ли переданный объект объектом класса Solution.
Метод equals должен возвращать true в случае, если поля first и last равны у переданного объекта и текущего(не забудь что они могут быть равны null).
Должно быть обеспечено корректное поведение HashSet с типом элементов Solution.

package com.javarush.task.task21.task2104; public class Drob { private int up; private int down; public Drob(int up, int down) { this.up = up; this.down = down; } public boolean equals(Object obj) { if (obj == null) return false; if (obj.getClass() != this.getClass()) return false; Drob other = (Drob) obj; return this.up * other.down == this.down * other.up; } public static void main(String[] args) { Drob first = new Drob(6, 10); Drob seconds = new Drob(12, 20); System.out.println(first.equals(seconds)); } }

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

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

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

Кирилл

Уровень 35

11 сентября 2019, 18:53

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

public boolean equals(Object n) {


        if (this == n){
            return true;
        }
        if (!(n instanceof Solution)){
            return false;
        }

        Solution other = (Solution) n;
        if (first!=null && !first.equals(other.first)){
            return false;
        }
        if (last!=null && !last.equals(other.last)){
            return false;
        }
        return true;
    }

Стас Пасинков Software Developer в Zipy

Master

11 сентября 2019, 23:05

решение

вот вам для тестов

Solution a = new Solution(null, null);
Solution b = null;
// false

Solution a = new Solution(null, null);
Solution b = a;
// true

Solution a = new Solution(null, null);
Solution b = new Solution(null, null);
// true

Solution a = new Solution("aaa", null);
Solution b = new Solution(null, null);
// false

Solution a = new Solution("aaa", "bbb");
Solution b = new Solution(null, null);
// false

Solution a = new Solution(null, null);
Solution b = new Solution("aaa", null);
// false

Solution a = new Solution(null, null);
Solution a = new Solution("aaa", "bbb");
// false

в принципе, этого достаточно. варианты с разными/одинаковыми не-нулл значениями у вас пока учитываются

Стас Пасинков Software Developer в Zipy

Master

11 сентября 2019, 23:12

у вас два последних вроде true выдают когда поправите - проверьте чтобы третий не сломался и выдавал true

Кирилл

Уровень 35

12 сентября 2019, 11:18

В следующих попытках решил проблему 3 пункта, но потом вылезла проблема с последним и 3 пункт выполнялся в том случае, если итоговый возврат был false,чтобы сработал 3 пункт, достаточно было просто указать на то, что объект не содержит ссылки на пустоту, а потом сравнить его с текущим и тогда true возвращалось как надо. В итоге я использовал две строки из другого кода с возвращаемым значением false ,а итоговый return изменил на true и валидатор пропустил решение. Так вот эквивалентна ли та часть кода, на которую я заменил свой, если нет, прошу объяснить написанное.Спасибо!

if (first != null ? !first.equals(solution.first) : solution.first != null) return false;
      if (last != null ? !last.equals(solution.last) : solution.last != null) return false;

Стас Пасинков Software Developer в Zipy

Master

13 сентября 2019, 15:10

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

Кирилл

Уровень 35

14 сентября 2019, 11:42

Я уже разобрался, спасибо за помощь. В представленном коде выше : обычный тернарный оператор, в моем коде который я указал изначально, не было учтен код после двоеточия.