1.代码评审的好处
- 提高代码质量。评审别人的代码,自己的水平也能提高
- 修复bug的代价最小。自己内部人指出问题,代价最小;其次是测试部门测来;最严重时来自用户反馈。
- 促进团队之间相互备份。评审别人的代码的同时,可以强制每个人尽可能熟悉别人的代码,可以形成团队成员之间的相互备份。
2.代码评审的关键---流程化
如何做代码评审?具有可操作性的经验:
- 规定一个固定的周期去做代码评审,比如两周一次。团队成员尽量在代码评审前完成自己模块的代码,能达到一个可被评审的状态。
- 一定要有个主审人提前去看代码,否则效果大打折扣。可以尝试主审人讲解为主,被评审的人答疑为辅的形式。
- 被审核的开发者,最好能主动提出哪些代码可能存在潜在的问题,以节省时间。
- 会议中提出的疑问点,会议中当场讨论。会议上的最终决议形成新的任务记录,来跟踪相关程序员修改。
- 会议上大家有机会头脑风暴。
- 每个人主要从以下3个方面去进行代码评审
- 变量名和函数名是否准确?
- 是否有明显的bug?
- 整体设计是否有优化的空间?
代码平生时提高软件整体质量的重要手段,也是提高程序员编程素养的捷径。我们每个人都应该从评审自己开始,进而去评审别人,提高项目质量的同时,也促进大家共同进步。