代码审查规范

代码审查规范

版本 V1.0

更新时间 2016

修改人 wp 备注 目录

引言2 目的2 说明2 代码审查2 检查点2 审查流程图3 流程概述4

1 / 4

具体流程4

建立任务4 桌面检查4 团体评审4 Bug修复4 Bug复检4

引言

目的

检查开发/前端人员是否遵守开发规范中的规定 检查开发/前端人员是否代码审定表中的错误 检查代码是否存在逻辑错误或安全问题

说明

代码检查每月进行一次，根据《代码审查表》的内容进行检查,结果计入绩效考核—开发质量项。

代码审查

检查点

参见《代码审查表》。

2 / 4

审查流程图

审查任务开始确定代码审查范围指定审查人/Bug类型审查人桌面检查YBug记录发现BugN提交检查过的数据团队评审/交叉检定确认BugY指定修改人，定决方案N删除Bug修改人修改Bug reopen审查人Bug复查确认Bug修改Y提交审查数据，Bug关闭N重新打开BugEND

3 / 4

流程概述

1. 建立任务：建立代码检查任务,指定需评审的项目或代码文件范围、参与评审的人员、定义问题类型及

严重级别等.

2. 桌面检查阶段：开始各审查人独自评审，将可能出现的问题加入代码管理的Bug列表.

3. 团队评审阶段：合并上步审查结果，统一讨论桌面检查阶段的问题，实施交叉检定，确定是否Bug，需

要修复的分配解决人员。

4. 问题修复阶段：修改人修复分配给自己的问题,修复后修改Bug状态,审查人复查无误后，关闭Bug。

具体流程 建立任务

1. 2. 3. 4.

选择一个要检查的项目。

确定此次检查的重点内容和主要关注的Bug类型。 新建OA流程发起新的检查任务，告知相关人员.

定义错误严重级别，划分各审查人检查范围，指定Bug文档位置。

桌面检查

1. 2. 3. 4. 5.

获取要检查的源代码更新,使用分析工具寻找Bug。

简单的风格类Bug如缩进、换行格式等，可直接修改后等待发布。 人工检查代码,查找使用工具无法找到的错误.

使用文档记录Bug.标记问题类型及严重性,出现位置和操作场景。 桌面检查完毕后,所有审查人将Bug文档合并至到源码管理工具.

团体评审

1. 2. 3. 4. 5.

审查成员在一起,从源码管理工具获取更新的Bug文档。

按照文档交叉检定其他人提交的Bug是否存在，Bug描述是否完整。 Bug文档在集体确认后，一起讨论代码中的Bug存在问题及处理优先级。 对高优先级的Bug优先分配人手，并制定解决方案.

团队评审完毕后，更新Bug文档，并将确定需要处理的Bug分发给指定修改人.

Bug修复

1. 个人回到自己的工位，更新最新的Bug文档,并获取指定给自己的Bug。 2. 将分配给自己的问题,按解决方案的指定逐一修复，并进行测试。

3. 将修改后的源代码提交至管理工具,修改指定给自己的Bug状态，修改Bug文档说明。

Bug复检

1. 2. 3. 4. 5.

审查人更新最新的Bug文档,依据文档逐一检查Bug修改。

如有功能变动或逻辑变化，需要加入测试人员的黑盒测试，进行二次确认。 Bug修改未完成的，退回到Bug修复。

已修复的Bug，将复检结果更新至Bug文档。

所有Bug修复、复检完成后，联同风格类Bug合并进行发布部署。

4 / 4