官網解釋:
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元等您想捐的金額吧,狠狠點擊下面給點支持吧,站長非常感激您!手機微信長按不能支付解決辦法:請將微信支付二維碼保存到相冊,切換到微信,然后點擊微信右上角掃一掃功能,選擇支付二維碼完成支付。
【本文對您有幫助就好】元
