Вопрос: почему моё решение не прошло, а подсмотренное прошло? Работало вроде как одинаково.
Моё:
public boolean equals(Object obj) {
if (obj == null)
return false;
if (this == obj)
return true;
if (obj instanceof Solution) {
Solution temp = (Solution) obj;
if (this.first != null && this.last != null && temp.first != null && temp.last != null) {
return this.first.equals(temp.first) && this.last.equals(temp.last);
}
}
return false;
}
Подсмотренное:
public boolean equals(Object o) {
if (o == null)
return false;
if (!(o instanceof Solution)){
return false;
}
if (this == o) return true;
Solution solution = (Solution) o;
if (first != null ? !first.equals(solution.first) : solution.first != null) return false;
if (last != null ? !last.equals(solution.last) : solution.last != null) return false;
return true;
}
package com.javarush.task.task21.task2104;
import java.util.HashSet;
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 o) {
if (o == null)
return false;
if (!(o instanceof Solution)){
return false;
}
if (this == o) return true;
Solution solution = (Solution) o;
if (first != null ? !first.equals(solution.first) : solution.first != null) return false;
if (last != null ? !last.equals(solution.last) : solution.last != null) return false;
return true;
}
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")));
}
}
расистскоезамечание относительно "instanceof". Допустим у тебя есть класс Котик с полями имя, возраст. И ты переопределил иквелс на основании этих полей. А потом сделал наследника - класс ЦветнойКотик, добавив поле цвет. И теперь у тебя "ЦветнойКотик instanceof Котик" - вернет тру. То есть ColoredCat("black", "vasya", 5) equals Cat("vasya", 5), который в свою очередь Cat("vasya", 5) equals ColoredCat("grey", "vasya", 5). Получается черный котеюшка равен серому. Мне equals() через getClass нравится больше. Но это наверно уже дело вкуса... наверно и так и так правильно. Зависит от конкретных целей.