建站
咨詢
售后
建站咨詢
新聞中心
| 本博文出自博客畢成功博主，有任何問(wèn)題請(qǐng)進(jìn)入博主頁(yè)面互動(dòng)討論！ 博文地址：http://passover.blog./2431658/1642176 |
為什么需要代碼審查
創(chuàng)新互聯(lián)建站是由多位在大型網(wǎng)絡(luò)公司、廣告設(shè)計(jì)公司的優(yōu)秀設(shè)計(jì)人員和策劃人員組成的一個(gè)具有豐富經(jīng)驗(yàn)的團(tuán)隊(duì)，其中包括網(wǎng)站策劃、網(wǎng)頁(yè)美工、網(wǎng)站程序員、網(wǎng)頁(yè)設(shè)計(jì)師、平面廣告設(shè)計(jì)師、網(wǎng)絡(luò)營(yíng)銷(xiāo)人員及形象策劃。承接：成都網(wǎng)站建設(shè)、成都做網(wǎng)站、網(wǎng)站改版、網(wǎng)頁(yè)設(shè)計(jì)制作、網(wǎng)站建設(shè)與維護(hù)、網(wǎng)絡(luò)推廣、數(shù)據(jù)庫(kù)開(kāi)發(fā),以高性價(jià)比制作企業(yè)網(wǎng)站、行業(yè)門(mén)戶平臺(tái)等全方位的服務(wù)。
最近看了一些文章，發(fā)現(xiàn)敏捷開(kāi)發(fā)的一些理念越來(lái)越多的團(tuán)隊(duì)在實(shí)踐，也覺(jué)得敏捷不再像最早提出的時(shí)候那么虛，有很多體現(xiàn)這個(gè)理念的工具涌現(xiàn)。其中，“如何提高代碼質(zhì)量”的討論一直很多，敏捷開(kāi)發(fā)中也有好多種提案，最廣為人知、但也最不靠譜的應(yīng)該就是結(jié)對(duì)編程了，只要沒(méi)被敏捷洗腦的人都清楚知道這個(gè)基本沒(méi)有實(shí)際可操作性，然而這個(gè)做法體現(xiàn)的觀點(diǎn)是多個(gè)人互相監(jiān)督可以把事情做的更好，這反而是完全沒(méi)有問(wèn)題的。所以還有一種方式就是代碼審查了，把兩人同時(shí)寫(xiě)代碼改成在不同的時(shí)間上一個(gè)人寫(xiě)、另外一個(gè)人看。這個(gè)實(shí)際開(kāi)發(fā)中是完全可以做到的，只是要留有審查的時(shí)間即可。
復(fù)查團(tuán)隊(duì)成員的代碼自己一直也是無(wú)意識(shí)的在做，經(jīng)常去看git log，但是這個(gè)方式效率真的很低，也沒(méi)有嚴(yán)格的規(guī)定，所以做的也比較隨意。恰巧最近看到一個(gè)論調(diào)，“越牛逼的團(tuán)隊(duì)，對(duì)于代碼審查的態(tài)度越嚴(yán)謹(jǐn)”，頓時(shí)引發(fā)共鳴，長(zhǎng)久以來(lái)在心里一直有一種這樣去檢查代碼的方式是不行的感覺(jué)，但是一直也沒(méi)找到合適的方式，而這次再聯(lián)想到代碼審查感覺(jué)如獲至寶，怎么一直都把它給忘了呢？！
如果你對(duì)代碼審查能帶來(lái)什么好處有疑問(wèn)的話，請(qǐng)仔細(xì)閱讀Phabricator官網(wǎng)的這篇文章。
代碼審查的方式
代碼審查主要有兩種方式：
1. pre-push：在提交合并代碼之前，先進(jìn)行審查，通過(guò)和才能合并。這是一種非常嚴(yán)格的審查方式，可以確保每個(gè)發(fā)布的代碼都是已經(jīng)被審查過(guò)的。這種放到在github上維護(hù)的開(kāi)源項(xiàng)目極其合適，代碼的所有者可以確保代碼是在自己的控制范圍。
2. post-push：代碼提交后，再審查之前的代碼。這是非常寬松的審查方式，審查的效果肯定是打折扣的，但是好處是可以忽略一些不必要的審查以節(jié)約時(shí)間。其實(shí)在國(guó)內(nèi)這種沒(méi)有太多工程師文化的地方，這種方式是比較好在早期推行的。
代碼審查的工具
這個(gè)事情在團(tuán)隊(duì)中實(shí)行的話，是一定需要有個(gè)工具的，相關(guān)的工具有很多，審查方式也各有偏重。這里工具主要是解決了這幾個(gè)問(wèn)題：
1. 有一個(gè)更為直觀的界面查看diff。
2. 可以基于工具進(jìn)行簡(jiǎn)單的標(biāo)記和通知，直接把標(biāo)記寫(xiě)在代碼里更利于溝通。
3. 可以知道哪些提交時(shí)已經(jīng)被誰(shuí)審查過(guò)了，方便審查的協(xié)作。
之前在sf寫(xiě)過(guò)一篇問(wèn)答可以參考。這里再例舉一些，供參考選擇。
1. Gerrit：google的產(chǎn)品，名氣很大，但是這個(gè)東西設(shè)計(jì)理念比較陳舊，據(jù)說(shuō)也沒(méi)有什么維護(hù)了，不推薦。
2. github pull request：這個(gè)當(dāng)然很好，典型的pre-push方式，但是個(gè)人用也沒(méi)太多協(xié)同的事情，團(tuán)隊(duì)用又覺(jué)得貴。其實(shí)感覺(jué)用bitbucket會(huì)經(jīng)濟(jì)實(shí)用些。
3. phabricator：facebook內(nèi)部使用并開(kāi)源出來(lái)的工具，功能超級(jí)強(qiáng)大，但相對(duì)的就是非常復(fù)雜，界面設(shè)計(jì)非常歐美的風(fēng)格，運(yùn)行速度也有點(diǎn)慢。東西還是很牛逼的，看你是不是喜歡了。
4. gitlab：如果是自己搭建的git server，這個(gè)是不錯(cuò)的選擇，相當(dāng)于自己弄了個(gè)github，就是配置環(huán)境會(huì)比較多工作量。
5. upsource：JetBrains的產(chǎn)品，只有post-push的方式，但是從安裝、界面、到使用都是挺不錯(cuò)的，唯一問(wèn)題就是10個(gè)人以上要收費(fèi)，而且還很貴。
我們從中的獲益
選用哪種方式，我覺(jué)得因團(tuán)隊(duì)文化、項(xiàng)目背景、效果預(yù)期而定,我們最后暫時(shí)選用的是upsource，目標(biāo)是先在團(tuán)隊(duì)中把代碼審查施行起來(lái)。
目前來(lái)看效果還可以。有了工具之后大家互相做代碼審查也方便很多，心理抗拒性也沒(méi)那么強(qiáng)。其實(shí)只要進(jìn)度不那么催，研發(fā)人員還是比較愿意去做這種事情的。審查過(guò)程中目前發(fā)現(xiàn)了一些代碼中的問(wèn)題，但是現(xiàn)在還不多，我想只要能在出現(xiàn)bug之前能解決掉一些問(wèn)題，就已經(jīng)有很大價(jià)值了。
分享題目：【博文推薦】如何提高團(tuán)隊(duì)代碼質(zhì)量——代碼審查的實(shí)踐
URL地址：http://www.5511xx.com/article/coipshg.html
