# Day 28 - Review code

因團隊制宜。
假設桌球王成立團隊，將會有以下`Pricinple`、`SOP`、`CheckList`、`Take away`。

## Principle
以Priority由上而下:

0. 確認邏輯**無誤**
1. 確認如何操作APP
2. **了解**背後運作邏輯
3. 減少重複
: 重複使用的邏輯包成function
: 不用改一個地方，四處都要改
4. 增加可讀
: 命名
: 結構
: 刪除沒有使用的

## SOP & Document
可執行 round N ，Ｎ大於等於1。

round N ，依複雜性，及當round吸收程度來決定N為多少。
每 round 有其`主題`，比如說某次Release，加了哪些Feature、操作步驟/流程、前端架構，後端API、網路連線(關聯系統)、Config.等等。

最後的吸收理解可以邊看文件邊Trace Code，整個走一次確認是否大略知道重點，以便未來查案或加值。

## Check List
- [ ] 了解user怎麼操作，通常哪些使用頻繁
- [ ] 前後端交握，哪些前端處理，傳遞給後端，後端Status code 回傳
- [ ] 上下游(關聯) 系統，如何溝通，正常異常判別
- [ ] log 識別 以及 量的控制

## Take away
Review Code 的同時其實可以看到更多或是更好的寫法或架構，

如果可以看出原開發沒有發現的邏輯問題，就能即時在上線前排除，
也能想想看有沒機會在不改變輸出的情況，改出易於維運的Code。

其實是 reviewer 與 commiter(developer) 之間的切磋成長!

## 相關參考
- [Code Review 怎麼做](https://tw.alphacamp.co/blog/code-review)