Code Review 在軟體開發是佔很重要的一個環節,通常聽到的情況是在要 commit code 前、pull request 之前,或是
fixing tracker 之前的一個動作,確保新增、修改的部分不會影響既有的架構,也不會產生額外的問題。
如果是 fixing tracker ,抓緊幾個問題:
- 為什麼要修?
- 修了哪些東西?
- 和之前差別是什麼?
- 有沒有影響到其他的地方?
- 為什麼修這樣就修的好?
今天某匿名大神同事分享了幾個重點
- Reviewer 要了解問題是什麼,如果不清楚,不是 requester 沒有寫清楚的話,要請 requester 解釋清楚
- 如果是 workaround 要問清楚採取 workaround 的原因
- Reviewer 從自己的理解,看看新的 design or implementation 有沒有問題。特別注意一般容易會被忽略的地方,eg, error handling, boundary, etc 以及會不會造成其他 side effects
- 要了解一下之前沒有被測到的原因
以往修 tracker 大家都是憑直覺修,但是像這樣具體列出來,拳拳到肉真的很有感覺!