Правильное использование Optional.ifPresent ()

Я пытаюсь понять ifPresent() метод OptionalAPI в Java 8.

У меня простая логика:

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>качестве аргумента. Вы передаете ему выражение, тип которого недействителен. Так что это не компилируется.

Потребитель предназначен для реализации в виде лямбда-выражения:

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()вызов метода будет выполняться только при наличии пользователя. Ваш код выполняет вызов метода напрямую и пытается передать свой недействительный результат в 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
})

Для меня это более интуитивно понятно.

Зачем писать сложный код, если можно сделать его простым?

В самом деле, если вы абсолютно собираетесь использовать Optionalкласс, то самый простой код - это то, что вы уже написали ...

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

Этот код имеет то преимущество, что он

1. удобочитаемый
2. легко отлаживать (точка останова)
3. не сложно

Тот Optionalфакт, что Oracle добавил этот класс в Java 8, не означает, что этот класс необходимо использовать во всех ситуациях.

Используйте flatMap. Если значение присутствует, flatMap возвращает последовательный Stream, содержащий только это значение, в противном случае возвращает пустой Stream. Так что в использовании нет необходимости ifPresent(). Пример:

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

Вы можете использовать ссылку на метод следующим образом:

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

Метод ifPresent()получает Consumerобъект в качестве параметра и (из JavaDoc ): «Если значение присутствует, вызвать указанного потребителя со значением». Значение - это ваша переменная user.

Или, если этот метод doSomethingWithUserнаходится в Userклассе, а его нет static, вы можете использовать ссылку на метод следующим образом:

user.ifPresent(this::doSomethingWithUser);