Optionalがついた値をついていない値と比較したい

Question

ご覧いただきありがとうございます。
送られてきたデータの請求番号の値が検索条件で入力された値と同じかどうかを
チェックする処理を実装しています。

送られてきたデータの方は

public class ItemEntity {
     private final Optional<InvoiceNumber> invoiceNumberOptional;
}

のようにOptional付きで定義されていますが
検索項目の方は

public class ItemFilterCondition {

     private final InvoiceNumber invoiceNumber;

}

とoptionalがついていません。

比較の処理を書いてみましたが値が一致してもデータがItemEntityResultに入りません。

if(ItemFilterCondition.getInvoiceNumber() != null) {
                ItemEntityResult = ItemEntitys.stream()
                        .filter(i -> i.getInvoiceNumberOptional().get().equals(ItemFilterCondition.getInvoiceNumber().toString())
                        .collect(Collectors.toList());
}

比較している値をログに出力してみたところ

ItemFilterCondition:InvoiceNumber(value=1212660-E)
ItemEntity:InvoiceNumber(value=1212660-E)

と出力されました。比較の仕方のどこに問題があるのでしょうか？

Comments

[neko_the_shadow]

invoiceNumberがNULLになることはありますか? その場合、invoiceNumberOptionalとはどのように比較しますか?

[ttpk]

>invoiceNumberがNULLになることはありますか?
あります。

>その場合、invoiceNumberOptionalとはどのように比較しますか?
shiketaさんの回答を参考にnullチェックを通った場合だけ比較するよう実装中です

Answer 1

せっかくなら、Optionalの特性をもっと活かしましょう。
Optionalに中身があるかどうかのチェックも、わざわざ中身を取り出すこともせずに比較できます。
(InvoiceNumberをequals比較できるようにしておくと楽かと思われます）

ItemEntityResult = ItemEntitys.stream()
                        .filter(i -> i.getInvoiceNumberOptional()
                                      .filter(x -> x.equals(ItemFilterCondition.getInvoiceNumber()))
                                      .isPresent())
                        .collect(Collectors.toList());

あるいは、検索項目の方をOptionalに入れてしまうという手もあると思います。
(先と同じく、InvoiceNumberをequals比較できること前提)

ItemEntityResult = ItemEntitys.stream()
                        .filter(i -> i.getInvoiceNumberOptional()
                                      .equals(Optional.ofNullable(ItemFilterCondition.getInvoiceNumber())))
                        .collect(Collectors.toList());

Comments

[ttpk]

お返事遅くなってしまいすみません。
絞り込む条件がぐっとわかりやすくなりましたね！
ありがとうございます。
ItemEntityResultにデータが入ることを確認できました

Answer 2

i.getInvoiceNumberOptional().get().equals(ItemFilterCondition.getInvoiceNumber().toString())

この部分ですが、InvoiceNumberオブジェクトとStringオブジェクトを比較しているので、値が不一致と判定されているのだと思います。

またOptional.getはOptionalオブジェクトが空だった場合、Exceptionをスローします。ここもケアする必要があります。おそらく以下のようなコードでよいかと思います。

i.isPresent() && i.getInvoiceNumberOptional().get().toString().equals(ItemFilterCondition.getInvoiceNumber().toString())

Comments

[ttpk]

ありがとうございます！
ItemEntityResultにデータが入ることを確認できました！

Answer 3

ドキュメントにあたりませう。

1. isPresent()して、
2. trueならget()して、
3. equals()する。

もしくは、

1. ifPresent()で済ませる。