用java 8替换null check可选

Question

我试图用可选的替换下面的代码块,只是想检查这样做会有什么不好的影响,如性能或其他任何东西？

现有代码:

UserObj userObj=new UserObj();
Object result = fetchDetails();
if (null != result) {
    userObj.setResult(result.toString());
}

使用Java 8 Optional:

UserObj userObj=new UserObj();
Optional.ofNullable(fetchDetails()).ifPresent(var -> userObj.setResult(var.toString()));

执行此更改只是为了使代码看起来简洁,因为我的代码中有很多空检查块.

Answer 1

首先,我认为你误解了目的Optional.它不仅仅是替换

if(obj != null){ ... }

重点Optional是为函数提供一种返回值的方法,以指示缺少返回值.请阅读这篇文章了解更多详情.

Optional在您的情况下正确使用ResultObj将从fetchDetails方法返回可选:

Optional<ResultObj> fetchDetails() {
   ...
}

然后,您只需Optional像以前一样链接获取的方法.

更新

如果您无法修改fetchDetails,仍然可以选择将其包装到您自己的方法中,如下所示:

Optional<ResultObj> fetchOptionalDetails() {
    return Optional.ofNullable(fetchDefails());
}

创建一个新方法会增加很小的开销,但代码将更具可读性:

fetchOptionalDetails().ifPresent(details -> /* do something */);