如何修复SonarQube检测到的Java代码空指针异常？

你的代码核心问题在于：GeneratedKeyHolder是用来捕获INSERT语句生成的自增主键的，而你执行的是UPDATE操作，这类操作不会生成新主键，因此generatedKeyHolder.getKey()必然返回null，调用intValue()时就会触发空指针异常。原代码用try-catch捕获NPE的方式属于被动补救，不符合代码规范，SonarQube的提示正是指出了这一点。

下面是几种规范的解决方案：

方案1：主动判空处理 #

先获取Key对象，显式判断是否为null后再返回对应值：

GeneratedKeyHolder generatedKeyHolder = new GeneratedKeyHolder();
String sql = "Update users set enable=true where username=:username";
SqlParameterSource paramSource = new MapSqlParameterSource().addValue("username", u.getUsername());
namedParameterJdbcTemplate.update(sql, paramSource, generatedKeyHolder);

Number key = generatedKeyHolder.getKey();
return key != null ? key.intValue() : 0;

方案2：用Optional优雅处理null（Java 8+） #

借助Optional类简化空值判断逻辑，代码更简洁：

GeneratedKeyHolder generatedKeyHolder = new GeneratedKeyHolder();
String sql = "Update users set enable=true where username=:username";
SqlParameterSource paramSource = new MapSqlParameterSource().addValue("username", u.getUsername());
namedParameterJdbcTemplate.update(sql, paramSource, generatedKeyHolder);

return Optional.ofNullable(generatedKeyHolder.getKey())
               .map(Number::intValue)
               .orElse(0);

额外优化：移除不必要的GeneratedKeyHolder #

既然是UPDATE操作，完全不需要GeneratedKeyHolder——这个类对UPDATE场景没有任何作用。可以直接利用update方法返回的受影响行数作为返回值，既解决空指针问题，又精简逻辑：

String sql = "Update users set enable=true where username=:username";
SqlParameterSource paramSource = new MapSqlParameterSource().addValue("username", u.getUsername());
int affectedRows = namedParameterJdbcTemplate.update(sql, paramSource);
return affectedRows;

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