【原】Eclipse Code Review（代码审查）工具介绍

Keywords : Code Review、Jupiter、ReviewClipse、代码评审、代码审查

最近组内一直在做代码改进，既然是代码级别的改进，我想比较直接有效的办法就是Code Review。

虽然以前开发部也做过代码评审，但基本上是程序员手动打包代码，评审人员对照打包和IDE中的代码提交缺陷，缺陷在平台上，代码在IDE中，我想评审效率和评审效果都会大打折扣。

既然发现了问题，目标很明确，找到一个即可以提高评审效率（即让评审人和被评审人都可以很方便的来参与评审），又可以保证评审效果的工具。

再结合我们的实际情况，我们是使用Eclipse IDE开发的，Eclipse的插件不计其数，不怕找不到，就怕你不找，因此从Eclipse Code Review Plugin 下手。

经过调研和试用，我找到两款比较好用的免费插件：ReviewClipse 和 Jupiter 。

简单介绍一下这两个工具的特点和在Qone中的使用场景

ReviewClipse

官方地址：http://www.inso.tuwien.ac.at/projects/reviewclipse/

这款插件是基于SVN版本比较机制进行代码Review的工具，通过配置svn中哪些人可以评审哪些人的指定类型代码，可以监控整个项目范围内指定人员指定类型代码文件的所有变更情况。通过这些变化代码的前后版本对比，可以发现被评审人的所有代码改动。

它比较适合在对代码改动比较敏感的阶段使用，如版本测试后期bug修改阶段或者mantis上小版本bug修改。或者进行较频繁的代码评审时使用，如项目经理想看新人每天提交的代码，让新人快速提升代码质量及改善代码风格。

这个工具应该说是项目经理进行代码修改影响分析的好工具，无需再自己对着SVN log，查谁改了哪些文件，然后再查影响分析了。贴个图先：


 


 
同时ReviewClipse还支持多人协同的代码评审，被评审人可以看到评审人给出的评审意见。

ReviewClipse的不足在我看来就是它只支持文件级别的代码评审，不能提供代码行级别的评审，并且不支持常规的代码评审流程。

Jupiter

jar及帮助文档下载地址：http://code.google.com/p/jupiter-eclipse-plugin/downloads/list

中文简要参考文档：http://fyting.iteye.com/blog/87489

Jupiter解决了ReviewClipse的不足，它提供了代码行级别的评审批注功能，方便评审参与人了解具体是哪些行代码存在问题。同时，它也比较 符合常规的评审流程，被评审人提供待审代码->评审人线下提出个人意见->组织讨论会讨论每个人提出的意见并确定问题及解决方案->被 评审人rework修改代码->评审人查看修改情况。

Jupiter支持在一个项目中的多次评审，多人协同参与评审，支持多种配置库如SVN、CVS等，支持简单的评审流程，支持问题跟踪。

我想Jupiter提供的这些简单的功能，已经完全可以胜任常规的代码评审需要，作为常规评审比较合适。并且评审人和被评审人都可以通过Eclipse操 作，对于我们程序员来说，很方便，大家天天都要对着Eclipse，Jupiter可以让代码评审成为编码的一部分工作而又不增加评审工作量。

Jupiter虽然对常规代码评审(Code Review)比较适合，但是对于像敏捷中的每日代码评审就显得有些力不从心，虽然也可以做到，但是每次评审需要指定评审文件，并且不能通过指定目录或者 通配符方式还是有些不便的，这一点它不如ReviewClipse那么直接精准。

先上一个Jupiter的图：



 
 
这两款工具完全可以结合起来使用，相信会减少不少代码评审的工作量。

两个工具的详细介绍，内容比较多，有必要再单独开2个帖子讨论，当然这也得看大家的回帖和参与情况了，否则没有发帖动力啊：）
暂时先介绍这么多…

评论

31 楼 jellyfish 2011-01-06  

If you use JIRA system, Crucible is also a good choice.

30 楼 wenshao 2011-01-06  

这两个都是很烂的工具，别报啥希望了。。。

29 楼 287854442 2010-08-19  

希望楼主继续介绍，持续关 注 中....

28 楼 soartju 2010-02-05  

kjj 写道

主贴颜色能不能改亮点,看着费劲得很

字体已经改成全黑。

27 楼 kjj 2010-02-05  

主贴颜色能不能改亮点,看着费劲得很

26 楼 sanyao 2010-02-05  

好东西，正准备开展相关工作呢，学习中

25 楼 soartju 2010-01-13  

sswh 写道

刚看到ReviewClipse网站下面的声明

You are not allowed to decompile or otherwise reduce the Software to a human perceivable form.

多谢提醒，是我搞错了，ReviewClipse不是开源的，原帖已经修改

24 楼 sswh 2010-01-13  

刚看到ReviewClipse网站下面的声明

You are not allowed to decompile or otherwise reduce the Software to a human perceivable form.

23 楼 sswh 2010-01-13  

“经过调研和试用，我找到两款比较好用的开源免费插件：ReviewClipse 和 Jupiter 。”

~~~~~~~~~~~~~~~~~~~~~~
ReviewClipse 是开源的吗？源码在哪儿可以下到？
在http://www.inso.tuwien.ac.at/projects/reviewclipse/
我没找到

谢谢

22 楼 myhousepoor 2010-01-11  

这个是好东西，值得研究一下

21 楼 ithero 2010-01-09  

很不错。呵呵，看来大家遵循的规范都差不多。这个工具很好用

20 楼 x_root 2010-01-08  

不错的工具，第二个东东我们组正准备使用，希望贴出来使用方法及经验出来。

19 楼 soartju 2010-01-08  

黑暗浪子 写道

snowing11812 写道

这代码写的 包括hql的构造 简直跟我们公司写的一模一样啊
哈哈哈 看来大家是遵循的一个规范啊

if else太多，建议使用metrics插件

谢谢，回头试试

18 楼 黑暗浪子 2010-01-08  

snowing11812 写道

这代码写的 包括hql的构造 简直跟我们公司写的一模一样啊
哈哈哈 看来大家是遵循的一个规范啊

if else太多，建议使用metrics插件

17 楼 橘子洲头 2010-01-08  

好东西，谢谢分享

16 楼 soartju 2010-01-08  

readythink 写道

中科院.....

看来截图暴漏太多内容了，呵呵

15 楼 readythink 2010-01-08  

中科院.....

14 楼 soartju 2010-01-08  

hankesi2000 写道

代码的检查工具是需要有一个，并且贯彻下去！

支持楼主对两个plugin写详细介绍，如果附上怎么带动组里执行的，就更好了！

谢谢支持，等我组里真正开始用起来后，我会整理出来与大家分享。

13 楼 prowl 2010-01-08  

改用idea了。。。

12 楼 kjj 2010-01-08  

是我梦寐以求的号东西，谢谢楼主了