代码检视交流

《代码检视交流》由会员上传分享，免费在线阅读，更多相关内容在行业资料-天天文库。

1、华为通讯质量部代码检视交流2021/10/26目录Review的目的Review流程代码Review计划代码Review关注点代码Review实施代码Review结果分析Page2Review的目的早期发现缺陷去除缺陷降低成本提高质量相对于通过review发现缺陷的rework成本，测试发现缺陷的rework成本是其14.5倍,而客户发现缺陷的rework成本是其68倍－－一家德国软件公司产品发布后发现缺陷的rework成本是在设计阶段的45倍Page3Review流程——角色PL作者讲解员REVIEW人员记录员组织者可兼任不可兼任各司其

2、职Page4Review流程——步骤出口准则介绍会议？1.计划阶段3.准备阶段2.介绍会议YN7.跟踪阶段6.返工阶段YN第三小时会议？4.Review会议5.第三小时会议入口准则Page5代码review计划有两种估计review工作量的方式，可以只选取其中一种方式，也可分别采取两种方式估计然后折衷两个结果：代码review工作量占编码活动总工作量（包括写代码、代码review和修改）的20%左右。在计划阶段进行工作量估计和制定WBS时可以用这种方式。根据代码量来估计，平均每千行代码的review工作量（包括预审和会议）建议在6人时左右

3、。这种方式可以用来在编码阶段制定详细review计划时对review计划进行调整。Page6代码review计划总体安排方式一：每完成一部分就立即review方式二：所有代码集中review方式三：分阶段集中review不管采取那种安排方式，都要遵循一条最基本的原则：review多少代码，就要投入多少工作量。Page7代码Review关注点代码实现与设计是否一致；代码运行时的出错、异常的检测、处理是否完备；代码的可读性和易理解性；特殊代码需要制定特殊的关注重点；将缺陷预防活动中分析总结的常见代码错误作为代码Review的重点;项目组根据组

4、织级Checklist或者编码规范制定的Checklist简化以后得到的项目组公共Checklist。详情请参见《DVPCHK02-CodeReviewChecklist.xls》Page8代码检视常见问题字符串拷贝没有判断长度，字符串结尾没有0外部接口传入的数组长度没有判断，特别是命令行接口的命令长度是可变的循环时没有仔细考虑原始数据的长度遍历链表/树等数据结构时，对表头或表尾操作错误空指针操作，特别是没有判断外部接口传入的指针的合法性资源（内存/文件/网络）泄漏，特别是函数异常退出时If语句中的&&,

5、

6、写反了If的Else考虑不周，

7、特别是多个elseSwitch-case语句的default,break存在问题缺乏对某些异常的处理（和业务相关，质量人员较难发现）变量初始化错误，使用memset将大块内存初始化为0，但实际上部分变量的初始值不应该为0（和业务相关，质量人员较难发现）Page9Review实施入口检查介绍会议（可选）预审Review会议第三小时会议问题跟踪Review过程监控请参见《CodeReviewGuideline.doc》3.2.2节Page10Review结果分析具体分析请参见《CodeReviewGuideline.doc》3.3.1.3节P

8、age11