序
豆瓣數據API和后台數據同步已經完成,我大概兩天沒去管這部分代碼,主要是因為產品、老大都不是很積極。因為這個畢竟不是非常重要的。
今天我對代碼進行了審查,也進行了一些修復,比如命名規范,代碼邏輯,還有就是讓整個代碼更加符合整個框架的規范(如文件分類,指定文件做它該做的事情)。於是有了想寫一篇“代碼審查”博文的沖動,故有了此文。
盯哨
作為程序員應該知道Code Review的重要性。我相信很多程序員都不喜歡Code Review,那你喜歡它嗎?
在公司兩年多,一直是在原有的代碼基礎上進行代碼修復和功能添加。這樣勢必會經常接觸其他人代碼,自然而然就養成Code Review的習慣。
在Code Review的時候,我經常感覺我后腦勺發涼,老是覺得有人(呵呵,一般是老大)在盯着我。
我有沒有偷懶?
我有沒有完全弄懂邏輯?
我寫的代碼羅不羅嗦?
我有沒有按照命名規范進行編碼?
我有沒有寫出漂亮而完美的代碼?...
過程
現在,每次寫完代碼,我會間隔一段時間翻看舊代碼,如果發現不好的地方,我就會去進行修復(這應該就是Code Review了)。
我是一名phper,我不太喜歡使用工具去做這個Review,我喜歡用眼睛看、用腦去思考。
有人說了,你這個不標准,效率不高。——呵呵,畢竟我工作量我自認為不大,所以很多時間我一直在Code Review。
每次看我的代碼就像看一件藝術品,我時常問自己,我寫的代碼我滿意嗎?還缺點什么?我還能寫的更好嗎?
感覺
Code Review是個美妙的過程,讓我的腦袋從混沌逐漸清晰。剛開始,我會頭疼,但是隨着你不斷的Review,整個思路越來越明朗,代碼越來越規范。
人腦是個非常奇特的東西,混沌到清晰是一個痛苦到愉悅的過程。如果長期不經過這個過程,你腦袋很容易銹到。
那么混沌是一個什么感覺?我比較喜歡我外甥的一句話,“腦袋瓜子里就像進入一堆漿糊似的”,他把這個混沌比喻成漿糊。我相信等你把漿糊弄干凈了,你腦袋也就清晰了。
再舉一個不好聽的比喻,就像你把大便從肚子里排除去后的那種感覺。大便越多,最后排出去之后,感覺越爽。所以,你代碼中,越有很多理不清的東西,等你真正理清完之后,你的腦子會非常舒服,整個身體都會非常輕松。
方法
我一般如何Code Review呢?
初期:
我首先找到一個切入點(也許是一個頁面,也許是一個定時執行的腳本main()方法),然后逐個去找類,找方法,直到最后輸出。——不停的去理這個過程。
然后再找下一個切入點。
后期:
我會挑選幾個比較常見的業務,沒有太明白的業務,然后尋找切入點,在進行以上循環。
推薦
