代码评审该如何做

代码评审该如何做

评审尤其是同行评审，是提前发现缺陷，降低开发成本的有效方法；但是很多企业并不重视，或者效果不佳。原因有很多：没有获得合适的专家、组织不合理、方法不当等。针对代码评审，我想简单地谈几点（其实其它工程文档的评审存在同样问题）：1、评审要计划。因为大家都很忙，各领域的专家也都有各自的工作，评审结果的好坏对他们工作绩效没有影响，所以专家投入多少精力来用于评审不受控，完全靠他们自己的职业精神。 所以，这就要求项目经理比较有影响力，能够争取到足够的、合适的专家来参与评审。专家最好要提前协调，当然项目经理首先要知道要请哪些专家（如系统专家、领域专家、测试人员、QA等），这些专家在哪里？能否获得专家资源的支持，是项目经理的项目管理能力一个重要体现。评审时间的投入比例，一般是第一次版本准备时间的3/5以上。2、评审要分层次和重点。作为项目成员，在代码编写完成后，首先是自检，这时发现的缺陷不计算；然后是项目组内的评审，这时发现的缺陷应计算；最后才是外部评审。应提前把相关文档和代码交给评审专家，让他们各自评审；根据项目和评审结果的情况，决定是否召开评审会议，评审会议以确认问题为主，而不是讨论解决方案。对于重要的核心的代码，要投入重点评审力量；另外，对于能力不足的成员所完成的代码，也要重点评审。每个角色也应明白各自评审重点，如QA重点放在编程规范、测试人员侧重在可测性、系统专家侧重在从整体来考虑（如对其它功能的影响、性能等）。3、问题的确认与跟踪。评审专家发现问题后，首先与该代码的完成人进行确认，如果大家达成一致认可这个问题，然后由代码完成人提出解决方案，这个解决方案要得到问题发现者的同意，然后编程人员编码实现该方案，并进行测试和验证，并将验证结果提交问题发现人，问题发现人确认无误后，该问题就可关闭。所以，评审专家发现问题后并没有结束，而是要负责跟踪该问题。总之，评审活动的组织其实比较复杂，也是项目经理项目管理能力的一个重要体现，项目经理对评审活动负有重要责任。

代码review主要就是事前的准备，执行，和后期的跟踪事前的准备也叫做review的输入，这个输入是在什么条件下可以做为输入，可以是个人检查完了，也可以是相互交叉的检查，这里有个检查的标准，如有个检查单什么的，等规定的输入完成之后，进行review的执行在review中，主要就是首先保证什么人参加，这是最主要的，如果找几个没有什么经验的，那review的效果肯定是打折扣的，review的多少时间跟踪，主要是问题的跟踪和后期的数据的度量你上面说的相互评审，主要是项目内部的review，review分好几种，有内部的，专家的，客户的review代码时主要看是否符合编码规范，特别是是否符合设计书（需求）的要求，逻辑是否正确，主要是专家级！！