Java: Manera limpia de evitar NullPointerException en las comprobaciones equivalentes

Question

Tengo un objeto de dirección para el que quiero crear un método igual. Podría haber hecho esto muy sencilla haciendo algo como lo siguiente (se ha reducido un poco):

public boolean equals(Object obj) 
{ 
    if (this == obj) 
     return true; 

    if (obj == null) 
     return false; 

    if (getClass() != obj.getClass()) 
     return false; 

    Address other = (Address) obj; 

    return this.getStreet().equals(other.getStreet()) 
     && this.getStreetNumber().equals(other.getStreetNumber()) 
     && this.getStreetLetter().equals(other.getStreetLetter()) 
     && this.getTown().equals(other.getTown()); 
} 

El problema es que algunos de estos podría ser nulo. En otras palabras, obtendré un NullPointerException si no hay una letra de calle en esta dirección.

¿Cómo puedo escribir esto de forma limpia teniendo en cuenta valores nulos?

Answer 1

Se puede utilizar un método de ayuda como

public static boolean isEqual(Object o1, Object o2) { 
    return o1 == o2 || (o1 != null && o1.equals(o2)); 
} 

Answer 2

Se podría hacer lo siguiente:

public boolean equals(Object obj) 
{ 
    if (this == obj) { 
     return true; 
    } 

    if (obj == null) { 
     return false; 
    } 

    if (getClass() != obj.getClass()) { 
     return false; 
    } 

    Address other = (Address) obj; 

    return equals(this.getStreet(),other.getStreet()) 
     && equals(this.getStreetNumber(), other.getStreetNumber()) 
     && equals(this.getStreetLetter(), other.getStreetLetter()) 
     && equals(this.getTown(), other.getTown()); 
} 

private boolean equals(Object control, Object test) { 
    if(null == control) { 
     return null == test; 
    } 
    return control.equals(test); 
} 

Java 7 introducido soporte incorporado para este caso de uso con la clase java.util.Objects ver:

• java.utils.Objects.equals(Object, Object)
• java.utils.Objects.deepEquals(Object, Object)

Answer 3

No hay una manera realmente limpia de hacerlo; la mejor opción es que tu IDE genere el código para ti. Eclipse puede hacerlo a través del menú contextual Source -> Generate hashCode() y equals().

Answer 4

Consideraría definir algunos de los métodos iguales como métodos de clase estáticos, como decir para los objetos de Street. De esta forma, nunca intente llamar al método .equals() en un nulo.

una función de ejemplo podría ser:

public static boolean equals(Object one, Object two) 

Además, es una buena práctica para poner los cheques como

if (obj == null) 
    return false; 

en el comienzo de una función.

Answer 5

Google Guava proporciona Objects.equal(Object, Object) que verifica la igualdad, teniendo en cuenta que ninguno de los parámetros podría ser nulo:

... 
return Objects.equal(this.getStreet(), other.getStreet()) 
    && Objects.equal(this.getStreetNumber(), other.getStreetNumber()) 
    && Objects.equal(this.getStreetLetter(), other.getStreetLetter()) 
    && Objects.equal(this.getTown(), other.getTown()); 

También vale la pena señalar que los objetos tienen otros métodos de ayuda para la implementación de hashCode() y toString().

Answer 6

Puede utilizar Objects.equal de Googles guayaba o la EqualsBuilder de apache commons

Answer 7

tengo una clase de ayuda del inspector w/un método estático:

public static boolean isEquals(final Object o1, final Object o2) { 
     return o1 == null ? o2 == null : o1.equals(o2); 
} 

así, en el método equals,

return Checker.isEquals(this.getStreet(), other.getStreet()) 
     && Checker.isEquals(this.getStreetNumber(), other.getStreetNumber()) 
     && Checker.isEquals(this.getStreetLetter(), other.getStreetLetter()) 
     && Checker.isEquals(this.getTown(), other.getTown()); 

Answer 8

Apache Commons Lang proporciona la clase de ayuda EqualsBuilder para comparar la igualdad. También hay uno para códigos hash.

return new EqualsBuilder() 
.append(this.getStreet(), other.getStreet()) 
.append(this.getStreetNumber(), other.getStreetNumber() 
.append(this.getStreetLetter(), other.getStreetLetter()) 
.append(this.getTown(), other.getTown())).isEquals();