Penggunaan yang tepat dari Opsional.ifPresent ()

Saya mencoba memahami ifPresent()metode OptionalAPI di Java 8.

Saya memiliki logika sederhana:

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

Tetapi ini menghasilkan kesalahan kompilasi:

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

Tentu saja saya bisa melakukan sesuatu seperti ini:

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

Tapi ini persis seperti nullcek yang berantakan .

Jika saya mengubah kodenya menjadi ini:

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

Kode menjadi semakin kotor, yang membuat saya berpikir untuk kembali ke nullcek lama .

Ada ide?

Optional<User>.ifPresent()mengambil Consumer<? super User>argumen sebagai. Anda memberikan ekspresi yang tipenya kosong. Jadi itu tidak bisa dikompilasi.

Konsumen dimaksudkan untuk diimplementasikan sebagai ekspresi lambda:

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

Atau bahkan lebih sederhana lagi, menggunakan referensi metode:

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

Ini pada dasarnya sama dengan

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

Idenya adalah bahwa doSomethingWithUser()pemanggilan metode hanya akan dijalankan jika pengguna hadir. Kode Anda menjalankan panggilan metode secara langsung, dan mencoba meneruskan hasil kosongnya ke ifPresent().

Selain jawaban @ JBNizet, kasus penggunaan umum saya ifPresentadalah untuk menggabungkan .isPresent()dan.get() :

Cara lama:

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

Jalan baru:

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

Ini, bagi saya, lebih intuitif.

Mengapa menulis kode yang rumit padahal Anda bisa membuatnya sederhana?

Memang, jika Anda benar-benar akan menggunakan Optionalkelas, kode paling sederhana adalah yang telah Anda tulis ...

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

Kode ini memiliki keuntungan menjadi

1. mudah dibaca
2. mudah untuk di-debug (breakpoint)
3. tidak rumit

Hanya karena Oracle telah menambahkan Optionalkelas di Java 8 tidak berarti bahwa kelas ini harus digunakan di semua situasi.

Gunakan flatMap. Jika ada nilai, flatMap mengembalikan Stream berurutan yang hanya berisi nilai itu, jika tidak, mengembalikan Stream kosong. Jadi tidak perlu digunakan ifPresent(). Contoh:

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

Anda dapat menggunakan referensi metode seperti ini:

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

Metode ifPresent()dapatkan Consumerobjek sebagai paremeter dan (dari JavaDoc ): "Jika ada nilai, panggil konsumen yang ditentukan dengan nilai tersebut." Nilai itu adalah variabel Anda user.

Atau jika metode ini doSomethingWithUserada di Userkelas dan tidak static, Anda dapat menggunakan referensi metode seperti ini:

user.ifPresent(this::doSomethingWithUser);