首页 > 代码库 > JAVA静态代码审查之checkstyle

JAVA静态代码审查之checkstyle

  技术总监来巡查,刚巧前段时间遇到了一个问题还没解决,就拉着大牛开问。结果,问题是解决了,还附带了另一个问题,或是要求出来,没啥技术含量,但是很麻烦的一个东西:代码格式。

  之前我写代码,因为屏幕比较小,所以就尽量一行写的不写两行,每一行都是一个逻辑片段,后期审阅时超级方便,但“这只是我自己的习惯而已”,可我觉着这么写,超级爽啊!但是,但是,但是,技术总监严厉要求,要按照公司自己的一套标准来写。

  后来我自己也想了下,我这么写的原因是,自己开发的笔记本的屏幕比较小,代码看不全所以才这么来的,可是,别人,特别是后期的运维人员在进行操作时,他们的机器情况跟我的完全不一样啊,万一他们的是90英寸超级大屏呢?想象一下,我的代码就漂浮在屏幕的最上边,想一条虫子,难看至极。

  哎,好吧,咱哥们还是按照公司的要求来吧,万一这个项目的接盘侠是个武力至上的人,那就等着被提刀满街追了。

  废话说了好多,就是心里不爽吐吐槽,现在,把一套checkstyle标准贴出来,以备后来装逼用。

  1 <?xml version="1.0" encoding="UTF-8"?>
  2 <!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.3//EN" "http://www.puppycrawl.com/dtds/configuration_1_3.dtd">
  3 
  4 <!--
  5     This configuration file was written by the eclipse-cs plugin configuration editor
  6 -->
  7 <!--
  8     Checkstyle-Configuration: Vastio Checks
  9     Description: none
 10 -->
 11 <module name="Checker">
 12   <property name="severity" value="http://www.mamicode.com/warning"/>
 13   <module name="TreeWalker">
 14     <module name="JavadocMethod">
 15       <property name="severity" value="http://www.mamicode.com/ignore"/>
 16       <property name="suppressLoadErrors" value="http://www.mamicode.com/true"/>
 17       <metadata name="net.sf.eclipsecs.core.lastEnabledSeverity" value="http://www.mamicode.com/inherit"/>
 18     </module>
 19     <module name="JavadocType">
 20       <property name="severity" value="http://www.mamicode.com/ignore"/>
 21       <metadata name="net.sf.eclipsecs.core.lastEnabledSeverity" value="http://www.mamicode.com/inherit"/>
 22     </module>
 23     <module name="JavadocVariable">
 24       <property name="severity" value="http://www.mamicode.com/ignore"/>
 25       <metadata name="net.sf.eclipsecs.core.lastEnabledSeverity" value="http://www.mamicode.com/inherit"/>
 26     </module>
 27     <module name="ConstantName">
 28       <property name="applyToPrivate" value="http://www.mamicode.com/false"/>
 29     </module>
 30     <module name="LocalFinalVariableName"/>
 31     <module name="LocalVariableName"/>
 32     <module name="MemberName"/>
 33     <module name="MethodName"/>
 34     <module name="PackageName"/>
 35     <module name="ParameterName"/>
 36     <module name="StaticVariableName"/>
 37     <module name="TypeName"/>
 38     <module name="AvoidStarImport"/>
 39     <module name="IllegalImport"/>
 40     <module name="RedundantImport"/>
 41     <module name="UnusedImports"/>
 42     <module name="MethodLength"/>
 43     <module name="ParameterNumber"/>
 44     <module name="LineLength">
 45       <property name="max" value="http://www.mamicode.com/160"/>
 46       <property name="tabWidth" value="http://www.mamicode.com/4"/>
 47     </module>
 48     <module name="EmptyForIteratorPad"/>
 49     <module name="MethodParamPad"/>
 50     <module name="NoWhitespaceAfter"/>
 51     <module name="NoWhitespaceBefore"/>
 52     <module name="OperatorWrap"/>
 53     <module name="ParenPad"/>
 54     <module name="TypecastParenPad"/>
 55     <module name="WhitespaceAfter"/>
 56     <module name="WhitespaceAround">
 57       <property name="tokens" value="http://www.mamicode.com/BAND,BAND_ASSIGN,BOR,BOR_ASSIGN,BSR,BSR_ASSIGN,BXOR,BXOR_ASSIGN,COLON,DIV,DIV_ASSIGN,EQUAL,GE,GT,LAND,LCURLY,LE,LITERAL_ASSERT,LITERAL_CATCH,LITERAL_DO,LITERAL_ELSE,LITERAL_FINALLY,LITERAL_FOR,LITERAL_IF,LITERAL_RETURN,LITERAL_SYNCHRONIZED,LITERAL_TRY,LITERAL_WHILE,LOR,LT,MINUS,MINUS_ASSIGN,MOD,MOD_ASSIGN,NOT_EQUAL,PLUS,PLUS_ASSIGN,QUESTION,RCURLY,SL,SLIST,SL_ASSIGN,SR,SR_ASSIGN,STAR,STAR_ASSIGN,LITERAL_ASSERT,TYPE_EXTENSION_AND,WILDCARD_TYPE"/>
 58     </module>
 59     <module name="ModifierOrder"/>
 60     <module name="RedundantModifier"/>
 61     <module name="AvoidNestedBlocks"/>
 62     <module name="EmptyBlock"/>
 63     <module name="LeftCurly"/>
 64     <module name="NeedBraces"/>
 65     <module name="RightCurly"/>
 66     <module name="AvoidInlineConditionals"/>
 67     <module name="EmptyStatement"/>
 68     <module name="EqualsHashCode"/>
 69     <module name="HiddenField">
 70       <property name="ignoreConstructorParameter" value="http://www.mamicode.com/true"/>
 71       <property name="ignoreSetter" value="http://www.mamicode.com/true"/>
 72     </module>
 73     <module name="IllegalInstantiation">
 74       <property name="classes" value="http://www.mamicode.com/java.lang.Boolean"/>
 75     </module>
 76     <module name="InnerAssignment"/>
 77     <module name="MissingSwitchDefault"/>
 78     <module name="RedundantThrows">
 79       <property name="severity" value="http://www.mamicode.com/ignore"/>
 80       <property name="suppressLoadErrors" value="http://www.mamicode.com/true"/>
 81       <metadata name="net.sf.eclipsecs.core.lastEnabledSeverity" value="http://www.mamicode.com/inherit"/>
 82     </module>
 83     <module name="SimplifyBooleanExpression"/>
 84     <module name="SimplifyBooleanReturn"/>
 85     <module name="FinalClass"/>
 86     <module name="HideUtilityClassConstructor"/>
 87     <module name="InterfaceIsType"/>
 88     <module name="VisibilityModifier"/>
 89     <module name="ArrayTypeStyle"/>
 90     <module name="TodoComment"/>
 91     <module name="UpperEll"/>
 92     <module name="OperatorWrap"/>
 93   </module>
 94   <module name="JavadocPackage">
 95     <property name="severity" value="http://www.mamicode.com/ignore"/>
 96     <metadata name="net.sf.eclipsecs.core.lastEnabledSeverity" value="http://www.mamicode.com/inherit"/>
 97   </module>
 98   <module name="NewlineAtEndOfFile"/>
 99   <module name="Translation"/>
100   <module name="FileLength"/>
101   <module name="FileTabCharacter">
102     <property name="severity" value="http://www.mamicode.com/ignore"/>
103     <metadata name="net.sf.eclipsecs.core.lastEnabledSeverity" value="http://www.mamicode.com/inherit"/>
104   </module>
105   <module name="RegexpSingleline">
106     <property name="format" value="http://www.mamicode.com/s+$"/>
107     <property name="message" value="http://www.mamicode.com/Line has trailing spaces."/>
108   </module>
109 </module>

  具体的代码含义懒得分析了,直接拿来用就ok了。

附上用法:

  Eclipse 或者 IntelliJ IDEA 等 IDE 中有对应的 Plugin。可搜索相应的 Plugin Market 安装。

  以 IntelliJ IDEA 为例。

  菜单选择 File>Setting,Plugins 选项卡中搜索 checkstyle 与 findbugs 进行安装。安装可能需要FQ,可也去IDEA官网的插件页面里去下载,然后手动添加到IDEA中去。

  菜单选择 File>Setting, Other Settings 中,会有安装后的Plugin的选项。

  CheckStyle 选项中新增 Configuration File, 选择 "Use a local CheckStyle file", File: 中选择本项目中的 checkstyle.xml 文件,Description 随意命名,比如 "Vastio Java Style"。

  将新增加的 Configuration File 设为 Active。

  最后,确认一下 Setting 中的 Editor:Inspections:CheckStyle 中的 Real-time scan 是打开的。

JAVA静态代码审查之checkstyle