FindBugs插件报NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE问题咨询

你遇到的NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE错误的核心触发原因是：FindBugs识别到LocalSessionFactoryBean.getObject()方法的返回值存在为null的可能性，而你在最初的代码中没有对这个返回值做非空校验，直接传入了reader.setSessionFactory()方法。你之前做的bean != null校验仅仅保证了LocalSessionFactoryBean实例本身非空，完全没有覆盖它的返回值可能为null的情况，因此会触发告警。

• 可以解决问题的写法：
  你将bean.getObject()的返回值赋值给了局部变量sessionFactory，之后对这个局部变量做了非空校验，再直接传入方法。FindBugs的静态分析逻辑可以确认经过非空校验的同一个局部变量在后续使用时必然是非空的，因此会判定这个路径不存在null风险，关闭告警。

if(bean!=null) {
    SessionFactory sessionFactory = bean.getObject();
    if(sessionFactory!=null)
    reader.setSessionFactory(sessionFactory);
}

• 无法解决问题的写法：
  你在分支判断和参数传入的位置两次调用了bean.getObject()方法。FindBugs的静态分析不会默认假设同一个实例的同一个方法多次调用会返回相同结果：即使第一次调用返回非空，也存在第二次调用返回null的可能性（比如多线程场景下实例内部状态被其他线程修改）。因此FindBugs仍然判定你传入setSessionFactory的参数存在null风险，会继续触发告警。

if(bean!=null) {
    if(bean.getObject()!=null)
    reader.setSessionFactory(bean.getObject());
}

如果你的业务场景可以确认getObject()多次调用一定会返回相同结果，也可以通过给该方法添加@Nonnull注解、配置FindBugs忽略规则的方式消除告警，不过存局部变量的写法通用性更强，也符合静态检查的最佳实践。

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