title: Findbugs、checkStyle、PMD常见错误总结

2019-05-12 10:46:01

记录的是在软件工程基础实验中使用到的代码审计工具(基于Eclipse)的一些能够审计到的错误类型，大部分来源于网络，在这里归纳，方便以后查看。这里用到的工具主要是checkstyle、findbugs、PMD，同时在实践中实际使用工具进行代码评审所发现的问题通过图片展示出来。

CheckStyle:

1 Type is missing a javadoc commentClass

  缺少类型说明

2 “{” should be on the previous line

  “{” 应该位于前一行。解决方法：把“{”放到上一行去

3 Methos is missing a javadoc comment

  方法前面缺少javadoc注释。解决方法：添加javadoc注释，类似下述这样：

set default mock parameter.（方法说明）
@param additionalParameters parameter additional(参数名称)
@return data manager(返回值说明)
@throws Exception if has error(异常说明)

4 Expected @throws tag for “Exception”

在注释中希望有@throws的说明

解决方法：在方法前得注释中添加这样一行：* @throws Exception if has error(异常说明)

5 “.” Is preceeded with whitespace “.”

前面不能有空格。解决方法：把“.”前面的空格去掉

6 “.” Is followed by whitespace“.”

后面不能有空格。解决方法：把“.”后面的空格去掉

7 “=” is not preceeded with whitespace

“=” 前面缺少空格。解决方法：在“=”前面加个空格

8“=” is not followed with whitespace“=”

后面缺少空格。解决方法：在“=”后面加个空格

9“}” should be on the same line“}”

应该与下条语句位于同一行。解决方法：把“}”放到下一行的前面

10 Unused @param tag for “unused”

没有参数“unused”，不需注释

解决方法：“* @param unused parameter additional(参数名称)” 把这行unused参数的注释去掉“

11 Variable “CA” missing javadoc

变量“CA”缺少javadoc注释

解决方法：在“CA“变量前添加javadoc注释：/** CA. */（注意：一定记得加上“.”）

12 Line longer than 80characters

行长度超过80  。解决方法：把它分成多行写。必要时候，可以ctrl+shif+f

13 Line contains a tab character

行含有”tab” 字符。

14 Redundant “Public” modifier

冗余的“public” 修饰符 。解决方法：去掉“public”修饰符

15 Final modifier out of order with the JSL suggestion

Final modifier的顺序错误

16 Avoid using the “.*” form of import

import格式避免使用“.*”。解决方法：可以先不添加import，再需要用到时直接“ctrl + o”导入即可。

17 Redundant import from the same package

从同一个包中Import内容，多余，删掉即可

18 Unused import-java.util.list

Import进来的java.util.list没有被使用。解决方法：去掉导入的多余的类

19 Duplicate import to line 13

重复Import同一个内容。解决方法：去掉导入的多余的类

20 Import from illegal package

从非法包中 Import内容

21 “while” construct must use “{}”

“while” 语句缺少“{}”

22 Variable “sTest1” must be private and have accessor method

变量“sTest1”应该是private的，并且有调用它的方法

23 Variable “ABC” must match pattern “¹[a-zA-Z0-9]*$”

变量“ABC”不符合命名规则“^[a-z][a-zA-Z0-9]*$”，

解决方法：把这个命名改成符合规则的命名 “aBC”

24 “(” is followed by whitespace

“(” 后面不能有空格 25“)”is proceeded by whitespace“)” 前面不能有空格。

解决方法：把前面或者后面的空格去掉

25 First sentence should end with a period

解决方法：你的注释的第一行文字结束应该加上一个"."。

26 Redundant throws: ‘NameNotFoundException’ is subclass of ‘NamingException’.

'NameNotFoundException '是'NamingException'的子类重复抛出异常。

解决方法：如果抛出两个异常，一个异常类是另一个的子类，那么只需要写父类
去掉NameNotFoundException异常，对应的javadoc注释异常注释说明也需要去掉

27 Parameter docType should be final.

参数docType应该为final类型， 解决方法：在参数docType前面加个final

28 Line has trailing spaces.

多余的空行 ，解决方法：去掉这行空行

29 Must have at least one statement.

至少一个声明

解决方法：} catch (NumberFormatException nfe) {
LOG.error(“Auto Renews the agreement failed”, nfe);//异常捕捉里面不能为空，在异常里面加一句话。如打印等等

30 ‘>’ is not followed by whitespace.并且又有 ‘(’ is preceded with whitespace.

定义集合和枚举的时候的时候，最后一个“>”后面要有空格，“（”前面不容许有空格。
解决方法：去掉泛型

31 Got an exception - java.lang.RuntimeException: Unable to get class information for @throws tag ‘SystemException’.

原因：不合理的throws。

解决方法：要确保某些类型，如某些类、接口不被throws。把声明的异常去掉。在实现类中抛出异常

1. 这是CheckStyle报的错。通常需要Refreh, clean/build这个Project。
  如果不行，可以尝试clean all projects, restart Eclipse.
1. 因为编译好的类没有在checkstyle的classpath中。所以, 只要将编译好的class配置到在checkstyle/的classpath中就没有这个问题了。另外, 还发现checkstyle的line length好像也有点问题, 明明没有超过120个字符, 却还是报错.无奈, 我把Eclipse中java > code style > formatter中的Maximum line with改成了100, 然后format一下, 基本就没有问题了。

32 File does not end with a newline.

解决方法：删掉报错的类，新建一个同名的类，把代码全部复制过去

33 Utility classes should not have a public or default constructor.

接口中的内部类中不应该有公共的或者默认的构造方法

解决方法：在内部类中，定义一个私有的构造方法，然后内部类声明为final类型。如果前面有static，那么final还必须放在static之后

34 Variable ‘functionCode’ must be private and have accessor methods.

变量要改成private然后提供访问的方法

解决方法：给这些变量的修饰符改成private，然后提供set，get方法，并加上对应的方法javadoc注释、参数注释。并在返回值和参数类型前添加final。并把调用了这个变量的地方改成通过方法访问

35 ‘X’ hides a field.

public class Foo
{
    private int bar;

    public Foo(int bar)
    {
        this.bar = bar;
    }

    public final int getBar()
    {
        return bar;
    }
}

全局private int bar;和局部public Foo(int bar)的bar变量名字重复。
解决方法：把方法里面的参数名称改变下就可以了。

36 Got an exception - Unexpected character 0xfffd in identifier

这是因为CheckStyle不能识别制定的编码格式。

网上参考解决方法：

1. Eclipse中可以配置，在Other–>checker中可以指定
1. 可以修改checkstyle配置文件：

<module name="Checker">

<property name="severity" value="warning"/>

<property name="charset" value="UTF-8"/>

<module name="TreeWalker">

如果是UTF-8的话，就添加加粗斜体的那条语句，就可以了。

37 Got an exception - java.lang.RuntimeException: Unable to get class information for @throws tag whatever.

解决方法：选中CheckSytle的JavaDoc --> Method JavaDoc --> logLoadErrors。如果是CheckStyle自己加载时出错的，打个Log就可以了，不要整出Errors吓人。还有一处也可能包出同样的错误。Coding Problems --> Redundant Throws --> logLoadErrors选中即可

38 Expected @param tag for ‘dataManager’.

 缺少dataManager参数的注释

解决方法：在注释中添加@param dataManager DataManager

39 Parameter X should be final.

public class Foo
{
    private int bar;

    public Foo(int bar)
    {
        this.bar = bar;
    }

    public final int getBar()
    {
        return bar;
    }
}

解释：public Foo(int bar)的局部变量，被认为是不可改变的，检查需要加上final关键字定义public Foo(final int bar)此错误，可以忽略不检查。

40 Redundant ‘X’ modifier.

public interface CacheHRTreeService extends Manager {

/**
  * Organization Tree
  * @param orgDto
  * @return
  * @throws Exception
  */
public void setOrganization(OrganizationDTO orgDto) throws Exception;

/**
  * Organization Tree
  * @return
  * @throws Exception
  */
public OrganizationDTO getOrganization() throws Exception;

解释：多余的字段。public OrganizationDTO getOrganization() throws Exception;此时public为多余的字段，因为interface定义的时候，就是public的,需要检查。

41 Class X should be declared as final.

解释：对于单例设计模式，要求返回唯一的类对象。但是HRFactory和ContextFactory为优化的两个类，不需求检查,其他的单例类，依然需要进行检查。

42 Method ‘addChildrenId’ is not designed for extension - needs to be abstract, final or empty.

解释：通过父类继承的，此类有点特殊可以忽略此类。

43 Variable ‘id’ must be private and have accessor methods.

解释：BaseHRDTO类，为父类，属性给子类继承，比较特殊。但是其他的类，声名需要加上范围’private’关键字。需要检查。

44 Array brackets at illegal position.

解释：代码写法，习惯不一样。需要检查，仅仅提示。

Findbugs:

FindBugs找出的bug有3中颜色:
1. 黑色的臭虫标志是分类
2. 红色的臭虫表示严重bug发现后必须修改代码
3. 橘黄色的臭虫表示潜在警告性bug 尽量修改
每次使用Find Bugs（文件，文件夹上右键--> Find Bugs-->Find Bugs）命令时，如果改动了代码，需要先Build Project;
修改完相关bug之后，在文件上右键-Find Bugs-->Clear Bug Marker清除Find Bugs的标志

PMD：

1. 潜在的bug：空的try/catch/finally/switch语句
2. 未使用的代码：未使用的局部变量、参数、私有方法等
3. 可选的代码：String/StringBuffer的滥用
4. 复杂的表达式：不必须的if语句、可以使用while循环完成的for循环
5. 重复的代码：拷贝/粘贴代码意味着拷贝/粘贴bugs
6. 循环体创建新对象：尽量不要再for或while循环体内实例化一个新对象
7. 资源关闭：Connect，Result，Statement等使用之后确保关闭掉

a-z ↩︎

柳妹要坚持

发布了25 篇原创文章 · 获赞 42 · 访问量 8982

私信关注

Findbugs、checkStyle、PMD常见错误总结