Java equals() ordenando

Question

Si trato de hacer un .equals() en una cadena nula en java, se lanzará una excepción de puntero nulo. Me pregunto, si yo estoy tratando de comparar si una cadena es igual a alguna cadena constante, puedo hacer lo siguiente:

MY_CONSTANT_STRING.equals(aStringVariable) 

Sé que va a funcionar, pero es el código realmente pobres?

Answer 1

Esta es una expresión idiomática de Java llamada en broma Yoda condition.

Personalmente prefiero manejar el caso nulo explícitamente, pero el modo Yoda se usa mucho y cualquier programador experimentado en Java debería ser capaz de entender lo que está sucediendo inmediatamente. Está bien de usar.

Answer 2

Lo que tienes está bien. Incluso es posible usar un literal String.

if("value".equals(variable)) { 
    ... 

Si no te gusta eso, siempre se puede comprobar de forma explícita para null y la igualdad, y combinar los dos cheques con &&. El cortocircuito del operador se asegurará de que nunca obtenga un NPE.

if((variable != null) && variable.equals("value")) { 
    ... 

Answer 3

es esta realmente mal código?

No, esta es la forma en que muchas personas codificarían la declaración para evitar NPE.

Answer 4

No, generalmente se hace para evitar NPE. Sin embargo, generalmente prefiero hacer una comprobación explícita de nulo.

Answer 5

Guardaría el código "CONSTANT.equals (possibleNull)" sin la prueba nula solo si es una condición normal que la variable podría ser nula, por ejemplo porque acaba de salir de un mapa de propiedades.

Del mismo modo que puede salirse con la no comprobación de nulo en instanceof controles - como:

Food dinner = map.get("dinner"); 
if (dinner instanceof Soup) { 
     ((Soup)blah).eat(); 
} // We don't care if it is a Fish or null 

Pero si realmente no esperaba nula, debe comprobar de forma explícita para que en una prueba si es independiente, y manejarlo apropiadamente. En general, es mejor detectar estos errores de datos más temprano que tarde.

Answer 6

Si usted está preocupado por la calidad de su código escriba una clase de ayuda que se encarga de prueba de la igualdad:

public class ObjectHelper { 
    public static boolean testEquality(Object o1, Object o2) { 
     if (o1 == null && o2 == null) return true; 
     if (o1 == null) return false; 
     return o1.equalts(o2); 
    } 
} 

Entonces utilizar de esta manera:

if (ObjectHelper.testEquality(aStringVariable, My_CONSTANT_STRING)) 

Su llamada constante PODRÍA dejar de ser constante. Puede leerse desde un archivo de configuración en el futuro.