代碼規范那點事

最近這個月,從制作 CocoaPods 私有庫,到搭建 Git 服務器,大多數時間都花在了調研和試錯上。不過調研試錯這個事總是能讓你感覺到自己懂的只是皮毛這個事實(所以當你覺得天下無敵的時候那就去調研,TA 會讓你爆棚的自信心損失殆盡 ????)。以下文字就先記錄下關于團隊統一代碼風格這件事。

正文

對于團隊統一代碼風格的好處想必大家都應該清楚,以下我就羅列幾點相對比較重要的:

  1. 促進團隊合作,提高代碼的可讀性;
  2. 有助于 Code Review ;
  3. 甚至可以降低 Bug 出現的概率;
  4. 自行 YY .....;

代碼規范想必有部分同學會把它和個人習慣混為一談,其實代碼規范和個人習慣壓根不是一個層面上的東西。代碼規范針對的是團隊,而個人習慣僅僅針對你自己。制定代碼規范的目的是為了提高團隊協作的效率以及代碼的可維護性。因此,個人習慣和代碼規范擦出火花的時候,很明顯你應該遵守規范而不是你自己所謂的那些習慣。當然規范并不是一塵不變的,當產生火花的時候完全可以提議修改代碼規范,但是在規范修改之前,你必須要遵守舊的規范。因為團隊利益大于一切。

舉個栗子,
A 習慣寫 single line 的條件語句。

if (a) return;     

而 B 自己的習慣是這樣,并且 B 還會使用一些格式化代碼(Xcode 的 clang-format,js 的 jsFormat 等)的插件來提升開發效率。

if (a) {
    return ;
}   

那么當 B 去修改 A 的代碼時,看著不順眼,就會把 A 的所有 single line 都消滅掉,因為 B 有插件,秒秒鐘解決戰斗。這就會導致 A 和 B 沖突的概率大大增加,甚至會造成很多不必要代碼合并的成本。想必大家看完這個栗子應該很清楚一個團隊制定代碼規范的重要性了。

一個團隊的代碼規范首先需要制定一份文檔,當出現任何風格沖突的時候,以文檔為準。因此團隊在開始之初就制定了一份規范

制定規范文檔這是第一步,在這個基礎上能不能再提高些效率呢?
大多數 iOS 開發者應該都知道 Xcode 的插件 Clang Format。它是基于 clang-format 命令行工具的一個 Xcode 插件。

clang-format是基于clang的一個命令行工具。這個工具能夠自動化格式C/C++/Obj-C代碼,支持多種代碼風格:Google, Chromium, LLVM, Mozilla, WebKit,也支持自定義風格(通過編寫.clang-format文件)。

我們可以通過自定義 .clang-format 文件來實現自定義代碼風格。這是目前使用的一份配置文件。具體每項參數可以查閱這份文檔

通過這個插件已經可以實現通過快捷鍵或者在文件保存時格式化代碼了,其實做到這一步已經節省了很多開發成本,但是這種方案也是存在一定缺陷。如果通過快捷鍵去格式化,容易遺漏;保存的時候去格式化,對于像我這樣時不時會按 Command + S 的人來說,Xcode 會變得稍顯卡頓。那么有沒有現成的方案可以進一步優化體驗?既然之前的方案有問題,就有必要繼續尋找更優解。

一輪 Google 之后,spacecommander 貌似符合大部分需求。它利用 Git Hooks 可以在 commit 之前驗證代碼風格符合規范,只有符合規范的代碼才允許提交,同時也提供 Shell 腳本來格式化一個文件,或者一整個 Git 倉庫。我猜想通過 spacecommander ,可以在不改變自己編碼風格(當然只限于格式,具體的命名規范,注釋規范還得參照具體規范文檔)的前題下,可以實現代碼風格統一。因為你開發過程中可以按照個人習慣來,commit 之前使用 spacecommander 提供的腳本對文件進行格式化。這樣只需要所有開發人員統一 spacecommander 的 .clangformat 配置文件就可以了。看到這里是不是發現這個輪子是不是剛好命中痛點。那么接下來我就簡單描述下使用步驟:

  1. fork spacecommander
  2. 修改其中的 .clangformat 文件以滿足自己團隊的編碼風格,當然有能力也可以修改其中的 Shell 腳本,自定義一些功能;
  3. clone 到本地一個較為安全的目錄(別一不小心刪掉了...);
  4. 為了之后方便使用,可以把幾個腳本對應設置一個 alias;
  5. cd 到項目根目錄,執行 clangformatinit ,進行初始化(添加了一個指向本地 spacecommander 倉庫的 .clangformat 替身以及在 .git/hooks 中的 pre-commit, hook 相關的可以參見 Pro Git
  6. 在提交代碼之前,spacecommander 都會通過 pre-commit 這個 Hook 來校驗修改過的文件,校驗通過才允許提交。

如何設置 alias 簡化命令

// 使用zsh 則修改 ~/.zshrc;bash 則修改~/.bash_profile
// 初始化
alias clangformatinit="/Users/SpaceCommander_iOS/setup-repo.sh"
// 格式化對應文件
alias clangformat="/Users/SpaceCommander_iOS/format-objc-file.sh"
// 格式化整個倉庫
alias clangformatall="/Users/SpaceCommander_iOS/format-objc-files-in-repo.sh    

如果需要 spacecommander 忽略某個目錄下的文件的格式,則可以通過修改 spacecommander/lib/common-lib.sh 腳本來實現。默認它已經忽略了 Pods 和 Carthage 目錄。

// common-lib.sh 簡化版

function objc_files_to_format() {
    optional_base_sha="$1"
    directories_to_check
    files=$(git diff --cached --name-only $optional_base_sha --diff-filter=ACM -- $locations_to_diff | grep -e '\.m$' -e '\.mm$' -e '\.h$' -e '\.hh$')
    directories_to_ignore
    echo "$files" | grep -v 'Pods/' | grep -v 'Carthage/' >&1
}
 
function all_valid_objc_files_in_repo() {
    directories_to_check
    files=$(git ls-tree --name-only --full-tree -r HEAD -- $locations_to_diff | grep -e '\.m$' -e '\.mm$' -e '\.h$' -e '\.hh$')
    directories_to_ignore
    echo "$files" | grep -v 'Pods/' | grep -v 'Carthage/' >&1
}    

就此初步解決了 iOS 開發的代碼風格問題 ———— 一份代碼規范以及自定義的 spacecommander。

小結

這個問題看似有點微不足道,但的確花了毛一天的時間,才找到了較為合適的解決方案。但是在我看來是很值的。不僅能減少后續開發維護過程中由于代碼風格引起的麻煩(感謝 spacecommander 這個輪子),而且在調研的過程中,也接觸到了一些新的知識,比如 Shell 腳本語言。所以我很喜歡去干這種事情 ????。 接下來有時間會整理一篇 CocoaPods 私有庫相關的博客 ????。

有其它的解決方案,歡迎分享 ????

最后編輯于
?著作權歸作者所有,轉載或內容合作請聯系作者
平臺聲明:文章內容(如有圖片或視頻亦包括在內)由作者上傳并發布,文章內容僅代表作者本人觀點,簡書系信息發布平臺,僅提供信息存儲服務。
  • 序言:七十年代末,一起剝皮案震驚了整個濱河市,隨后出現的幾起案子,更是在濱河造成了極大的恐慌,老刑警劉巖,帶你破解...
    沈念sama閱讀 229,698評論 6 539
  • 序言:濱河連續發生了三起死亡事件,死亡現場離奇詭異,居然都是意外死亡,警方通過查閱死者的電腦和手機,發現死者居然都...
    沈念sama閱讀 99,202評論 3 426
  • 文/潘曉璐 我一進店門,熙熙樓的掌柜王于貴愁眉苦臉地迎上來,“玉大人,你說我怎么就攤上這事。” “怎么了?”我有些...
    開封第一講書人閱讀 177,742評論 0 382
  • 文/不壞的土叔 我叫張陵,是天一觀的道長。 經常有香客問我,道長,這世上最難降的妖魔是什么? 我笑而不...
    開封第一講書人閱讀 63,580評論 1 316
  • 正文 為了忘掉前任,我火速辦了婚禮,結果婚禮上,老公的妹妹穿的比我還像新娘。我一直安慰自己,他們只是感情好,可當我...
    茶點故事閱讀 72,297評論 6 410
  • 文/花漫 我一把揭開白布。 她就那樣靜靜地躺著,像睡著了一般。 火紅的嫁衣襯著肌膚如雪。 梳的紋絲不亂的頭發上,一...
    開封第一講書人閱讀 55,688評論 1 327
  • 那天,我揣著相機與錄音,去河邊找鬼。 笑死,一個胖子當著我的面吹牛,可吹牛的內容都是我干的。 我是一名探鬼主播,決...
    沈念sama閱讀 43,693評論 3 444
  • 文/蒼蘭香墨 我猛地睜開眼,長吁一口氣:“原來是場噩夢啊……” “哼!你這毒婦竟也來了?” 一聲冷哼從身側響起,我...
    開封第一講書人閱讀 42,875評論 0 289
  • 序言:老撾萬榮一對情侶失蹤,失蹤者是張志新(化名)和其女友劉穎,沒想到半個月后,有當地人在樹林里發現了一具尸體,經...
    沈念sama閱讀 49,438評論 1 335
  • 正文 獨居荒郊野嶺守林人離奇死亡,尸身上長有42處帶血的膿包…… 初始之章·張勛 以下內容為張勛視角 年9月15日...
    茶點故事閱讀 41,183評論 3 356
  • 正文 我和宋清朗相戀三年,在試婚紗的時候發現自己被綠了。 大學時的朋友給我發了我未婚夫和他白月光在一起吃飯的照片。...
    茶點故事閱讀 43,384評論 1 372
  • 序言:一個原本活蹦亂跳的男人離奇死亡,死狀恐怖,靈堂內的尸體忽然破棺而出,到底是詐尸還是另有隱情,我是刑警寧澤,帶...
    沈念sama閱讀 38,931評論 5 363
  • 正文 年R本政府宣布,位于F島的核電站,受9級特大地震影響,放射性物質發生泄漏。R本人自食惡果不足惜,卻給世界環境...
    茶點故事閱讀 44,612評論 3 348
  • 文/蒙蒙 一、第九天 我趴在偏房一處隱蔽的房頂上張望。 院中可真熱鬧,春花似錦、人聲如沸。這莊子的主人今日做“春日...
    開封第一講書人閱讀 35,022評論 0 28
  • 文/蒼蘭香墨 我抬頭看了看天上的太陽。三九已至,卻和暖如春,著一層夾襖步出監牢的瞬間,已是汗流浹背。 一陣腳步聲響...
    開封第一講書人閱讀 36,297評論 1 292
  • 我被黑心中介騙來泰國打工, 沒想到剛下飛機就差點兒被人妖公主榨干…… 1. 我叫王不留,地道東北人。 一個月前我還...
    沈念sama閱讀 52,093評論 3 397
  • 正文 我出身青樓,卻偏偏與公主長得像,于是被迫代替她去往敵國和親。 傳聞我的和親對象是個殘疾皇子,可洞房花燭夜當晚...
    茶點故事閱讀 48,330評論 2 377

推薦閱讀更多精彩內容

  • Spring Cloud為開發人員提供了快速構建分布式系統中一些常見模式的工具(例如配置管理,服務發現,斷路器,智...
    卡卡羅2017閱讀 134,816評論 18 139
  • WebSocket-Swift Starscream的使用 WebSocket 是 HTML5 一種新的協議。它實...
    香橙柚子閱讀 24,008評論 8 183
  • 今天老婆給我分享了一篇文章,說的是31歲的生性活潑開朗的妻子懷抱兩個幼兒跳樓自殺了。 看完之后我覺得真是不可思議,...
    Cplusplus墨水閱讀 725評論 0 0
  • 簡介 一個叫嫣然月的女孩兒在茫茫的黑夜中被一輛像是被安排過的車撞了,不幸的是她成了植...
    有辭而別閱讀 172評論 0 0
  • 大一時候胖胖的女老師在講臺上看著我們說, 哎,整天寫這種什么數據得到什么結論的東西,整個人也變得干巴巴的不會說好聽...
    雪琪呀閱讀 227評論 0 0