序
豆瓣數(shù)據(jù)API和后臺數(shù)據(jù)同步已經(jīng)完成,我大概兩天沒去管這部分代碼,主要是因為產(chǎn)品、老大都不是很積極。因為這個畢竟不是非常重要的。
今天我對代碼進行了審查,也進行了一些修復(fù),比如命名規(guī)范,代碼邏輯,還有就是讓整個代碼更加符合整個框架的規(guī)范(如文件分類,指定文件做它該做的事情)。于是有了想寫一篇“代碼審查”博文的沖動,故有了此文。
盯哨
作為程序員應(yīng)該知道Code Review的重要性。我相信很多程序員都不喜歡Code Review,那你喜歡它嗎?
在公司兩年多,一直是在原有的代碼基礎(chǔ)上進行代碼修復(fù)和功能添加。這樣勢必會經(jīng)常接觸其他人代碼,自然而然就養(yǎng)成Code Review的習(xí)慣。
在Code Review的時候,我經(jīng)常感覺我后腦勺發(fā)涼,老是覺得有人(呵呵,一般是老大)在盯著我。
我有沒有偷懶?
我有沒有完全弄懂邏輯?
我寫的代碼羅不羅嗦?
我有沒有按照命名規(guī)范進行編碼?
我有沒有寫出漂亮而完美的代碼?...

過程
現(xiàn)在,每次寫完代碼,我會間隔一段時間翻看舊代碼,如果發(fā)現(xiàn)不好的地方,我就會去進行修復(fù)(這應(yīng)該就是Code Review了)。
我是一名phper,我不太喜歡使用工具去做這個Review,我喜歡用眼睛看、用腦去思考。
有人說了,你這個不標準,效率不高。——呵呵,畢竟我工作量我自認為不大,所以很多時間我一直在Code Review。
每次看我的代碼就像看一件藝術(shù)品,我時常問自己,我寫的代碼我滿意嗎?還缺點什么?我還能寫的更好嗎?

感覺
Code Review是個美妙的過程,讓我的腦袋從混沌逐漸清晰。剛開始,我會頭疼,但是隨著你不斷的Review,整個思路越來越明朗,代碼越來越規(guī)范。
人腦是個非常奇特的東西,混沌到清晰是一個痛苦到愉悅的過程。如果長期不經(jīng)過這個過程,你腦袋很容易銹到。
那么混沌是一個什么感覺?我比較喜歡我外甥的一句話,“腦袋瓜子里就像進入一堆漿糊似的”,他把這個混沌比喻成漿糊。我相信等你把漿糊弄干凈了,你腦袋也就清晰了。
再舉一個不好聽的比喻,就像你把大便從肚子里排除去后的那種感覺。大便越多,最后排出去之后,感覺越爽。所以,你代碼中,越有很多理不清的東西,等你真正理清完之后,你的腦子會非常舒服,整個身體都會非常輕松。

方法
我一般如何Code Review呢?
初期:
我首先找到一個切入點(也許是一個頁面,也許是一個定時執(zhí)行的腳本main()方法),然后逐個去找類,找方法,直到最后輸出。——不停的去理這個過程。
然后再找下一個切入點。
后期:
我會挑選幾個比較常見的業(yè)務(wù),沒有太明白的業(yè)務(wù),然后尋找切入點,在進行以上循環(huán)。

推薦

喜歡編程
浙公網(wǎng)安備 33010602011771號