2014代码审查规.docVIP

信息技术中心网站技术部 代码审查规范 版本 更新时间 修改人 备注 V1.0 2013 ？？ 文档创建  目 录 引言 3 目的 3 说明 3 代码审查 3 检查点 3 审查流程图 4 流程概述 5 具体流程 5 建立任务 5 桌面检查 5 团体评审 5 Bug修复 5 Bug复检 5 参考资料  引言 目的 检查开发/前端人员是否遵守开发规范中的规定 检查开发/前端人员是否代码审定表中的错误 检查代码是否存在逻辑错误或安全问题 说明 代码检查每月进行一次，根据《代码审查表》的内容进行检查，结果计入绩效考核-开发质量项。 代码审查 检查点 参见《代码审查检定表》。 审查流程图 流程概述 建立任务：建立代码检查任务，指定需评审的项目或代码文件范围、参与评审的人员、定义问题类型及严重级别等。 桌面检查阶段：开始各审查人独自评审，将可能出现的问题加入代码管理的Bug列表。 团队评审阶段：合并上步审查结果，统一讨论桌面检查阶段的问题，实施交叉检定，确定是否Bug，需要修复的分配解决人员。 问题修复阶段：修改人修复分配给自己的问题，修复后修改Bug状态，审查人复查无误后，关闭Bug。 具体流程 建立任务 选择一个要检查的项目。 确定此次检查的重点内容和主要关注的Bug类型。 新建OA流程发起新的检查任务，告知相关人员。 定义错误严重级别，划分各审查人检查范围，指定Bug文档位置。 桌面检查 获取要检查的源代码更新，使用分析工具寻找Bug。 简单的风格类Bug如缩进、换行格式等，可直接修改后等待发布。 人工检查代码，查找使用工具无法找到的错误。 使用文档记录Bug。标记问题类型及严重性，出现位置和操作场景。 桌面检查完毕后，所有审查人将Bug文档合并至到源码管理工具。 团体评审 审查成员在一起，从源码管理工具获取更新的Bug文档。 按照文档交叉检定其他人提交的Bug是否存在，Bug描述是否完整。 Bug文档在集体确认后，一起讨论代码中的Bug存在问题及处理优先级。 对高优先级的Bug优先分配人手，并制定解决方案。 团队评审完毕后，更新Bug文档，并将确定需要处理的Bug分发给指定修改人。 Bug修复 个人回到自己的工位，更新最新的Bug文档，并获取指定给自己的Bug。 将分配给自己的问题，按解决方案的指定逐一修复，并进行测试。 将修改后的源代码提交至管理工具，修改指定给自己的Bug状态，修改Bug文档说明。 Bug复检 审查人更新最新的Bug文档，依据文档逐一检查Bug修改。 如有功能变动或逻辑变化，需要加入测试人员的黑盒测试，进行二次确认。 Bug修改未完成的，退回到Bug修复。 已修复的Bug，将复检结果更新至Bug文档。 所有Bug修复、复检完成后，联同风格类Bug合并进行发布部署。 1 / 5