Prawidłowe użycie Optional.ifPresent ()

Próbuję zrozumieć ifPresent()metodę OptionalAPI w Javie 8.

Mam prostą logikę:

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

Ale to powoduje błąd kompilacji:

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

Oczywiście mogę zrobić coś takiego:

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

Ale to jest dokładnie jak zagracony nullczek.

Jeśli zmienię kod na ten:

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

Kod staje się brudniejszy, co sprawia, że ​​myślę o powrocie do starego nullczeku.

Jakieś pomysły?

Optional<User>.ifPresent()przyjmuje Consumer<? super User>jako argument. Przekazujesz mu wyrażenie, którego typ jest nieważny. Więc to się nie kompiluje.

Konsument ma zostać zaimplementowany jako wyrażenie lambda:

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

Lub jeszcze prościej, używając odwołania do metody:

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

To w zasadzie to samo, co

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

Chodzi o to, że doSomethingWithUser()wywołanie metody zostanie wykonane tylko wtedy, gdy użytkownik jest obecny. Twój kod wykonuje wywołanie metody bezpośrednio i próbuje przekazać wynik void do ifPresent().

Oprócz odpowiedzi @ JBNizet, moim ogólnym przypadkiem użycia ifPresentjest połączenie .isPresent()i .get():

Stara droga:

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

Nowy sposób:

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

Dla mnie jest to bardziej intuicyjne.

Po co pisać skomplikowany kod, skoro można to uprościć?

Rzeczywiście, jeśli absolutnie zamierzasz używać tej Optionalklasy, najprostszym kodem jest to, co już napisałeś ...

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

Ten kod ma zalety bycia

1. czytelny
2. łatwe do debugowania (punkt przerwania)
3. nie trudne

To, że Oracle dodała Optionalklasę w Javie 8, nie oznacza, że ​​ta klasa musi być używana w każdej sytuacji.

Użyj flatMap. Jeśli wartość jest obecna, flatMap zwraca sekwencyjny Stream zawierający tylko tę wartość, w przeciwnym razie zwraca pusty Stream. Więc nie ma potrzeby używania ifPresent(). Przykład:

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

Możesz użyć odwołania do metody w następujący sposób:

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

Metoda ifPresent()get Consumerobject jako parometr i (z JavaDoc ): "Jeśli wartość jest obecna, wywołaj określonego konsumenta z wartością." Wartość to twoja zmienna user.

Lub jeśli ta metoda doSomethingWithUserznajduje się w Userklasie, a tak nie jest static, możesz użyć odwołania do metody w następujący sposób:

user.ifPresent(this::doSomethingWithUser);