代码评审该如何做

代码评审该如何进行？

就是同事之间互相较交换代码查看吗？

具体的应该如何做？互换查看代码时都看些什么？
---------------------------------
代码评审应该是项目以外的专家对其关键代码进行的检查吧！
应该是代码编写者讲解自己的代码由专家或项目组其他成员及项目经理来作评审，其间有不了解之处可随时提问，并提出意见。主要采用关键代码检查，部分代码抽查的原则。
我倒是有代码检查单可是不知道如何添加附件，你可以告诉我如何添加，我在传上来。
---------------------------------
写代码一般注意些什么,你们所审查的代码都达到标准没,每个公司都有标准,但是都差不多的
---------------------------------
评审尤其是同行评审，是提前发现缺陷，降低开发成本的有效方法；但是很多企业并不重视，或者效果不佳。原因有很多：没有获得合适的专家、组织不合理、方法不当等。
针对代码评审，我想简单地谈几点（其实其它工程文档的评审存在同样问题）：
1、评审要计划。因为大家都很忙，各领域的专家也都有各自的工作，评审结果的好坏对他们工作绩效没有影响，所以专家投入多少精力来用于评审不受控，完全靠他们自己的职业精神。
   所以，这就要求项目经理比较有影响力，能够争取到足够的、合适的专家来参与评审。专家最好要提前协调，当然项目经理首先要知道要请哪些专家（如系统专家、领域专家、测试人员、QA等），这些专家在哪里？能否获得专家资源的支持，是项目经理的项目管理能力一个重要体现。评审时间的投入比例，一般是第一次版本准备时间的3/5以上。
2、评审要分层次和重点。作为项目成员，在代码编写完成后，首先是自检，这时发现的缺陷不计算；然后是项目组内的评审，这时发现的缺陷应计算；最后才是外部评审。应提前把相关文档和代码交给评审专家，让他们各自评审；根据项目和评审结果的情况，决定是否召开评审会议，评审会议以确认问题为主，而不是讨论解决方案。对于重要的核心的代码，要投入重点评审力量；另外，对于能力不足的成员所完成的代码，也要重点评审。每个角色也应明白各自评审重点，如QA重点放在编程规范、测试人员侧重在可测性、系统专家侧重在从整体来考虑（如对其它功能的影响、性能等）。
3、问题的确认与跟踪。评审专家发现问题后，首先与该代码的完成人进行确认，如果大家达成一致认可这个问题，然后由代码完成人提出解决方案，这个解决方案要得到问题发现者的同意，然后编程人员编码实现该方案，并进行测试和验证，并将验证结果提交问题发现人，问题发现人确认无误后，该问题就可关闭。所以，评审专家发现问题后并没有结束，而是要负责跟踪该问题。
总之，评审活动的组织其实比较复杂，也是项目经理项目管理能力的一个重要体现，项目经理对评审活动负有重要责任。
---------------------------------
代码review主要就是事前的准备，执行，和后期的跟踪
事前的准备也叫做review的输入，这个输入是在什么条件下可以做为输入，可以是个人检查完了，也可以是相互交叉的检查，这里有个检查的标准，如有个检查单什么的，等规定的输入完成之后，进行review的执行
在review中，主要就是首先保证什么人参加，这是最主要的，如果找几个没有什么经验的，那review的效果肯定是打折扣的，review的多少时间
跟踪，主要是问题的跟踪和后期的数据的度量
你上面说的相互评审，主要是项目内部的review，review分好几种，有内部的，专家的，客户的
review代码时主要看是否符合编码规范，特别是是否符合设计书（需求）的要求，逻辑是否正确，主要是专家级！！