Substituindo o método java equals () - não está funcionando?

Corri para um problema interessante (e muito frustrante) com o equals()método hoje, que causou o travamento do que eu pensava ser uma classe bem testada e causou um bug que levou muito tempo para ser rastreado.

Apenas para completar, eu não estava usando um IDE ou depurador - apenas um bom editor de texto à moda antiga e System.out. O tempo era muito limitado e era um projeto escolar.

De qualquer forma -

Eu estava desenvolvendo um carrinho de compras básico que poderia conter um ArrayListdos Bookobjetos . A fim de implementar as addBook(), removeBook()e hasBook()métodos do carrinho, eu queria verificar se o Bookjá existia no Cart. Então eu vou -

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

Tudo funciona bem nos testes. Crio 6 objetos e os preencho com dados. Faça muitas adições, remoções, operações com () Carte tudo funciona bem. Eu li que você pode ter equals(TYPE var)ouequals(Object o) { (CAST) var } presumiu que, como estava funcionando, não importava muito.

Então eu corri para um problema - eu precisava para criar um Bookobjeto com única a IDnele de dentro da classe Book. Nenhum outro dado seria inserido nele. Basicamente o seguinte:

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

De repente, o equals(Book b)método não funciona mais. Demorou MUITO tempo para rastrear sem um bom depurador e assumindo que a Cartclasse foi testada e correta corretamente. Depois de trocar o equals()método para o seguinte:

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

Tudo começou a funcionar novamente. Existe uma razão pela qual o método decidiu não usar o parâmetro Book, mesmo que claramente fosse um Bookobjeto? A única diferença parecia ser instanciada na mesma classe e preenchida apenas com um membro dos dados. Estou muito, muito confusa. Por favor, lançar alguma luz?

Em Java, o equals()método herdado Objecté:

public boolean equals(Object other);

Em outras palavras, o parâmetro deve ser do tipo Object. Isso é chamado de substituição ; seu método public boolean equals(Book other)faz o que é chamado de sobrecarga para o equals()método.

Os métodos ArrayListsubstituídos são usados equals()para comparar o conteúdo (por exemplo, para os métodos contains()e equals()) e não para os sobrecarregados. Na maioria do seu código, chamar o que não substituiu corretamente Objectos iguais foi bom, mas não compatível ArrayList.

Portanto, não substituir o método corretamente pode causar problemas.

Eu substituo é igual ao seguinte sempre:

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

O uso da @Overrideanotação pode ajudar bastante com erros tolos.

Use-o sempre que achar que está substituindo o método de uma superclasse ou interface. Dessa forma, se você fizer da maneira errada, receberá um erro de compilação.

Se você usa eclipse, basta ir ao menu superior

Fonte -> Gerar equals () e hashCode ()

Ligeiramente fora de tópico para sua pergunta, mas provavelmente vale a pena mencionar de qualquer maneira:

O Commons Lang possui alguns métodos excelentes que você pode usar para substituir iguais e hashcode. Confira EqualsBuilder.reflectionEquals (...) e HashCodeBuilder.reflectionHashCode (...) . Me salvou muita dor de cabeça no passado - embora, é claro, se você quiser apenas "igual" ao ID, ele pode não se encaixar nas suas circunstâncias.

Concordo também que você deve usar a @Overrideanotação sempre que substituir valores iguais (ou qualquer outro método).

Outra solução rápida que salva o código padrão é a anotação Lombok EqualsAndHashCode . É fácil, elegante e personalizável. E não depende do IDE . Por exemplo;

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;

Veja as opções disponíveis para personalizar quais campos usar nos iguais. Lombok está disponível em maven . Basta adicioná-lo com o escopo fornecido :

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

no Android Studio é alt + insert ---> equals e hashCode

Exemplo:

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

Considerar:

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

a instanceOfdeclaração é freqüentemente usada na implementação de iguais.

Esta é uma armadilha popular!

O problema é que o uso instanceOfviola a regra da simetria:

(object1.equals(object2) == true) se e apenas se (object2.equals(object1))

se o primeiro igual é verdadeiro e o objeto2 é uma instância de uma subclasse da classe à qual o obj1 pertence, o segundo igual retornará falso!

se a classe considerada à qual ob1 pertence for declarada como final, esse problema não poderá surgir, mas, em geral, você deve testar da seguinte maneira:

this.getClass() != otherObject.getClass(); Caso contrário, retorne false; caso contrário, teste os campos para comparar a igualdade!

recordId é propriedade do objeto

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