这是一个创建于 4306 天前的主题，其中的信息可能已经有所发展或是发生改变。
无意中看到 http://www.v2ex.com/t/38858 这个thread，讨论如何进行code review。
回想两年前在微软某组工作时，发送一个code review需要经过以下几步，
(source depot是微软自己包裹的perforce)
1. 通过source depot找到changelist的编号
2. 通过DPK命令生成一个.dpk文件 (在本机)
3. 将.dpk文件拷贝到\\transfer\transfer\your_name
4. 手动生成一个full path, e.g. \\transfer\transfer\your_name\changelist_num.dpk
5. 打开outlook，发一封邮件给reviewers，将full path粘贴到邮件正文。
除了DPK， 还有诸如bbpack, jjpack, 等等xxpack，yypack各种各样的pack云云
对于code reviewers来说，没有统一的工具来打开dpk. 默认的是windiff. 可用性很低。我个人使用一个叫odd的内部工具。
添加comment就更搞笑了。首先新建一封邮件，在odd中，对于某个要发表评论的change点右键，选择“copy”，再在邮件中选择“paste”，这时change会被拷贝到邮件中，并添加了一个小文本框，reviewer在小文本框中填写comments。如果change比较多，那么这个复制粘贴的过程会狠冗长。最后可以在邮件中说些overall这个CR如何的话。邮件中全是change，毫无code context。
后来内部做了一个叫做code flow的工具，把这个过程略自动化了，不知道现在有没有改善。
回想两年前在微软某组工作时，发送一个code review需要经过以下几步，
(source depot是微软自己包裹的perforce)
1. 通过source depot找到changelist的编号
2. 通过DPK命令生成一个.dpk文件 (在本机)
3. 将.dpk文件拷贝到\\transfer\transfer\your_name
4. 手动生成一个full path, e.g. \\transfer\transfer\your_name\changelist_num.dpk
5. 打开outlook，发一封邮件给reviewers，将full path粘贴到邮件正文。
除了DPK， 还有诸如bbpack, jjpack, 等等xxpack，yypack各种各样的pack云云
对于code reviewers来说，没有统一的工具来打开dpk. 默认的是windiff. 可用性很低。我个人使用一个叫odd的内部工具。
添加comment就更搞笑了。首先新建一封邮件，在odd中，对于某个要发表评论的change点右键，选择“copy”，再在邮件中选择“paste”，这时change会被拷贝到邮件中，并添加了一个小文本框，reviewer在小文本框中填写comments。如果change比较多，那么这个复制粘贴的过程会狠冗长。最后可以在邮件中说些overall这个CR如何的话。邮件中全是change，毫无code context。
后来内部做了一个叫做code flow的工具，把这个过程略自动化了，不知道现在有没有改善。
 |
1
jedia9264 2013-02-06 10:52:15 +08:00
咋个这么原始呢？堂堂微软啊。他自己不是有 TFS，都不用啊？
|
 |
2
AlexYang 2013-02-06 15:43:29 +08:00
VS2008里新开发得Team Foundation Server不知道有没有code review的功能。
现在开源社区的CI工具链都已经很自动化了。 |
 |
3
missdeer 2013-02-06 17:37:54 +08:00
ms不同的项目组用不同的vcs。现在那个code flow集成了一些工具，已经非常强大了，可以多轮迭代review，可以comment。。。而且也不是发邮件发起code review了，在bugger里改status就会把diff整出来搞。。。
|
 |
4
200 2013-02-07 11:53:04 +08:00 via iPad
去年实习的时候Team用的是codeflow，现在codeflow很好用，可以和sd集成，自动生成dpk
|
 |
5
yueq OP |
 |
6
weibest 2014-03-16 21:30:47 +08:00
感觉就是上个世纪的工具，又慢又难用。
哎，日薄西山了~ |
Select Language