代码审查(补充说明).ppt_第1页
代码审查(补充说明).ppt_第2页
代码审查(补充说明).ppt_第3页
代码审查(补充说明).ppt_第4页
代码审查(补充说明).ppt_第5页
已阅读5页,还剩7页未读 继续免费阅读

下载本文档

版权说明:本文档由用户提供并上传,收益归属内容提供方,若内容存在侵权,请进行举报或认领

文档简介

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

温馨提示

  • 1. 本站所有资源如无特殊说明,都需要本地电脑安装OFFICE2007和PDF阅读器。图纸软件为CAD,CAXA,PROE,UG,SolidWorks等.压缩文件请下载最新的WinRAR软件解压。
  • 2. 本站的文档不包含任何第三方提供的附件图纸等,如果需要附件,请联系上传者。文件的所有权益归上传用户所有。
  • 3. 本站RAR压缩包中若带图纸,网页内容里面会有图纸预览,若没有图纸预览就没有图纸。
  • 4. 未经权益所有人同意不得将文件中的内容挪作商业或盈利用途。
  • 5. 人人文库网仅提供信息存储空间,仅对用户上传内容的表现方式做保护处理,对用户上传分享的文档内容本身不做任何修改或编辑,并不能对任何下载内容负责。
  • 6. 下载文件中如有侵权或不适当内容,请与我们联系,我们立即纠正。
  • 7. 本站不保证下载资源的准确性、安全性和完整性, 同时也不承担用户因使用这些下载资源对自己和他人造成任何形式的伤害或损失。

评论

0/150

提交评论