回答編集履歴

動作を間違えていたので修正

2021/11/05 18:06

投稿

swordone

スコア20675

answer CHANGED Viewed

@@ -1,20 +1,18 @@
-せっかくなら、Streamの機能やOptionalの特性をもっと活かしましょう。
+せっかくなら、Optionalの特性をもっと活かしましょう。
-Optionalに中身があるかどうかのチェックも、わざわざ中身を取り出すことも
+Optionalに中身があるかどうかのチェックも、わざわざ中身を取り出すこともせずに比較できます。
-することなく比較できます。
 (InvoiceNumberをequals比較できるようにしておくと楽かと思われます）
 ```java
 ItemEntityResult = ItemEntitys.stream()
-                        .map(i -> i.getInvoiceNumberOptional()
+                        .filter(i -> i.getInvoiceNumberOptional()
                                       .filter(x -> x.equals(ItemFilterCondition.getInvoiceNumber()))
-                                      .orElse(null))
+                                      .isPresent())
-                        .filter(x -> x != null)
                         .collect(Collectors.toList());
 ```
-上記コードではOptionalからListにまとめるにあたって、排除する要素をnullにして除外していますが、Java9以降ならその処理もOptionalで済ませることができ、nullが必要なくなります。
+あるいは、検索項目の方をOptionalに入れてしまうという手もあると思います。
+(先と同じく、InvoiceNumberをequals比較できること前提)
 ```java
 ItemEntityResult = ItemEntitys.stream()
-                        .flatMap(i -> i.getInvoiceNumberOptional()
+                        .filter(i -> i.getInvoiceNumberOptional()
-                                      .filter(x -> x.equals(ItemFilterCondition.getInvoiceNumber()))
+                                      .equals(Optional.ofNullable(ItemFilterCondition.getInvoiceNumber())))
-                                      .stream())
                         .collect(Collectors.toList());
 ```