不妥協的河蟹之道

1. 不妥協的河蟹之道
你今天 code review 了嗎？

2. ⾃自我介紹
• 阿官 - Ken Kuan
• Hooloop - Extreme Programmer
• 專注於
• iOS, Android, Scrum, CI, Testing

3. 何謂不妥協的河蟹之道？

4. 不妥協的軟體品質

5. 和諧的⼯工作氣氛

6. 軟體品質 ⼯工作氣氛

7. What is Code Review
• 就是程式碼審查
• 分為：
• 直接 review
• 透過⼯工具 (email, github)
• Pair Programming

8. 低密度
⾼高密度時間⻑⾧長
時間短
直接
間接

9. How we review
Review!
GitFlow + GitHub

10. Live Demo

11. 究竟，Code Review 如
何讓團隊不妥協⼜又和諧？

12. Code Review 三⼤大妙⽤用
• 改善團隊氣氛
• 強化團隊學習
• 維持軟體品質

13. Code Review 妙⽤用⼀一
改善團隊氣氛

14. 情境模擬
• ⼀一早上班更新程式碼後，看到了些許錯誤，你該
怎麼辦呢？
A. 默默幫他改掉
B. ⾛走過去拍拍他肩膀
C. 裝作沒看到

15. A、默默幫他改掉
你 肇事者

16. B、⾛走過去拍拍他肩膀
• 你：經過⼀一番掙扎
開放式辦公室

17. 肇事者⼼心理三部曲
真謝謝啊 夠了喔！ 你針對我啊

18. C、裝作沒看到
• 肇事者 - 毫無影響
• 你 - 悶久了也會內傷
軟體品質低落

19. ⻑⾧長此以往......

21. Code Review 如何解決上
述問題？
權⼒力 責任

22. 權⼒力
• Code Review 賦予團隊成員 review code 的權
⼒力！
• 有沒有權⼒力差很多嗎？

23. 賦予⽼老⿃鳥、菜⿃鳥同等權⼒力
Reviewer Author

24. 讓我們回到剛剛的情境

25. • 程式碼都還沒進來
• 可以在系統上直接反應
A、默默幫他改掉

26. B、⾛走過去拍拍他肩膀
• 你、肇事者：⼀一切照規定來！

27. C、裝作沒看到
• Reviewer 不只是擁有 review code 的權⼒力，更有
認真 review 的責任

28. 責任
• 賦予團隊成員 review 權⼒力的同時， reviewer 也擁
有承擔該段 code 的責任。
Code 的好壞，⼈人⼈人有責！

29. 情境⼆二：⼯工程師最害怕
的夢魘

31. 腥⾵風⾎血⾬雨的檢討會議
找出兇⼿手！

32. 找出兇⼿手以後？

33. 贏了⾯面⼦子，輸了裡⼦子
• 不犯錯 = 各⾃自為政
• 出了問題，找代罪羔⽺羊

34. 以不犯錯為最⾼高指導原則
犧牲軟體品質！

35. 難道，犯錯真的是肇事者
⼀一個⼈人的問題嗎？

36. Scrum
• 團隊！

37. 責任分擔
• Reviewer 和 Author 同時 OWN 該段 code
• Reviewer 責任更⼤大
• 不再推卸責任
• 增加責任感

38. 另⼀一個好處：信任
• reviewer 給予的建議，是為了產品更好
• ⾝身為 author ，也會更加信任 reviewer 的
feedback
• 培養團隊成員的信任感，達到良好的合作模式

39. 最終⺫⽬目標：和諧
• 真正的和諧是⼀一起為了共同的⺫⽬目標⽽而努⼒力，⽽而⾮非
沈默通過⼀一個提案
• 責任、信任
• 指責 -> 解決問題
• 了解彼此個性

40. Code Review 妙⽤用 2
強化團隊學習

41. 只有新⼿手需要學習？
• ⽼老⿃鳥更需要學習
軟體的開發技術與觀念與時俱進，往往不
到⼀一年就過時了！

42. ⽼老⿃鳥學習的障礙
• 科學實驗指出：年紀⼤大學習慢不是因為⽣生理退化，
⽽而是因為被⾃自⾝身經驗、觀念束縛！

43. 最好的學習⽅方式
• 看別⼈人寫的 code

44. 如果沒有 code review
• 什麼時候會看別⼈人的 code ？
• 寫新功能剛好需要寫到
• Demo 出包，找戰犯修 bug 的時候
幾乎沒有！

45. 學習最⼤大的動⼒力
強迫看 code
• 在責任感的驅使下，硬著頭⽪皮也要看懂
• 學習 > 指導
• 透過交流進⽽而理解別⼈人想法

46. Code Review 妙⽤用三
維持軟體品質

47. 對 Code Review 的質疑

48. 浪費時間？

49. code review 發現問題
的機率？

50. 強化軟體品質
相當⾼高！

51. 改善品質 = 節省時間

52. 結論
• 改善團隊氣氛
• 強化團隊學習
• 維持軟體品質

53. 有了 code review 以後

54. kenkuan@hooloop.com