错误NULL CHECK SonarQube

Question

在为存储过程赋值时，调用的Nullcheck的SonarQube中有一个错误。

 call.setBigDecimal(ONE, bank != null ? bank : BigDecimal.ZERO);

正是在上述摘录中。如果去掉三元数，他也会犯同样的错误。我在函数之前做了一个IF，同样的错误也发生了。

在此输入图像描述

 public CarrierReturnDTO getMobileCarriers(BigDecimal bank) throws SQLException {
    CarrierReturnDTO carrierReturnListDTO = new CarrierReturnDTO();
    List<CarrierDTO> mobileCarrierList = new ArrayList<CarrierDTO>();

    DataSource dataSource = jdbcTemplate.getDataSource();
    if (dataSource != null) {
        try (Connection connection = dataSource.getConnection()) {

            if (connection != null) {
                try (CallableStatement call = connection.prepareCall(Constants.COMBO_OPERCEL)) {

                    call.setBigDecimal(ONE, bank != null ? bank : BigDecimal.ZERO);
                    call.registerOutParameter(TWO, OracleTypes.CURSOR);
                    call.execute();

                    ResultSet rs = (ResultSet) call.getObject(TWO);

                    while (rs.next()) {
                        CarrierDTO mobileCarrier = new CarrierDTO();
                        mobileCarrier.setMobileCarrierCode(rs.getBigDecimal(Constants.COD_EMP_OPER));
                        mobileCarrier.setMobileCarrier(rs.getString(Constants.NOM_EMP_OPER));
                        mobileCarrier.setAmountDigitChecked(rs.getBigDecimal(Constants.QTD_DIG_VERIFIC));
                        mobileCarrier.setFlagDoubleDigit(rs.getString(Constants.FLG_DUP_DIGIT));
                        mobileCarrier.setBankCode(rs.getString(Constants.BCO_CNV_ALTAIR));
                        mobileCarrier.setBranchCode(rs.getString(Constants.AGE_CNV_ALTAIR));
                        mobileCarrier.setAccountCode(rs.getString(Constants.CTA_CNV_ALTAIR));

                        mobileCarrierList.add(mobileCarrier);
                    }

                    rs.close();
                }
            }
        }
    }
    carrierReturnListDTO.setCarriers(mobileCarrierList);
    return carrierReturnListDTO;

}

Answer 1

您已启用SonarQube的NullCheck。如果在变量不能为null的位置使用签入，则会出现错误。在下面的文档中，这意味着您永远不会将null传递给getMobileCarriers（）。为了保证代码的健壮性，让该检查为空是一个好主意（因为您可能从其他地方使用该方法，甚至在某些时候从外部使用该方法）。因此，我建议查看SonarQube的设置，要么关闭此检查，要么修改它，使其不会跨方法调用执行此分析。