Optional.ifPresent () öğesinin doğru kullanımı

Java 8'de API'nin ifPresent()yöntemini anlamaya çalışıyorum Optional.

Basit mantığım var:

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

Ancak bu, bir derleme hatasıyla sonuçlanır:

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

Tabii ki bunun gibi bir şey yapabilirim:

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

Ancak bu tam olarak dağınık bir nullçek gibi .

Kodu şununla değiştirirsem:

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

Kod gittikçe kirleniyor ve bu da bana eski nullçeke dönmeyi düşündürüyor .

Herhangi bir fikir?

Optional<User>.ifPresent()Consumer<? super User>argüman olarak alır . Ona türü geçersiz bir ifade geçiriyorsunuz. Böylece bu derlemez.

Bir Tüketici, lambda ifadesi olarak uygulanmak üzere tasarlanmıştır:

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

Veya bir yöntem referansı kullanarak daha da basit:

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

Bu temelde aynı şey

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

Buradaki fikir, doSomethingWithUser()yöntem çağrısının yalnızca kullanıcı mevcutsa yürütüleceğidir. Kodunuz yöntem çağrısını doğrudan yürütür ve void sonucunu öğesine iletmeye çalışır ifPresent().

@ JBNizet'in cevabına ek olarak, genel kullanım durumum ifPresentbirleştirmek .isPresent()ve .get():

Eski yol:

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

Yeni yol:

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

Bu benim için daha sezgisel.

Basitleştirebilecekken neden karmaşık kod yazasınız?

Nitekim Optionaldersi kesinlikle kullanacaksanız , en basit kod zaten yazmış olduğunuz koddur ...

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

Bu kodun olmanın avantajları vardır

1. okunabilir
2. hata ayıklaması kolay (kesme noktası)
3. zor değil

Sırf Oracle'ın Optionalsınıfı Java 8'e eklemiş olması, bu sınıfın her durumda kullanılması gerektiği anlamına gelmez.

FlatMap'i kullanın. Bir değer varsa, flatMap yalnızca bu değeri içeren sıralı bir Akış döndürür, aksi takdirde boş bir Akış döndürür. Yani kullanmaya gerek yok ifPresent(). Misal:

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

Yöntem referansını şu şekilde kullanabilirsiniz:

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

Yöntem ifPresent()get Consumerobject as a paremeter and (from JavaDoc ): "Bir değer varsa, belirtilen tüketiciyi değerle çağırın." Değer sizin değişkeninizdir user.

Bu yöntem Ya doSomethingWithUseriçindedir Usersınıf ve değil staticböyle bir yöntem başvuru kullanabilirsiniz:

user.ifPresent(this::doSomethingWithUser);