正确使用Optional.ifPresent（）

我试图了解Java 8中API 的ifPresent()方法Optional。

我有简单的逻辑：

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>作为参数。您正在向其传递类型为void的表达式。因此，它不会编译。

使用者应被实现为lambda表达式：

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()and 结合使用.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
})

对我来说，这更直观。

使用flatMap。如果存在值，则flatMap返回仅包含该值的顺序Stream，否则返回空Stream。所以没有必要使用ifPresent()。例：

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

为什么要编写简单的复杂代码？

确实，如果您绝对要使用Optional该类，那么最简单的代码就是您已经编写的代码...

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

该代码的优点是

1. 可读的
2. 易于调试（断点）
3. 不棘手

甲骨文已经Optional在Java 8中添加了该类，并不意味着必须在所有情况下都使用该类。

您可以使用如下方法引用：

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

方法ifPresent()获取Consumer对象作为参数，并且（来自JavaDoc）：“如果存在值，则使用该值调用指定的使用者。” 值是您的变量user。

或者，如果此方法doSomethingWithUser在User类中，而不是static，则可以使用如下方法引用：

user.ifPresent(this::doSomethingWithUser);