代码审查(补充说明).ppt

代码审查 补充说明,代码审查的目的,提高软件的质量减少测试和维护软件的工作量提高编码人员的技术水平和代码审查能力,代码审查=代码自查+代码互查,编码,编译通过,进行代码自查,提交自查结果、原始代码,讲解代码,进行代码审查,提交审查结果,选择审查者,确认审查者,调试完毕，提交单元测试,Y 有错？N,讨论复查结果查者,此段时间可以审查其他人代码、（有限地）调试程序、编制测试程序；但不能进行任何测试。,编码人员：开始,编码,编译通过,进行代码自查,提交自查结果、原始代码,讲解代码,互查者：开始,进行代码互查,提交互查结果,选择互查者,确认互查者,调试完毕，提交单元测试,讨论互查结果,此段时间可以审查其他人代码、（有限地）调试程序、编制测试程序；但不能进行任何测试。,修改,对于固定的审查关系，这些步骤不需要。,代码审查的时机,代码自查在编译后、调试前进行 代码互查在自查后、任何测试前进行。,编译,自查,互查,测试,调试,代码审查的工作量,在编码期中每天进行代码审查 自查当天所有代码 互查当天的核心代码 每天（自查+互查+填表）时间不应超过1 小时。可以酌情调整,所有代码都必须进行自查。,对代码审查人员的要求,有认真负责的精神；要了解代码的需求和设计；就事论事，不要进行人身攻击。,如何选择互查人员,交叉使用者；该模块的替补开发者；直接技术领导或负责人。,不合理的审查关系,新手给高手审查；由无关人员审查；审查关系固定的时间太长。,什么代码需要互审,交付用户使用的程序中重要的、复杂的、需要重用的或者底层部分；用于验证设计的原型程序的核心部分；长期使用的仿真软件；长期使用的自动测试程序中的核心部分。,代码审查关注什么样的缺陷,代码与需求不符合（如不符合业务逻辑，没有考虑功 能支持的用户量）；代码逻辑与设计逻辑不符合；程序可读性不好；程序没有考虑性能、内存占用、关键资源的共享使用、可维护性等 因素；代码技术错误（空指针未检查、new/delete不匹配、文件未关闭、数组越界、异常处理错误等）；代码语义错误或模糊；低级错误（变量误用、拼写错误、符号错误等）；,谢谢!,