CODE REVIEW规范管理制度word文档.docxVIP

Code review规范/制度 word文档 关于Code Review Code Review的目的 Code Review是一种用来确认方案设计和代码实现的质量保证机制,通过这个机制我们可以对代码、测试过程和注释进行检查。 Code Review主要用来在软件工程过程中改进代码质量,通过Code Review可以达到如下目的: 在项目早期就能够发现代码中的BUG 帮助初级开发人员学习高级开发人员的经验,达到知识共享 避免开发人员犯-些很常见,很普通的错误 保证项目组人员的良好沟通 项目或产品的代码更容易维护 Code Review的前提 进入Code Review需要检查的条件如下: Code Review人员是否理解了Code Review的概念和Code Review将做什么如果做Code Review的人员不能理解Code Review对项目成败和代码质量的重要程度,他们的做法可能就会是应付了事。代码是否已经正确的build , build的目的使得代码已经不存在基本语法错误 我们总不希望高级开发人员或是主管将时间浪费在检查连编译都通不过的代码上吧。 代码执行时功能是否正确 Code Review人员也不负责检查代码的功能是否正确,也就是说,需要复查的代码必须由开发人员或质量人员负责该代码的功能的正确性。 Review人员是否理解了代码 做复查的人员需要对该代码有一个基本的了解,其功能是什么,拿-方面的代码,涉及到数据库或是通讯,这样才能采取针对性的检查 开发人员是否对代码做了单元测试 这一点也是为了保证Code Review前一些语法和功能问题已经得到解决 , Code Review人员可以将精力集中在代码的质量上。 Code Review需要做什么 Code Review主要检查代码中是否存在以下方面问题: 代码的一致性、编码风格、代码的安全问题、代码冗余、 是否正确设计以满足需求(性能、功能)等等 完整性检查( Completeness ) 代码是否完全实现了设计文档中提出的功能需求 代码是否已按照设计文档进行了集成和Debug 代码是否已创建了需要的数据库, 包括正确的初始化数据 代码中是否存在任何没有定义或没有引用到的变量、常数或数据类型 一致性检查( Consistency ) 代码的逻辑是否符合设计文档 代码中使用的格式、符号、结构等风格是否保持一致 正确性检查( Correctness ) 代码是否符合制定的标准 所有的变量都被正确定义和使用 所有的注释都是准确的 所有的程序调用都使用了正确的参数个数 可修改性检查( Modifiability ) 代码涉及到的常量是否易于修改(如使用配置、定义为类常量、使用专门的常量类等) 代码中是否包含了交叉说明或数据字典,以描述程序是如何对变量和常量进行访问的 代码是否只有一个出口和一个入口(严重的异常处理除外) 可预测性检查( Predictability ) 代码所用的开发语言是否具有定义良好的语法和语义 是否代码避免了依赖于开发语言缺省提供的功能 代码是否无意中陷入了死循环 代码是否了无穷递归 健壮性检查( Robustness ) 代码是否采取措施运行时错误(如数组边界溢出、被零除、值越界、堆栈溢出等)