团队成员之间的 code review 大家都用什么工具？

以前在windows平台上的TFS，做CodeReview非常简单：
开发者在自己机器上改好代码后生成一个shelveset，这个shelveset直接就保存在服务器上了
邮件发送给reviewer告知shelveset名称
reviewer通过tfsodd等工具，打开shelveset,查看代码变化，提意见。reviewer本机并不需要有服务器端代码
review结束后（也许几个来回），开发者check-in shelveset。结束。
也有时候，有些shelveset并不check-in，但一样能留在服务器上以备今后查看。

现在的小公司里，也不知道这群毕业了直接就在这里的小朋友，是没人教不会用呢，还是真的Mac下没有好的code review工具，目前他们要么是直接跑到对方座位上看（于是匆匆而过看得很不仔细），要么就是看天书般的svn diff得到的文件（我觉得这个实在需要超能力才能读）。

我粗粗搜了一下，没找到什么方便的工具，只有若干零星功能的，要合并在一起用才好。
所以我在想，是不是该写个脚本或app。
我大致想要一下几个功能。大家帮我看看，有没有能满足我要求的推荐一下。要没有我就只能自己动手了。
0，每个人在机器上都有一个专门用来做code review 的 workspace。配制好CRWorkSpace的路径参数给下面的工具用
1，开发者在写完程序，敲 cr -create <shelvesetName> <message>
于是
1.1 程序检查开发者当前代码是否update过，没有，提醒并帮助svn update，如有必要弹出FileMerge
1.2 svn diff > shelvesetName.patch, 上传patch到公司的share folder
2, cr -email <shelvesetName> <reviewer's email>发邮件给指定email。邮件中包含一条用以apply patch的 cr -review <shelvesetName> 命令
3，收到邮件的同学copy & paste & run 这条命令
3.1 将程序检查code review workspace的代码版本，sync到和patch一致
3.2 apply patch
3.3 弹出 SmartSVN（免费版）以查看代码自之前一个版本起的变动（可以吗？若不行， reviewer 本机还得准备两个work space，用文件比较工具来查看）
4，开发者执行 cr -commit <shelvesetName>提交代码

基本上就这样，还算蛮简单的。
和原来用TFS相比，就差一个reviewer本机必须有一个用来做code review的专门workspace。这个在以后有多个brunch的时候会遇到麻烦（我原先的项目，最多的时候有十多个brunch同时在做平行开发）

各位都在用什么工具，有可以让我偷懒不自己写的么？

谢啦！

sampeng

2012-06-09 11:34:24 +08:00

@Elfe 其实很简单，我们没把事情弄那么复杂。
每个人的代码都是相对独立的。在有良好的代码结构的条件下，直接看他的代码即可。
我工作的两家公司都是如此。
虽然比你的只关注改动部分看的代码要多的多。但好处是全局性的观察。
很多时候review可以看出代码的修改是否影响到了我或者其他程序员的接口功能。。。。。
改接口却不管下层调用的情况太多了。。

而且有些优化性质的第一次看是没感觉的，第二次看突然会觉得某个地方需要优化一下的情况也蛮多的。。所以整体看可以解决这个问题。。

然后就直接写邮件告知。。。也没出现啥问题啊。。。。。
工具用最简单的就行了。。关键在人和做的过程。搞复杂了会打击积极性的。。尤其是有很多人参与的情况下，你喜欢用工具不代表别人也用。那选择大众化的未尝不可。

当然，你说的那种直接跑工位上去看就比较搞笑了。

我还本来想自己做一个review的工具，后来也是转念一想，还真没什么必要。。。。需求太少了。

github你可以试试，简单而高效。批注很给力。当然，前提是公司敢于把代码放上去。虽然github也有私有库。