发个无聊的贴子，看看大家怎么看code review的

最近公司在搞流程改进，而code review就不可避免地成了技术小组关心的一个重要问题，因为公司和CMMI一直在推，效果却非常不理想

正是因为我们做的效果不理想，而网上牛人们却在不停地推它，所以，我们就宁可相信这个code review是个很好的东东，是我们自身没有做对而已

下面是我和公司一同事讨论得出的一个方案，供大家参考（我们考虑的重点是可行性、实际效果和执行效率）
1．  开发刚开始的阶段，由TL和PM召集大家坐在一起进行几次review（至少3-5次）
2．  在开发过程中，如果开发人员觉得哪段代码写得不是太好时可以请senior/TL/designer帮忙review
3．  senior/TL/designer在某一模块完成之前，必需完成该模块的review


对方案的一些解释：

1. 第一条的目的是在项目开始的时候，把一些代码规范和参照设计文档写代码的习惯传达给大家
2. 第二条的目的是让开发人员有这样的一个机会请senior/TL/designer来帮忙查看一下代码，提前发现一些问题。这一条需要开发人员的主动性，如果他真的想把代码写好，在遇到问题时，我相信是一定会有这个主动性的。另外，即使开发人员不能提问，那还有第三条来保证代码质量。
3. 第三条的目的是TL/Senior/Designer出于对项目一致性或质量的要求，会去做出review，看代码是否符合设计，是否符合编码规范，是否有哪些写法值得提醒开发人员注意。在这一点上，要求review工作在某一个模块完成的时候同时完成，不应该在开发人员开发完成后再做review，让开发人员把模块都做完后来返工是件不太好的事。这也就要求PM在排计划的时候，一定要给TL/Senior/Designer安排合适的时间来完成review工作

所以，我们必须要有统一、稳定的编码规范(format, 目录结构等)，供review的参考的设计，开发人员提出review的请求后能得到积极的响应，并给出有效的建议

下面的链接是ozzzzzz大师发起的讨论，遗憾的是我不能找到他的那篇关于code review的文章，希望有人能分享一下
http://www.iteye.com/topic/2217

评论

16 楼 kimmking 2008-06-19  

传统的公司中，pair不可想象，

试问：领导者如何会乐意见到每时每刻都有一个人在“闲”着！

15 楼 kimmking 2008-06-19  

seen 写道

等等 为什么2是不需要的？难不成老板要说，我付给你8小时工资，你就得干8小时的活？

如果老板这么说，我会忍不住亲他一口。
无休止的无意义的无薪酬的加班。
我已经忍无可忍。

14 楼 spiritfrog 2008-06-19  

pair感觉上比code review容易推广，
但事实上又看到code review的模式多一些。也许大家都更相信集体力量大， pair的人员组合是个问题， 应该是一个水平高点的带水平低一点的，很难找到合适资源吧。

13 楼 leton2008 2008-06-18  

我个人倾向于多种方式结合的模式。有这几个要求:
1,只有在代码需要check in的时候才需要进行检查。结对人员必须无条件暂停工作进行code review.
2,代码检查分3个阶段进行:
  a,自动工具检查。使用IDE的插件功能进行检查。这类工具有Check Style,PMD,jTest等工具。我个人比较喜欢PMD，因为可以更容易组合规则，PMD支持大部分的IDE (Eclipse,IDEA,JBuilder,NetBeans....).
  b,个人自检。个人经过一定的修正后，个人再对照公司的代码规范和代码编写建议进行自检。
  c,Pair Review,结对检查。结对人员也可以再参考公司的规范和建议进行检查，同时还需要针对开发任务书进行代码检查。

但是以现行的很多公司来说。这样实施的开发成本较高。就跟做文档评审先需要进行文档评审准备会一样。但是如果能坚持实施，我个人认为对项目和组员都是有很大的帮助的。

之前也在寻找如何解决code review的途径，参考过的资料包括:
http://www.ibm.com/developerworks/cn/java/j-cq06306/index.html
“追求代码质量: 用代码度量进行重构(http://www.ibm.com/developerworks/cn/java/j-cq05306/index.html)”（Andrew Glover, developerWorks，2006 年 5 月）：用 Extract Method 模式进行重构和简化复杂代码。
这是IBM开发者社区的系列文章，我觉得很有帮助的。

12 楼 fuwang 2008-06-18  

你们说的是什么类型的公司啊，国内绝大多数公司都不可能实行pair，
code review都是很稀有的事情。
老板要省成本，而且最喜欢在开发这一块省。

11 楼 andyhu1007 2008-06-18  

seen 写道

gigix 写道

seen 写道

等等 为什么2是不需要的？难不成老板要说，我付给你8小时工资，你就得干8小时的活？

想休息可以休息，但是要清楚每段时间是在做什么。跟同伴说一声“我现在要休息半小时”有助于更好的管理时间。

ok。。。我只好说，我不适合这种管理方式
当然 我还是有兴趣尝试那么几次pair的 感觉就像玩大家一起来找茬-我经常跟老婆两个人一起玩这个游戏

跟老婆pair。。。寒。。。

不过习惯这个问题，我倒觉得人是应该有适应能力的。没试之前，最好不要说自己不适合。试了再说。

10 楼 seen 2008-06-17  

gigix 写道

seen 写道

等等 为什么2是不需要的？难不成老板要说，我付给你8小时工资，你就得干8小时的活？

想休息可以休息，但是要清楚每段时间是在做什么。跟同伴说一声“我现在要休息半小时”有助于更好的管理时间。

ok。。。我只好说，我不适合这种管理方式
当然 我还是有兴趣尝试那么几次pair的 感觉就像玩大家一起来找茬-我经常跟老婆两个人一起玩这个游戏

9 楼 gigix 2008-06-17  

seen 写道

等等 为什么2是不需要的？难不成老板要说，我付给你8小时工资，你就得干8小时的活？

想休息可以休息，但是要清楚每段时间是在做什么。跟同伴说一声“我现在要休息半小时”有助于更好的管理时间。

8 楼 seen 2008-06-17  

等等 为什么2是不需要的？难不成老板要说，我付给你8小时工资，你就得干8小时的活？

7 楼 gigix 2008-06-17  

seen 写道

gigix 写道

seen 写道

pair会不会太不自由了？

为什么你需要自由？

两种意义上的自由
1是编程的自由 在有一个能跑的结构之前 我不喜欢别人来打扰我的思路
2是个人自由 也许我写了一个头文件之后想去倒杯水 想看看今天的比赛直播 或者想打个电话 可如果有个同事就在身边等着我。。。

2是不需要的
1在绝大多数时候也是不需要的，真的需要的时候你可以暂时不pair嘛

6 楼 seen 2008-06-17  

gigix 写道

seen 写道

pair会不会太不自由了？

为什么你需要自由？

两种意义上的自由
1是编程的自由 在有一个能跑的结构之前 我不喜欢别人来打扰我的思路
2是个人自由 也许我写了一个头文件之后想去倒杯水 想看看今天的比赛直播 或者想打个电话 可如果有个同事就在身边等着我。。。

5 楼 gigix 2008-06-17  

seen 写道

pair会不会太不自由了？

为什么你需要自由？

4 楼 seen 2008-06-17  

pair会不会太不自由了？

3 楼 Godlikeme 2008-06-17  

pair是不错的实践。

2 楼 yamakasy 2008-06-17  

spyker 写道

我以前一个项目的code review，是在自己代码测试后，交由tl进行
每个人都会被review
也就是说code review是tl的一项工作

基本上是这样！check style 可以通过工具实现。

1 楼 andyhu1007 2008-06-16  

code review: 费时，费力，交流麻烦。推荐pair。