代码评审的目的
第一个目的:确保要发布质量可靠的代码,换句话说,它对于开发过程中的下个阶段是否应该提高代码的质量是个严峻的考验。代码评审能非常有效地发现所有类型的错误,包括那些由于不正确的结构引起的错误,那些不适合商业进程的错误,还有那些简单的冗余错误。这就是为什么它对于代码质量来说是个有效的试金石。
第二个目的:作为教学工具帮助开发人员学会何时并且如何应用技术来提高代码的质量、一致性和维护性。经过反复仔细地评估代码,开发人员有机会掌握不同的和潜在的更好的编码方法。
评审前准备
1、 确定评审代码的checklist;
2、 会议前两天通知评审会议时间、地点、人员及评审代码;
代码评审会议
1) 代码编写人介绍本次代码评审业务逻辑、设计思路、现有问题(10分钟)
2) 强调评审的特别关注点(5分钟)
4) 深入地评审代码(55分钟)
a)代码架构
b)数据结构、算法设计
1) 代码编写人介绍本次代码评审业务逻辑、设计思路、现有问题(10分钟)
2) 强调评审的特别关注点(5分钟)
4) 深入地评审代码(55分钟)
a)代码架构
b)数据结构、算法设计
c) 代码规范
d)异常、容错处理
5) 总结优点与改进点(10分钟)
评审结果跟进
代码负责人在评审三天后提交改进报告于代码评审人员;
评审注意事项
代码评审作为代码分析方法中的一种,以参与者处理代码开始,评审过程仿佛被设计成证明谁是更好的程序员一样。代码评审常常会变成人身的攻击,在这里人们瞄准对象进行攻击;换句话说,编写代码的开发人员在被复查。更好的途径是采用学习的方法,所有的评审工作都被当作讨论的内容来讨论并向其他人学习。就是说这种方法应该是有教育意义
d)异常、容错处理
5) 总结优点与改进点(10分钟)
评审结果跟进
代码负责人在评审三天后提交改进报告于代码评审人员;
评审注意事项
代码评审作为代码分析方法中的一种,以参与者处理代码开始,评审过程仿佛被设计成证明谁是更好的程序员一样。代码评审常常会变成人身的攻击,在这里人们瞄准对象进行攻击;换句话说,编写代码的开发人员在被复查。更好的途径是采用学习的方法,所有的评审工作都被当作讨论的内容来讨论并向其他人学习。就是说这种方法应该是有教育意义