Code Review 在軟體開發是佔很重要的一個環節，通常聽到的情況是在要 commit code 前、pull request 之前，或是 fixing tracker 之前的一個動作，確保新增、修改的部分不會影響既有的架構，也不會產生額外的問題。

如果是 fixing tracker ，抓緊幾個問題：

• 為什麼要修？
• 修了哪些東西？
• 和之前差別是什麼？
• 有沒有影響到其他的地方？
• 為什麼修這樣就修的好？

今天某匿名大神同事分享了幾個重點

1. Reviewer 要了解問題是什麼，如果不清楚，不是 requester 沒有寫清楚的話，要請 requester 解釋清楚
2. 如果是 workaround 要問清楚採取 workaround 的原因
3. Reviewer 從自己的理解，看看新的 design or implementation 有沒有問題。特別注意一般容易會被忽略的地方，eg, error handling, boundary, etc 以及會不會造成其他 side effects
4. 要了解一下之前沒有被測到的原因

以往修 tracker 大家都是憑直覺修，但是像這樣具體列出來，拳拳到肉真的很有感覺！