java uso Evitando!=Declaraciones nulas




uso de excepción en java (20)

Si no se permiten valores indefinidos:

Puede configurar su IDE para advertirle sobre una posible anulación de referencia nula. Por ejemplo, en Eclipse, consulte Preferencias> Java> Compilador> Errores / Advertencias / Análisis nulo .

Si se permiten valores indefinidos:

Si desea definir una nueva API en la que los valores no definidos tengan sentido , use el Patrón de opción (puede ser familiar para los lenguajes funcionales). Tiene las siguientes ventajas:

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

Java 8 tiene una clase Optional incorporada (recomendada); para versiones anteriores, hay alternativas de biblioteca, por ejemplo, la Optional Guava o la Option FunctionalJava . Pero al igual que muchos patrones de estilo funcional, el uso de Opción en Java (incluso 8) da como resultado una gran cantidad de repeticiones, 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 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.

https://code.i-harness.com

Uso object != null mucho para evitar NullPointerException .

¿Hay una buena alternativa a esto?

Por ejemplo:

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

Esto evita una NullPointerException , cuando se desconoce si el objeto es null o no.

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


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 el object no es nulo.

Si es un método interno (que no es 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 simplemente pasa el valor, y el siguiente método lo pasa, etc. podría resultar problemático. En ese caso, es posible que desee comprobar 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
}

Así que 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 normalmente el idioma.


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 sabes que b nunca puede ser nulo, simplemente puedes cambiarlo. Es más útil para iguales: en lugar de foo.equals("bar"); mejor hacer "bar".equals(foo); .


Con Java 8 viene la nueva clase java.util.Optional que posiblemente resuelva algunos de los problemas. Al menos, se puede decir que mejora la legibilidad del código y, en el caso de las API públicas, el contrato de la API es más claro para el desarrollador del cliente.

Funcionan así:

Un objeto opcional para un tipo dado ( Fruit ) se crea como el tipo de retorno de un método. Puede estar vacío o contener un objeto Fruit :

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 Fruta determinada:

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

Puede utilizar el operador map() para realizar un cálculo o extraer un valor de un objeto opcional. orElse() permite proporcionar un respaldo 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 podría estar vacío y el riesgo de olvidarse de la verificación es limitado.

En una API creada desde cero utilizando Optional siempre que un valor devuelto esté vacío, y devolviendo un objeto plano solo cuando no pueda ser null (convención), el código del cliente podría abandonar las comprobaciones nulas en valores de retorno de objetos simples ...

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

Optional ofrece otros métodos convenientes, como orElse que permiten el uso de un valor predeterminado, y ifPresent que funciona con expresiones lambda .

Los invito a leer este artículo (mi fuente principal para escribir esta respuesta) en el que la NullPointerException (y en general el puntero nulo) es problemática, así como la solución (parcial) presentada por Optional están bien explicadas: Objetos opcionales de Java .


El marco de las colecciones de Google ofrece una forma buena y elegante de lograr el cheque nulo.

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

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

En lugar de un 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 el seguimiento de la pila y trabaje a través del error.


Para mí, esto me parece un problema bastante 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 defensivamente realizan una verificación excesiva de los nulos. Además, cuando escriben su propio código, tienden a confiar en que devuelvan nulos para indicar algo, por lo que requieren que la persona que llama compruebe si hay nulos.

Para poner esto de otra manera, hay dos instancias donde aparece la comprobación nula:

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

  2. Donde no es una respuesta válida.

(2) es fácil. Utilice declaraciones de aserción (aserciones) o permita fallos (por ejemplo, NullPointerException ). Las aserciones son una característica de Java muy 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 cuya salida del método toString() se incluirá en el error.

Una declaración de afirmació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 -ea a la JVM. Puede habilitar y deshabilitar aserciones para clases individuales y paquetes. Esto significa que puede validar el código con las aserciones durante el desarrollo y la prueba, y deshabilitarlas en un entorno de producción, aunque mis pruebas han demostrado casi sin 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 ocurrirá si usa aserciones. La única diferencia es que, con las afirmaciones, podría suceder 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, estás atascado. Si null es una respuesta válida, debe verificarla.

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

Con no-colecciones podría ser más difícil. Considera esto como un ejemplo: si tienes estas interfaces:

public interface Action {
  void doSomething();
}

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

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

Una solución alternativa es nunca devolver el valor nulo y, en su lugar, utilizar 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, tal vez sea totalmente apropiado que el método findAction () lance una excepción con un mensaje de error significativo, especialmente en este caso en el que confía en la información del usuario. Sería mucho mejor para el método findAction lanzar una excepción que para el método de llamada explotar 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 try / catch es demasiado feo, en lugar de hacer nada, su acción predeterminada debe 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 @Nullable y @NotNull .

Se puede utilizar en método 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 compilará (en IntelliJ IDEA).

Cuando usas la primera función helloWorld() en otro fragmento de código:

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

Ahora el compilador de IDEA de IntelliJ le dirá que la comprobación es inútil, ya que la función helloWorld() no volverá a ser null , nunca.

Usando parametro

void someMethod(@NotNull someParameter) { }

si escribes algo como:

someMethod(null);

Esto no compilará.

Último ejemplo usando @Nullable

@Nullable iWantToDestroyEverything() { return null; }

Haciendo esto

iWantToDestroyEverything().something();

Y puedes 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 que sus contratos sean más sólidos. Desafortunadamente, no es compatible con todos los compiladores.

En IntelliJ IDEA 10.5 y en adelante, agregaron soporte para cualquier otra implementación de @Nullable @NotNull .

Ver publicación en el blog Más flexible y configurable @ Nullable / @ NotNull anotaciones .


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


Wow, casi odio agregar otra respuesta cuando tenemos 57 formas diferentes de recomendar el NullObject pattern , pero creo que a algunas personas interesadas en esta pregunta les gustaría saber que hay una propuesta para Java 7 sobre la mesa para agregar "null". "Manejo seguro": una sintaxis optimizada para la lógica if-not-equal-null.

El ejemplo dado por Alex Miller se ve así:

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

El ?. significa solo desvincular 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 aman esta propuesta, pero también hay oposición, ya que en realidad fomentará un mayor uso de null como valor centinela.

Actualización: Se proposed una proposed para un operador seguro en nulo en Java 7 en el marco de Project Coin. La sintaxis es un poco diferente a la del ejemplo anterior, pero es la misma noción.

Actualización: La propuesta del operador de seguridad nula no se convirtió en Project Coin. Por lo tanto, no verás esta sintaxis en Java 7.


¿Puedo contestarlo más generalmente?

Por lo general, nos enfrentamos a este problema cuando los métodos obtienen los parámetros de la forma que no esperábamos (la mala llamada es un error del programador). Por ejemplo: esperas obtener un objeto, en vez de eso obtienes un nulo. Usted espera obtener una Cadena con al menos un carácter, en lugar de obtener una Cadena vacía ...

Así que 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 recibimos parámetros válidos, antes de realizar cualquier otra función.

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

Para implementar este patrón en java, puede usar anotaciones de java principales 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 del validador de hibernación)

public class Car {

   @NotNull
   private String manufacturer;

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

   @Min(2)
   private int seatCount;

   // ...
}

En última instancia, la única forma de resolver completamente este problema es mediante el uso de un lenguaje de programación diferente:

  • En Objective-C, puedes hacer el equivalente de invocar un método nily no pasará absolutamente nada. Esto hace que la mayoría de las comprobaciones nulas no sean necesarias, pero puede hacer que los errores sean mucho más difíciles de diagnosticar.
  • En Nice , un lenguaje derivado de Java, hay dos versiones de todos los tipos: una versión potencialmente nula y una versión no nula. Sólo 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 necesarios los controles nulos y dónde no lo son.

Además de usar assertpuedes usar lo siguiente:

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

Esto es un poco mejor que:

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



    .....
}

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 o no contener un valor no nulo. Java 8 ha brindado 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 que un valor está presente o ausente. Sin embargo, la ventaja en comparación con las referencias nulas es que la clase <T> Opcional lo obliga a pensar en el caso cuando el valor no está presente. Como consecuencia, puede evitar excepciones de puntero nulo no deseadas.

En el ejemplo anterior, tenemos una fábrica de servicios para el hogar que devuelve un identificador a varios dispositivos disponibles en el hogar. Pero estos servicios pueden o no estar disponibles / funcionales; significa que puede resultar en una NullPointerException. En lugar de agregar una ifcondición nula antes de usar cualquier servicio, envolvámoslo en <Servicio> Opcional.

ENVÍO A OPCIÓN <T>

Consideremos un método para obtener una referencia de un servicio de una fábrica. En lugar de devolver la referencia de 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 ve, Optional.ofNullable()proporciona una manera fácil de obtener la referencia envuelta. 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.

¿CUÁNTO EXACTAMENTE LO AYUDA A EVITAR UNA COMPROBACIÓN NULA?

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

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

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

@FunctionalInterface
public interface Consumer<T>

Representa una operación que acepta un solo argumento de entrada y no devuelve ningún resultado. A diferencia de la mayoría de las otras interfaces funcionales, se espera que el Consumidor opere a través de efectos secundarios. Es tan 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 muy a menudo 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. Optional.orElse (defaultObj) devuelve defaultObj si el Opcional tiene un valor nulo. Vamos a usar 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 lo mismo o cree un nuevo servicio nuevo. Opcional <T> .orElse (T) ayuda a devolver un valor predeterminado.

Finalmente, aquí está nuestro NPE y nuestro código nulo de verificación sin cargo:

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 el código nulo de verificación sin cargo ... ¿En serio? .


He intentado el NullObjectPatternpero para mí no siempre es la mejor manera de ir. Hay ocasiones en que una "no acción" no es apropiada.

NullPointerExceptiones una excepción de Runtime 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. Por supuesto, puede tener los valores de argumento en el constructor, pero al reducir el alcance, no permite que la clase cliente pase un valor no válido. Si necesita modificar los valores, siempre puede crear uno nuevo object. Verifica 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 entender y aplicar esta sugerencia.

¡Byte!


Ignoro las respuestas que sugieren el uso de objetos nulos en cada situación. Este patrón puede romper el contrato y enterrar los problemas más y más profundamente en lugar de resolverlos, sin mencionar que el uso inadecuado creará otra pila 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 debe tomar una decisión al respecto, debe haber una llamada anterior que garantice el estado.

También tenga en cuenta que ese patrón de objeto nulo tendrá hambre de memoria si se utiliza sin cuidado. Para esto, la instancia de NullObject debe compartirse entre los propietarios y no debe ser una instancia de Unigue para cada uno de estos.

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

Vale la pena considerar estos casos para evitar resultados absurdos.


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

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


Nulo no es un 'problema'. Es una parte integral de un conjunto complete herramientas de modelado. El software apunta a modelar la complejidad del mundo y null lleva su carga. Nulo indica 'No hay datos' o 'Desconocido' en Java y similares. Por eso es apropiado usar nulos para estos propósitos. No prefiero el patrón 'Objeto nulo'; Creo que surge el problema de ' quién cuidará a los guardianes '.
Si me preguntas cuál es el nombre de 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)t quiere ser) resuelto allí mismo y delegarlo en un lugar más alto en la pila para reintentar o reportar un error de acceso de datos al usuario.

  1. Para una 'pregunta desconocida' dé 'respuesta desconocida'. (Esté a salvo de nulos cuando esto sea correcto desde el punto de vista comercial). Verificar los argumentos para null una vez dentro de un método antes de que el uso libere a varias personas que llaman 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 al 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 de Java que viene (mirando hacia adelante)

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

    Si bien es más bien 'flujo comercial normal' no encontrar una foto almacenada en la base de datos para alguna persona, solía usar pares como abajo para otros casos

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

    Y no deteste escribir <alt> + <shift> + <j>(genere javadoc en Eclipse) y escriba tres palabras adicionales para su API pública. Esto será más que suficiente para todos menos 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, debería preferir una API nula segura de Java (en caso de que se publique en otros 10 años), pero NullPointerExceptiones una subclase de Exception. Por lo tanto, es una forma de Throwableque indica las condiciones que una aplicación razonable podría querer capturar ( javadoc ). Para usar la primera ventaja más grande de las excepciones y el código separado de manejo de errores del código 'regular' (de acuerdo con los creadores de Java ) es apropiado, como para mí, capturar 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. Está a la altura de la lógica empresarial. Si no encuentro un álbum de fotos no te mostraré fotos. ¿Qué pasa si appContext no está inicializado? La lógica de negocios de este método soporta esto. Si la misma lógica debería ser más estricta, lanzar una excepción es parte de la lógica de negocios y se debe utilizar la comprobación explícita de nulos (caso 3). La nueva API de Java Null-safe se adapta mejor aquí para especificar de manera selectiva lo que implica y lo que no implica que se inicialice para fallar en caso de errores de programador.

    P. El código redundante se puede ejecutar y los recursos innecesarios se pueden tomar.
    A. Podría ocurrir si getPhotoByName()intentara abrir una conexión de base de datos, crear PreparedStatementy usar el nombre de la persona como un parámetro de SQL por fin. El enfoque para una pregunta desconocida da una respuesta desconocida (caso 1) que funciona aquí. Antes de tomar 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 del 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! ( source ), 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" que se puede usar en algún lugar. Considera 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 rápido (y no tan rápido para leer la documentación), me gustaría decir que nunca he detectado 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 se ajusta para ser un Errorque ThreadDeath? Pero no lo es.

  3. Verifique 'No hay 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 inicializa en tiempo de ejecución no manejado, NullPointerException eliminará el hilo actual y será procesado por Thread.defaultUncaughtExceptionHandler (para que defina y use su registrador favorito u otro Thread.defaultUncaughtExceptionHandler notificación). Si no está configurado, ThreadGroup#uncaughtException imprimirá stacktrace en el error del sistema. 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 debería corregir un error en algún lugar de la inicialización.


Simplemente nunca uses null. No lo permitas

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

Una vez que adopté esta práctica, noté que los problemas parecían solucionarse solos. Cogió cosas mucho antes en el proceso de desarrollo solo por accidente y se dio cuenta de que tenía un punto débil ... y, lo que es más importante, ayuda a encapsular las inquietudes de los diferentes módulos, los diferentes módulos pueden "confiar" entre sí, y no dejar de ensuciar los Código con if = null elseconstructos!

Esta es una programación defensiva y resulta en un código mucho más limpio a largo plazo. Siempre desinfecte los datos, p. Ej. Aquí mediante la aplicación de estándares rígidos, y los problemas desaparecen.

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 mini pruebas unitarias que siempre se están ejecutando, incluso en producción, y cuando las cosas fallan, sabes por qué, en lugar de un NPE aleatorio, tienes que descifrarlo.


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




null