建站
咨詢
售后
建站咨詢
新聞中心
身處在一個(gè)卓越開(kāi)明的開(kāi)發(fā)團(tuán)隊(duì)，你被安排了一整天的時(shí)間，什么都不干，只做代碼審查。然而，在活動(dòng)開(kāi)始兩小時(shí)前，你發(fā)現(xiàn)自己把近視眼鏡忘家里了，整個(gè)早上你看到的都是模糊的影像和顏色。你該怎么辦？
創(chuàng)新互聯(lián)服務(wù)項(xiàng)目包括七里河網(wǎng)站建設(shè)、七里河網(wǎng)站制作、七里河網(wǎng)頁(yè)制作以及七里河網(wǎng)絡(luò)營(yíng)銷策劃等。多年來(lái)，我們專注于互聯(lián)網(wǎng)行業(yè)，利用自身積累的技術(shù)優(yōu)勢(shì)、行業(yè)經(jīng)驗(yàn)、深度合作伙伴關(guān)系等，向廣大中小型企業(yè)、政府機(jī)構(gòu)等提供互聯(lián)網(wǎng)行業(yè)的解決方案，七里河網(wǎng)站推廣取得了明顯的社會(huì)效益與經(jīng)濟(jì)效益。目前，我們服務(wù)的客戶以成都為中心已經(jīng)輻射到七里河省份的部分城市，未來(lái)相信會(huì)繼續(xù)擴(kuò)大服務(wù)區(qū)域并繼續(xù)獲得客戶的支持與信任！
正確的做法是，回家取你的眼鏡，因?yàn)椴叫惺昼娋湍艿郊遥缓蠼裉鞎?huì)跟往常一樣愉快的度過(guò)。但是，如果說(shuō)，是你在早上準(zhǔn)備離家上班時(shí)，發(fā)現(xiàn)一群兇猛好斗的大黃蜂在放眼鏡的抽屜里筑起了蜂巢，擋住了你拿眼鏡的途徑，而且它們看起來(lái)很不喜歡被打攪。那你怎么辦呢？
另一個(gè)正確的做法，很顯然，是假裝你帶了隱形眼鏡，免得讓自己很尷尬。而且你知道自己有不看任何代碼細(xì)節(jié)、只看大概就能說(shuō)出很多讓人欽佩的意見(jiàn)的能力。
代碼審查 例一
我們都認(rèn)可代碼責(zé)任應(yīng)該絕對(duì)的分離。任何類都應(yīng)該只做它自己職責(zé)范圍內(nèi)的事情。但是，你的這個(gè) UserCreator很可能有點(diǎn)過(guò)了。如果這個(gè) UserCreator對(duì)象只做這一點(diǎn)事，那其實(shí)Users 對(duì)象自己就應(yīng)該創(chuàng)建自己。另外一點(diǎn)不好的是，創(chuàng)建一個(gè)簡(jiǎn)單的Users，你還需要在成堆的小文件中找出它的創(chuàng)建者對(duì)象，不宜操作，也不宜理解。
代碼審查 例二
看看這些一大堆的方法函數(shù)，卻偽裝成一個(gè)類，我可以看到，從技術(shù)上講，這些方法都是在各自做自己的事情。雖然沒(méi)有任何的文字信息提示或暗示，我也能猜出你沒(méi)有很好的給這個(gè)類寫測(cè)試程序。如果給我一杯濃咖啡、一個(gè)下午時(shí)間，我想我可以分析出中間這20行的代碼是用來(lái)判斷應(yīng)該給哪個(gè)用戶發(fā)送郵件，但我還是希望你將這部分代碼放到def users_to_send_emails_to函數(shù)里，免得我再去動(dòng)腦子。
代碼審查 例三
很好，在這個(gè)類里，你的方法都非常的簡(jiǎn)潔短小，這是一個(gè)進(jìn)步。然而，你做的是有點(diǎn)過(guò)了。雖然Ruby解釋器并不在意你的代碼邏輯在每個(gè)只有一行的方法間來(lái)回跳躍，但人腦解釋器會(huì)在意。也許有人會(huì)喜歡上下翻屏看代碼，但如果換成我，讓我在手臂上記下哪個(gè)方法應(yīng)該調(diào)用哪個(gè)方法，那我更喜歡將這些小方法組合到一起。
代碼審查 例四
可以看出，你非常關(guān)心這個(gè)類是否被放在了合適的命名空間里。非常好，使用命名空間是個(gè)好事。但在這個(gè)文件里，你嵌套了6層?？雌饋?lái)你試圖在一個(gè)小小的地方里裝下太多的東西。你要么應(yīng)該想想不要分那么多層(是的，我可以看到這里有兩個(gè)輔助類，應(yīng)該放到它們自己的空間里，但如果把它放到其它地方會(huì)有不好嗎？)，要么拆分一些代碼，放到自己的根空間下。
代碼審查 例五
非常詳細(xì)的注釋，佩服！這段代碼需要好幾個(gè)章節(jié)的文字來(lái)輔助理解，佩服！
代碼審查 例六
仔細(xì)看一下這第二個(gè)方法。如果一個(gè)方法需要8個(gè)參數(shù)才能讓它知道干什么事情、如何干事情，那么，這個(gè) 方法有點(diǎn)太累了。請(qǐng)重構(gòu)它，從它的肩膀上消減一些負(fù)擔(dān)，拿走一些壓力。把它一分為二(或更多)，或者，也許將一些參數(shù)當(dāng)成類的初始化參數(shù)，可能會(huì)更有意義些。這8個(gè)參數(shù)你會(huì)同時(shí)都用到嗎？所以，不要期望你的方法這樣。
所以，這就是當(dāng)你忘了拿近視眼鏡、或直視太陽(yáng)太久時(shí)做代碼審查的方法。如果你有豐富的編程經(jīng)驗(yàn)，我相信你也能舉出很多有趣的例子。有人可能會(huì)反對(duì)，說(shuō)這些只是一些小事情，還有很多更重要的事情需求考慮。然而，清爽、簡(jiǎn)潔、良好格式的代碼是你需要的，如果你不去用心控制好你的代碼結(jié)構(gòu)，它終究會(huì)給你帶來(lái)煩惱。
英文原文：
Code review without your glasses
譯文出自：http://www.vaikan.com/code-review-without-your-eyes/
分享標(biāo)題：如果代碼審查時(shí)你忘記了拿近視眼鏡
鏈接URL：http://m.5511xx.com/article/dhjcopc.html
