Ghi đè phương thức java bằng () - không hoạt động?

Tôi đã gặp phải một vấn đề thú vị (và rất bực bội) với equals()phương pháp ngày hôm nay, điều khiến cho thứ mà tôi nghĩ là một lớp được kiểm tra tốt bị sập và gây ra một lỗi khiến tôi mất nhiều thời gian để theo dõi.

Để hoàn thiện, tôi đã không sử dụng IDE hoặc trình gỡ lỗi - chỉ là trình soạn thảo văn bản lỗi thời và System.out. Thời gian rất hạn chế và đó là một dự án trường học.

Dù sao đi nữa -

Tôi đã phát triển một giỏ mua hàng cơ bản mà có thể chứa một ArrayListsố Bookđối tượng . Để thực hiện addBook(), removeBook()và hasBook()phương pháp của giỏ hàng, tôi muốn kiểm tra xem Bookđã tồn tại trong Cart. Vì vậy, tôi đi -

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

Tất cả đều hoạt động tốt trong thử nghiệm. Tôi tạo 6 đối tượng và điền chúng với dữ liệu. Thực hiện nhiều thao tác thêm, xóa, có () trên Cartvà mọi thứ đều hoạt động tốt. Tôi đọc rằng bạn có thể có equals(TYPE var)hoặcequals(Object o) { (CAST) var } nhưng cho rằng vì nó đang hoạt động nên nó không quá quan trọng.

Sau đó, tôi chạy vào một vấn đề - Tôi cần thiết để tạo ra một Bookđối tượng với chỉ những IDở trong đó từ bên trong lớp Book. Không có dữ liệu khác sẽ được nhập vào nó. Về cơ bản như sau:

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);
}

Đột nhiên, equals(Book b)phương pháp không còn hoạt động. Điều này đã mất một thời gian RẤT lâu để theo dõi mà không có trình sửa lỗi tốt và giả sử rằng Cartlớp đã được kiểm tra và sửa chữa đúng. Sau khi hoán đổi equals()phương thức như sau:

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

Mọi thứ bắt đầu hoạt động trở lại. Có lý do nào để phương thức quyết định không lấy tham số Sách mặc dù rõ ràng nó là một Bookđối tượng? Sự khác biệt duy nhất dường như là nó được khởi tạo từ trong cùng một lớp và chỉ chứa đầy một thành viên dữ liệu. Tôi rất rất bối rối. Xin vui lòng, làm sáng tỏ một số?

Trong Java, equals()phương thức được kế thừa từ Object:

public boolean equals(Object other);

Nói cách khác, tham số phải là loại Object. Điều này được gọi là ghi đè ; phương thức của bạn public boolean equals(Book other)thực hiện những gì được gọi là quá tải cho equals()phương thức.

Việc ArrayListsử dụng equals()các phương thức được ghi đè để so sánh nội dung (ví dụ: nội dung contains()và equals()phương thức), không bị quá tải. Trong hầu hết các mã của bạn, việc gọi một mã không ghi đè chính xác Objectbằng là tốt, nhưng không tương thích ArrayList.

Vì vậy, không ghi đè phương thức chính xác có thể gây ra vấn đề.

Tôi ghi đè bằng mọi lúc sau:

@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...
}

Việc sử dụng @Overridechú thích có thể giúp một tấn với những sai lầm ngớ ngẩn.

Sử dụng nó bất cứ khi nào bạn nghĩ rằng bạn đang ghi đè phương thức của siêu lớp 'hoặc giao diện. Bằng cách đó, nếu bạn làm sai cách, bạn sẽ gặp lỗi biên dịch.

Nếu bạn sử dụng nhật thực, chỉ cần vào menu trên cùng

Nguồn -> Tạo bằng () và hashCode ()

Hơi lạc đề với câu hỏi của bạn, nhưng dù sao thì nó cũng có thể được đề cập đến:

Commons Lang đã có một số phương pháp tuyệt vời mà bạn có thể sử dụng để ghi đè bằng và mã băm. Hãy xem EqualsBuilder.reflectionEquals (...) và HashCodeBuilder.reflectionHashCode (...) . Đã cứu tôi rất nhiều đau đầu trong quá khứ - mặc dù tất nhiên nếu bạn chỉ muốn làm "bằng" trên ID thì nó có thể không phù hợp với hoàn cảnh của bạn.

Tôi cũng đồng ý rằng bạn nên sử dụng @Overridechú thích bất cứ khi nào bạn ghi đè bằng (hoặc bất kỳ phương pháp nào khác).

Một giải pháp nhanh khác giúp tiết kiệm mã soạn sẵn là chú thích Lombok EqualsAndHashCode . Nó là dễ dàng, thanh lịch và tùy biến. Và không phụ thuộc vào IDE . Ví dụ;

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;

Xem các tùy chọn có sẵn để tùy chỉnh các trường sẽ sử dụng trong bằng. Lombok có sẵn trong maven . Chỉ cần thêm nó với phạm vi được cung cấp :

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

trong Android Studio là alt + insert ---> bằng và hashCode

Thí dụ:

    @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();
}

Xem xét:

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

các instanceOftuyên bố thường được sử dụng trong việc thực hiện bình đẳng.

Đây là một cạm bẫy phổ biến!

Vấn đề là việc sử dụng instanceOfvi phạm quy tắc đối xứng:

(object1.equals(object2) == true) nếu và chỉ nếu (object2.equals(object1))

nếu bằng thứ nhất là đúng và object2 là một thể hiện của một lớp con của lớp mà obj1 thuộc về, thì bằng thứ hai sẽ trả về false!

nếu lớp được coi là nơi ob1 thuộc về được khai báo là cuối cùng, thì vấn đề này không thể phát sinh, nhưng nói chung, bạn nên kiểm tra như sau:

this.getClass() != otherObject.getClass(); nếu không, trả về false, nếu không thì kiểm tra các trường để so sánh cho bằng nhau!

recordId là tài sản của đối tượng

@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;
    }