Нульова перевірка в поліпшеному циклі

Який найкращий спосіб захистити від null в циклі for для Java?

Це здається некрасивим:

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

Або

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

Не може бути іншого способу. Чи повинні вони помістити його в саму forконструкцію, якщо вона є нульовою, тоді не запускайте цикл?

Вам слід краще перевірити, звідки ви отримуєте цей список.

Порожній список - все, що вам потрібно, тому що порожній список не вийде з ладу.

Якщо ви отримаєте цей список з іншого місця та не знаєте, чи це нормально чи ні, ви можете створити корисний метод і використовувати його так:

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

І звичайно safeбуло б:

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

Ви потенційно можете написати хелперний метод, який повернув порожню послідовність, якщо ви перейшли в нуль:

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

Потім використовуйте:

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

Я не думаю, що насправді б це зробив - зазвичай я використовував вашу другу форму. Зокрема, важливо "або кинути екс" - якщо це дійсно не повинно бути нульовим, ви обов'язково повинні кинути виняток. Ви знаєте, що щось пішло не так, але ви не знаєте ступеня шкоди. Покиньте рано.

Вже 2017 рік, і тепер ви можете використовувати колекції Apache Commons4

Використання:

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

Ви можете зробити те саме, що перевіряє нуль для інших класів колекції CollectionUtils.emptyIfNull.

З Java 8 Optional:

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

Використання ArrayUtils.nullToEmptyз commons-langбібліотеки для масивів

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

Ця функціональність існує в commons-langбібліотеці, яка входить до більшості 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;
}

Це те саме, що відповідь, яку дав @OscarRyz, але заради мари DRY , я вважаю, що це варто відзначити. Дивіться сторінку проекту commons-lang . Ось документація та джерелоnullToEmpty API

Додано запис, який потрібно включити commons-langу ваш проект, якщо його ще немає.

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

На жаль, commons-langне надає цю функціональність для Listтипів. У цьому випадку вам доведеться скористатися допоміжним методом, як згадувалося раніше.

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

Якщо ви отримуєте це Listз виклику методу, який ви реалізуєте, тоді не повертайтеся null, поверніть порожнє List.

Якщо ви не можете змінити реалізацію, тоді ви застрягли з nullчеком. Якщо цього не повинно бути null, то киньте виняток.

Я б не звертався за допоміжним методом, який повертає порожній список, оскільки він може бути корисним кілька разів, але тоді ви звикли б називати його в кожному циклі, який ви створюєте, можливо, приховуючи деякі помилки.

Я змінив вищевказану відповідь, тому вам не потрібно робити передачу з Object

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

а потім просто зателефонувати за списком

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

Пояснення: MyOwnObject: Якщо List<Integer>тоді, MyOwnObject буде цілим у цьому випадку.

Інший спосіб ефективного захисту від нуля в циклі for for - обернути свою колекцію за допомогою Google Guava, Optional<T>оскільки це, сподіваємось, робить можливість фактично порожньої колекції зрозумілою, оскільки, як очікується, клієнт перевірить, чи існує колекція Optional.isPresent().

Для всіх, хто не зацікавлений у написанні власного статичного методу безпеки нуля, ви можете використовувати: commons-lang's org.apache.commons.lang.ObjectUtils.defaultIfNull(Object, Object). Наприклад:

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

ObjectUtils.defaultIfNull JavaDoc

Використання, CollectionUtils.isEmpty(Collection coll)метод, який є безпечним для перевірки, чи вказана колекція порожня.

для цього import org.apache.commons.collections.CollectionUtils.

Залежна залежність

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