[FindBugs分析记录]Redundant nullcheck of o,w

系统 1985 0

官网解释:

  This method contains a redundant check of a known non-null value against the constant null.

这种方法包含了一个称为非空对空值的不断重复检查。

什么代码会引起这个问题呢?先看下面:

      
        public
      
      
        static
      
      
        boolean
      
      
         isNull(Object o) {

        
      
      
        if
      
       (
      
        null
      
       ==
      
         o)

            
      
      
        return
      
      
        null
      
       ==
      
         o;

        
      
      
        if
      
       (o 
      
        instanceof
      
      
         String) {

            
      
      
        return
      
      
         StringUtils.isBlank((String) o);

        }

        return o == null;
      
      
        

    }
      
    

倒数第二行代码:return o == null;执行到这一步的时候已经表明o不为空,现在就是拿一个已知非空的值和null进行比较,findbus认为这是种运行期存在安全隐患的代码。

这里的建议是:防止这种情况出现,直接返回false.

      
        public
      
      
        static
      
      
        boolean
      
      
         isNull(Object o) {

        
      
      
        if
      
       (
      
        null
      
       ==
      
         o)

            
      
      
        return
      
      
        null
      
       ==
      
         o;

        
      
      
        if
      
       (o 
      
        instanceof
      
      
         String) {

            
      
      
        return
      
      
         StringUtils.isBlank((String) o);

        }

        
      
      
        return
      
      
        false
      
      
        ;

    }
      
    

 

[FindBugs分析记录]Redundant nullcheck of o,which is known to be non-null


更多文章、技术交流、商务合作、联系博主

微信扫码或搜索:z360901061

微信扫一扫加我为好友

QQ号联系: 360901061

您的支持是博主写作最大的动力,如果您喜欢我的文章,感觉我的文章对您有帮助,请用微信扫描下面二维码支持博主2元、5元、10元、20元等您想捐的金额吧,狠狠点击下面给点支持吧,站长非常感激您!手机微信长按不能支付解决办法:请将微信支付二维码保存到相册,切换到微信,然后点击微信右上角扫一扫功能,选择支付二维码完成支付。

【本文对您有帮助就好】

您的支持是博主写作最大的动力,如果您喜欢我的文章,感觉我的文章对您有帮助,请用微信扫描上面二维码支持博主2元、5元、10元、自定义金额等您想捐的金额吧,站长会非常 感谢您的哦!!!

发表我的评论
最新评论 总共0条评论