06cb6420cb1c5039ebc52141dc533746
Code Review体系与团队文化

Code Review不只是一种管理方法,也是开发者特有的沟通方式,更是一种团队文化。Code Review机制是否健全是评价一个研发团队技术氛围好坏的重要参考。

Code Review的意义

  • 交叉排查缺陷 - 绝大多数BUG都可以在代码层面被发现,甚至测试难以覆盖到的深层次BUG也可以通过团队成员相互审核而避免
  • 提高代码质量 - Code Review意味着开发者要接受团队成员的建议与监督,在完成功能的基础之上不断完善代码结构
  • 建立团队意识 - 代码是团队财产,团队成员在相互督促与改进中共同成长

Code Review的体系

  • Daily Code Review - 开发者完成初步结构设计,或者完成一个相对完整的小模块都可以提交PR让团队成员Review
  • 需求Code Review - 评估需求完成度,与其它需求的潜在冲突
  • 上线Code Review - 上线前Review,重点排查配置问题,安全问题,代码冲突
  • 重点代码走读 - 每个迭代/双周/月度对具有代表性的代码集体走读,重点在于解决团队共性问题,讨论改进方法

是整个Code Review体系中最重要的是Daily Code Review,绝大多数问题应该在每天的Code Review中沟通解决,也就是功夫在平时。

Code Review有几个重要的参考指标:

  • 单次提交Review的代码量 - 在思路相对完整的情况下越少越好
  • 评论的数量,参与程度 - 每个PR至少2人Review通过之后才能合并代码,但参与人数也不宜过多,避免盲目通过的情况
  • 更新的次数 - 通常情况下Code Review不应该直接通过,平均要Update至少一次
top Created with Sketch.