การใช้ 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
})

สำหรับฉันแล้วสิ่งนี้ง่ายกว่า

ทำไมต้องเขียนโค้ดที่ซับซ้อนในเมื่อคุณทำให้มันง่าย

อันที่จริงถ้าคุณจะใช้Optionalคลาสอย่างแน่นอนรหัสที่ง่ายที่สุดคือสิ่งที่คุณเขียนไปแล้ว ...

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

รหัสนี้มีข้อดีคือ

1. อ่านได้
2. ง่ายต่อการดีบัก (เบรกพอยต์)
3. ไม่ยุ่งยาก

เพียงเพราะ Oracle ได้เพิ่มOptionalคลาสใน Java 8 ไม่ได้หมายความว่าจะต้องใช้คลาสนี้ในทุกสถานการณ์

ใช้ flatMap หากมีค่าอยู่ flatMap จะส่งคืนสตรีมตามลำดับที่มีเฉพาะค่านั้นมิฉะนั้นจะส่งคืนสตรีมว่าง ifPresent()ดังนั้นจึงไม่มีความจำเป็นต้องใช้ ตัวอย่าง:

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

คุณสามารถใช้การอ้างอิงวิธีการดังนี้:

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

เมธอดifPresent()รับConsumerอ็อบเจกต์เป็น paremeter และ (จากJavaDoc ): "หากมีค่าอยู่ให้เรียกใช้ผู้บริโภคที่ระบุด้วยค่า" userราคามันเป็นตัวแปรของคุณ

หรือถ้าวิธีนี้doSomethingWithUserอยู่ในUserคลาสและไม่ใช่staticคุณสามารถใช้การอ้างอิงเมธอดดังนี้:

user.ifPresent(this::doSomethingWithUser);