Null sprawdza w rozszerzonej pętli for

Jaki jest najlepszy sposób ochrony przed zerami w pętli for w Javie?

To wydaje się brzydkie:

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

Lub

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

Inaczej może nie być. Czy powinni umieścić go w forsamej konstrukcji, jeśli jest zerowy, to nie uruchamiaj pętli?

Powinieneś lepiej sprawdzić, skąd masz tę listę.

Potrzebujesz tylko pustej listy, ponieważ pusta lista nie zawiedzie.

Jeśli otrzymasz tę listę z innego miejsca i nie wiesz, czy jest w porządku, czy nie, możesz utworzyć metodę narzędziową i użyć jej w następujący sposób:

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

I oczywiście safebyłoby:

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

Możesz potencjalnie napisać metodę pomocniczą, która zwróci pustą sekwencję, jeśli przekażesz wartość null:

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

Następnie użyj:

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

Chociaż nie sądzę, żebym to zrobił - zwykle używałbym twojej drugiej formy. W szczególności ważne jest „or throw ex” - jeśli naprawdę nie powinno być null, zdecydowanie powinieneś zgłosić wyjątek. Wiesz, że coś poszło nie tak, ale nie znasz rozmiaru szkód. Przerwij wcześniej.

Jest już 2017 rok i możesz teraz korzystać z Apache Commons Collections4

Użycie:

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

Możesz wykonać to samo sprawdzanie bezpieczeństwa null dla innych klas kolekcji za pomocą CollectionUtils.emptyIfNull.

W Javie 8 Optional:

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

Użyj ArrayUtils.nullToEmptyz commons-langbiblioteki dla tablic

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

Ta funkcjonalność istnieje w commons-langbibliotece, która jest zawarta w większości projektów 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;
}

To jest to samo, co odpowiedź udzielona przez @OscarRyz, ale ze względu na mantrę DRY uważam, że warto o tym wspomnieć. Zobacz stronę projektu Common-Lang . Oto dokumentacja i źródłonullToEmpty API

Maven należy dołączyć do commons-langprojektu, jeśli jeszcze nie jest.

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

Niestety, commons-langnie zapewnia tej funkcji dla Listtypów. W takim przypadku musisz użyć metody pomocniczej, jak wspomniano wcześniej.

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

Jeśli otrzymujesz to Listz wywołania metody, którą implementujesz, nie zwracaj null, zwróć pusty List.

Jeśli nie możesz zmienić implementacji, utkniesz z nullczekiem. Jeśli tak nie jest null, zgłoś wyjątek.

Nie wybrałbym metody pomocniczej, która zwraca pustą listę, ponieważ czasami może się przydać, ale wtedy przyzwyczaiłbyś się do wywoływania jej w każdej pętli, którą wykonujesz, prawdopodobnie ukrywając niektóre błędy.

Zmodyfikowałem powyższą odpowiedź, więc nie musisz rzucać z Object

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

a następnie po prostu zadzwoń do listy według

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

Wyjaśnienie: MyOwnObject: Jeśli List<Integer>to MyOwnObject będzie w tym przypadku Integer.

Innym sposobem skutecznego zabezpieczenia się przed wartością zerową w pętli for jest umieszczenie kolekcji w Google Guava, Optional<T>ponieważ można mieć nadzieję, że dzięki temu możliwość efektywnego opróżnienia kolekcji jest jasna, ponieważ klient powinien sprawdzić, czy kolekcja jest obecna Optional.isPresent().

Dla każdego, kto nie jest zainteresowany pisaniem własnej statycznej metody bezpieczeństwa zerowego, możesz użyć: commons-lang's org.apache.commons.lang.ObjectUtils.defaultIfNull(Object, Object). Na przykład:

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

ObjectUtils.defaultIfNull JavaDoc

Use, CollectionUtils.isEmpty(Collection coll)metoda, która jest null-bezpiecznym sprawdzaniem, czy określona kolekcja jest pusta.

za to import org.apache.commons.collections.CollectionUtils.

Zależność Mavena

<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.