IDEA学习篇——findbugs 插件 下载、安装、使用详解

你可以自己去下载最新的，也许我提供的已经不是最新的了呢？下载链接页面的网址：

https://plugins.jetbrains.com/plugin/3847-findbugs-idea

这个是安装完之后，重启之后的截图，看到他就在下面啦。可以使用啦。

你要是不想下载的话，或者页面打不开的话，可以使用如下，我的某度云盘的分享，

我的某度云盘的地址如下：

链接：http://pan.baidu.com/s/1bpKPzWF 密码：3z6h

下面是实际在项目里面使用的情况

简单说明如下：

自己鼠标移动到右边的所有图表，看看每个图标都是什么意思。我就大概说一下。

第一竖列的红色图标是分析当前的这个文件。

下面是分析是某一个类，鼠标所在的那个类

再往下是某个包，某个模块，整个项目，所有修改过的文件，等等吧，看意思都知道大概是啥意思。

第二数列呢，则是控制分析结果的展示情况，自己点点试试就好

下面就整个实际找到的代码的问题。我单独分析我的某个service文件，看代码里面的问题。如下图所示：

看他的意思就是说我代码里面把这个声明为静态属性，但是这个SimpleDateFormat是线程不安全的。

在多线程使用的时候，可能就会有意想不到的问题。所以，他就找到个问题啦。

然后，咱就改咯。

这么改完，之后，再分析以下 这个service文件。看看什么情况，但是在这之前，可能你有疑惑，怎么把代码里面这些黄色的提示给关掉。

点这个差号就 可以啦，代码就恢复成原来的样子啦。

下面看再次分析的结果。

附：一些常见的错误信息

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