天天看點

編寫高品質的代碼—從命名入手(命名…

 

 編寫高品質的代碼—從命名入手(命名規範)

編寫高品質的代碼—從命名入手(命名…

筆者從事開發多年,有這樣一種感覺,檢視一些開源項目,如Spring、Apache

Common等源碼是一件賞心悅目的事情,究其原因,無外兩點:1)代碼品質非常高;2)命名特别規範(這可能跟老外的英語水準有關)。

  要寫高品質的代碼,不是一件容易的事,需要長年累月的鍛煉,是一個量變到質變的過程,但要寫好命名,隻需要有比較好的英語文法基礎和一種自我意識即可輕松達到。本博文将會結合本人的開發經驗,總結出若幹命名規則,這些命名規則純屬個人的使用習慣,不代表是一種理想的規則,在這裡列舉出來,供大家交流讨論。

  1.

切忌使用沒有任何意義的英語字母進行命名

for(int i=0; i<10; i++)

{

  //...

}

 這是在很多教Java基本文法的書上常見的代碼片斷,作為教學材料,這樣寫無可厚非,但作為真正的代碼編寫,程式員必須要養成良好的習慣,不要使用這種沒有任何含義的命名方式,這裡可以使用“index”。

  2.

切忌使用拼音,甚至是拼音首字母組合

cishu =5;  //

循環的次數

zzje = 1000.00;  //

轉賬金額

筆者在做代碼檢查的時候,無數次遇到過這樣的命名,使人哭笑不得。

  3. 要使用英文,而且要使用準确的英語,無論是拼寫還是文法

名詞單數,必須使用單數英文,如Account、Customer。

對于數組,清單等對象集合的命名,必須使用複數,而且最好按照英文的文法基礎知識使用準确的複數形式,如

List accounts、Set strategies。

對于boolean值的屬性,很多開發人員習慣使用isXXX,如isClose(是否關閉),但這裡有兩點建議:1)最好不要帶“is”,因為JavaBean的規範,為屬性生成get/set方法的時候,會用“get/set/is”,上面的例子,生成get/set方法就會變成“getIsClose/isIsClose/getIsClose”,非常别扭;2)由于boolean值通常反映“是否”,是以準确的用法,應該是是用“形容詞”,上面的例子,最終應該被改為

closed,那麼get/set方法就是“getClosed/isColsed/setClosed”,非常符合英語閱讀習慣。

  4.

方法名的命名,需要使用“動賓結構短語”或“是動詞+表語結構短語”

  筆者曾看到過千奇百怪的方法命名,有些使用名詞,有些甚至是“名詞+動詞”,而且,如果賓語是一個對象集合,還是最好使用複數。

createOrder(Order order) //good

  orderCreate(Order

order) //bad

  removeOrders(List

orders) //good 

  removeOrder(List

5.

對于常見的“增删改查”方法,命名最好要謹慎

增加:最常見使用create和add,但最好根據英語的語義進行區分,這有助于了解,create代表建立,add代表增加。比如,要建立一個Student,用createStudent要比用addStudent好,為什麼?想想如果有個類叫Clazz(班級,避開Java關鍵字),現在要把一個Student加入到一個Clazz,Clazz很容易就定義了一個

addStudent(Student student)的方法,那麼就比較容易混淆。

修改:常見的有alter、update、modify,個人覺得modify最準确。

查詢:對于擷取單個對象,可以用get或load,但個人建議用get,解釋請見第7點的說明;對于不分條件列舉,用list;對于有條件查詢,用search(最好不要用find,find在英文了強調結果,是“找到”的意思,你提供一個“查詢”方法,不保證輸入的條件總能“找到”結果)。

删除:常見的有delete和remove,但删除建議用delete,因為remove有“移除”的意思,參考Clazz的例子就可以了解,從班級移除一個學生,會用removeStudent。

  6.

甯願方法名冗長,也不要使用讓人費解的簡寫

  筆者曾經遇到一個方法,判斷“支付賬戶是否與收款賬戶相同”,結果我看到一個這樣的命名:

checkIsOrderingAccCollAccSame(...)

很難了解,我馬上把它改為:

isOrderingAccountSameAsCollectionAccount(...)

 雖然有點長,但非常容易閱讀,而且這種情況總是出現得比較少。

  7. 如果你在設計業務系統,最好不要使用技術化的術語去命名

  筆者曾經工作的公司曾經制訂這樣的命名規則,接口必須要以“I”開頭,資料傳輸對象必須以“DTO”作為字尾,資料通路對象必須以“DAO”作為字尾,領域對象必須以“DO”作為字尾。我之是以不建議這種做法,是希望設計人員從一開始就引導開發人員,要從“業務”出發考慮問題,而不要從“技術”出發。是以,接口不需要非得以“I”開頭,隻要其實作類以“Impl”結尾即可(注:筆者認為接口是與細節無關的,與技術無關,但實作類是實作相關的,用技術化術語無可口非);而資料傳輸對象,其實無非就是儲存一個對象的資訊,是以可以用“**Info”,如CustomerInfo;領域對象本身就是業務的核心,是以還是以其真實名稱出現,比如Account、Customer;至于“DAO”,這一個詞來源于J2ee的設計模式,筆者在之前的項目使用“***Repository”命名,意味“***的倉庫”,如AccountRepository,關于“Repository”這個詞的命名,是來源于Eric

Evans的《Domain-Driven Design》一書的倉庫概念,Eric

Evans對Repository的概念定義是:領域對象的概念性集合,個人認為這個命名非常的貼切,它讓程式員完全從技術的思維中擺脫出來,站在業務的角度思考問題。說到這裡,可能有人會反駁:像Spring、Hibernate這些優秀的架構,不是都在用“I”作為接口開頭,用“DAO”來命名資料通路對象嗎?沒錯!但千萬别忽略了語義的上下文,Spring、Hibernate架構都是純技術架構,我這裡所說的場景是設計業務系統。

  8. 成員變量不要重複類的名稱

  例如,很多人喜歡在Account對象的成員變量中使用accountId,accountNumber等命名,其實沒有必要,想想成員變量不會孤立的存在,你引用accountId,必須是account.accountId,用account.id已經足夠清晰了。

  “勿以善小而不為,勿以惡小而為之”、“細節決定成敗”,有太多的名言告訴我們,要注重細節。一個優秀的程式員,必須要有堅實的基礎,而對于命名規則這樣容易掌握的基礎,我們何不現行?

歡迎學習本文,未經部落客允許,禁止轉載!