【问题标题】:Handle null condition (In sonar Change this condition so that it does not always evaluate to "false")处理空条件(在声纳中更改此条件,使其不总是评估为“假”)
【发布时间】:2016-11-25 23:47:37
【问题描述】:
public boolean validateEmployeeRequestForm(String empCode) {
           if ( null == empCode ) {
                return false;
            }
           return true;
    }

在第二行if ( null == empCode ) 它给了我Sonar 下面的异常。

更改此条件,使其不会总是评估为“假”。

如何处理?

【问题讨论】:

  • 如何/在哪里调用该方法? 'validateEmployeeRequestForm' 可能永远不会将'null' 作为参数?你考虑过空字符串吗?
  • 你在调用它之前是否真的为该参数设置了一个值?这是您正在评估的实际代码,还是只是“看起来像”?
  • 我认为更大的问题是如果 if 条件失败,该方法不会返回 any 值。
  • @TimBiegeleisen 可能看起来如此,但不是。这段代码不会编译,所以声纳不太可能看到它
  • @Stultuske 是的,我希望这甚至无法编译。

标签: java sonarqube


【解决方案1】:
public boolean validateEmployeeRequestForm(String empCode) {
       return empCode != null;
}

【讨论】:

    猜你喜欢
    • 2017-06-12
    • 2018-05-07
    • 2018-08-01
    • 2021-01-05
    • 2018-02-10
    • 2018-08-17
    • 2016-05-14
    • 2020-01-11
    • 2017-02-02
    相关资源
    最近更新 更多