代码评审该如何做

懒月亮...

代码评审该如何进行？

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

具体的应该如何做？互换查看代码时都看些什么？

大家谁有代码评审的检查单？能不能帮我一下

多多谢谢了

xsnzhq

代码评审应该是项目以外的专家对其关键代码进行的检查吧！
应该是代码编写者讲解自己的代码由专家或项目组其他成员及项目经理来作评审，其间有不了解之处可随时提问，并提出意见。主要采用关键代码检查，部分代码抽查的原则。
我倒是有代码检查单可是不知道如何添加附件，你可以告诉我如何添加，我在传上来。

猫猫的拖鞋

写代码一般注意些什么,你们所审查的代码都达到标准没,每个公司都有标准,但是都差不多的~

zhongmg108

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

chengxq

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

jinjinjin19.

代码评审，一般有哪些人参加呢？

懒月亮...

看到这么多热心的帮助，真心的感谢大家！！！

像版主说的“，特别是是否符合设计书（需求）的要求，逻辑是否正确，”

我想问：检查逻辑是不是需要大量的时间?
另外，检查逻辑的方法，就是仔细研读程序员的代码么?还有别的方法么？

懒月亮...

呀，偶也不会上传呢；麻烦把代码检查单给我发下邮箱吧，好么？bbbld@sohu.com

多谢~~~~~~~~~~~~~~~

chengxq

呀，偶也不会上传呢；麻烦把代码检查单给我发下邮箱吧，好么？bbbld@sohu.com多谢~~~~~~~~~~~~~~~

附件上传方法

[ 本帖最后由 chengxq 于 2008-10-14 13:15 编辑 ]

chengxq

原帖由 jinjinjin19. 于 2008-10-13 15:17 发表
代码评审，一般有哪些人参加呢？

这个主要是根据在项目计划中，相关人员的介入计划来的，在计划中就已经规定了。
参加人员主要是相应的担当开发人员，项目组长，项目经理，以及一些技术专家。
这个时候，高层和客户可以不参加代码的评审

xsnzhq

代码评审检查单，仅供参考

xsnzhq

刚刚不会添加附件，现在好了，大家来取吧！

懒月亮...

代码评审会议中，QA该干些什么？应该关注哪些方面？

总感觉自己在代码评审时无能为力阿~~~~~~

懒月亮...

谢谢了

一个好人~~~~~~

irene0331223

帮顶13楼的问题，有同样的疑问，希望明白的给个回答，谢谢各位！

zxbcug

注释，命名，参数等主要方面的review吧

mgs2-007

原帖由 懒月亮... 于 2008-10-15 14:20 发表
代码评审会议中，QA该干些什么？应该关注哪些方面？

总感觉自己在代码评审时无能为力阿~~~~~~

主要关注评审的效率、有效性、评审的目的。
效率方面：评审准备是否充足，有没有提出一定的问题，编码规约和历史经验的一些问题是否在评审之前就被淘出来，评审时关注评审的速率、评审的问题数、各种分类问题的问题数。
评审的目的和评审效率方面息息相关，按评审的角色来看，如果相关业务人员或客户评审需要考虑需求满足情况逻辑方面是否符合等等，技术人员评审是主要看实现方式等等。QA要注意的是评审是否针对评审准备的问题进行有目的的讨论、针对核心代码进行再评审。
不知理解是否正确请版主指正。

chengxq

如果QA是与测试无关的角色的话，主要是看review的过程上保证review的有效性
至于如何review，我相信公司的组织级都有相关的规定
人：参加的人员要求
时间：review 的时间
过程：事前，事中，事后
这些东西是否符合公司的流程规范
没有一个好的过程，就没有一个好的产品

evensky

看看先~~~

shboyzone

好东西 谢谢，支持楼主