应避免直接调用Optional.get()，因其在空值时抛出未检查的NoSuchElementException，易致线上崩溃；正确做法是使用orElse、orElseGet、orElseThrow或flatMap等安全方法。

直接调用 optional.get() 获取值，最大的坑就是——Optional 为空时抛出 NoSuchElementException，而且这个异常是运行时异常，编译器不检查，很容易在上线后突然炸掉。

空 Optional 调用 get() 必然崩溃

Optional.get() 的设计本意不是“安全取值”，而是“我确定它有值，你帮我拿一下”。一旦里面是 Optional.empty()，立刻抛 NoSuchElementException，没有任何商量余地。

• 常见误用：把 Optional.ofNullable(xxx) 当成“防空包装”，然后无脑 .get()
• 典型场景：数据库查不到记录返回 Optional.empty()，后续直接 .get() → 500 错误
• 调试困难：堆栈里看不到空值来源，只看到 get() 这一行报错

别用 get() 做判空逻辑

有人写成这样：

if (opt.isPresent()) { return opt.get(); } else { return defaultVal; }

这等于手动重复了 orElse() 的工作，还多写两行、多一次判断、多一次对象访问，完全违背 Optional 的设计初衷。

Java Maven Common

Java Maven 项目审查工作流的通用输入处理。当 Java Maven 项目以 ZIP压缩包或 GitLab 仓库 URL 形式提供且需要...时使用。

下载

立即学习“Java免费学习笔记（深入）”；

• 正确替代：opt.orElse(defaultVal)、opt.orElseGet(() -> computeDefault())
• 需要抛异常？用 opt.orElseThrow(() -> new XxxException("xxx"))，语义清晰且可控

链式调用中 get() 会中断流式表达

Optional 的优势在于支持 map、filter、flatMap 等函数式操作，形成可读的空安全链条。一旦中间插入 .get()，就强行解包，后续无法再链式处理，还可能引入新的空指针风险。

• 错误示范：opt.map(User::getProfile).get().getAvatar() —— 第二个 .get() 可能 NPE
• 正确写法：opt.flatMap(u -> Optional.ofNullable(u.getProfile())) .map(Profile::getAvatar) .orElse(null)

真正该用 get() 的场景极少

官方文档明确说：get() 应仅用于“已通过 isPresent() 确认非空”的上下文（比如单元测试断言、内部断言工具），生产代码几乎不该出现。

• 测试中可用：assertThat(opt).isPresent(); String val = opt.get();
• 业务逻辑中请默认忽略 get()，优先考虑 orElse 系列、ifPresent 或直接用传统 if 判空