Не понимаю данную запись

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

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

Обсуждается

Ребят, вроде ерунда, но застрал. Понять не могу эту запись

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

мы ведь сверху проверили параметр "о" в коде

if (!(o instanceof Solution)) return false;

это условие ведь проверяет на равенство и нулл переданного объекта . Кто нибудь по другому чуть по понятнее может написать? Или с написать так

if (first != null) {
           if (!first.equals(solution.first)) return false;
} else
    return false;

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

Требования:

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

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(Object n) { // // if(this == n) return true; // // if(n == null || getClass() != n.getClass()) return false; // // Solution s = (Solution) n; // // return s.first.equals(first) && s.last.equals(last); // // // // // //return n.first.equals(first) && n.last.equals(last); // } @Override public boolean equals(Object o) { if (this == o) return true; if( o instanceof Solution) return true; if(o == null) return false; Solution solution = (Solution) o; // return Objects.equals(first, solution.first) && Objects.equals(last, solution.last); return ((solution.first.equals(first) && first != null) || first == solution.first) && ((solution.last.equals(last) && last != null) || last == solution.last); } @Override public int hashCode() { return Objects.hash(first,last); } 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"))); } }

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

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

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

Денис Enterprise Java Developer

7 марта, 13:03

полезный

На каком основании делается вот этот вывод?

if( o instanceof Solution) return true;

Ты и я оба люди, но вряд ли мы равны. Вот тут написан ужас, это невозможно прочесть, смешались в кучу кони, люди... в твоём случае ветвление и тернарник которые строго говоря не очень то и нужны.

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

Учитывая, что equals возвращает логический тип тебе достаточно составить выражение которое бы проверяло равенство двух объектов, его и вернуть. Кроме того, если это последний return то у тебя не обрабатывается условие когда под if будет false. Я вижу ты в курсе про класс Objects, ну так и используй его:

@Override
public boolean equals(Object o) {
    if (o == null) return false;
    if (o == this) return true;

    if (o instanceof Solution) {
        Solution test = (Solution) o;
        return Objects.equals(first, test.first) &&
                Objects.equals(last, test.last);
    }
    return false;
}

В более свежих версиях языка можно использовать паттерн переменную для инстансоф.

if (o == this) return true;
if (o instanceof Solution test) {
    return Objects.equals(first, test.first) &&
            Objects.equals(last, test.last);
}
return false;

Anonymous #3303140

Уровень 2

7 марта, 12:55

полезный

Надпись сверху не имеет к этому отношения, т.к. одно дело, когда переменная o содержит null, а другое - когда o содержит ссылку на Solution, у которого поле first (ну или second) равно null. Если o содержит null, то объекты (this и o) одинаковыми быть не могут, сразу возвращаем null. Если одно из полей (или оба) равны null, то это ещё ничего не значит, нужно сравнивать все поля. Например, два таких объекта Solution(null, "x") и Solution(null, "x") будут равны, потому что состояние всех их полей одинаковое: у обоих поле first равно null, а поле second равно "x". Эта запись с тернарным оператором эквивалентна следующей:

if (this.first != null) {
    if (!this.first.equals(solution.first) {
        return false;
    }
} else { //this.first == null
    if (solution.first != null) {
        return false;
    }
}

Таким образом в первом внешнем ифе мы проверили, что в случае, если значение this.first не равно null, то значение полей this.first и solution.first одинаковое, и в противном случае вернули false. А во втором, проверили тот случай, когда this.first == null, и если у второго объекта solution.first не равно null, то опять-таки поля разные, а значит и объекты (this и solution) тоже разные. В совокупности мы убедились что в том или другом случае значение полей одинаковое. Это значит, что можно переходить к сравнению других полей. Эти две ветки можно объединить в одну при помощи такого условия:

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

Abakar

Уровень 33

Expert

вчера, 03:03

ПОнимаю и не понимаю одновременно( Допустим this.first == 1 и solution.first == 1, то условие ниже вернёт true или там это не предусмотрено? Во внутреннем if не вижу else для случая true. Вроде мелочь, но что то подзабыл

if (this.first != null) {
    if (!this.first.equals(solution.first) {
        return false;
    }
}
}

Anonymous #3303140

Уровень 2

вчера, 08:30

А тебе не надо возвращать true в этом случае. Ты как скульптор, твоя задача отсечь всё лишнее. Ты проверил все возможные случаи, когда что-то может различаться, и в каждом из них вернул false. Если метод не прервал своё выполнение, значит поля действительно не различаются, и можно переходить к сравнению следующей пары полей. То есть после всех этих сравнений ты в голове уже держишь, что this.first и solution.first одинаковые, их больше сравнивать не придётся. При этом тебя даже не интересует после этого, null там или настоящее значение - главное, что они совпадают в обоих объектах. После этого можно аналогичным образом сравнить вторую пару полей, затем третью, пятую, десятую, пока не сравнишь все. Если все сравнил, то молодец - объекты идентичны. Грубый псевдокод:

if (первые поля разные) {
    return false;
}

// если одинаковые - проверяем дальше

if (вторые поля разные) {
    return false;
}

// если одинаковые - проверяем дальше

...

if (n-ные поля разные) {
    return false;
}
// все поля попарно одинаковые
return true;

При этом последняя пара полей получается "решающая", чуть-чуть можно сократить код:

<pre class="language-java line-numbers"><code>
if (первые поля разные) {
    return false;
}

// если одинаковые - проверяем дальше

if (вторые поля разные) {
    return false;
}

// если одинаковые - проверяем дальше

...

// сравниваем последнюю пару
return (последние поля одинаковые?)
</code></pre>

Anonymous #3303140

Уровень 2

вчера, 08:33

При этом в каждом из случаев выше "одинаковость" включает в себя и случаи, когда оба одновременно равны null, и когда оба не равны null и содержат одно и то же. При этом утильный метод Objects.equals внутри себя уже содержит проверку на null и позволяет безопасно сравнить два любых объекта. Из-за того, что запись получается довольно лаконичной, можно все сравнения объединить через && и сразу написать в команде return:

// проверки на класс, приведение типа

return Objects.equals(this.a, another.a)
        && Objects.equals(this.b, another.b)
        && ...
        && Objects.equals(this.xxx, another.xxx);