Comprobación nula en un bucle for mejorado

¿Cuál es la mejor manera de protegerse contra nulo en un bucle for en Java?

Esto parece feo:

if (someList != null) {
    for (Object object : someList) {
        // do whatever
    }
}

O

if (someList == null) {
    return; // Or throw ex
}
for (Object object : someList) {
    // do whatever
}

Puede que no haya otra forma. ¿Deberían haberlo puesto en la forconstrucción en sí, si es nulo, entonces no ejecute el ciclo?

Debería verificar mejor de dónde obtiene esa lista.

Una lista vacía es todo lo que necesita, porque una lista vacía no fallará.

Si obtiene esta lista de otro lugar y no sabe si está bien o no, podría crear un método de utilidad y usarlo así:

for( Object o : safe( list ) ) {
   // do whatever 
 }

Y por supuesto safesería:

public static List safe( List other ) {
    return other == null ? Collections.EMPTY_LIST : other;
}

Potencialmente, podría escribir un método auxiliar que devuelva una secuencia vacía si pasa nulo:

public static <T> Iterable<T> emptyIfNull(Iterable<T> iterable) {
    return iterable == null ? Collections.<T>emptyList() : iterable;
}

Luego use:

for (Object object : emptyIfNull(someList)) {
}

Sin embargo, no creo que realmente haga eso, generalmente usaría su segunda forma. En particular, "o throw ex" es importante: si realmente no debería ser nulo, definitivamente debe lanzar una excepción. Sabes que algo salió mal, pero no sabes el alcance del daño. Abortar temprano.

Ya es 2017, y ahora puedes usar Apache Commons Collections4

El uso:

for(Object obj : ListUtils.emptyIfNull(list1)){
    // Do your stuff
}

Puede hacer la misma comprobación de seguridad nula para otras clases de colección con CollectionUtils.emptyIfNull.

Con Java 8 Optional:

for (Object object : Optional.ofNullable(someList).orElse(Collections.emptyList())) {
    // do whatever
}

Uso ArrayUtils.nullToEmptyde la commons-langbiblioteca para matrices

for( Object o : ArrayUtils.nullToEmpty(list) ) {
   // do whatever 
}

Esta funcionalidad existe en la commons-langbiblioteca, que se incluye en la mayoría de los proyectos Java.

// ArrayUtils.nullToEmpty source code 
public static Object[] nullToEmpty(final Object[] array) {
    if (isEmpty(array)) {
        return EMPTY_OBJECT_ARRAY;
    }
    return array;
}

// ArrayUtils.isEmpty source code
public static boolean isEmpty(final Object[] array) {
    return array == null || array.length == 0;
}

Esto es lo mismo que la respuesta dada por @OscarRyz, pero por el bien del mantra DRY , creo que vale la pena señalar. Vea la página del proyecto commons-lang . Aquí está la documentación y la fuente de la nullToEmptyAPI

Entrada de Maven para incluir commons-langen su proyecto si aún no lo está.

<dependency>
    <groupId>org.apache.commons</groupId>
    <artifactId>commons-lang3</artifactId>
    <version>3.4</version>
</dependency>

Desafortunadamente, commons-langno proporciona esta funcionalidad para los Listtipos. En este caso, tendría que usar un método auxiliar como se mencionó anteriormente.

public static <E> List<E> nullToEmpty(List<E> list)
{
    if(list == null || list.isEmpty())
    {
        return Collections.emptyList();
    }
    return list;
}

Si obtiene eso Listde una llamada a un método que implementa, no devuelva null, devuelva un vacío List.

Si no puede cambiar la implementación, entonces está atascado con la nullverificación. Si no debería ser así null, lanza una excepción.

No elegiría el método auxiliar que devuelve una lista vacía porque puede ser útil algunas veces, pero luego te acostumbrarías a llamarlo en cada bucle que hagas, posiblemente ocultando algunos errores.

He modificado la respuesta anterior, por lo que no necesita lanzar desde Object

public static <T> List<T> safeClient( List<T> other ) {
            return other == null ? Collections.EMPTY_LIST : other;
}

y luego simplemente llame a la Lista por

for (MyOwnObject ownObject : safeClient(someList)) {
    // do whatever
}

Explicación: MyOwnObject: si es List<Integer>así, MyOwnObject será Integer en este caso.

Otra forma de protegerse eficazmente contra un nulo en un bucle for es envolver su colección con Google Guava's Optional<T>ya que esto, se espera, deja en claro la posibilidad de una colección efectivamente vacía, ya que se esperaría que el cliente verifique si la colección está presente Optional.isPresent().

Para cualquiera que no esté interesado en escribir su propio método de seguridad nulo estático, puede usar: commons-lang's org.apache.commons.lang.ObjectUtils.defaultIfNull(Object, Object). Por ejemplo:

    for (final String item : 
    (List<String>)ObjectUtils.defaultIfNull(items, Collections.emptyList())) { ... }

ObjectUtils.defaultIfNull JavaDoc

Use, CollectionUtils.isEmpty(Collection coll)método que es Null-safe check si la colección especificada está vacía.

para esto import org.apache.commons.collections.CollectionUtils.

Dependencia Maven

<dependency>
    <groupId>org.apache.commons</groupId>
    <artifactId>commons-collections4</artifactId>
    <version>4.0</version>
</dependency>

for (Object object : someList) {

   // do whatever
}  throws the null pointer exception.