为何SonarQube对Optional三元表达式报“先调用isPresent()”警告？

别担心，SonarQube这里完全没搞错——咱们来拆解下你的代码问题：

你写的代码是：

String name = user.isPresent() ? "<default>" : user.get().getName();

仔细看三元运算符的逻辑：

• 当user.isPresent()为true（也就是Optional里有值），你返回了<default>；
• 当user.isPresent()为false（Optional是空的），你反而去调用user.get().getName()！

这就大错特错了——空Optional调用get()会直接抛出NoSuchElementException，SonarQube的警告“Call "user.isPresent()" before accessing the value.”其实是在提醒你：你现在的代码逻辑里，访问值（user.get()）的时机是在Optional为空的时候，这完全违背了Optional的设计初衷。

正确的写法应该是这样： #

把三元运算符的分支调换过来，确保只有当Optional存在值的时候才去获取name：

String name = user.isPresent() ? user.get().getName() : "<default>";

更优雅的Optional最佳实践 #

其实用Optional自带的map和orElse方法会更简洁，也完全符合SonarQube的校验规则：

String name = user.map(User::getName).orElse("<default>");

map方法会在Optional有值的时候执行getName()，空的时候直接跳过，orElse提供默认值，既安全又易读。

内容的提问来源于stack exchange，提问作者RainHunter