Правильне використання 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
})

Для мене це більш інтуїтивно.

Використовуйте flatMap. Якщо значення присутнє, flatMap повертає послідовний потік, що містить лише це значення, інакше повертає порожній потік. Тому немає необхідності у використанні 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 додав Optionalклас у Java 8, не означає, що цей клас повинен використовуватися в усіх ситуаціях.

Ви можете використовувати посилання на метод так:

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

Метод ifPresent()get Consumerobject як параметр і (з JavaDoc ): "Якщо значення є, викличте вказаного споживача зі значенням." Значення це ваша змінна user.

Або якщо цей метод doSomethingWithUserє в Userкласі, а його немає static, ви можете використати посилання на метод таким чином:

user.ifPresent(this::doSomethingWithUser);