團隊開發(fā)中，期望 Code Review 來約束的是代碼結(jié)構(gòu)和潛在邏輯問題，這時代碼規(guī)范這種『細節(jié)』就給 review 過程帶來不小的噪音偶宫。
特別是團隊目前的情況：歷史負擔(dān)較重侈贷，大量經(jīng)幾手的舊代碼風(fēng)格不一致，團隊成員新組建抡谐，對代碼規(guī)范裁奇、好的范式還沒達成默契。新寫代碼的規(guī)范尚在磨合中麦撵，遑論手動改寫大量的舊代碼刽肠。
這是重要不緊急的問題，最近項目略有空閑免胃，是對這類問題下手的好時機音五；）
目標
- 提升代碼可讀性，打破『破窗效應(yīng)』
- 用工具約束代碼規(guī)范羔沙，盡可能不影響原有開發(fā)流程的同時放仗，盡可能減少 code review 中代碼規(guī)范方面的噪音，讓開發(fā)精力放在該放的地方
- Reformat 歷史代碼撬碟，一勞永逸的解決這個問題
破窗效應(yīng)
『如果那些窗不被修理好诞挨，可能將會有破壞者破壞更多的窗戶。...又或想像一條人行道有些許紙屑呢蛤，如果無人清理惶傻，不久后就會有更多垃圾，最終人們會視為理所當然地將垃圾順手丟棄在地上其障∫遥』
——維基百科
一些調(diào)研
本以為不會太復(fù)雜，實際執(zhí)行起來發(fā)現(xiàn)問題也不少：
- 拜蘋果的封閉和垃圾的 Xcode 所致，舊插件不可用 ClangFormat-Xcode 蜈敢、BBUncrustifyPlugin-Xcode 辜荠，Xcode8 插件又未成火候，均不好用
-
XcodeClangFormat for Xcode8 調(diào)試
.clang-format并不方便 - SwiftLint 評價很高抓狭，可惜目前項目 7w lines ObjC伯病，Swift 在下半年計劃里
看了一圈，最終決定選擇 square 家的方案：https://github.com/square/spacecommander
spacecommander = clang-format + ObjC 定制規(guī)范 + git 工作流
spacecommander 在 clang-format 基礎(chǔ)上加入幾條定制規(guī)范否过，解決了 block 縮進問題午笛，對 ObjC 更友好。內(nèi)置腳本苗桂，很方便的添加 git pre-commit hook药磺，format 代碼也算方便，不影響原有工作流煤伟。源代碼很少癌佩，一些 shell 和 python，很容易看懂便锨。
這些跟團隊需要很契合围辙，項目有些老，有些小問題鸿秆，縫縫補補能用就好酌畜。
如何執(zhí)行
- Fork https://github.com/square/spacecommander 項目（方便后續(xù)定制）怎囚，clone 到本地
- cd 到需要格式化的項目根目錄卿叽，執(zhí)行
/path/to/spacecommander/setup-repo.sh。這時你會看到項目目錄下創(chuàng)建了一個軟鏈接.clang-format（并加入到.gitignore) 同時創(chuàng)建.git/hooks/pre-commit恳守，commit 時起效果的秘密就在這里 - 隨意修改項目中的一個 .m 文件考婴，stage file，commit催烘，你會得到這樣的提示：
git -c diff.mnemonicprefix=false -c core.quotepath=false -c credential.helper=sourcetree commit -q -F /var/folders/7w/5_0xvl794n960s_wsr3h68340000gn/T/SourceTreeTemp.GSJLdQ
?? Format and stage individual files:
"/Users/zengming/ming-dev/gg-projects/spacecommander"/format-objc-file.sh 'ios-common/GGLib/GGBase/GGModels/GGHotelAPIModel.m' && git add 'ios-common/GGLib/GGBase/GGModels/GGHotelAPIModel.m';
?? Format and stage all affected files:
"/Users/zengming/ming-dev/gg-projects/spacecommander"/format-objc-files.sh -s
?? There were formatting issues with this commit, run the?? above?? command to fix.
?? Commit anyway and skip this check by running git commit --no-verify
Completed with errors, see above
- 新修改的 GGHotelAPIModel.m 文件代碼規(guī)范檢查不通過沥阱，所以 commit 失敗。解決起來也很容易伊群，按上面的提示在終端輸入
"/Users/zengming/ming-dev/gg-projects/spacecommander"/format-objc-files.sh -s考杉，再 commit，搞定舰始！
spacecommander 的原理是定制好 .clang-format崇棠，執(zhí)行 commit 前運行 format-objc-file-dry-run 檢查當前要提交的文件，如果不符合規(guī)范丸卷，則提示修改枕稀。（默認只檢查已修改并 Stage 的文件）
如果要格式化項目所有文件，可以嘗試在項目中運行 /path/to/spacecommander/format-objc-files-in-repo
一些細節(jié)問題
Block 格式化不符合預(yù)期？
參見這里：https://bugs.llvm.org//show_bug.cgi?id=23585
clang-format: Support nested block formatting with ColumnLimit=0.
改到可用萎坷，這點還不大符合我的預(yù)期凹联。誰有更好的方法嗎？
// 格式化后的效果
[UIAlertView bk_showAlertViewWithTitle:@"溫馨提示"
message:push.alert
cancelButtonTitle:@"忽略"
otherButtonTitles:@[ @"查看" ]
handler:^(UIAlertView *alertView, NSInteger buttonIndex) {
if (buttonIndex == 0) {
return;
}
...
}];
// 我期望的效果
[UIAlertView bk_showAlertViewWithTitle:@"溫馨提示"
message:push.alert
cancelButtonTitle:@"忽略"
otherButtonTitles:@[ @"查看" ]
handler:^(UIAlertView *alertView, NSInteger buttonIndex)
{
if (buttonIndex == 0) {
return;
}
...
}];
直接使用 .clang-format ?
最開始我也是直接配 .clang-format哆档，總有些配置不合預(yù)期蔽挠，改起來調(diào)試也不是那么方便。spacecommander 的基礎(chǔ)配置是一個能看的配置虐呻，推薦在它的基礎(chǔ)上改象泵，何況 git hook 都寫好了何必自己折騰。
.clang-format 全部配置細節(jié)參見這里：https://clang.llvm.org/docs/ClangFormatStyleOptions.html
spacecommander 的一些小問題
- spacecommander 一些配置是挺老的斟叼，自帶的 clang-format 是 v3.8（官方最新是 v5）偶惠，這點沒碰到問題，略過
- 如果你運行
format-objc-files.sh -s時 python 報錯朗涩，這是因為 spacecommander 定制的規(guī)則 py 使用的是 python2, 切到 python2 下運行即可 - spacecommander 其中一個定制
MacroSemicolonAppender.py格式化時會把我的文件改錯（錯加分號）忽孽，我沒細究，禁掉即可（禁掉方式參見我的 fork）
總結(jié)
- 代碼規(guī)范其實是很重要的事谢床，看整齊的代碼給大家好的心情兄一，這非常重要
- 代碼規(guī)范這種重要卻又瑣碎的臟活累活還是交個自動化工具來解決，讓團隊精力集中在其它重要的事情上
- 格式化配置細節(jié)還有有些問題识腿，關(guān)系不大出革，不怕工具不好，先有渡讼，再慢慢改進
- 這只是代碼規(guī)范骂束，跟代碼邏輯，架構(gòu)沒半毛錢關(guān)系
- 代碼整齊了成箫，看著真舒服不少??
