FindBugs-IDEA 插件的使用

转载请注明出处:http://blog.csdn.net/feibendexiaoma/article/details/72821781
前言
Findbugs 很多人都并不陌生,Eclipse 中有插件可以帮助查找代码中隐藏的 bug,IDEA 中也有这款插件。这个插件可以帮助我们查找隐藏的 bug, 比较重要的功能就是查找潜在的 null 指针。
在编写代码的过程中, 我们可能不会一直记得检查空的引用, 在我们测试时可能很难发现问题, 但是应用上线之后,面对大量的用户,很多问题就会浮现出来。所以在编码时,使用 findbugs 检查一下很有必要。

安装
findbugs_01
findbugs_02

findbugs_03

安装完之后,重启 studio, 会发现左下角会出现 findbugs 的图标
findbugs_04

可以分析单个文件,包下面的所有文件,整个 module 下的文件,整个 project 下的文件,右键想要分析的文件名 / 包名 /module 名 /project

findbugs_05

分析完之后就会出现结果面板

findbugs_06

点击对应的 item 在右边会定位到具体的代码

findbugs_07

根据需要可以进行更改,其中 Correctness 这个错误使我们重点关注的对象,这里大多是空指针的错误,根据提示进行处理。

附:一些常见的错误信息
Bad practice 代码中的一些坏习惯

Class names should start with an upper case letter 主要包括类名的命名,以大写字母开头
Method names should start with a lower case letter 方法名以小写字母开头
Field names should start with a lower case letter 字段名以小写字母开头
equals()method does not check for null argument equals() 方法应该检查非空
Class defines equals()and uses Object.hashCode() 一个类覆写了 equals 方法,没有覆写 hashCode 方法,使用了 Object 对象的 hashCode 方法
Method ignores exceptional return value 方法忽略返回值的异常信息
Equals method should not assume anything about the type of its argument equals(Object o) 方法不能对参数 o 的类型做任何的假设。比较此对象与指定的对象。当且仅当该参数不为 null,并且是表示与此对象相同的类型的对象时,结果才为 true。
Comparison of String objects using == or != 用 == 或者!= 去比较 String 类型的对象
Method might ignore exception 方法可能忽略异常
Method invokes System.exit()在方法中调用 System.exit(…) 语句,考虑用 RuntimeException 来代替
Method ignores result of InputStream.read() InputStream.read 方法忽略返回的多个字符,如果对结果没有检查就没法正确处理用户读取少量字符请求的情况。
Dodgy code 糟糕的代码

Switch statement found where default case is missing Switch 没有默认情况下执行的 case 语句
Switch statement found where one case falls through to the next case Switch 语句中一个分支执行后又执行了下一个分支。通常 case 后面要跟 break 或者 return 语句来跳出。
Dead store to local variable 该指令为局部变量赋值,但在其后的没有对她做任何使用。通常,这表明一个错误,因为值从未使用过。
Write to static field from instance method 在实例方法写入静态字段
Redundant nullcheck of value known to be non-null 方法中对不为空的值进行为空的判断。
Method uses the same code for two branches 此方法使用相同的代码,以实现两个有条件的分支。检查以确保这是不是一个编码错误
Exception is caught when Exception is not thrown 在 try/catch 块中捕获异常,但是异常没有在 try 语句中抛出而 RuntimeException 又没有明确的被捕获
Integral division result cast to double or float 整形数除法强制转换为 double 或者 float 类型。
Possible null pointer dereference due to return value of called method 方法的返回值没有进行是否为空的检查就重新赋值,这样可能会出现空指针异常。
Useless object created 对象创建了并没有用
Unread public/protected field 没有用到的字段
Internationalization 关于代码国际化相关方面的

Consider using Locale parameterized version of invoked method
使用平台默认的编码格式对字符串进行大小写转换,这可能导致国际字符的转换不当。使用以下方式对字符进行转换
Performance 关于代码性能相关方面的

Boxing/unboxing to parse a primitive 类型转换 比如字符串转换成 int 应该使用 Integer.parseInt(“”) 代替 Integer.valueOf(“”)
Method concatenates string using + in aloop
每次循环里的字符串 + 连接,都会新产生一个 string 对象,在 java 中,新建一个对象的代价是很昂贵的,特别是在循环语句中,效率较低
解决办法:使用 StringBuffer 或者 StringBuilder 重用对象。
Private method is never called 私有方法没有被调用
Explicit garbage collection;extremely dubious except in benchmarking code
在代码中显式的调用垃圾回收命名,这样做并不能起作用。在过去,有人在关闭操作或者 finalize 方法中调用垃圾回收方法导致了很多的性能浪费。这样大规模回收对象时会造成处理器运行缓慢。
Unread field:should this field be static? 没有用到的 static 字段
should be a static inner class 此内部类应该使用 static 修饰
Experimental

Method may fail to clean up stream or resource on checked exception
这种方法可能无法清除(关闭,处置)一个流,数据库对象,或其他资源需要一个明确的清理行动
解决方法:流的关闭都写在 finally 里面
Malicious code vulnerability 关于恶意破坏代码相关方面的

May expose internal representation by incorporating reference to mutable object
此代码把外部可变对象引用存储到对象的内部表示。如果实例受到不信任的代码的访问和没有检查的变化危及对象和重要属性的安全。存储一个对象的副本,在很多情况下是更好的办法。
Field isn’t final but should be 此字段前应该加 final
Field isn’t final and can’t be protected from malicious code 此字段前应该加 final
Field should be package protected
一个静态字段是可以被恶意代码或其他的包访问修改。可以把这种类型的字段声明为 final 类型的以防止这种错误。
Multithreaded correctness 关于代码正确性相关方面的

Static DateFormat DateFormat 在多线程中本身就是不安全的,如果在线程范围中共享一个 DateFormat 的实例而不使用一个同步的方法在应用中就会出现一些奇怪的行为。
Call to static DateFormat DateFormats 多线程使用本事就是不安全的, 改进方法:需要创建多实例或线程同步
Correctness 关于代码正确性相关方面的
Nullcheck of value previously dereferenced 此代码之前废弃 null 值检查。解决办法 进行 null 检查
Possible null pointer dereference 可能为 null
Null pointer dereference 对象赋为 null 值后 没有被重新赋值
Possible null pointer dereference in method on exception path 在异常 null 值处理分支调用的方法上,可能存在对象去除引用操作
value is null and guaranteed to be dereferenced on exception path exception 分支上,存在引用一个 null 对象的方法,引发空指针异常。
Self comparison of value with itself 方法中对一个局部变量自身进行比较运算,并可说明错误或逻辑错误。请确保您是比较正确的事情。
An apparent infinite recursive loop 明显的无限迭代循环, 将导致堆栈溢出.


作者:飞奔的小付
来源:CSDN
原文:https://blog.csdn.net/feibendexiaoma/article/details/72821781
版权声明:本文为博主原创文章,转载请附上博文链接!