|
精华帖 (0) :: 良好帖 (0) :: 新手帖 (0) :: 隐藏帖 (0)
|
|
|---|---|
| 作者 | 正文 |
|
发表时间：2008-06-22
最近在看effective java有时候我可以review一下自己的代码了
|
| 返回顶楼 | |
|
发表时间：2008-06-22
leton2008 写道 我个人倾向于多种方式结合的模式。有这几个要求:
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开发者社区的系列文章，我觉得很有帮助的。 项目组新人多，需要使用正式的小组评审。等基本上路以后，采用结对评审。 我也赞成采用自检的方式，这是一个好的程序员和一般程序员的重要差别。一般程序员，写代码没有规划，就是敲代码，敲完之后马上Run,不行就Debug。好的程序员会有简单的规划，写完之后会看上两遍，一般会挑出一些错误。然后再Run,实在不行才debug。 |
| 返回顶楼 | |
|
发表时间：2008-06-23
当前公司的现状，code review还是比较现实的做法。正式的评审比非正式的效果要好，非正式的大家都是混混了事。成本是提高了，但不能只看成本，要看成本的收益是多少
|
| 返回顶楼 | |
|
发表时间：2008-06-23
现实的条件下，能把核心的和重要的代码进行REVIEW就不错了。
|
| 返回顶楼 | |
|
发表时间：2008-06-24
欲速则不达，很多老板都不懂的。
|
| 返回顶楼 | |
|
发表时间：2008-06-25
1.代码规范性检查，CheckStyle , PMD
2.重复代码检查，PMD cpd 3.单元测试，测试覆盖率 4.持续集成，每日构建，让这些步骤自动运行。 |
| 返回顶楼 | |
|
发表时间：2008-06-26
code review啊，让项目成员坐在一起嗑嗑瓜子，喝喝茶，顺带着抓抓虫子。。。。。。挺好啊
|
| 返回顶楼 | |
|
发表时间：2008-06-28
什么pair/review这些方法都不好,好的办法要从中国传统智慧中去寻找.
南北朝时北朝有个皇帝赫连勃勃,要建首都(统万城)的城墙.为了最大程度保证质量,他把团队分为两组,一组施工,一组质监.施工组建完一段墙后,质监组过来做检查,检查办法是用铁枪使劲往城墙上戳,那么结果无非两个,一是把墙戳个坑,或者是墙毫发无损. 对于第一种情况,皇帝的措施是这样的:把施工组的人全部砍头.第二种情况的处理大家可能猜到了,就是把质监组的人杀掉. 采取这种措施的成效非常喜人:近2000年过去了,这个统万城依然屹立不倒. 诸位认为这个质保措施用于软件业,会有什么效果? |
| 返回顶楼 | |
|
发表时间：2008-06-28
楼上，如果放在软件业，效果很明显，就是没有人呆在执行这个残忍措施的公司。不能乱套经验。
其实要不要review，采取何种方式review，是和楼主的项目特点、团队具体情况密切相关的，看看有谁有成功的review经验，并且项目、团队也和楼主类似。 |
| 返回顶楼 | |
|
发表时间：2008-06-30
xuby 写道 什么pair/review这些方法都不好,好的办法要从中国传统智慧中去寻找.
南北朝时北朝有个皇帝赫连勃勃,要建首都(统万城)的城墙.为了最大程度保证质量,他把团队分为两组,一组施工,一组质监.施工组建完一段墙后,质监组过来做检查,检查办法是用铁枪使劲往城墙上戳,那么结果无非两个,一是把墙戳个坑,或者是墙毫发无损. 对于第一种情况,皇帝的措施是这样的:把施工组的人全部砍头.第二种情况的处理大家可能猜到了,就是把质监组的人杀掉. 采取这种措施的成效非常喜人:近2000年过去了,这个统万城依然屹立不倒. 诸位认为这个质保措施用于软件业,会有什么效果? 提问:修墙的人死的多还是质检的人死的多? 
|
| 返回顶楼 | |
