最近经常思考团队开发的一些东西,其中代码风格不统一是最常见的问题之一。按理说,大家协商和沟通一下,风格统一一下就可以了,其实不然,因为这是个个性张扬的时代!
工作code review中用了CheckStyle小半年了,觉得很好很强大很方便,大家写出来的代码就像一个人写出来的一样,值此辞旧迎新,特简做说明,与尔同飨。
1.简介
官方网站:http://checkstyle.sourceforge.net/
CheckStyle提供了一个帮助JAVA开发人员遵守某些编码规范的工具。它能够自动化代码规范检查过程,从而使得开发人员从这项重要,但是枯燥的任务中解脱出来。
CheckStyle检验的主要内容
(1). Javadoc注释
(2). 命名约定
(3). 标题
(4). Import语句
(5). 体积大小
(6). 空白
(7). 修饰符
(8). 块
(9). 代码问题
(10). 类设计
(11). 混合检查(包活一些有用的比如非必须的System.out和printstackTrace)
2.定制
官方提供的代码规范往往太过严格,在工作中使用不太现实,所以有必要根据具体情况来定制具体的代码规范,CheckStyle对代码规范的定制提供了很多大灵活性。
下面我们来定义一些基本的规范,后续有增加我们再修改。
(1). 不要tab键;
(2). 避免重复的import, 多余的import和import *
(3). 常量全部大写字母(static final);
(4). 成员变量以m开头;
(5). 当有多重修饰符时,修饰符采用以下顺序:(public,protected,private,abstract,static,final,transient,volatile,synchronized,native,strictfp)
... ...
其他的我们后面慢慢的修改。
3.配置文件
在运行checkstyle时,需要一个参数(注:我的ubuntu服务器上安装的checkstyle5.4版本,最新的是5.5版本):
这个配置文件就是用来定义你自己定制的代码规范,你可以参考官方说明:http://checkstyle.sourceforge.net/availablechecks.html
这里,我们根据上面说的,来写这样一个code_check.xml :
<?xml version="1.0"?> <!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.2//EN" "http://www.puppycrawl.com/dtds/configuration_1_2.dtd"> <module name="Checker"> <!-- 检查文件是否以一个新行结束--> <module name="NewlineAtEndOfFile"/> <!-- 检查文件中是否含有tab键--> <module name="FileTabCharacter"/> <module name="TreeWalker"> <!-- 常量全部用大写--> <module name="ConstantName"/> <!-- 避免.*,重复多余的和不使用的import--> <module name="AvoidStarImport"/> <module name="RedundantImport"/> <module name="UnusedImports"/> <!-- 成员变量格式为:m+大写+字母--> <module name="MemberName"> <property name="format" value="^m[A-Z][a-zA-Z0-9]*$"/> </module> <!-- 检查代码块:起始大括号和if等同行,不能有空的代码块,结束大括号另起一行--> <module name="LeftCurly"/> <module name="NeedBraces"/> <module name="RightCurly"/> <!-- 当有多重修饰符时,修饰符采用以下顺序: (public,protected,private,abstract,static,final, transient,volatile,synchronized,native,strictfp) --> <module name="ModifierOrder"/> </module> </module>
更多检查规范中文说明参考园子里地址: http://www.cnblogs.com/liugang/archive/2010/10/26/1860903.html
4.使用checkstyle
插件工具的使用我就不介绍了,我只想起个抛砖引玉的作用,checkstyle命令非常简单,这里我只说明3个参数:-c, -r, -o。
/******************* ***CheckStyle Usage** ******************/ /*** *参数 *1. -c 配置文件,并验证文件 *2. -o 输出结果 *3. -r 遍历目录 **/ //用我们定制的code_checks.xml检查文件 $>checkstyle -c ~/GitProj/world/code_checks.xml ~/GitProj/world/floworld/src/com/tianxia/app/floworld/appreciate/AppreciateLatestActivity.java //用我们定制的code_checks.xml检查目录下的所有源文件,并把结果输出到result.txt中 $>checkstyle -c ~/GitProj/world/code_checks.xml -r ~/GitProj/world/floworld/src/com/tianxia/app/floworld/appreciate/
我们打开result.txt来看下输出结果:
又是tab又是命名不规范,非常的准确。
5.与Git挂钩
快12点了,我先把文章发表出来,这部分随后我再补写。