1. 拼錯字:WARM,WARN傻傻分不清科科,Code Review這樣就結束了?所以,你也滿意了嗎?真有那麼簡單,我還需要特別寫一篇網誌來屁嗎?
2. 沒寫註解
3. Code很臭,斷行縮排不照規矩
4. 變數命名不依照團隊編碼規範
2014年1月26日 星期日
005 整數陣列搜尋大法 (參考解)
回應的人太多了,就如同我當年的仰慕者可以從總統府排隊排到西門町那樣的多,篇幅有限,以下僅列出前四大回應
2014年1月14日 星期二
005 整數陣列搜尋大法 (找碴篇)
大家來找碴之 Code Review 好好玩
試想一個情境,同事A被要求撰寫一個小模組,規格如下
在一個整數陣列中搜尋特定整數;
若尋得,返回該值位於陣列中的位移值;
若無得,返回-1
因此,同事A花了幾分鐘,寫出了以下這幾段程式碼,並且很貼心的提供兩種search實作方式,給一般人使用的while 版本 (binarySearch_int) 或給神人使用的 recursive 版本 (binarySearch_int_R).
現在問題來了,本團隊有採用 Code Review + Code Inspection 的習慣;現在請模擬你正處在會議室中,你是否有能力透過人腦編譯器你的專業能力,找出程式碼中可能有的問題或給予其他實質的建議?
訂閱:
文章 (Atom)