Code Review的前提
==============
- 重视代码review
(Code Review人员是否理解了Code Review的概念和Code Review将做什么如果做Code Review的人员不能理解Code Review对项目成败和代码质量的重要程度,他们的做法可能就会是应付了事。)
- 代码是否已经正确的build,build的目的使得代码已 经不存在基本语法错误
(我们总不希望高级开发人员或是主管将时间浪费在检查连编译都通不过的代码上吧。 )
- 代码执行时功能是否正确
(Code Review人员也不负责检查代码的功能是否正确,也就是说,需要复查的代码必须由开发人员或质量人员负责该代码的功能的正确性。 )
- 开发人员是否对代码做了单元测试
(这一点也是为了保证Code Review前一些语法和功能问题已经得到解决,Code Review人员可以将精力集中在代码的质量上。 )
Code Review需要注意什么?
==================
- 完整性检查(Completeness)
-
代码是否完全实现了设计文档中提出的功能需求
-
代码是否已按照设计文档进行了集成和Debug
-
代码是否已创建了需要的数据库,包括正确的初始化数据
-
代码中是否存在任何没有定义或没有引用到的变量、常数或数据类型</