Download as PDF, PPTX
Uploaded byKen Kuan
不妥協的河蟹之道
本文探讨了代码审查的重要性，强调通过不妥协的代码质量和和谐的工作氛围来提升团队协作。代码审查不仅能改善团队气氛和增强学习能力，还能维持软件质量。最终目标是通过责任和信任来实现团队的和谐合作。
不妥協的河蟹之道
- 1.
- 2. ⾃自我介紹 • 阿官 -Ken Kuan • Hooloop - Extreme Programmer • 專注於 • iOS, Android, Scrum, CI, Testing
- 3.
- 4.
- 5.
- 6.
- 7. What is CodeReview • 就是程式碼審查 • 分為： • 直接 review • 透過⼯工具 (email, github) • Pair Programming
- 8.
- 9. How we review Review! GitFlow+ GitHub
- 10.
- 11.
- 12. Code Review 三⼤大妙⽤用 •改善團隊氣氛 • 強化團隊學習 • 維持軟體品質
- 13.
- 14. 情境模擬 • ⼀一早上班更新程式碼後，看到了些許錯誤，你該 怎麼辦呢？ A. 默默幫他改掉 B.⾛走過去拍拍他肩膀 C. 裝作沒看到
- 15.
- 16.
- 17.
- 18. C、裝作沒看到 • 肇事者 -毫無影響 • 你 - 悶久了也會內傷 軟體品質低落
- 19.
- 21.
- 22. 權⼒力 • Code Review賦予團隊成員 review code 的權 ⼒力！ • 有沒有權⼒力差很多嗎？
- 23.
- 24.
- 25.
- 26.
- 27. C、裝作沒看到 • Reviewer 不只是擁有review code 的權⼒力，更有 認真 review 的責任
- 28. 責任 • 賦予團隊成員 review權⼒力的同時， reviewer 也擁 有承擔該段 code 的責任。 Code 的好壞，⼈人⼈人有責！
- 29.
- 31.
- 32.
- 33. 贏了⾯面⼦子，輸了裡⼦子 • 不犯錯 =各⾃自為政 • 出了問題，找代罪羔⽺羊
- 34.
- 35.
- 36.
- 37. 責任分擔 • Reviewer 和Author 同時 OWN 該段 code • Reviewer 責任更⼤大 • 不再推卸責任 • 增加責任感
- 38. 另⼀一個好處：信任 • reviewer 給予的建議，是為了產品更好 •⾝身為 author ，也會更加信任 reviewer 的 feedback • 培養團隊成員的信任感，達到良好的合作模式
- 39. 最終⺫⽬目標：和諧 • 真正的和諧是⼀一起為了共同的⺫⽬目標⽽而努⼒力，⽽而⾮非 沈默通過⼀一個提案 • 責任、信任 •指責 -> 解決問題 • 了解彼此個性
- 40. Code Review 妙⽤用2 強化團隊學習
- 41.
- 42.
- 43.
- 44. 如果沒有 code review •什麼時候會看別⼈人的 code ？ • 寫新功能剛好需要寫到 • Demo 出包，找戰犯修 bug 的時候 幾乎沒有！
- 45. 學習最⼤大的動⼒力 強迫看 code • 在責任感的驅使下，硬著頭⽪皮也要看懂 •學習 > 指導 • 透過交流進⽽而理解別⼈人想法
- 46.
- 47.
- 48.
- 49.
- 50.
- 51.
- 52. 結論 • 改善團隊氣氛 • 強化團隊學習 •維持軟體品質
- 53.
- 54.