Xiaoven / codegex

A light-weight tools like spotbugs
GNU Lesser General Public License v2.1
1 stars 0 forks source link

implement RV_DONT_JUST_NULL_CHECK_READLINE #188

Closed zhouyinn closed 2 years ago

zhouyinn commented 2 years ago

code review :)

BTW https://github.com/renaud-vmg/findbugs/commit/95f24de27c9eb0b12115ca4ce89b190b1a9566e0 源代码有filter一个java/io/LineNumberReader不是很明白

Xiaoven commented 2 years ago

BTW renaud-vmg/findbugs@95f24de 源代码有filter一个java/io/LineNumberReader不是很明白

LineNumberReader 增加了下面两个功能:

常用的用法如下:

while ( (line = lineNumberReader.readLine()) != null ) {
         System.out.println(lineNumberReader.getLineNumber());
}

别的 reader 调用 readLine 方法,目的只能是想要用读取到的line的内容。但是LineNumberReader的目的完全可以是:我只想要行号,不关心line的具体内容是什么,调用 readLine 方法只是为了让 lineNumber++ 拿到正确的行号而已。

但是我们无法判断 reader 的类型,所以没有办法消除这种误报