Code Review 的巔峰
Code Review
張大胖工作快一年了，這一年看了不少書(shū)，尤其是編程實(shí)踐方面，例如單元測(cè)試、重構(gòu)、持續(xù)集成等等， 和公司的開(kāi)發(fā)現(xiàn)狀一對(duì)比， 就發(fā)現(xiàn)很多軟件開(kāi)發(fā)實(shí)踐不夠規(guī)范，比如說(shuō)Code Review完全就沒(méi)有。
張大胖是那種遇到問(wèn)題不逃避，不推脫，反而迎難而上的人，他心想既然現(xiàn)在公司沒(méi)有這個(gè)流程，我可以提議，甚至推動(dòng)把Code Review這件事情給建立起來(lái)。
他在中午吃飯時(shí)“勾搭”了一下部門經(jīng)理，把想法匯報(bào)了一下，成功取得了經(jīng)理的支持和授權(quán)：推動(dòng)Code Review流程的建立。
張大胖先收集了很多資料發(fā)給大家，用于宣傳Code Review的好處 ， 他制作了一個(gè)精美的PPT，召集會(huì)議給大家預(yù)熱講解。
隨后他便結(jié)合公司的源碼管理工具， 建立了強(qiáng)制Code Review的制度，沒(méi)有經(jīng)過(guò)Code Review的代碼，不允許進(jìn)入代碼庫(kù)。 至少有兩個(gè)人Review過(guò)代碼才算通過(guò)。
CheckStyle 和連坐
張大胖滿心期待這樣的流程能夠發(fā)現(xiàn)Bug ，提高代碼質(zhì)量，可是試運(yùn)行了一周以后，靜悄悄的，大家通過(guò)Code Review糾結(jié)的盡是一些代碼風(fēng)格的問(wèn)題：縮進(jìn)不夠了，空格太少，一行的字符太多了等等， 這對(duì)于提高代碼質(zhì)量并沒(méi)有很大的幫助，還浪費(fèi)了不少時(shí)間。
張大胖思考了一陣，決定引入工具來(lái)自動(dòng)化地做這些瑣碎的事情，比如Checkstyle，用工具把這些代碼風(fēng)格問(wèn)題消滅在程序員的本地代碼中， 不改正Checkstyle發(fā)現(xiàn)的問(wèn)題，連代碼評(píng)審這一步都走不到。
這下子大家必須得動(dòng)動(dòng)腦子去Review代碼了吧！張大胖想。
但人都是懶惰的，想改變習(xí)慣是很難的。 又過(guò)了一周，那幾個(gè)資深的骨干說(shuō)：哎呀，太忙了，自己的代碼還寫(xiě)不完，怎么可能Review別人的代碼？
Code Review的效果自然也好不到哪里去。 這時(shí)候一直在觀察的部門經(jīng)理對(duì)張大胖做出了強(qiáng)有力的支持，他宣布了一項(xiàng)制度：連坐。 即代碼的質(zhì)量由寫(xiě)代碼的人和Review代碼的人共同負(fù)責(zé)。
Check List
張大胖心想這個(gè)制度肯定會(huì)傷害一些人，但是也應(yīng)該會(huì)調(diào)到大家的積極性，將來(lái)等到大家習(xí)慣養(yǎng)成了，就可以廢除這個(gè)連坐了。
果不其然，Review代碼發(fā)現(xiàn)的有效問(wèn)題開(kāi)始增多。只是大家的喜好不同，關(guān)注的重點(diǎn)不同，老李關(guān)注可讀性，老王習(xí)慣看各種異常處理和次要分支條件，老方喜歡看類的設(shè)計(jì)是否合理， 大周最關(guān)心代碼對(duì)別的模塊的影響，還有性能問(wèn)題......
張大胖是個(gè)有心人，他把這些問(wèn)題總結(jié)，分析了一下，形成了一份Code Review Check List，列舉出來(lái)Code Review應(yīng)該關(guān)注的點(diǎn)， 請(qǐng)各位資深大牛審查，補(bǔ)充以后便正式頒布實(shí)施。
大家都挺認(rèn)可這份CheckList， 積極性也提高了，張大胖想，這下可以消停一陣了。
代碼量控制
沒(méi)想到新問(wèn)題很快就出現(xiàn)了，有些人不喜歡“小步快跑”，總是在工作了一周以后，達(dá)到階段性目標(biāo)以后才愿意提交代碼Review， 這就帶來(lái)了兩個(gè)問(wèn)題： 一個(gè)是Reviewer 一下子面對(duì)大量的代碼，看不過(guò)來(lái)，很容易喪失信心，于是草草掃過(guò)了事。 第二個(gè)是如果發(fā)現(xiàn)了嚴(yán)重問(wèn)題，再讓作者返工，也是很難的。
張大胖趕緊和大家商量，不要一下子提交這么多代碼，而是要小步快跑，控制每次Review代碼的數(shù)量。 大家同意每次Review的代碼要控制在200至400行以內(nèi)。 讓Reviewer有意愿去做真正的Review，而不是流于形式。
另外對(duì)于特別復(fù)雜、特別重要的模塊，要專門把相關(guān)人等“糾集”到一起，在會(huì)議室中一行一行做嚴(yán)格的代碼評(píng)審。
經(jīng)過(guò)這么一番折騰，大家終于養(yǎng)成了習(xí)慣，原來(lái)的“連坐”機(jī)制也廢除了，改成了獎(jiǎng)勵(lì)機(jī)制，給與那些在代碼評(píng)審中發(fā)現(xiàn)重要Bug的人以精神和物質(zhì)的獎(jiǎng)勵(lì)。
張大胖發(fā)現(xiàn)，其實(shí)Code Review改變了人的思想意識(shí)，原來(lái)為了快速完成功能，大家寫(xiě)代碼的時(shí)候都沒(méi)有思考太多，現(xiàn)在不一樣了，一想到自己的代碼馬上就要被別人審視， 就像開(kāi)源軟件一樣，寫(xiě)得小心翼翼了。還會(huì)對(duì)照著CheckList好好想想，盡自己最大努力寫(xiě)好代碼，看起來(lái)在開(kāi)發(fā)上花費(fèi)的時(shí)間唱了，但是Bug大量減少了。慢慢地，自己的水平也提高了。
結(jié)對(duì)編程： Code Review的巔峰
一天晚上上網(wǎng)瀏覽時(shí)，張大胖發(fā)現(xiàn)了一個(gè)有趣的東西：極限編程。
這個(gè)極限編程是敏捷軟件開(kāi)發(fā)領(lǐng)域的一個(gè)“奇葩”， 它的宗旨就是：如果一個(gè)軟件開(kāi)發(fā)的實(shí)踐很好，我們就把它做到極致！
所以既然測(cè)試是好的，我們就先寫(xiě)測(cè)試，再寫(xiě)代碼 ，這就是TDD。
既然Code Review這么好，那我們就在一個(gè)程序員寫(xiě)代碼的時(shí)候，安排另外一個(gè)人時(shí)時(shí)刻刻去Review代碼，定期讓這兩個(gè)人的角色互換， 這就是結(jié)對(duì)編程。
看到這里，張大胖激動(dòng)地一下子跳了起來(lái)： 臥草，這才是Code Review的巔峰啊！
第二天上午，張大胖興沖沖去找經(jīng)理，希望在部門實(shí)施結(jié)對(duì)編程， 可這一次經(jīng)理并沒(méi)有像上次那樣全力支持，只是答應(yīng)他小范圍嘗試。
張大胖找了和自己水平差不多的小李和自己結(jié)對(duì)編程，嘗試共同完成一個(gè)需求。張大胖一會(huì)兒當(dāng)Driver， 主要控制鍵盤(pán)寫(xiě)代碼；一會(huì)兒當(dāng)Navigator，關(guān)注宏觀目標(biāo)，思考問(wèn)題，并且Review。
一天時(shí)間下來(lái)，他倆的共同感受就是：真TMD累！
小李開(kāi)玩笑地說(shuō)：“下次你給我安排個(gè)女生一起‘結(jié)對(duì)編程’吧， 男女搭配，干活不累！”
原來(lái)“單干”的時(shí)候遇到問(wèn)題了，總是想休息一下，拿起手機(jī)看看新聞，刷刷朋友圈，時(shí)間很快就過(guò)去了。雖說(shuō)一天工作9個(gè)多小時(shí)，但是真正高效率的編程時(shí)間可能還不到4個(gè)小時(shí)。 現(xiàn)在可好，旁邊一直有個(gè)人盯著，精神時(shí)刻處于高度緊張之中，不是在寫(xiě)代碼，就是在思考怎么解決問(wèn)題。
寫(xiě)出代碼質(zhì)量確實(shí)不錯(cuò)，Code Review時(shí)以及測(cè)試階段幾乎沒(méi)有發(fā)現(xiàn)問(wèn)題。
向經(jīng)理匯報(bào)時(shí)，張大胖做了如下的總結(jié)：
1. 心態(tài)得開(kāi)放，愿意“暴露”自己，并且接受別人的建議
對(duì)很多程序員來(lái)講，并不愿意暴露自己編程的思路和技術(shù)水平，這也算是程序員的小隱私了。結(jié)對(duì)編程就得要求思維的透明化，別人得充分了解你的思路，從而能了解你的水平。 尤其是自己寫(xiě)每一行代碼都被人盯著，隨時(shí)接受“審判” ， 如果沒(méi)有開(kāi)發(fā)的心態(tài)，是做不到的。
2. 當(dāng)Driver在寫(xiě)代碼時(shí)，不能頻繁地被打斷。
張大胖有幾次在專心致志地寫(xiě)一個(gè)復(fù)雜的功能時(shí)，小李老是提示他變量名命名不合適，其實(shí)他自己也知道，只是還沒(méi)改，這讓他很不爽。
3. 不能太霸道
小李有一段時(shí)間一直霸占著鍵盤(pán)不放，自己說(shuō)了好幾次他才戀戀不舍地把鍵盤(pán)遞過(guò)來(lái)。
4. 兩個(gè)人的水平應(yīng)該差不太多。
如果差得太多，就變成了一個(gè)人教另外一個(gè)人編程了。
經(jīng)理聽(tīng)了以后說(shuō)道：“結(jié)對(duì)編程看起來(lái)很美，執(zhí)行起來(lái)還是有難度的，很多人不愿意在寫(xiě)代碼的時(shí)候被別人盯著，不愿意'暴露'自己。它的見(jiàn)效周期也比較長(zhǎng)，管理層不愿意看到兩個(gè)人干一份活兒， 我們還是先放下吧。”
雖然心有不甘，但是張大胖承認(rèn)經(jīng)理說(shuō)得很對(duì)，暫時(shí)放棄吧。
年終的時(shí)候，張大胖由于對(duì)Code Review這個(gè)制度的建設(shè)而獲得了公司的獎(jiǎng)勵(lì)，他總結(jié)了一些成功的經(jīng)驗(yàn)：
1. 領(lǐng)導(dǎo)的支持
2. 民主的決策
3. 能用工具完成的，就別讓人再麻煩了。
4. CheckList很有用
5. 每次review適量的代碼
【本文為51CTO專欄作者“劉欣”的原創(chuàng)稿件，轉(zhuǎn)載請(qǐng)通過(guò)作者微信公眾號(hào)coderising獲取授權(quán)】
