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

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

Это, как по мне, более интуитивно понятно.

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

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

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

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

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

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

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

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

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

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

user.ifPresent(this::doSomethingWithUser);

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

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