Android学习系列(24)--App代码规范之使用CheckStyle

十度 Android 2015年12月01日 收藏

最近经常思考团队开发的一些东西,其中代码风格不统一是最常见的问题之一。按理说,大家协商和沟通一下,风格统一一下就可以了,其实不然,因为这是个个性张扬的时代!
工作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 :

  1. <?xml version="1.0"?>
  2. <!DOCTYPE module PUBLIC
  3. "-//Puppy Crawl//DTD Check Configuration 1.2//EN"
  4. "http://www.puppycrawl.com/dtds/configuration_1_2.dtd">
  5.  
  6. <module name="Checker">
  7.  
  8. <!-- 检查文件是否以一个新行结束-->
  9. <module name="NewlineAtEndOfFile"/>
  10. <!-- 检查文件中是否含有tab键-->
  11. <module name="FileTabCharacter"/>
  12. <module name="TreeWalker">
  13.  
  14. <!-- 常量全部用大写-->
  15. <module name="ConstantName"/>
  16.  
  17. <!-- 避免.*,重复多余的和不使用的import-->
  18. <module name="AvoidStarImport"/>
  19. <module name="RedundantImport"/>
  20. <module name="UnusedImports"/>
  21. <!-- 成员变量格式为:m+大写+字母-->
  22. <module name="MemberName">
  23. <property name="format" value="^m[A-Z][a-zA-Z0-9]*$"/>
  24. </module>
  25.  
  26. <!-- 检查代码块:起始大括号和if等同行,不能有空的代码块,结束大括号另起一行-->
  27. <module name="LeftCurly"/>
  28. <module name="NeedBraces"/>
  29. <module name="RightCurly"/>
  30. <!-- 当有多重修饰符时,修饰符采用以下顺序:
  31. (public,protected,private,abstract,static,final,
  32. transient,volatile,synchronized,native,strictfp) -->
  33. <module name="ModifierOrder"/>
  34. </module>
  35.  
  36. </module>

更多检查规范中文说明参考园子里地址: http://www.cnblogs.com/liugang/archive/2010/10/26/1860903.html

4.使用checkstyle
插件工具的使用我就不介绍了,我只想起个抛砖引玉的作用,checkstyle命令非常简单,这里我只说明3个参数:-c, -r, -o。

  1. /*******************
  2. ***CheckStyle Usage**
  3. ******************/
  4. /***
  5. *参数
  6. *1. -c 配置文件,并验证文件
  7. *2. -o 输出结果
  8. *3. -r 遍历目录
  9. **/
  10.  
  11. //用我们定制的code_checks.xml检查文件
  12. $>checkstyle -c ~/GitProj/world/code_checks.xml
  13. ~/GitProj/world/floworld/src/com/tianxia/app/floworld/appreciate/AppreciateLatestActivity.java
  14.  
  15. //用我们定制的code_checks.xml检查目录下的所有源文件,并把结果输出到result.txt中
  16. $>checkstyle -c ~/GitProj/world/code_checks.xml
  17. -r ~/GitProj/world/floworld/src/com/tianxia/app/floworld/appreciate/

我们打开result.txt来看下输出结果:

又是tab又是命名不规范,非常的准确。

5.与Git挂钩
快12点了,我先把文章发表出来,这部分随后我再补写。