1、关于Code Review
1.1 Code Review的目的
Code Review主要用来在软件工程过程中改进代码质量,通过Code Review可以达到如下目的目的:
(1)在项目早期就能够发现代码中的BUG
(2)帮助初级开发人员学习高级开发人员的经验,达到知识共享
(3)避免开发人员犯一些很常见,很普通的错误
(4)保证项目组人员的良好沟通
(5)项目或产品的代码更容易维护
1.2 Code Review的前提
进入Code Review需要检查的条件如下:
(1)Code Review人员是否理解了Code Review的概念和Code Review将做什么
如果做Code Review的人员不能理解Code Review对项目成败和代码质量的重要程度,他们的做法可能就会是应付了事。
(2)代码是否已经正确的build,build的目的使得代码已经不存在基本语法错误
我们总不希望高级开发人员或是主管将时间浪费在检查连编译都通不过的代码上吧。
(3)代码执行时功能是否正确
Code Review人员也不负责检查代码的功能是否正确,也就是说,需要复查的代码必须由开发人员或质量人员负责该代码的功能的正确性。
(4)Review人员是否理解了代码
做复查的人员需要对该代码有一个基本的了解,其功能是什么,是拿一方面的代码,涉及到数据库或是通讯,这样才能采取针对性的检查
(5)开发人员是否对代码做了单元测试
这一点也是为了保证Code Review前一些语法和功能问题已经得到解决,Code Review人员可以将精力集中在代码的质量上。
1.3 Code Review需要做什么
Code Review主要检查代码中是否存在以下方面问题:
代码的一致性、编码风格、代码的安全问题、代码冗余、是否正确设计以满足需求(性能、功能)等等
1.3.1 完整性检查(Completeness)
代码是否完全实现了设计文档中提出的功能需求
代码是否已按照设计文档进行了集成和Debug
代码是否已创建了需要的数据库,包括正确的初始化数据
代码中是否存在任何没有定义或没有引用到的变量、常数或数据类型
1.3.2 一致性检查(Consistency)
代码的逻辑是否符合设计文档
代码中使用的格式、符号、结构等风格是否保持一致