Zastąpienie metody równa się java () - nie działa?

equals()Dzisiaj napotkałem interesujący (i bardzo frustrujący) problem z tą metodą, który spowodował awarię klasy, którą uważałem za dobrze przetestowaną, i spowodowanie błędu, którego wytropienie zajęło mi bardzo dużo czasu.

Dla kompletności nie używałem IDE ani debuggera - tylko dobry, staroświecki edytor tekstu i System.out. Czas był bardzo ograniczony i był to projekt szkolny.

W każdym razie -

Ja rozwijał podstawowy koszyk, które mogą zawierać ArrayListod Bookobiektów . W celu wprowadzenia w życie addBook(), removeBook()oraz hasBook()metod Koszyka, chciałem sprawdzić, czy Bookistniała już w Cart. Więc idę -

public boolean equals(Book b) {
    ... // More code here - null checks
    if (b.getID() == this.getID()) return true;
    else return false;
}

W testowaniu wszystko działa dobrze. Tworzę 6 obiektów i wypełniam je danymi. Czy wiele dodaje, usuwa, ma operacje () na Carti wszystko działa dobrze. Czytałem, że możesz albo mieć, equals(TYPE var)alboequals(Object o) { (CAST) var } założyć, że skoro to działa, nie ma to większego znaczenia.

Potem napotkałem problem - musiałem stworzyć Bookobiekt zawierający tylko ten obiekt z IDklasy Book. Żadne inne dane nie zostałyby do niego wprowadzone. Zasadniczo następujące:

public boolean hasBook(int i) {
    Book b = new Book(i);
    return hasBook(b);
}

public boolean hasBook(Book b) {
    // .. more code here
    return this.books.contains(b);
}

Nagle equals(Book b)metoda przestaje działać. To zajęło BARDZO dużo czasu, aby wyśledzić bez dobrego debuggera i przy założeniu, że Cartklasa została odpowiednio przetestowana i poprawna. Po przejściu equals()metody do następującego:

public boolean equals(Object o) {
    Book b = (Book) o;
    ... // The rest goes here   
}

Wszystko znów zaczęło działać. Czy istnieje powód, dla którego metoda zdecydowała się nie przyjmować parametru Book, mimo że najwyraźniej był to Bookobiekt? Jedyna różnica polegała na tym, że został utworzony z tej samej klasy i wypełniony tylko jednym składnikiem danych. Jestem bardzo zdezorientowany. Proszę, rzuć trochę światła?

W Javie equals()dziedziczona jest metoda Object:

public boolean equals(Object other);

Innymi słowy, parametr musi być typu Object. Nazywa się to nadpisywaniem ; metoda public boolean equals(Book other)robi to, co nazywa się przeciążenia do equals()metody.

Do porównywania zawartości (np. Metod i ) ArrayListużywa equals()metod nadpisanych , a nie przeciążonych. W większości twojego kodu wywołanie tego, który nie przesłania poprawnie równości, było w porządku, ale nie było zgodne z .contains()equals()ObjectArrayList

Dlatego niepoprawne zastąpienie metody może spowodować problemy.

Za każdym razem nadpisuję:

@Override
public boolean equals(Object other){
    if (other == null) return false;
    if (other == this) return true;
    if (!(other instanceof MyClass)) return false;
    MyClass otherMyClass = (MyClass)other;
    ...test other properties here...
}

Użycie @Overrideadnotacji może pomóc tonie w głupich błędach.

Użyj go, gdy myślisz, że nadpisujesz metodę superklasy lub interfejsu. W ten sposób, jeśli zrobisz to w niewłaściwy sposób, otrzymasz błąd kompilacji.

Jeśli używasz zaćmienia, po prostu przejdź do górnego menu

Source -> Generate equals () and hashCode ()

Trochę nie na temat twojego pytania, ale i tak prawdopodobnie warto o tym wspomnieć:

Commons Lang ma kilka doskonałych metod, których możesz użyć do nadpisywania równości i hashcode. Sprawdź EqualsBuilder.reflectionEquals (...) i HashCodeBuilder.reflectionHashCode (...) . W przeszłości zaoszczędziło mi to wielu bólu głowy - chociaż oczywiście, jeśli chcesz po prostu zrobić „równa się” na legitymacji, może to nie pasować do twoich okoliczności.

Zgadzam się również, że powinieneś używać @Overrideadnotacji zawsze, gdy zastępujesz równa się (lub jakakolwiek inna metoda).

Innym szybkim rozwiązaniem, które zapisuje standardowy kod, jest adnotacja Lombok EqualsAndHashCode . Jest łatwy, elegancki i konfigurowalny. I nie zależy od IDE . Na przykład;

import lombok.EqualsAndHashCode;

@EqualsAndHashCode(of={"errorNumber","messageCode"}) // Will only use this fields to generate equals.
public class ErrorMessage{

    private long        errorNumber;
    private int         numberOfParameters;
    private Level       loggingLevel;
    private String      messageCode;

Zobacz dostępne opcje, aby dostosować, które pola mają być używane w równych. Lombok jest dostępny w Maven . Po prostu dodaj go z podanym zakresem:

<dependency>
    <groupId>org.projectlombok</groupId>
    <artifactId>lombok</artifactId>
    <version>1.14.8</version>
    <scope>provided</scope>
</dependency>

w Android Studio jest alt + insert ---> equals i hashCode

Przykład:

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

    Proveedor proveedor = (Proveedor) o;

    return getId() == proveedor.getId();

}

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

Rozważać:

Object obj = new Book();
obj.equals("hi");
// Oh noes! What happens now? Can't call it with a String that isn't a Book...

instanceOfoświadczenie jest często używany w realizacji równymi.

To popularna pułapka!

Problem w tym, że używanie instanceOfnarusza zasadę symetrii:

(object1.equals(object2) == true) wtedy i tylko wtedy gdy (object2.equals(object1))

jeśli pierwsze równa się jest prawdziwe, a obiekt2 jest instancją podklasy klasy, do której należy obiekt1, to drugie równa się zwróci fałsz!

jeśli rozpatrywana klasa, do której należy ob1, jest zadeklarowana jako ostateczna, to ten problem nie może powstać, ale generalnie należy przetestować w następujący sposób:

this.getClass() != otherObject.getClass(); jeśli nie, zwróć false, w przeciwnym razie przetestuj pola, aby porównać je pod kątem równości!

recordId jest właściwością obiektu

@Override
    public boolean equals(Object obj) {
        if (this == obj)
            return true;
        if (obj == null)
            return false;
        if (getClass() != obj.getClass())
            return false;
        Nai_record other = (Nai_record) obj;
        if (recordId == null) {
            if (other.recordId != null)
                return false;
        } else if (!recordId.equals(other.recordId))
            return false;
        return true;
    }