Evitar! = Declaraciones nulas

Yo uso object != nullmucho para evitar NullPointerException.

¿Hay una buena alternativa a esto?

Por ejemplo, a menudo uso:

if (someobject != null) {
    someobject.doCalc();
}

Esto comprueba una NullPointerExceptionpara el someobjectobjeto en el fragmento anterior.

Tenga en cuenta que la respuesta aceptada puede estar desactualizada; consulte https://stackoverflow.com/a/2386013/12943 para obtener un enfoque más reciente.

Esto para mí suena como un problema razonablemente común que los desarrolladores junior a intermedios tienden a enfrentar en algún momento: o no saben o no confían en los contratos en los que están participando y sobreprotegen defensivamente los nulos. Además, cuando escriben su propio código, tienden a confiar en los valores nulos devueltos para indicar algo que requiere que la persona que llama compruebe los valores nulos.

Para decirlo de otra manera, hay dos casos en los que aparece la comprobación nula:

1. Donde nulo es una respuesta válida en términos del contrato; y

2. Donde no es una respuesta válida.

(2) es fácil. Utilice assertsentencias (aserciones) o permita fallas (por ejemplo, NullPointerException ). Las aserciones son una característica de Java altamente infrautilizada que se agregó en 1.4. La sintaxis es:

assert <condition>

o

assert <condition> : <object>

donde <condition>es una expresión booleana y <object>es un objeto cuyo toString()resultado del método se incluirá en el error.

Una assertdeclaración arroja un Error( AssertionError) si la condición no es verdadera. Por defecto, Java ignora las aserciones. Puede habilitar aserciones pasando la opción -eaa la JVM. Puede habilitar y deshabilitar aserciones para clases y paquetes individuales. Esto significa que puede validar el código con las aserciones durante el desarrollo y las pruebas, y deshabilitarlas en un entorno de producción, aunque mis pruebas casi no han demostrado un impacto en el rendimiento de las aserciones.

No usar aserciones en este caso está bien porque el código simplemente fallará, que es lo que sucederá si usa aserciones. La única diferencia es que con las afirmaciones puede ocurrir antes, de una manera más significativa y posiblemente con información adicional, lo que puede ayudarlo a descubrir por qué sucedió si no lo esperaba.

(1) es un poco más difícil. Si no tienes control sobre el código al que estás llamando, entonces estás atascado. Si nulo es una respuesta válida, debe verificarlo.

Sin embargo, si controlas el código (y este suele ser el caso), entonces es una historia diferente. Evite usar nulos como respuesta. Con los métodos que devuelven colecciones, es fácil: devolver colecciones vacías (o matrices) en lugar de nulos casi todo el tiempo.

Con no colecciones puede ser más difícil. Considere esto como un ejemplo: si tiene estas interfaces:

public interface Action {
  void doSomething();
}

public interface Parser {
  Action findAction(String userInput);
}

donde Parser toma datos sin procesar del usuario y encuentra algo que hacer, tal vez si está implementando una interfaz de línea de comando para algo. Ahora puede hacer que el contrato que devuelve sea nulo si no hay una acción adecuada. Eso lleva a la comprobación nula de la que estás hablando.

Una solución alternativa es no devolver nunca nulo y en su lugar usar el patrón de objeto nulo :

public class MyParser implements Parser {
  private static Action DO_NOTHING = new Action() {
    public void doSomething() { /* do nothing */ }
  };

  public Action findAction(String userInput) {
    // ...
    if ( /* we can't find any actions */ ) {
      return DO_NOTHING;
    }
  }
}

Comparar:

Parser parser = ParserFactory.getParser();
if (parser == null) {
  // now what?
  // this would be an example of where null isn't (or shouldn't be) a valid response
}
Action action = parser.findAction(someInput);
if (action == null) {
  // do nothing
} else {
  action.doSomething();
}

a

ParserFactory.getParser().findAction(someInput).doSomething();

que es un diseño mucho mejor porque conduce a un código más conciso.

Dicho esto, quizás sea completamente apropiado que el método findAction () arroje una excepción con un mensaje de error significativo, especialmente en este caso en el que confía en la entrada del usuario. Sería mucho mejor para el método findAction lanzar una excepción que para que el método de llamada explote con una simple NullPointerException sin explicación.

try {
    ParserFactory.getParser().findAction(someInput).doSomething();
} catch(ActionNotFoundException anfe) {
    userConsole.err(anfe.getMessage());
}

O si cree que el mecanismo de prueba / captura es demasiado feo, en lugar de no hacer nada, su acción predeterminada debería proporcionar comentarios al usuario.

public Action findAction(final String userInput) {
    /* Code to return requested Action if found */
    return new Action() {
        public void doSomething() {
            userConsole.err("Action not found: " + userInput);
        }
    }
}

Si usa (o planea usar) un IDE de Java como JetBrains IntelliJ IDEA , Eclipse o Netbeans o una herramienta como findbugs, puede usar anotaciones para resolver este problema.

Básicamente, tienes @Nullabley @NotNull.

Puede usar métodos y parámetros, como este:

@NotNull public static String helloWorld() {
    return "Hello World";
}

o

@Nullable public static String helloWorld() {
    return "Hello World";
}

El segundo ejemplo no se compilará (en IntelliJ IDEA).

Cuando usa la primera helloWorld()función en otro código:

public static void main(String[] args)
{
    String result = helloWorld();
    if(result != null) {
        System.out.println(result);
    }
}

Ahora el compilador IntelliJ IDEA le dirá que la verificación es inútil, ya que la helloWorld()función no volverá nullnunca.

Usando parámetro

void someMethod(@NotNull someParameter) { }

si escribes algo como:

someMethod(null);

Esto no se compilará.

Último ejemplo usando @Nullable

@Nullable iWantToDestroyEverything() { return null; }

Haciendo esto

iWantToDestroyEverything().something();

Y puede estar seguro de que esto no sucederá. :)

Es una buena manera de dejar que el compilador verifique algo más de lo que suele hacer y hacer cumplir sus contratos para que sean más fuertes. Desafortunadamente, no es compatible con todos los compiladores.

En IntelliJ IDEA 10.5 y posteriores, agregaron soporte para cualquier otra @Nullable @NotNullimplementación.

Ver publicación del blog Anotaciones más flexibles y configurables de @ Nullable / @ NotNull .

Si no se permiten valores nulos

Si su método se llama externamente, comience con algo como esto:

public void method(Object object) {
  if (object == null) {
    throw new IllegalArgumentException("...");
  }

Luego, en el resto de ese método, sabrás que objectno es nulo.

Si es un método interno (no parte de una API), solo documente que no puede ser nulo, y eso es todo.

Ejemplo:

public String getFirst3Chars(String text) {
  return text.subString(0, 3);
}

Sin embargo, si su método solo transfiere el valor y el siguiente método lo transfiere, etc., podría ser problemático. En ese caso, es posible que desee verificar el argumento como se indica arriba.

Si se permite nulo

Esto realmente depende. Si encuentro que a menudo hago algo como esto:

if (object == null) {
  // something
} else {
  // something else
}

Entonces me ramifico y hago dos cosas completamente diferentes. No hay un fragmento de código feo, porque realmente necesito hacer dos cosas diferentes dependiendo de los datos. Por ejemplo, ¿debo trabajar en la entrada o debo calcular un buen valor predeterminado?

En realidad, es raro que use el idioma " if (object != null && ...".

Puede ser más fácil darle ejemplos, si muestra ejemplos de dónde usa habitualmente el idioma.

Wow, casi odio agregar otra respuesta cuando tenemos 57 formas diferentes de recomendar la NullObject pattern, pero creo que a algunas personas interesadas en esta pregunta les gustaría saber que hay una propuesta sobre la mesa para Java 7 para agregar "null-safe manejo ": una sintaxis simplificada para la lógica si no es igual a nula.

El ejemplo dado por Alex Miller se ve así:

public String getPostcode(Person person) {  
  return person?.getAddress()?.getPostcode();  
}  

Los ?.medios solo desreferencian el identificador izquierdo si no es nulo; de lo contrario, evalúe el resto de la expresión como null. Algunas personas, como Dick Wall, miembro de Java Posse, y los votantes de Devoxx realmente adoran esta propuesta, pero también hay oposición, alegando que en realidad fomentará un mayor uso nullcomo valor centinela.

Actualización: se ha presentado una propuesta oficial para un operador nulo-seguro en Java 7 bajo Project Coin. La sintaxis es un poco diferente al ejemplo anterior, pero es la misma noción.

Actualización: la propuesta del operador nulo-seguro no se convirtió en Project Coin. Por lo tanto, no verá esta sintaxis en Java 7.

Si no se permiten valores indefinidos:

Puede configurar su IDE para advertirle sobre posibles desreferencias nulas. Por ejemplo, en Eclipse, vea Preferencias> Java> Compilador> Errores / Advertencias / Análisis nulo .

Si se permiten valores indefinidos:

Si desea definir una nueva API donde los valores indefinidos tienen sentido , use el Patrón de opción (puede ser familiar a partir de lenguajes funcionales). Tiene las siguientes ventajas:

• Se establece explícitamente en la API si existe o no una entrada o salida.
• El compilador lo obliga a manejar el caso "indefinido".
• La opción es una mónada , por lo que no hay necesidad de una verificación nula detallada, solo use map / foreach / getOrElse o un combinador similar para usar el valor de manera segura (ejemplo) .

Java 8 tiene una Optionalclase incorporada (recomendado); para las versiones anteriores, hay alternativas de la biblioteca, por ejemplo guayaba 's Optionalo FunctionalJava ' s Option. Pero al igual que muchos patrones de estilo funcional, el uso de Opción en Java (incluso 8) da como resultado un poco de repetitivo, que puede reducir utilizando un lenguaje JVM menos detallado, por ejemplo, Scala o Xtend.

Si tiene que lidiar con una API que podría devolver nulos , no puede hacer mucho en Java. Xtend y Groovy tienen el operador de Elvis ?: y el operador de anulación de referencia de seguridad nula ?. , pero tenga en cuenta que esto devuelve nulo en caso de una referencia nula, por lo que simplemente "difiere" el manejo adecuado de nulo.

Solo para esta situación:

No verificar si una variable es nula antes de invocar un método igual (un ejemplo de comparación de cadenas a continuación):

if ( foo.equals("bar") ) {
 // ...
}

resultará en un NullPointerExceptionif foono existe.

Puede evitar eso si compara sus Stringmensajes de esta manera:

if ( "bar".equals(foo) ) {
 // ...
}

Con Java 8 viene la nueva java.util.Optionalclase que podría resolver parte del problema. Al menos se puede decir que mejora la legibilidad del código y, en el caso de las API públicas, hace que el contrato de la API sea más claro para el desarrollador del cliente.

Ellos trabajan así:

Se Fruitcrea un objeto opcional para un tipo dado ( ) como el tipo de retorno de un método. Puede estar vacío o contener un Fruitobjeto:

public static Optional<Fruit> find(String name, List<Fruit> fruits) {
   for (Fruit fruit : fruits) {
      if (fruit.getName().equals(name)) {
         return Optional.of(fruit);
      }
   }
   return Optional.empty();
}

Ahora mire este código donde buscamos una lista de Fruit( fruits) para una instancia de Fruit dada:

Optional<Fruit> found = find("lemon", fruits);
if (found.isPresent()) {
   Fruit fruit = found.get();
   String name = fruit.getName();
}

Puede usar el map()operador para realizar un cálculo en un objeto opcional o extraer un valor de este. orElse()le permite proporcionar una reserva para los valores faltantes.

String nameOrNull = find("lemon", fruits)
    .map(f -> f.getName())
    .orElse("empty-name");

Por supuesto, la verificación del valor nulo / vacío sigue siendo necesaria, pero al menos el desarrollador es consciente de que el valor puede estar vacío y el riesgo de olvidar verificar es limitado.

En una API construida desde cero utilizando Optionalsiempre que un valor de retorno puede estar vacío, y devolviendo un objeto plano solo cuando no puede ser null(convención), el código del cliente puede abandonar las verificaciones nulas en valores de retorno de objetos simples ...

Por supuesto Optional, también podría usarse como argumento de método, tal vez una mejor manera de indicar argumentos opcionales que 5 o 10 métodos de sobrecarga en algunos casos.

Optionalofrece otros métodos convenientes, como el orElseque permite el uso de un valor predeterminado y ifPresentque funciona con expresiones lambda .

Los invito a leer este artículo (mi fuente principal para escribir esta respuesta) en el que la NullPointerExceptionsolución problemática (y en general puntero nulo) así como la solución (parcial) presentada Optionalestán bien explicadas: objetos opcionales Java .

Dependiendo del tipo de objetos que esté verificando, puede usar algunas de las clases en los comunes de apache como: apache commons lang y colecciones de apache commons

Ejemplo:

String foo;
...
if( StringUtils.isBlank( foo ) ) {
   ///do something
}

o (según lo que necesite verificar):

String foo;
...
if( StringUtils.isEmpty( foo ) ) {
   ///do something
}

La clase StringUtils es solo una de muchas; Hay bastantes buenas clases en los comunes que hacen una manipulación segura nula.

Aquí sigue un ejemplo de cómo puede usar la validación nula en JAVA cuando incluye la biblioteca apache (commons-lang-2.4.jar)

public DOCUMENT read(String xml, ValidationEventHandler validationEventHandler) {
    Validate.notNull(validationEventHandler,"ValidationHandler not Injected");
    return read(new StringReader(xml), true, validationEventHandler);
}

Y si está utilizando Spring, Spring también tiene la misma funcionalidad en su paquete, consulte la biblioteca (spring-2.4.6.jar)

Ejemplo sobre cómo usar este classf estático de spring (org.springframework.util.Assert)

Assert.notNull(validationEventHandler,"ValidationHandler not Injected");

• Si considera que un objeto no debe ser nulo (o es un error), use una aserción.
• Si su método no acepta parámetros nulos, dígalo en el javadoc y use una aserción.

Debe verificar el objeto! = Nulo solo si desea manejar el caso donde el objeto puede ser nulo ...

Hay una propuesta para agregar nuevas anotaciones en Java7 para ayudar con los parámetros nulos / no nulos: http://tech.puredanger.com/java7/#jsr308

Soy fanático del código de "falla rápida". Pregúntese: ¿está haciendo algo útil en el caso de que el parámetro sea nulo? Si no tiene una respuesta clara de lo que debe hacer su código en ese caso ... Es decir, nunca debe ser nulo en primer lugar, luego ignórelo y permita que se arroje una NullPointerException. El código de llamada tendrá tanto sentido de un NPE como lo haría con una IllegalArgumentException, pero será más fácil para el desarrollador depurar y comprender qué salió mal si se lanza un NPE en lugar de que su código intente ejecutar alguna otra contingencia inesperada lógica, que en última instancia da como resultado que la aplicación falle de todos modos.

El marco de colecciones de Google ofrece una forma buena y elegante de lograr la verificación nula.

Hay un método en una clase de biblioteca como este:

static <T> T checkNotNull(T e) {
   if (e == null) {
      throw new NullPointerException();
   }
   return e;
}

Y el uso es (con import static):

...
void foo(int a, Person p) {
   if (checkNotNull(p).getAge() > a) {
      ...
   }
   else {
      ...
   }
}
...

O en tu ejemplo:

checkNotNull(someobject).doCalc();

A veces, tiene métodos que operan en sus parámetros que definen una operación simétrica:

a.f(b); <-> b.f(a);

Si sabe que b nunca puede ser nulo, simplemente puede intercambiarlo. Es más útil para iguales: en lugar de foo.equals("bar");hacerlo mejor"bar".equals(foo); .

En lugar del patrón de objeto nulo, que tiene sus usos, puede considerar situaciones en las que el objeto nulo es un error.

Cuando se lanza la excepción, examine la traza de la pila y resuelva el error.

Nulo no es un "problema". Es una parte integral de un conjunto completo de herramientas de modelado. El software tiene como objetivo modelar la complejidad del mundo y el nulo lleva su carga. Nulo indica 'Sin datos' o 'Desconocido' en Java y similares. Por lo tanto, es apropiado utilizar valores nulos para estos fines. No prefiero el patrón 'Objeto nulo'; Creo que surge el problema de " quién protegerá a los guardianes ".
Si me preguntas cómo se llama mi novia, te diré que no tengo novia. En el lenguaje Java devolveré nulo. Una alternativa sería lanzar una excepción significativa para indicar algún problema que no puede ser (o no

1. Para una 'pregunta desconocida', dé una 'respuesta desconocida'. (Sea nulo-seguro cuando esto sea correcto desde el punto de vista comercial) Verificar los argumentos para nulo una vez dentro de un método antes de su uso alivia a varias personas que llaman de verificarlos antes de una llamada.

   public Photo getPhotoOfThePerson(Person person) {
       if (person == null)
           return null;
       // Grabbing some resources or intensive calculation
       // using person object anyhow.
   }

   Anterior conduce a un flujo lógico normal para no obtener una foto de una novia inexistente de mi biblioteca de fotos.

   getPhotoOfThePerson(me.getGirlfriend())

   Y encaja con la nueva API Java que viene (mirando hacia adelante)

   getPhotoByName(me.getGirlfriend()?.getName())

   Si bien es más bien 'flujo comercial normal' no encontrar fotos almacenadas en la base de datos para alguna persona, solía usar pares como a continuación para otros casos

   public static MyEnum parseMyEnum(String value); // throws IllegalArgumentException
   public static MyEnum parseMyEnumOrNull(String value);

   Y no detesta escribir <alt> + <shift> + <j>(generar javadoc en Eclipse) y escribir tres palabras adicionales para su API pública. Esto será más que suficiente para todos, excepto para aquellos que no leen la documentación.

   /**
    * @return photo or null
    */

   o

   /**
    * @return photo, never null
    */

2. Este es un caso bastante teórico y, en la mayoría de los casos, debe preferir la API segura nula de Java (en caso de que se lance en otros 10 años), pero NullPointerExceptiones una subclase de una Exception. Por lo tanto, es una forma de Throwableque indica las condiciones que una aplicación razonable puede querer atrapar ( javadoc ). Para usar la primera ventaja más de las excepciones y separar el código de manejo de errores del código 'regular' ( según los creadores de Java ) es apropiado, como para mí, atrapar NullPointerException.

   public Photo getGirlfriendPhoto() {
       try {
           return appContext.getPhotoDataSource().getPhotoByName(me.getGirlfriend().getName());
       } catch (NullPointerException e) {
           return null;
       }
   }

   Pueden surgir preguntas:

   P. ¿Qué pasa si getPhotoDataSource()devuelve nulo?
   A. Depende de la lógica de negocios. Si no encuentro un álbum de fotos, no te mostraré fotos. ¿Qué pasa si appContext no se inicializa? La lógica de negocios de este método lo soporta. Si la misma lógica debe ser más estricta, entonces lanzar una excepción es parte de la lógica de negocios y se debe usar la verificación explícita de nulo (caso 3). los nueva API Java Null-safe encaja mejor aquí para especificar selectivamente lo que implica y lo que no implica que se inicialice para fallar rápidamente en caso de errores del programador.

   P. Se podría ejecutar código redundante y se podrían obtener recursos innecesarios.
   R. Podría tener lugar si getPhotoByName()intentara abrir una conexión de base de datos, crear PreparedStatementy usar el nombre de la persona como un parámetro SQL por fin. El enfoque para una pregunta desconocida da una respuesta desconocida (caso 1) funciona aquí. Antes de obtener recursos, el método debe verificar los parámetros y devolver el resultado 'desconocido' si es necesario.

   P. Este enfoque tiene una penalización de rendimiento debido a la apertura de cierre de prueba.
   A. El software debe ser fácil de entender y modificar en primer lugar. Solo después de esto, uno podría pensar en el rendimiento, ¡y solo si es necesario! y donde sea necesario! ( fuente ) y muchos otros).

   PD. Este enfoque será tan razonable de usar como el código de manejo de errores separado del principio del código "regular" es razonable de usar en algún lugar. Considere el siguiente ejemplo:

   public SomeValue calculateSomeValueUsingSophisticatedLogic(Predicate predicate) {
       try {
           Result1 result1 = performSomeCalculation(predicate);
           Result2 result2 = performSomeOtherCalculation(result1.getSomeProperty());
           Result3 result3 = performThirdCalculation(result2.getSomeProperty());
           Result4 result4 = performLastCalculation(result3.getSomeProperty());
           return result4.getSomeProperty();
       } catch (NullPointerException e) {
           return null;
       }
   }
   
   public SomeValue calculateSomeValueUsingSophisticatedLogic(Predicate predicate) {
       SomeValue result = null;
       if (predicate != null) {
           Result1 result1 = performSomeCalculation(predicate);
           if (result1 != null && result1.getSomeProperty() != null) {
               Result2 result2 = performSomeOtherCalculation(result1.getSomeProperty());
               if (result2 != null && result2.getSomeProperty() != null) {
                   Result3 result3 = performThirdCalculation(result2.getSomeProperty());
                   if (result3 != null && result3.getSomeProperty() != null) {
                       Result4 result4 = performLastCalculation(result3.getSomeProperty());
                       if (result4 != null) {
                           result = result4.getSomeProperty();
                       }
                   }
               }
           }
       }
       return result;
   }

   PPS Para aquellos que votan negativamente (y no tan rápido para leer la documentación), me gustaría decir que nunca he visto una excepción de puntero nulo (NPE) en mi vida. Pero esta posibilidad fue diseñada intencionalmente por los creadores de Java porque NPE es una subclase de Exception. Tenemos un precedente en la historia de Java cuando ThreadDeathes un Errorno, ya que es en realidad un error de aplicación, sino únicamente porque no tenía la intención de ser atrapado! ¡Cuánto NPE cabe para ser un Errorque ThreadDeath! Pero no lo es.

3. Verifique 'Sin datos' solo si la lógica de negocios lo implica.

   public void updatePersonPhoneNumber(Long personId, String phoneNumber) {
       if (personId == null)
           return;
       DataSource dataSource = appContext.getStuffDataSource();
       Person person = dataSource.getPersonById(personId);
       if (person != null) {
           person.setPhoneNumber(phoneNumber);
           dataSource.updatePerson(person);
       } else {
           Person = new Person(personId);
           person.setPhoneNumber(phoneNumber);
           dataSource.insertPerson(person);
       }
   }

   y

   public void updatePersonPhoneNumber(Long personId, String phoneNumber) {
       if (personId == null)
           return;
       DataSource dataSource = appContext.getStuffDataSource();
       Person person = dataSource.getPersonById(personId);
       if (person == null)
           throw new SomeReasonableUserException("What are you thinking about ???");
       person.setPhoneNumber(phoneNumber);
       dataSource.updatePerson(person);
   }

   Si appContext o dataSource no se inicializan en tiempo de ejecución no controlado, NullPointerException matará el subproceso actual y será procesado por Thread.defaultUncaughtExceptionHandler (para que defina y use su registrador favorito u otro mecanismo de notificación). Si no se establece, ThreadGroup # uncaughtException imprimirá stacktrace en el sistema err. Uno debe monitorear el registro de errores de la aplicación y abrir el problema de Jira para cada excepción no manejada que de hecho es un error de la aplicación. El programador debe corregir el error en algún lugar de las cosas de inicialización.

Java 7 tiene una nueva java.util.Objectsclase de utilidad en la que hay un requireNonNull()método. Todo lo que hace es arrojar un NullPointerExceptionsi su argumento es nulo, pero limpia un poco el código. Ejemplo:

Objects.requireNonNull(someObject);
someObject.doCalc();

El método es más útil para verificar justo antes de una asignación en un constructor, donde cada uso del mismo puede guardar tres líneas de código:

Parent(Child child) {
   if (child == null) {
      throw new NullPointerException("child");
   }
   this.child = child;
}

se convierte

Parent(Child child) {
   this.child = Objects.requireNonNull(child, "child");
}

Finalmente, la única forma de resolver completamente este problema es usar un lenguaje de programación diferente:

• En Objective-C, puede hacer el equivalente a invocar un método nil, y no pasará absolutamente nada. Esto hace que la mayoría de las verificaciones nulas sean innecesarias, pero puede hacer que los errores sean mucho más difíciles de diagnosticar.
• En Niza , un lenguaje derivado de Java, hay dos versiones de todos los tipos: una versión potencialmente nula y una versión no nula. Solo puede invocar métodos en tipos no nulos. Los tipos potencialmente nulos se pueden convertir en tipos no nulos mediante la comprobación explícita de nulos. Esto hace que sea mucho más fácil saber dónde son necesarias las comprobaciones nulas y dónde no.

"Problema" común en Java de hecho.

Primero, mis pensamientos sobre esto:

Considero que es malo "comer" algo cuando se pasó NULL donde NULL no es un valor válido. Si no está saliendo del método con algún tipo de error, significa que nada salió mal en su método, lo que no es cierto. Entonces, probablemente devuelva nulo en este caso, y en el método de recepción vuelva a verificar si es nulo, y nunca termina, y termina con "if! = Null", etc.

Entonces, en mi humilde opinión, nulo debe ser un error crítico que impide una mayor ejecución (es decir, donde nulo no es un valor válido).

La forma en que resuelvo este problema es la siguiente:

Primero, sigo esta convención:

1. Todos los métodos públicos / API siempre verifican que sus argumentos sean nulos
2. Todos los métodos privados no verifican si son nulos, ya que son métodos controlados (solo deje morir con excepción de puntero nulo en caso de que no se haya manejado anteriormente)
3. Los únicos otros métodos que no comprueban nulo son los métodos de utilidad. Son públicos, pero si los llama por alguna razón, sabe qué parámetros pasa. Esto es como tratar de hervir agua en la tetera sin proporcionar agua ...

Y finalmente, en el código, la primera línea del método público es así:

ValidationUtils.getNullValidator().addParam(plans, "plans").addParam(persons, "persons").validate();

Tenga en cuenta que addParam () devuelve self, para que pueda agregar más parámetros para verificar.

El método validate()arrojará marcado ValidationExceptionsi alguno de los parámetros es nulo (marcado o desmarcado es más un problema de diseño / sabor, pero mi ValidationExceptionestá marcado).

void validate() throws ValidationException;

El mensaje contendrá el siguiente texto si, por ejemplo, "planes" es nulo:

" Se encontró un valor de argumento ilegal nulo para el parámetro [planes] "

Como puede ver, el segundo valor en el método addParam () (cadena) es necesario para el mensaje del usuario, porque no puede detectar fácilmente el nombre de la variable pasada, incluso con reflexión (de todos modos no es tema de esta publicación ...).

Y sí, sabemos que más allá de esta línea ya no encontraremos un valor nulo, por lo que simplemente invocamos métodos con seguridad en esos objetos.

De esta manera, el código es limpio, fácil de mantener y legible.

Hacer esa pregunta señala que puede estar interesado en las estrategias de manejo de errores. El arquitecto de su equipo debe decidir cómo trabajar los errores. Hay varias formas de hacerlo:

1. permita que las Excepciones se propaguen: atrapelas en el 'bucle principal' o en alguna otra rutina de administración.

   • compruebe las condiciones de error y trátelas adecuadamente

Claro que también eche un vistazo a la Programación Orientada a Aspectos: tienen buenas formas de insertar if( o == null ) handleNull()en su código de bytes.

Además de usar assert, puede usar lo siguiente:

if (someobject == null) {
    // Handle null here then move on.
}

Esto es un poco mejor que:

if (someobject != null) {
    .....
    .....



    .....
}

Simplemente no uses nulo. No lo permitas

En mis clases, la mayoría de los campos y las variables locales tienen valores predeterminados no nulos, y agrego declaraciones de contrato (afirmaciones siempre activadas) en todas partes del código para asegurarme de que esto se aplique (ya que es más sucinto y más expresivo que dejarlo) aparecer como un NPE y luego tener que resolver el número de línea, etc.).

Una vez que adopté esta práctica, noté que los problemas parecían solucionarse. Descubriría cosas mucho antes en el proceso de desarrollo solo por accidente y se daría cuenta de que tenía un punto débil ... y, lo que es más importante, ayuda a encapsular las preocupaciones de los diferentes módulos, los diferentes módulos pueden 'confiar' entre sí y no dejar más basura. código conif = null else construcciones!

Esta es una programación defensiva y da como resultado un código mucho más limpio a largo plazo. Siempre desinfecte los datos, por ejemplo, haciendo cumplir estándares rígidos, y los problemas desaparecerán.

class C {
    private final MyType mustBeSet;
    public C(MyType mything) {
       mustBeSet=Contract.notNull(mything);
    }
   private String name = "<unknown>";
   public void setName(String s) {
      name = Contract.notNull(s);
   }
}


class Contract {
    public static <T> T notNull(T t) { if (t == null) { throw new ContractException("argument must be non-null"); return t; }
}

Los contratos son como pruebas de mini unidades que siempre se ejecutan, incluso en producción, y cuando las cosas fallan, ya sabes por qué, en lugar de un NPE aleatorio, tienes que resolverlo de alguna manera.

Guava, una biblioteca central muy útil de Google, tiene una API agradable y útil para evitar nulos. Encuentro UsingAndAvoidingNullExplained muy útil.

Como se explica en la wiki:

Optional<T>es una forma de reemplazar una referencia T anulable con un valor no nulo. Un Opcional puede contener una referencia T no nula (en cuyo caso decimos que la referencia está "presente") o puede no contener nada (en cuyo caso decimos que la referencia está "ausente"). Nunca se dice que "contenga nulo".

Uso:

Optional<Integer> possible = Optional.of(5);
possible.isPresent(); // returns true
possible.get(); // returns 5

Este es un problema muy común para todos los desarrolladores de Java. Por lo tanto, hay soporte oficial en Java 8 para abordar estos problemas sin código desordenado.

Java 8 ha introducido java.util.Optional<T>. Es un contenedor que puede contener o no un valor no nulo. Java 8 ha dado una forma más segura de manejar un objeto cuyo valor puede ser nulo en algunos de los casos. Está inspirado en las ideas de Haskell y Scala .

En pocas palabras, la clase Opcional incluye métodos para tratar explícitamente los casos en los que un valor está presente o ausente. Sin embargo, la ventaja en comparación con las referencias nulas es que la clase Opcional <T> lo obliga a pensar en el caso cuando el valor no está presente. Como consecuencia, puede evitar excepciones de puntero nulo no intencionadas.

En el ejemplo anterior, tenemos una fábrica de servicios para el hogar que devuelve una manija a varios dispositivos disponibles en el hogar. Pero estos servicios pueden o no estar disponibles / funcionales; significa que puede dar lugar a una NullPointerException. En lugar de agregar un nuloif condición antes de usar cualquier servicio, envuélvala a Opcional <Servicio>.

ENVOLVER A LA OPCIÓN <T>

Consideremos un método para obtener una referencia de un servicio de una fábrica. En lugar de devolver la referencia del servicio, envuélvala con Opcional. Le permite al usuario de la API saber que el servicio devuelto puede o no estar disponible / funcional, usar a la defensiva

public Optional<Service> getRefrigertorControl() {
      Service s = new  RefrigeratorService();
       //...
      return Optional.ofNullable(s);
   }

Como puede ver, Optional.ofNullable()proporciona una manera fácil de envolver la referencia. Hay otras formas de obtener la referencia de Opcional, ya sea Optional.empty()&Optional.of() . Uno para devolver un objeto vacío en lugar de volver a sintonizar nulo y el otro para envolver un objeto no anulable, respectivamente.

Entonces, ¿cómo exactamente ayuda a evitar un cheque nulo?

Una vez que haya ajustado un objeto de referencia, Opcional proporciona muchos métodos útiles para invocar métodos en una referencia ajustada sin NPE.

Optional ref = homeServices.getRefrigertorControl();
ref.ifPresent(HomeServices::switchItOn);

Opcional. Si Presente invoca al Consumidor dado con una referencia si es un valor no nulo. De lo contrario, no hace nada.

@FunctionalInterface
public interface Consumer<T>

Representa una operación que acepta un único argumento de entrada y no devuelve ningún resultado. A diferencia de la mayoría de las otras interfaces funcionales, se espera que Consumer opere a través de efectos secundarios. Es muy limpio y fácil de entender. En el ejemplo de código anterior, HomeService.switchOn(Service)se invoca si la referencia de retención Opcional no es nula.

Usamos el operador ternario con mucha frecuencia para verificar la condición nula y devolver un valor alternativo o valor predeterminado. Opcional proporciona otra forma de manejar la misma condición sin marcar nulo. Opcional. O Else (defaultObj) devuelve defaultObj si el Opcional tiene un valor nulo. Usemos esto en nuestro código de muestra:

public static Optional<HomeServices> get() {
    service = Optional.of(service.orElse(new HomeServices()));
    return service;
}

Ahora HomeServices.get () hace lo mismo, pero de una mejor manera. Comprueba si el servicio ya está inicializado o no. Si es así, devuelva el mismo o cree un nuevo servicio nuevo. Opcional <T> .orElse (T) ayuda a devolver un valor predeterminado.

Finalmente, aquí está nuestro NPE, así como el código nulo sin cheques:

import java.util.Optional;
public class HomeServices {
    private static final int NOW = 0;
    private static Optional<HomeServices> service;

public static Optional<HomeServices> get() {
    service = Optional.of(service.orElse(new HomeServices()));
    return service;
}

public Optional<Service> getRefrigertorControl() {
    Service s = new  RefrigeratorService();
    //...
    return Optional.ofNullable(s);
}

public static void main(String[] args) {
    /* Get Home Services handle */
    Optional<HomeServices> homeServices = HomeServices.get();
    if(homeServices != null) {
        Optional<Service> refrigertorControl = homeServices.get().getRefrigertorControl();
        refrigertorControl.ifPresent(HomeServices::switchItOn);
    }
}

public static void switchItOn(Service s){
         //...
    }
}

La publicación completa es NPE, así como un código de verificación nulo ... ¿En serio? .

Me gustan los artículos de Nat Pryce. Aquí están los enlaces:

• Evitar nulos con envío polimórfico
• Evitar nulos con estilo "Dile, no preguntes"

En los artículos también hay un enlace a un repositorio de Git para un Java Maybe Type que me parece interesante, pero no creo que por sí solo pueda disminuir la hinchazón del código de verificación. Después de investigar un poco en Internet, creo que ! = La expansión de código nulo podría reducirse principalmente mediante un diseño cuidadoso.

He intentado el NullObjectPatternpero para mí no siempre es el mejor camino a seguir. A veces hay una "no acción" que no es apropiada.

NullPointerExceptiones una excepción en tiempo de ejecución que significa que es culpa de los desarrolladores y con suficiente experiencia le dice exactamente dónde está el error.

Ahora a la respuesta:

Intente hacer que todos sus atributos y sus accesores sean lo más privados posible o evite exponerlos a los clientes. Puede tener los valores de argumento en el constructor, por supuesto, pero al reducir el alcance no permite que la clase del cliente pase un valor no válido. Si necesita modificar los valores, siempre puede crear uno nuevo object. Comprueba los valores en el constructor solo una vez y en el resto de los métodos puede estar casi seguro de que los valores no son nulos.

Por supuesto, la experiencia es la mejor manera de comprender y aplicar esta sugerencia.

¡Byte!

Probablemente la mejor alternativa para Java 8 o posterior es usar la Optionalclase.

Optional stringToUse = Optional.of("optional is there");
stringToUse.ifPresent(System.out::println);

Esto es especialmente útil para largas cadenas de posibles valores nulos. Ejemplo:

Optional<Integer> i = Optional.ofNullable(wsObject.getFoo())
    .map(f -> f.getBar())
    .map(b -> b.getBaz())
    .map(b -> b.getInt());

Ejemplo sobre cómo lanzar una excepción en nulo:

Optional optionalCarNull = Optional.ofNullable(someNull);
optionalCarNull.orElseThrow(IllegalStateException::new);

Java 7 introdujo el Objects.requireNonNullmétodo que puede ser útil cuando se debe verificar que algo no sea nulo. Ejemplo:

String lowerVal = Objects.requireNonNull(someVar, "input cannot be null or empty").toLowerCase();

¡Puedo responderlo de manera más general!

Nosotros usualmente enfrentamos a este problema cuando los métodos obtienen los parámetros de la manera que no esperábamos (la llamada al método incorrecta es culpa del programador). Por ejemplo: espera obtener un objeto, en su lugar, obtiene un valor nulo. Esperas obtener una Cadena con al menos un carácter, en su lugar obtienes una Cadena vacía ...

Entonces no hay diferencia entre:

if(object == null){
   //you called my method badly!

}

o

if(str.length() == 0){
   //you called my method badly again!
}

Ambos quieren asegurarse de que hayamos recibido parámetros válidos, antes de realizar cualquier otra función.

Como se menciona en algunas otras respuestas, para evitar los problemas anteriores, puede seguir el patrón Diseño por contrato . Consulte http://en.wikipedia.org/wiki/Design_by_contract .

Para implementar este patrón en java, puede usar anotaciones centrales de java como javax.annotation.NotNull o usar bibliotecas más sofisticadas como Hibernate Validator .

Solo una muestra:

getCustomerAccounts(@NotEmpty String customerId,@Size(min = 1) String accountType)

Ahora puede desarrollar de forma segura la función principal de su método sin necesidad de verificar los parámetros de entrada, protegen sus métodos de parámetros inesperados.

Puede ir un paso más allá y asegurarse de que solo se puedan crear pojos válidos en su aplicación. (muestra del sitio de validación de hibernación)

public class Car {

   @NotNull
   private String manufacturer;

   @NotNull
   @Size(min = 2, max = 14)
   private String licensePlate;

   @Min(2)
   private int seatCount;

   // ...
}

No hago caso de las respuestas que sugieren usar los objetos nulos en cada situación. Este patrón puede romper el contrato y enterrar los problemas cada vez más en vez de resolverlos, sin mencionar que si se usa de manera inapropiada, se creará otro montón de código repetitivo que requerirá mantenimiento futuro.

En realidad, si algo devuelto por un método puede ser nulo y el código de llamada tiene que tomar una decisión al respecto, debería haber una llamada anterior que garantice el estado.

También tenga en cuenta que ese patrón de objeto nulo requerirá memoria si se usa sin cuidado. Para esto, la instancia de un NullObject debe compartirse entre los propietarios y no ser una instancia única para cada uno de ellos.

Además, no recomendaría usar este patrón donde el tipo está destinado a ser una representación de tipo primitiva, como entidades matemáticas, que no son escalares: vectores, matrices, números complejos y objetos POD (datos antiguos simples), que están destinados a mantener el estado en forma de tipos incorporados de Java. En el último caso, terminaría llamando a métodos getter con resultados arbitrarios. Por ejemplo, ¿qué debería devolver un método NullPerson.getName ()?

Vale la pena considerar tales casos para evitar resultados absurdos.

1. Nunca inicialice las variables a nulo.
2. Si (1) no es posible, inicialice todas las colecciones y matrices para vaciar colecciones / matrices.

Haciendo esto en su propio código y puede evitar! = Comprobaciones nulas.

La mayoría de las veces, las comprobaciones nulas parecen proteger los bucles sobre colecciones o matrices, así que simplemente inicialícelas vacías, no necesitará ninguna comprobación nula.

// Bad
ArrayList<String> lemmings;
String[] names;

void checkLemmings() {
    if (lemmings != null) for(lemming: lemmings) {
        // do something
    }
}



// Good
ArrayList<String> lemmings = new ArrayList<String>();
String[] names = {};

void checkLemmings() {
    for(lemming: lemmings) {
        // do something
    }
}

Hay una pequeña sobrecarga en esto, pero vale la pena por un código más limpio y menos NullPointerExceptions.

Este es el error más común ocurrido para la mayoría de los desarrolladores.

Tenemos varias formas de manejar esto.

Enfoque 1:

org.apache.commons.lang.Validate //using apache framework

notNull (objeto objeto, mensaje de cadena)

Enfoque 2:

if(someObject!=null){ // simply checking against null
}

Enfoque 3:

@isNull @Nullable  // using annotation based validation

Enfoque 4:

// by writing static method and calling it across whereever we needed to check the validation

static <T> T isNull(someObject e){  
   if(e == null){
      throw new NullPointerException();
   }
   return e;
}

public static <T> T ifNull(T toCheck, T ifNull) {
    if (toCheck == null) {
           return ifNull;
    }
    return toCheck;
}