SonarLint提示Optional.ofNullable可能抛NPE，实际未抛是否为误报？

这并非你的代码存在实际风险，而是SonarLint的静态分析局限性导致的假阳性警告，你的测试结果是符合预期的。

为什么测试null时不会抛出NPE？ #

你的代码逻辑是安全的：

• 当test为null时，Optional.ofNullable(test)会生成空Optional，后续map操作后仍然是空Optional，isPresent()返回false，不会进入if代码块，自然不会执行test.x.length()。
• 当test不为null但test.x为null时，map(test1 -> test1.x)会生成空Optional，同样不会进入if块。
• 只有当test不为null且test.x不为null时，才会进入if块，此时test必然是非空状态，test.x也非空，调用length()不会触发NPE。

为什么SonarLint会给出警告？ #

SonarLint的数据流分析没有识别到Optional.ofNullable(test).map(...).isPresent()这个条件等价于test != null && test.x != null。它仅看到test是方法参数（没有被显式的if(test != null)判断），因此判定test仍可能为null，从而触发警告。

消除警告的优化方案 #

推荐使用更符合Optional设计意图的写法，既避免警告，也让代码更简洁：

private static void printXLength(Test test) {
  Optional.ofNullable(test)
          .map(Test::x)
          .ifPresent(x -> System.out.println(x.length()));
}

或者显式添加非空判断，让SonarLint明确识别变量状态：

private static void printXLength(Test test) {
  if (test != null && test.x != null) {
    System.out.println(test.x.length());
  }
}

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