代码检视交流.ppt_第1页
代码检视交流.ppt_第2页
代码检视交流.ppt_第3页
代码检视交流.ppt_第4页
代码检视交流.ppt_第5页
已阅读5页，还剩7页未读， 继续免费阅读

下载本文档

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

文档简介

2020/5/18,华为通讯质量部,代码检视交流,Page2,目录,Review的目的Review流程代码Review计划代码Review关注点代码Review实施代码Review结果分析,Page3,Review的目的,早期发现缺陷去除缺陷降低成本提高质量,相对于通过review发现缺陷的rework成本，测试发现缺陷的rework成本是其14.5倍,而客户发现缺陷的rework成本是其68倍一家德国软件公司产品发布后发现缺陷的rework成本是在设计阶段的45倍,Page4,Review流程角色,可兼任,不可兼任,各司其职,Page5,Review流程步骤,Page6,代码review计划,有两种估计review工作量的方式，可以只选取其中一种方式，也可分别采取两种方式估计然后折衷两个结果：代码review工作量占编码活动总工作量（包括写代码、代码review和修改）的20%左右。在计划阶段进行工作量估计和制定WBS时可以用这种方式。根据代码量来估计，平均每千行代码的review工作量（包括预审和会议）建议在6人时左右。这种方式可以用来在编码阶段制定详细review计划时对review计划进行调整。,Page7,代码review计划,总体安排方式一：每完成一部分就立即review方式二：所有代码集中review方式三：分阶段集中review不管采取那种安排方式，都要遵循一条最基本的原则：review多少代码，就要投入多少工作量。,Page8,代码Review关注点,代码实现与设计是否一致；代码运行时的出错、异常的检测、处理是否完备；代码的可读性和易理解性；特殊代码需要制定特殊的关注重点；将缺陷预防活动中分析总结的常见代码错误作为代码Review的重点;项目组根据组织级Checklist或者编码规范制定的Checklist简化以后得到的项目组公共Checklist。详情请参见DVPCHK02-CodeReviewChecklist.xls,Page9,代码检视常见问题,字符串拷贝没有判断长度，字符串结尾没有0外部接口传入的数组长度没有判断，特别是命令行接口的命令长度是可变的循环时没有仔细考虑原始数据的长度遍历链表/树等数据结构时，对表头或表尾操作错误空指针操作，特别是没有判断外部接口传入的指针的合法性资源（内存/文件/网络）泄漏，特别是函数异常退出时If语句中的&,|写反了If的Else考虑不周，特别是多个elseSwitch-case语句的default,break存在问题缺乏对某些异常的处理（和业务相关，质量人员较难发现）变量初始化错误，使用memset将大块内存初始化为0，但实际上部分变量的初始值不应该为0（和业务相关，质量人员较难发现）,Page10,Review实施,入口检查介绍会议（可选）预审Review会议第三小时会议问题跟踪Review过程监控请参见CodeReviewGuideline.doc
人人文库> 全部分类> 应用文书 > 技术指导

温馨提示

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

最新文档

评论

0/150

提交评论