对代码评审的一些理解

前几天我们进行了第一次代码评审，整个项目评审过程从下午1点半持续到晚上7点，过程中杨总引导我们从各个角度去看项目代码中存在的问题，最终将会议中出现的问题落实成《代码评审检查表》，以备开发人员自查，成果显著。
    这是我们各项目启动以来第一次进行代码评审，在接到这个任务之前我也没做过或参加过代码评审，为了能够更好的完成这项任务，我在网上查了一下如何做代码评审。在这里分享一下我参与过代码评审之后对代码评审的一些理解。

    杨总一直都跟我们强调，要知其然知其所以然，我们首先需要知道什么是 Code Review,我们使用它的目的是什么。

目的

    Code Review是一种用来确认方案设计和代码实现的质量保证机制，通过这个机制我

们可以对代码，测试过程和注释进行检查。Code Review主要用来在软件工程过程中改进代码质量，通过Code Review

可以达到如下目的：

    1.在项目早期就能够发现代码中的BUG   
    2.帮助初级开发人员学习高级开发人员的经验，达到知识共享  
    3.避免开发人员犯一些很常见，很普通的错误  
    4.保证项目组人员的良好沟通  
    5.项目代码更容易维护



前提


    了解了Code Review的目的之后，下一步就是如何做Code Review了。但在做Code Review前我们还有事要做，所谓预则立，不预则废，就是说如果在进入Code Review之前我们不做些准备工作，Code Review很容易就变得没有意义或是流于形式。
进入Code Review需要检查的条件如下：  
    1.评审人员是否理解了Code Review的概念和Code Review将做什么 如果做Code Review的人员不能理解Code Review对项目成败和代码质量的重要程度，他们的做法可能就会是应付了事。  
    2.代码是否已经正确的build，build的目的使得代码已经不存在基本语法错误 我们总不希望各位参评人员将时间浪费在检查连编译都通不过的代码上吧。 
    3.代码执行时功能是否正确  评审人员也不负责检查代码的功能是否正确，也就是说，需要复查的代码必须由开发人员或质量人员负责该代码的功能的正确性。 
    4.参评人员是否理解了代码  做复查的人员需要对该代码有一个基本的了解，其功能是什么，这样才能采取针对性的检查 
    5.开发人员是否对代码做了单元测试  这一点也是为了保证Code Review前一些语法和功能问题已经得到解决，评审人员可以将精力集中在代码的质量上。 

做什么

    Code Review主要检查代码中是否存在以下方面问题：

•     编码格式规范（包括代码排版、一致性）
•     代码冗余
•     是否符合设计准则
•     可能作为系统架构人员来讲，还应关注开发人员的开发过程及代码是否会脱离系统框架的约束范围。
  

怎么做

    具体在实践的时候要怎么去做，除了对以上内容的理解之外，还请参考CheckList中的检查项。

    希望以上内容对大家会有所帮助。​