Optional.ifPresent（）の適切な使用法

Java 8のAPI のifPresent()メソッドを理解しようとしていOptionalます。

私は単純なロジックを持っています：

Optional<User> user=...
user.ifPresent(doSomethingWithUser(user.get()));

しかし、これによりコンパイルエラーが発生します。

ifPresent(java.util.functionError:(186, 74) java: 'void' type not allowed here)

もちろん、私はこのようなことをすることができます：

if(user.isPresent())
{
  doSomethingWithUser(user.get());
}

しかし、これは雑然としたnull小切手とまったく同じです。

コードをこれに変更すると：

 user.ifPresent(new Consumer<User>() {
            @Override public void accept(User user) {
                doSomethingWithUser(user.get());
            }
        });

コードが汚れてきているので、以前のnullチェックに戻ることを考えさせられます。

何か案は？

Optional<User>.ifPresent()かかるConsumer<? super User>引数として。型がvoidの式を渡しています。したがって、コンパイルされません。

コンシューマーはラムダ式として実装することを目的としています。

Optional<User> user = ...
user.ifPresent(theUser -> doSomethingWithUser(theUser));

または、メソッド参照を使用してさらに簡単にします。

Optional<User> user = ...
user.ifPresent(this::doSomethingWithUser);

これは基本的に同じです

Optional<User> user = ...
user.ifPresent(new Consumer<User>() {
    @Override
    public void accept(User theUser) {
        doSomethingWithUser(theUser);
    }
});

この考え方は、doSomethingWithUser()メソッド呼び出しはユーザーが存在する場合にのみ実行されるというものです。コードはメソッド呼び出しを直接実行し、そのvoid結果をに渡そうとしますifPresent()。

@ JBNizetの答えに加えて、私のための一般的なユースケースがifPresent結合することである.isPresent()と.get()。

古い方法：

Optional opt = getIntOptional();
if(opt.isPresent()) {
    Integer value = opt.get();
    // do something with value
}

新しい方法：

Optional opt = getIntOptional();
opt.ifPresent(value -> {
    // do something with value
})

これは、私には、より直感的です。

flatMapを使用します。値が存在する場合、flatMapはその値のみを含む順次Streamを返し、それ以外の場合は空のStreamを返します。したがって、を使用する必要はありませんifPresent()。例：

list.stream().map(data -> data.getSomeValue).map(this::getOptinalValue).flatMap(Optional::stream).collect(Collectors.toList());

単純にすることができるのに、なぜ複雑なコードを書くのですか？

実際、Optionalクラスを絶対に使用する場合、最も単純なコードは、すでに記述したものです...

if (user.isPresent())
{
    doSomethingWithUser(user.get());
}

このコードには、

1. 読みやすい
2. デバッグが簡単（ブレークポイント）
3. トリッキーではない

OracleがOptionalJava 8でクラスを追加したからといって、このクラスをすべての状況で使用する必要があるとは限りません。

次のようなメソッド参照を使用できます。

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

メソッドifPresent()はConsumerオブジェクトをパラメーターとして取得し、（JavaDocから）：「値が存在する場合は、その値で指定されたコンシューマーを呼び出します。」値それはあなたの変数ですuser。

または、このメソッドdoSomethingWithUserがUserクラス内にあり、そうでない場合はstatic、次のようにメソッド参照を使用できます。

user.ifPresent(this::doSomethingWithUser);