Как переписать метод Equals

Question

Я видел коллегу написать это следующим образом:

@Override 
public boolean equals(Object obj) { 
    if (this == obj) { 
     return true; 
    } 
    if (obj == null) { 
     return false; 
    } 
    if (getClass() != obj.getClass()) { 
     return false; 
    } 
    final ImageCode other = (ImageCode) obj; 
    if ((this.code == null) ? (other.code != null) : !this.code.equals(other.code)) { 
     return false; 
    } 
    return true; 
} 

Тогда я предложил изменить его для этого:

@Override 
public boolean equals(Object obj) { 
    if (this == obj) { 
     return true; 
    } 
    if (!(obj instanceof Parent)){ 
     return false; 
    } 
    final ImageCode other = (ImageCode) obj; 
    if ((this.code == null) ? (other.code != null) : !this.code.equals(other.code)) { 
     return false; 
    } 
    return true; 
} 

, но он сказал мне, что это неправильно, и что instanceof не может использоваться для метода equals. Я не понимаю, почему он это сказал.

Answer 1

Вы можете использовать instanceof в equals. Java использует его все время. См. String

public boolean equals(Object anObject) { 
    if (this == anObject) { 
     return true; 
    } 
    if (anObject instanceof String) { 
     String anotherString = (String)anObject; 

     // More code ... 
     return true; 
    } 
    return false; 
} 

Answer 2

@dejvuth, Если anObject является экземпляр String, то почему мы должны типаж. Вы можете напрямую использовать его. Я думаю, что все в порядке, не проверяя instanceOf. Просто вам может понадобиться ввести листинг объекта. Здесь вы можете проверить мой код.