java8でOptionalを使い何らかの値がnullの場合の処理の書き方。

Question

前提・実現したいこと

java8を使用して、Optionalの値が存在する場合、Oprionalの値がnullの場合の処理を分岐させたい。

該当のソースコード

List<String> filePaths = 「いくつかのファイルパス文字列」;
Optional<String> targetFilePath = filePaths.stream().filter(path -> path.contain("hoge")).findFirst();
if(!tfFileOpt.isPresent()) {
  // 処理対象のファイルが取得できなかった場合
  // 例えば
  // 1. ログ出力
  // 2. DBのユーザステータスをエラーに更新。
  // 3.  throw new BusinessException(); // 上位層へ業務処理が失敗したことを通達
}

// 以下、対象ファイルが取得できた場合。
// 例えば
// 1. 対象ファイルの読み込み。 ここで、targetFilePath.get()を使う?
// 2. 読み込んだファイルの中身を解析&業務処理。
// 3. etc... ...

この処理がよろしくないっぽいことはわかるのですが、どうすればいいかわかりません。皆さんどうするものなのでしょうか?

よろしくないと思っている点。
・Optionalを取得して、わざわざisPresent()で値のあるなしを判定している。→ nullチェックやってるのと変わらないから、Optionlの思想(?)に反している。
※ ググるとOptional取得して、if(optional.isParent){ ... } は論外。と書かれていることが多かったので。
・正常業務で、値を使いたいときいちいちOptionlにget()をかけるのが微妙。かと言って、isPresent()で判定した後、正常処理の最初に、ローカル変数に入れるのも微妙。そもそも、Optionalを取得する流れの中で、getElse()とかで取得したり、nullの場合の処理も記述できないの?
・仮に、Optionalを取得する流れの中で値があるなら取得する、ない場合は処理を行うのができた場合、ない場合の処理はラムダ式(?)で書くことになるが、その中でログを出したりDB操作することに違和感がある。

Answer 1

おっしゃる通り、Java8の時点でのOptionalは少々使い勝手が悪いです。
それを受けてか、Java9でOptionalは少し改良されました。

・正常業務で、値を使いたいときいちいちOptionlにget()をかけるのが微妙。かと言って、isPresent()で判定した後、正常処理の最初に、ローカル変数に入れるのも微妙。そもそも、Optionalを取得する流れの中で、getElse()とかで取得したり、nullの場合の処理も記述できないの?

ifPresentOrElseというメソッドが追加され、ない場合の処理をRunnableで渡せます。

Answer 2

Optionalが後付けで登場したこともあって、そこまで使い勝手が良くないのは間違いありません（Kotlinでは、nullになりうる値のnullチェックを済ませれば、あとはnullなしの値として扱えるようになっています）。ただ、

nullチェックやってるのと変わらないから、Optionlの思想(?)に反している。

こちらについては、「nullチェックしなければ例外になる」という意味のクラスなので、そんなに間違ってはいないです。

Answer 3

ifPresent()で何も問題ないと思いますが。

String targetPath = filePaths.stream().filter(path -> path.contains("hoge")).findFirst().orElse("");
if(targetPath.isEmpty()) {
    //null
} else {
    //not null
}

これならnullチェックぽくないかな？

Answer 4

Optionalが一番気持ちよく使えるのは関数型プログラミングのモナド的な利用、例えば複数の関数間に無用な判定を介さずにそのまま状態を伝達出来て必要な箇所で「値がないかも知れないことを忘れようがないように表現できる」点なのだろうと自分は思っています。

Optionalによりスッキリ記述できる場面というのはよく用いられる次のような場合で、それはJavaであっても一応使えるようになっていると思います。

• 値がない場合でもデフォルト値を仮定したい場合

orElseとかcall by name的にしたければorElseGetを使えばよいですね。

• 値がないことが例外である場合

orElseThrowを用いればよいですね。call by name的に例外を書けるのでエラーでないケースのために無駄に例外インスタンスを生成するような気持ち悪さもありません。ここだけはJavaの検査例外とFunctionalInterfaceの相性の悪さが回避されており検査例外を指定すればorElseThrowは「その検査例外をハンドリングするかthrows句を書くかの選択を強制する働きを担い」RuntimeExceptionを選べば「そうしなくてよい」ように振舞ってくれますし。

• 値がある場合だけそれに基づいた計算をして新たな値にする

mapやflatMapで比較的気持ちよくかけるのではないでしょうか。

質問者さんが挙げておられるうちの一部は上記で書けるものもありますね。しかし残りは「値があるかないかで制御を変える」ことなので、nullチェックしようがOptional#isPresentでやろうが記述上のスッキリ感に違いはでてこないと思います。mapなどを使えば「値がある場合のみ副作用を起こす」ような書き方はできますが、それは質問者さんがおっしゃるよう自分も「違和感」を持ちます。

---

ちなみに書き味（？）の点では確かにisPresentと書いたのにgetをかかなきゃいけないのは少々どんくさい感じはあります。その辺りはmaisumakunさんがおっしゃるとおり言語仕様を拡張することなくライブラリーで実現している関係上しかたないことだと思います。考えてみればこれ以外にも「こうなっていたらなぁ」と思える点はいくつかあると思います。例えば

Base o = ...;
if (o instanceof Derived) {
  ((Derived)o).derivedMethod();
}

などは他の言語のように

Base o = ...;
if (o instanceof Derived) {
  o.derivedMethod(); // oはDerivedのインスタンスであることが明らかなのでキャスト不要
}

と書けたらなぁと思うことがありますよね。そう考えると(個人的には)Optional#getぐらいは許容範囲に思えてきます。（自分の場合、何事もほどほどで満足してしまうせいかも知れません。）

なんだか「もっと苦労している人もいるんだから自分も頑張ろう」みたいな精神論になってきて結論が何なのかがぼんやりしてしまいました。スミマセン・・・

Comments

[usukuti_hanako]

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

Optionalを使う際の例や、多言語での場合との比較など、例示が多く
また、個人的な使用感やそれに対する割り切りかたなど、が入っているのがありがたかったので、ベストアンサーに選ばせて頂きました。

他の回答者様も、ご意見ありがとうございます。

ググって、トップに出てくる記事をいくつか読んだ所、
isPresentで値の有無を判定するのは論外。
isPresentは使わない!という記事が多くOptionalの使い方がよくわからなくなってしまっていました。

```
if(optional.isPresent()){
// 正常処理
String target = optional.isPresent();
} else {
// 業務エラー処理
}
```
のような形で進めていこうと思います。

みなさんありがとうございました。

Answer 5

isPresent()使わず無理やり書いてみました

public class Main {
    public static void main(String[] args) throws Exception {
        Optional<String> src = Optional.of("hoge");
        
        
        Runnable r = src.map(x ->
            (Runnable)new Runnable(){
                public void run(){
                    // 正常フロー
                    System.out.println("Some");
                    System.out.println(x);
                }
            }
        ).orElse((Runnable)new Runnable(){
                    public void run(){
                    // 異常フロー
                        System.out.println("None");
                    }
                });
        
        r.run();
    }
}