Cách sử dụng phù hợp Optional.ifPresent ()

Tôi đang cố gắng hiểu ifPresent()phương thức của OptionalAPI trong Java 8.

Tôi có logic đơn giản:

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

Nhưng điều này dẫn đến lỗi biên dịch:

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

Tất nhiên tôi có thể làm điều gì đó như sau:

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

Nhưng điều này giống hệt như một nulltấm séc lộn xộn .

Nếu tôi thay đổi mã thành này:

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

Mã ngày càng bẩn khiến tôi nghĩ đến việc quay lại nullséc cũ .

Bất kỳ ý tưởng?

Optional<User>.ifPresent()lấy một Consumer<? super User>đối số làm đối số. Bạn đang chuyển cho nó một biểu thức có kiểu là void. Vì vậy, điều đó không biên dịch.

Người tiêu dùng dự định được triển khai dưới dạng biểu thức lambda:

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

Hoặc đơn giản hơn, sử dụng tham chiếu phương thức:

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

Về cơ bản, điều này giống với

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

Ý tưởng là doSomethingWithUser()cuộc gọi phương thức sẽ chỉ được thực hiện nếu người dùng có mặt. Mã của bạn thực hiện lệnh gọi phương thức trực tiếp và cố gắng chuyển kết quả vô hiệu của nó cho ifPresent().

Ngoài câu trả lời của @ JBNizet, trường hợp sử dụng chung của tôi ifPresentlà kết hợp .isPresent()và .get():

Cách cũ:

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

Cách mới:

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

Điều này, đối với tôi, là trực quan hơn.

Sử dụng flatMap. Nếu có giá trị, flatMap trả về một Luồng tuần tự chỉ chứa giá trị đó, nếu không sẽ trả về một Luồng trống. Vì vậy không có nhu cầu sử dụng ifPresent(). Thí dụ:

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

Tại sao phải viết mã phức tạp khi bạn có thể làm cho nó đơn giản?

Thật vậy, nếu bạn hoàn toàn định sử dụng Optionallớp, mã đơn giản nhất là những gì bạn đã viết ...

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

Mã này có những ưu điểm là

1. đọc được
2. dễ gỡ lỗi (điểm ngắt)
3. không khó

Chỉ vì Oracle đã thêm Optionallớp trong Java 8 không có nghĩa là lớp này phải được sử dụng trong mọi tình huống.

Bạn có thể sử dụng tham chiếu phương pháp như sau:

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

Phương thức ifPresent()lấy Consumerđối tượng dưới dạng paremeter và (từ JavaDoc ): "Nếu có giá trị, hãy gọi người tiêu dùng được chỉ định với giá trị." Giá trị nó là biến của bạn user.

Hoặc nếu phương thức doSomethingWithUsernày nằm trong Userlớp và không phải static, bạn có thể sử dụng tham chiếu phương thức như sau:

user.ifPresent(this::doSomethingWithUser);