1.进度计划
小组代码走查活动时间进度安排如下所示:
| 工作任务 | 时间安排 |
| 制定编码规范文档 | 2019年6月8日 |
| 制定代码走查CheckList,提交待评审项目 | 2019年6月8日 |
| 评审人员执行个人走查,利用工具记录发现的问题 | 2019年6月9日 |
| 小组走查会议,完成缺陷记录报告, | 2019年6月9日 |
| 开发人员完成代码修改 | 2019年6月10日 |
| 评审人员再次走查修改过的代码 | 2019年6月11日 |
| 跟踪发现的问题直至问题关闭 | 2019年6月11日 |
待评审物名称:九宫格程序源代码
3.成员角色
| 成员 | 角色 | 提交成果物 |
| 陶浩然 | 组长 | 代码走查报告 |
| 肖仲浩 | 质量保证人员 | 代码需求分析、代码检查单 |
| 陶浩然 | 开发人员 | 修改前后代码,展示PPT |
| 肖仲浩 | 评审人员 | 填写后代码检查单,走查改进建议 |
| 邓卓轩 | 评审人员 | 填写后代码检查单,走查过程描述 |
质量保证人员:分析代码走查需求,制定CheckList,记录代码走查会议以及完成问题记录报告;
开发人员:完成代码,在代码走查中引领走查人员读代码,走查结束后并根据走查的问题记录报告完成代码修改;
评审人员:依据编程规范和CheckList执行代码走查,记录发现的问题。
4.基本原则
4.1代码评审原则
1.努力达到一个合适的检查速度
2.有足够的时间、以适当的速度、仔细地检查,但不宜超过60~90分钟
3.在复审前,代码作者应该对代码进行注释
4.建立量化的目标并获得相关的指标数据,从而不断改进流程
5.使用检查表(checklist)肯定能改进双方(作者和复审者)的结果
6.验证缺陷是否真正被修复
7.管理人员要营造良好的氛围(文化),使大家可以积极地对待缺陷的发现,发现足够多的缺陷,只关心问题是什么、怎样引起的,而不关心是谁写的代码
8.自我约束:即使没有时间完成所有代码的检查,也应该尽可能去做,哪怕是一部分
9.轻量级的code review是高效率的、可行的,并能有效地发现缺陷
4.2评审指导文档
1.《代码走查需求分析》
2.《代码走查单》
5.走查过程定义
5.1代码走查计划准备阶段
主要活动:
1.开发人员提交待评审代码及其需求文档,提出走查申请;
2.组长审核及批准走查申请;
3.QA制定走查计划、代码检查单及规范文档,生成待评审包;
4.组长将待评审包上传。
出口准则:待评审包(包含源代码及其需求文档、代码检查单和规范文档)
5.2个人代码走查阶段
主要活动:
1.小组人员下载待评审包,预读代码;
2.组长制定走查任务,上传至;
3.评审人员获得走查任务文件,参照需求文档、代码检查单和规范,记录所发现问题,完成个人走查,将生成文件上传;
4.QA收集并整合文件,生成个人走查问题记录单。
出口准则:整合后文件,个人走查问题记录单。
5.3代码走查会议阶段
主要活动:
1.参会人员携带个人走查工作成果按时入场,由组长宣布小组走查会议开始;
2.由开发人员解释程序功能、实现方式、代码结构、主要业务和逻辑流程等,引导评审人员阅读代码,评审人员以轮流发言的方式提出个人走查时发现的问题,QA使用记录此过程中发现的问题,生成小组走查问题清单。
3.组长带领与会成员浏览小组走查问题清单,定义缺陷严重级别,确定缺陷是否被开启,是否需要修改。
4.QA生成最终问题清单并开启缺陷,向组长通报走查结果,并告知开发人员和评审人员。
5.小组成员提交评审日志。
出口准则:小组走查文件,代码走查问题清单,个人评审日志。
5.4缺陷修改与关闭
主要活动:
1.由开发人员对缺陷进行修改,记录缺陷状况。
2.再次召开小组走查会议,直至所有缺陷均被关闭或挂起。
3.开发人员整理工作成果,并存入开发库。
4.组长和QA编写代码走查报告,QA整理评审文档,并入库。
出口准则:修改后代码,缺陷跟踪矩阵,代码走查报告。下载本文