Czy powinienem pisać metody equals () w encjach JPA?

Tak, powinieneś zdefiniować odpowiednie metody equals() i hashcode(), ale nigdy nie pozwól, aby id było częścią żadnej z nich. (Zobacz moją ostatnią odpowiedź w podobnym pytaniu)

W dokumentacji Hibernate znajdują się informacje na ten temat.

Zazwyczaj pozwalamy IDE generować hashCode() i equals() dla nas. Bądź ostrożny. Podczas generowania tych metod dla encji JPA. Niektóre wersje equals() sprawdzania tożsamości klasy

// ... inside equals() - wrong approach for Entities (cause of generate proxies)
if (o == null || this.getClass() != o.getClass()) {
        return false;
}
// ...

Spowoduje to złamanie Twoich kolekcji z niektórymi bibliotekami JPA, ponieważ biblioteki te tworzą proxy do Twoich encji( podklas), jak na przykład MyGreatEntity_$$_javassist_7 w Hibernate.

In encje zawsze zezwalają na podklasy w equals().

Tak, powinieneś!

Jeśli nie nadpisujesz domyślnego Java.lang.Object equals i hashCode realizacja:

@Entity(name = "Book")
public class Book implements Identifiable<Long> {

    @Id
    @GeneratedValue
    private Long id;

    private String title;

    //Getters and setters omitted for brevity
}

Operacja merge zwróci inną instancję obiektu i umowa równości zostanie zerwana Jak wyjaśniono w tym poście .

Najlepszym sposobem jest użycie klucza biznesowego, takiego jak:

@Entity
public class Book implements Identifiable<Long> {

    @Id
    @GeneratedValue
    private Long id;

    private String title;

    @NaturalId
    private String isbn;

    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (!(o instanceof Book)) return false;
        Book book = (Book) o;
        return Objects.equals(getIsbn(), book.getIsbn());
    }

    @Override
    public int hashCode() {
        return Objects.hash(getIsbn());
    }

    //Getters and setters omitted for brevity
}

Możesz również użyć identyfikatora dla równości, ale pamiętaj, że implementacja hashCode powinna zawsze zwracać tę samą wartość, co wyjaśniono w tym samym post, o którym już wspomniałem:

@Entity
public class Book implements Identifiable<Long> {

    @Id
    @GeneratedValue
    private Long id;

    private String title;

    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (!(o instanceof Book)) return false;
        Book book = (Book) o;
        return Objects.equals(getId(), book.getId());
    }

    @Override
    public int hashCode() {
        return 31;
    }

    //Getters and setters omitted for brevity
}

To jedyny sposób. Możesz spróbować Pojomatic library, która wykonuje ciężką pracę za Ciebie.