拡張されたforループでのnullチェック

Javaのforループでnullを防ぐ最善の方法は何ですか？

これは醜いようです：

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

または

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

他に方法はないかもしれません。彼らはforそれ自体を構成体に入れるべきでした、それがnullの場合はループを実行しないでください？

そのリストをどこから取得するかをよりよく確認する必要があります。

空のリストは失敗しないので、空のリストで十分です。

他の場所からこのリストを取得し、それが問題ないかどうかわからない場合は、ユーティリティメソッドを作成して次のように使用できます。

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

そしてもちろんsafe：

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

nullを渡した場合に空のシーケンスを返すヘルパーメソッドを作成する可能性があります。

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

次に使用します：

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

私は実際にそうするつもりはないと思います-私は通常あなたの2番目のフォームを使います。特に、「or throw ex」は重要です。本当にnullにしてはいけない場合は、必ず例外をスローする必要があります。あなたは何かがうまくいっていないことを知っていますが、被害の程度を知りません。早く打ち切る。

すでに2017年です。ApacheCommons Collections4を使用できます。

使い方：

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

を使用して、他のコレクションクラスに対して同じnullセーフチェックを実行できます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のマントラのために、注目に値すると思います。参照コモンズ・ラングのプロジェクトページを。ここにnullToEmptyAPI ドキュメントとソースがあります

commons-langプロジェクトに含めるMavenエントリ（まだ含まれていない場合）。

<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はIntegerになります。

forループでnullを効果的に防ぐ別の方法は、コレクションをGoogle GuavaでラップするOptional<T>ことです。これにより、コレクションが存在するかどうかをクライアントが確認することが期待されるため、コレクションが実質的に空の可能性が明らかになりますOptional.isPresent()。

独自の静的null安全メソッドを作成することに関心がない人は、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。

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.