1. 程式人生 > >玩轉Eclipse — 自動程式碼規範檢查工具Checkstyle

玩轉Eclipse — 自動程式碼規範檢查工具Checkstyle

大專案都需要小組中的多人共同完成,但是每個人都有自己的編碼習慣,甚至很多都是不正確的。那麼如何使小組所有開發人員都遵循某些編碼規範,以保證專案程式碼風格的一致性呢?如果硬性地要求每個開發人員在提交程式碼之前,都要對照的編碼規範將自己的程式碼檢查一遍,將是一個非常枯燥而且耗時的任務。Checkstyle是一個開原始碼分析工具,能夠幫助開發人員保證他們的程式碼遵循一定的程式碼規範。Checkstyle通過不斷地檢查你的程式碼,一旦發現有違反定義的程式碼規範的地方就立馬提示,以便開發人員能夠及時發現和修改不規範程式碼。Checkstyle在Eclipse中的外掛是eclipse-cs

1. 安裝Checkstyle外掛

        Eclipse選單欄上選擇 Help -> Install New Software...,進入如下安裝介面:

        點選“Add...”開啟Add Repository對話方塊,如圖輸入Name和Location,點選“OK”。         選擇剛剛新增的JAutodoc,一直點選“Next >”直到安裝結束。

2. 配置Checkstyle

        Eclipse選單欄上選擇 Window -> Preferences -> Checkstyle,進入如下設定介面:

        在Global Check Configurations框中,列出可以選擇的檢查配置(Check Configuration),Default欄標記為對勾的預設配置。

        如果想自己配置檢查選擇,可以點選“New...”按鈕新建一個配置,進入如下Check Configuration介面:

        檢查配置的型別,可以有多種選擇。Internal Configuration型別是檢查配置存在於Eclipse內部;External Configuration型別相當於匯入一個已有的xml配置檔案。Checkstyle配置都是xml檔案。

        如果想修改已有的配置,可以點選“Configure...”按鈕,開啟Checkstyle Configuration對話方塊,根據需要進行修改。

3. 使用Checkstyle

        Checkstyle會在程式碼開發過程中,不斷地檢查程式碼規範,一般檢查的內容包括:

  • Javadoc註釋
  • 命名約定
  • 標題
  • Import
  • 大小
  • 空白
  • 修飾符
  • 程式碼
  • 類設計

        下面這段程式碼經過Checkstyle檢查之後,不符合規範的程式碼,底色都會被標記為淺黃色進行提示。點選行頭的提示標記,就會顯示具體不符合規範的地方。示例如下圖所示:

Reference

為便於規範各位開發人員程式碼、提高程式碼質量,研發中心需要啟動程式碼評審機制。為了加快程式碼評審的速度,減少不必要的時間,可以加入一些程式碼評審的靜態檢查工具,另外需要為研發中心配置統一的編碼模板和程式碼格式化模板。

1、配置統一的編碼模板

1.1 配置編碼模板

  在Eclipse或MyEclipse中點選Window -> Preferences選單,點選左側的“Java”->“Code Style”->“Code Templates”,介面如下圖所示:      

         點選上圖右側的“Import”按鈕,在彈出的檔案選擇視窗選擇公司自己的編碼模板,例如eclipse_templates.xml檔案(僅提供參考,可自行修改)。在“Configure generated code and comments”區域有“Comments”和“Code”兩個選單,點開後可以看到各種型別的註釋和編碼模板定義:               

         可以點選上面的各種型別檢視該模板檔案的定義。

例如檔案註釋定義:     

檔案註釋定義中的檔案作者取自所在系統登入使用者,若不正確時,可點選“Edit”按鈕後進行修改。

選擇編碼模板檔案完成後點選下方的“Apply”按鈕應用該模板檔案,而後點選“OK”按鈕完成設定。

1.2 使用編碼模板

1.2.1 建立類時自動添加註釋

         在建立類時,勾選如下頁面的“Generate comments”選項,將會使新建立的類具有檔案註釋和類註釋:            

         生成的類的內容參考如下:

/* * XX公司原始碼,版權歸XX公司所有。 *  * 專案名稱 : XXXXXXX * 建立日期 : May 9, 2014 * 修改歷史 :  *     1. [May 9, 2014]建立檔案 by Xingxing,Xie */package cn.amigo.eip.common;/** * //TODO 新增類/介面功能描述 * @author Xingxing,Xie */public class CommentTest {     /**     * //TODO 新增方法功能描述     * @param args     */    public static void main(String[] args) {        // TODO Auto-generated method stub    }}

1.2.2 對已有類添加註釋

         對已有的類、方法添加註釋的方法:

   1) 選擇類的類名、方法名所在的行後,使用“ALT + SHIFT + J”快捷鍵;

   2) 在所在類或方法上輸入“/**”後按Enter鍵。

2、配置統一的程式碼格式化模板

2.1 配置程式碼格式化模板

在Eclipse或MyEclipse中點選Window -> Preferences選單,點選左側的“Java”->“Code Style”->“Formatter”,介面如下圖所示:          

點選上圖的“Import”按鈕,彈出格式化模板檔案選擇框,選擇程式碼格式化檔案,例如eclipse_formatter-profile.xml檔案(僅提供參考,可自行定義)。點選下方的“Apply”按鈕應用該模板檔案,而後點選“OK”按鈕完成設定。

在格式化檔案中定義了很多Java格式化規範,例如:

<setting id="org.eclipse.jdt.core.formatter.lineSplit" value="150"/>

上行用於設定格式化時在一行達到150個字元(包括空格)時需要換行。

又如:

<setting id="org.eclipse.jdt.core.formatter.tabulation.size" value="4"/>

上行表示格式化時使用4個空格代替tab鍵。

有興趣的人可以檢視《eip_eclipse_formatter-profile.xml》檔案的內容。

2.2 配置使用4個空格代替tab鍵

tab鍵在不同的IDE、編輯器(例如EditPlus、UltraEdit等)中使用的空格數不同,因此為了統一,開發人員統一使用4個空格代替tab鍵(checkstyle會配置做相應檢查)。

在Eclipse或MyEclipse中點選Window -> Preferences選單,點選左側的“General”->“Editors”->“Text Editors”,介面如下圖所示:       

         在上圖中勾選“Insert spaces for tabs”(使用空格代替tab鍵),並設定“Displayed tab width”為4(即使用4個空格代替tab鍵)。

2.3 去掉對程式碼註釋的格式化

         在使用“2.4 使用模板進行格式化”進行程式碼格式化時,會將程式碼註釋也進行格式化,引起格式錯亂,為避免其對註釋進行格式化,採用的方法為:點選“Java”->“Code Style”-> “Formatter”選單,在右側選中模板後點擊“Edit”按鈕,在彈出的介面中選擇“Comments”的tab,如下圖所示:       

         在上圖中對紅色區域的複選框取消勾選。

2.4 使用模板進行格式化

         若要對某個檔案進行格式化,可使用如下幾種方法:

1) 開啟檔案後,在程式碼區域採用“CTRL + SHIFT + F”快捷鍵;

2) 開啟檔案後,在程式碼區域點選右鍵選擇“Source”->“Format”功能。

    【說明】建議採用“CTRL + Shift + F”快捷鍵。

3、配置checkstyle外掛進行程式碼檢查

3.1 安裝Eclipse的checkstyle外掛

可以從sourceforge網站下載外掛,網址如下:

         該頁面有3.3.1 ~ 5.7.0版本的checkstyle外掛,因為5.0以上版本的checkstyle外掛在筆者的MyEclipse 6.5版本上無法使用,所以下載的4.4.2版本,高版本的MyEclipse可嘗試新版本的checkstyle試試是否能成功使用。

安裝checkstyle外掛的方法主要有如下三種:

一、在Eclipse下使用link方式連結外掛

在某個目錄(eg. D:\Program Files (x86)\externalplugins)建立checkstyle\eclipse子目錄,將之前下載的外掛下的features和plugins兩個檔案下拷貝到該目錄下(eg: D:\Program Files (x86)\externalplugins\checkstyle\eclipse)。

接著在eclipse的links目錄下新建checkstyle.xml檔案,在該檔案中新增一行寫上需要連結的外掛地址,例如:path=D:/Program Files (x86)/externalplugins/checkstyle。

【說明】此種方式便於管理外部Eclipse外掛。

二、下載checkstyle外掛資料夾後拷貝到Eclipse的和features和plugins目錄

此種方式將下載的checkstyle外掛的features目錄下的內容拷貝到Eclipse下的features目錄,將將下載的checkstyle外掛的plugins目錄下的內容拷貝到Eclipse下的plugins目錄。

三、使用 “Software Updates”->“Find and install”指定網址安裝外掛。

點選Eclipse或MyEclipse的“Help”->“Software Updates”->“Find and Install”,在彈出的視窗中選擇“Search for new features to install”單選框,接著在“Update sites to visit”視窗點選“New Remote Site”按鈕,在該視窗輸入外掛名稱和外掛下載地址後點擊“OK”按鈕完成下載。

3.2 設定checkstyle配置檔案

         Checkstyle外掛安裝完成後,需要配置研發中心的checkstyle配置檔案,配置檔案參考《eclipse_checkstyle.xml》檔案,該檔案定義了研發中心需要用到的一些檢查規則。檢查項都加了註釋,具體內容請參見附錄。

         點選“Window”->“Preferences”選單,在彈出視窗左側點選“Checkstyle”,可看到右側預設選中的是“Sun Checks”檢查規則,因為該規則要求比較嚴格,而且不適合本研發中心,所以需要自定義檢查檔案。

         點選“New”按鈕後,彈出“Check Configuration Properties”視窗,如下圖所示:           

         在上圖中選擇“Type”為“External Configuration File”(外部配置檔案),在“Location”行點選“Browse”按鈕瀏覽選擇checkstyle的配置檔案,例如eclipse_checkstyle.xml檔案(僅供參考,可自行定義),並輸入“Name”行的值,例如“eclipse_checkstyle”,點選“OK”按鈕完成匯入操作。回到Checkstyle主頁面選擇剛才新增的檔案,點選“Set As Default”將其設定為預設checkstyle檢查檔案。

3.3 使用checkstyle

         選擇某個專案,點選右鍵選擇“Checkstyle”,會出現如下選單:

   1) Configure project(s) from blueprint:指定工程加入Checkstyle檢查;

   2) Activate Checkstyle:啟用Checkstyle;

   3) Deactivate Checkstyle:取消啟用Checkstyle;

   4) Check code with Checkstyle:使用Checkstyle檢查程式碼;

   5) Clear Checkstyle violations:去掉Checkstyle的檢查提示。

  選擇“Check code with Checkstyle”對所選中的專案進行檢查,檢查後對有問題的類會使用警告或錯誤標識。參考提示資訊如下:          

         左側會顯示小圓圈,將滑鼠移動到上面時將給出提示資訊。

         使用checkstyle後,註釋、變數命名規範性、類中變數、建構函式、方法的順序的檢查、行的長度、檔案的長度、方法的長度等都不需要手工進行檢查,只需要在程式碼評審時使用Checkstyle檢查即可。因此,讓程式碼評審人員將側重點放在checkstyle檢查不出來的一些程式碼的優化問題。

3.4 checkstyle常見提示速查

         Checkstyle常見錯誤和警告提示見下表所示:

錯誤提示

錯誤說明

missing a javadoc comment

缺少類註釋

Line longer than X characters

行長度超過X個字元(包括空格)

Return count is X(max allowed 3)

一個方法內的返回數量是X(最大值只能為3)

Nested if-else depth is X(max allowed is 3)

最大的if-else巢狀層數為X(最大隻能為3)

Array brackets at illegal position

陣列的方括號“[]”的位置不正確(檢查陣列型別的定義是String[] args,而不是String args[])

Line matchs the illegal pattern 'System\.out\.println'

本行包含System.out.println語句

ctor def modifier at indentation level 8 not at corrent indentation 4

縮排不正確,一般是因為沒有在Eclipse中使用4個空格代替tab鍵引起。

'static' modifier out of order with the JLS suggestions

static修飾符沒有按照JLS的建議來排序(eg.寫成public final static...應該改成public static final)

Name 'X' must match pattern '^[A-Z][A-Z0-9][_A-Z0-9+]$'(正則表示式)

名稱不符合正則表示式'^[A-Z][A-Z0-9][_A-Z0-9+]$'(即為大寫字母,數字、下劃線等)。

一般在靜態變數沒有大寫時提示,包名不是全部訊息時提示,類名不是大寫開頭時提示,方法名不是小寫開頭時提示

Variable access definition in wrong order

變數定義順序不正確(例如在類成員變數定義時,將private型別的變數定義在public型別的變數之前)

Static variable definition in wrong order

靜態變數定義順序不正確(例如在建構函式之後定義靜態變數)

Instance variable definition in wrong order

成員變數定義順序不正確(例如在建構函式之後定義成員變數)

X is a magic number

X是一個魔術數字(非0、1、2的數字)

if construct must use '{}'

if結構必須使用'{}'

Got an exception - Unexpected character 0xfffd in identifier

因為沒有設定checkstyle配置檔案的charset為UTF-8,而類檔案使用UTF-8編碼,並且含有中文

“{” should be on the previous line

“{” 應該位於前一行

Methods is missing a javadoc comment

方法前面缺少javadoc註釋

Expected @throws tag for “Exception”

在註釋中希望有@throws的說明

“.” Is preceeded with whitespace

“.” 前面不能有空格

“.” Is followed by whitespace

“.” 後面不能有空格

“=” is not preceeded with whitespace“=”

前面缺少空格

“=” is not followed with whitespace

“=” 後面缺少空格

“}” should be on the same line

“}” 應該與下條語句位於同一行

Unused @param tag for “unused”

沒有引數“unused”,不需註釋

Variable “X” missing javadoc

變數“CA”缺少javadoc註釋

Line contains a tab character

行含有”tab” 字元

Redundant “Public” modifier

冗餘的“public” modifier

final modifier out of order with the JSL suggestion

final修飾符的順序錯誤

Avoid using the “.*” form of import

Import格式避免使用“.*”

Redundant import from the same package

從同一個包中Import內容

Unused import-X Import

import的X類沒有被使用

Duplicate import to line X

重複Import同一個內容

Import from illegal package

從非法包中 Import內容

“while” construct must use “{}”

“while” 語句缺少“{}”

Variable “X” must be private and have accessor method

變數“X”應該是private的,並且有呼叫它的方法

Variable “X” must match pattern “^[a-z][a-zA-Z0-9]*$”

變數“X”不符合命名規則“^[a-z][a-zA-Z0-9]*$”

“(” is followed by whitespace

“(” 後面不能有空格

“)” is proceeded by whitespace

“)” 前面不能有空格

4、附錄

4.1 eclipse_checkstyle.xml檔案內容

         Checkstyle配置檔案eclipse_checkstyle如下所示:

<?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">         <property name="severity" value="warning"/>    <module name="StrictDuplicateCode">        <property name="charset" value="utf-8" />       </module>       <module name="TreeWalker">           <!-- javadoc的檢查 -->           <!-- 檢查所有的interface和class -->           <module name="JavadocType" />           <!-- 命名方面的檢查 -->           <!-- 區域性的final變數,包括catch中的引數的檢查 -->           <module name="LocalFinalVariableName" />           <!-- 區域性的非final型的變數,包括catch中的引數的檢查 -->           <module name="LocalVariableName" />           <!-- 包名的檢查(只允許小寫字母) -->           <module name="PackageName">               <property name="format" value="^[a-z]+(\.[a-z][a-z0-9]*)*$" />           </module>           <!-- 僅僅是static型的變數(不包括static final型)的檢查 -->           <module name="StaticVariableName" />           <!-- 型別(Class或Interface)名的檢查 -->           <module name="TypeName" />           <!-- 非static型變數的檢查 -->           <module name="MemberName" />           <!-- 方法名的檢查 -->           <module name="MethodName" />           <!-- 方法的引數名 -->           <module name="ParameterName " />           <!-- 常量名的檢查 -->           <module name="ConstantName" />        <!-- 沒用的import檢查,比如:1.沒有被用到2.重複的3.import java.lang的4.import 與該類在同一個package的 -->           <module name="UnusedImports" />           <!-- 長度方面的檢查 -->           <!-- 檔案長度不超過1500行 -->           <module name="FileLength">               <property name="max" value="1500" />           </module>           <!-- 每行不超過150個字-->           <module name="LineLength">               <property name="max" value="150" />           </module>           <!-- 方法不超過150行 -->           <module name="MethodLength">               <property name="tokens" value="METHOD_DEF" />               <property name="max" value="150" />           </module>           <!-- 方法的引數個數不超過5個。 並且不對構造方法進行檢查-->           <module name="ParameterNumber">               <property name="max" value="5" />               <property name="tokens" value="METHOD_DEF" />           </module>           <!-- 空格檢查 -->           <!-- 允許方法名後緊跟左邊圓括號"(" -->           <module name="MethodParamPad" />           <!-- 在型別轉換時,不允許左圓括號右邊有空格,也不允許與右圓括號左邊有空格 -->           <module name="TypecastParenPad" />        <!-- 關鍵字 -->           <!--               每個關鍵字都有正確的出現順序。比如 public static final XXX 是對一個常量的宣告。如果使用 static             public final 就是錯誤的           -->           <module name="ModifierOrder" />           <!-- 多餘的關鍵字 -->           <module name="RedundantModifier" />           <!-- 對區域的檢查 -->           <!-- 不能出現空白區域 -->           <module name="EmptyBlock" />           <!-- 所有區域都要使用大括號 -->           <module name="NeedBraces" />        <!-- 多餘的括號 -->           <module name="AvoidNestedBlocks">               <property name="allowInSwitchCase" value="true" />           </module>           <!-- 編碼方面的檢查 -->           <!-- 不許出現空語句 -->           <module name="EmptyStatement" />         <!-- 不允許魔法數 -->           <module name="MagicNumber">               <property name="tokens" value="NUM_DOUBLE, NUM_INT" />           </module>        <!-- 多餘的throw -->           <module name="RedundantThrows" />         <!-- String的比較不能用!= 和 == -->           <module name="StringLiteralEquality" />           <!-- if最多巢狀3層 -->           <module name="NestedIfDepth">               <property name="max" value="3" />           </module>           <!-- try最多被巢狀2層 -->           <module name="NestedTryDepth">               <property name="max" value="2" />           </module>           <!-- clone方法必須呼叫了super.clone() -->           <module name="SuperClone" />           <!-- finalize 必須呼叫了super.finalize() -->           <module name="SuperFinalize" />           <!-- 不能catch java.lang.Exception -->           <module name="IllegalCatch">               <property name="illegalClassNames" value="java.lang.Exception" />          </module>           <!-- 確保一個類有package宣告 -->           <module name="PackageDeclaration" />           <!-- 一個方法中最多有3個return -->           <module name="ReturnCount">               <property name="max" value="3" />               <property name="format" value="^$" />           </module>           <!--               根據 Sun 編碼規範, class 或 interface 中的順序如下: 1.class 宣告。首先是 public,               然後是protected , 然後是 package level (不包括access modifier ) 最後是private .               (多個class放在一個java檔案中的情況) 2.變數宣告。 首先是 public, 然後是protected然後是 package             level (不包括access modifier ) 最後是private . (多個class放在一個java檔案中的情況)               3.建構函式 4.方法           -->           <module name="DeclarationOrder" />         <!-- 同一行不能有多個宣告 -->           <module name="MultipleVariableDeclarations" />           <!-- 不必要的圓括號 -->           <module name="UnnecessaryParentheses" />            <!-- 雜項 -->           <!-- 禁止使用System.out.println -->           <module name="GenericIllegalRegexp">               <property name="format" value="System\.out\.println" />               <property name="ignoreComments" value="true" />        </module>        <!-- 檢查並確保所有的常量中的L都是大寫的。因為小寫的字母l跟數字1太象了 -->           <module name="UpperEll" />           <!-- 檢查陣列型別的定義是String[] args,而不是String args[] -->           <module name="ArrayTypeStyle" />        <!-- 檢查java程式碼的縮排 預設配置:基本縮排 4個空格,新行的大括號:0。新行的case 4個空格 -->           <module name="Indentation" />       </module> </module>        

checkStyle結果分析

1.Missing a Javadoc comment:缺少JavaDoc註釋 2.First sentence should end with a period:你的註釋的第一行文字結束應該加上一個"." 3.Expected @throws tag for 'Exception':在註釋中希望有@throws的說明,在方法前得註釋中新增這樣一行:* @throws Exception if has error(異常說明) 4.Parameter docType should be final:引數docType應該為final型別 解決方法:在引數docType前面加個final 5.Variable “ABC” must match pattern “^[a-z][a-zA-Z0-9]*$”變數“ABC”不符合命名規則“^[a-z][a-zA-Z0-9]*$”解決方法:把這個命名改成符合規則的命名 “aBC” 6.Utility classes should not have a public or default constructor. 介面中的內部類中不應該有公共的或者預設的構造方法 解決方法:在內部類中,定義一個私有的構造方法,然後內部類宣告為final型別。如果前面有static,那麼final還必須放在static之後 7.'{' is not preceded with whitespace.大括號後面必須空一格

8.'public' modifier out of order with the JLS suggestions. public順序錯誤

9.Method 'deleteChild' is not designed for extension - needs to be abstract, final or empty. 不是拓展或繼承的方法,必須指定abstract,final或空

1Type is missing a javadoc commentClass 缺少型別說明

2“{” should be on the previous line“{” 應該位於前一行。解決方法:把“{”放到上一行去

3Methos 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“}” 應該與下條語句位於同一行。解決方法:把“}”放到下一行的前面

10Unused @param tag for “unused”沒有引數“unused”,不需註釋

解決方法:“* @param unused parameter additional(引數名稱)” 把這行unused引數的註釋去掉“

11Variable “CA” missing javadoc變數“CA”缺少javadoc註釋

解決方法:在“CA“變數前新增javad