提高Java代码质量的Eclipse插件之Checkstyle的使用详解

CheckStyle是SourceForge下的一个项目，提供了一个帮助JAVA开发人员遵守某些编码规范的工具。它能够自动化代码规范检查过程，从而使得开发人员从这项重要但是枯燥的任务中解脱出来。

CheckStyle检验的主要内容
·Javadoc注释
·命名约定
·标题
·Import语句
·体积大小
·空白
·修饰符
·块
·代码问题
·类设计
·混合检查（包活一些有用的比如非必须的System.out和printstackTrace）

从上面可以看出，CheckStyle提供了大部分功能都是对于代码规范的检查，而没有提供象PMD和Jalopy那么多的增强代码质量和修改代码的功能。但是，对于团队开发，尤其是强调代码规范的公司来说，它的功能已经足够强大。

Checkstyle插件可以集成到Eclipse IDE中去，能确保Java代码遵循标准代码样式。

一、CheckStyle 安装方法

方法一：

1.Eclipse中，选择Help->Software Updates->Find and Install

2.选择 Search for new features to install 选择Next

3.选择 New Remote Site

4.输入更新地址：http://eclipse-cs.sourceforge.net/update

5. 安装后重启即可

方法二：

1.下载eclipse插件

http://sourceforge.net/projects/eclipse-cs/

下载后，把里面的两个文件夹plugins和 features下面的文件分别拷贝到eclipse目录下面对应的plugins和features目录

2.如果你就这样直接启动eclipse可能会出现ClassNotFoundExceptions 等错误（一般不会出现此类错误），所以必须在启动eclipse的时候加上一个参数 -clean. 这样eclipse就可以更新它当前安装的许多插件信息

比如你得eclipse在d盘，那么就是 d:\eclipse\eclipse.exe -clean

二、CheckStyle 使用

安装成功后，选中工程，右键选择checkstyle->check code with checkstyle,检查错误即可

三、Checkstyle的结果输出

检查后会发现代码中出现很多放大镜图标，如下图所示，这就是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+shift+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-z][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.

2、因为编译好的类没有在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变量名字重复。
解决方法：把方法里面的参数名称改变下就可以了public Foo(int newBar)
    {
        this.bar = newBar;
    }。

36 Got an exception - Unexpected character 0xfffd in identifier

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

网上参考解决方法：

1、Eclipse中可以配置，在Other-->checker中可以指定

2、可以修改checkstyle配置文件：

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

如果是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.

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

结果输出解释参考：http://www.cnblogs.com/bluesky4485/archive/2011/11/30/2269198.html

posted @ 2013-07-20 11:40 Alexia(minmin) 阅读(30413) 评论(2) 编辑收藏举报

会员力量，点亮园子希望

刷新页面返回顶部

程序媛想事儿（Alexia）

在现实的社会中还想追求小幸福的那个傻菇凉...

提高Java代码质量的Eclipse插件之Checkstyle的使用详解

公告