【Java】streamを使ってソースの簡略化

Question

【Java】streamを使ってソースの簡略化

java初心者です。
streamAPIというものを最近知りました。
以下のようなソースコードをstreamを使って短くしたいです。イタリックテキスト

該当のソースコード

for (Map detail : /*List<Map<String,Object>>*/detailList) {
  for (Entry<String, Object> entry : detail.entrySet()) {
    if (Objects.isNull(entry.getValue())) {
      return false;
    }
  }
}
return true;

想定の挙動をしなかったコード

for (Map detail : /*List<Map<String,Object>>*/detailList) {
  if(detail.entrySet().stream().anyMatch(d -> Objects.isNull(d.getValue()))){
    return false;
  }
}
return true;

Comments

[swordone]

｢想定の挙動をしなかった｣の内容を具体的にお願いします。

[xebme]

Javaのバージョンを教えてください。次のようにMap<String, Object>にすると思います。
for (Map<String, Object> detail : /* List<Map<String,Object>> */detailList) {

Answer 1

まず「該当のソースコード」を修正しましょう。
xebmeさんの指摘通り、Mapにはジェネリクスを付けましょう。
判定するのは「Mapの値にnullが含まれるかどうか」であるので、
entryを取り出さずともMap#containsValueを使えます。

for (Map<String, Object> detail : /* List<Map<String,Object>> */detailList) {
    if (detail.containsValue(null)) {
        return false;
    }
}
return true;

これをStreamで書くとなると、こうなります。

return detailList.stream().noneMatch(d -> d.containsValue(null));

Comments

[thmimi]

回答ありがとうございます。

ここまで短くなるんですね。
一番短かったのでベストアンサーにさせていただきました。

Answer 2

openjdk version "1.8.0_222"
OpenJDK 64-Bit Server VM (build 25.222-b10, mixed mode)で実行しました。以下は皆同じ結果になります。

拡張for()

for (Map<String, Object> detail : /* List<Map<String,Object>> */detailList) {
    for (Entry<String, Object> entry : detail.entrySet()) {
        if (Objects.isNull(entry.getValue())) {
            return false;
        }
    }
}
return true;

拡張for()とstream()のミックスモード。

for (Map<String, Object> detail : /* List<Map<String,Object>> */detailList) {
    if (detail.entrySet().stream().anyMatch(d -> Objects.isNull(d.getValue()))) {
        return false;
    }
}
return true;

flatMap()とnoneMatch()を使用するワンライナー。

return detailList.stream().flatMap(d -> d.entrySet().stream()).noneMatch(d -> Objects.isNull(d.getValue()));

ワンライナーは別の書き方があると思います。バージョンやコードの見落としがあれば指摘してください。

Comments

[thmimi]

回答ありがとうございます。
一行で記述することができるんですね。驚きです。

次回からはJavaのバージョンを記載するようにします。
素早い回答ありがとうございました。