微软Visual Studio 11之代码评审

ZDNet至顶网软件频道消息 3月13日，微软发布了应用开发测试与敏捷开发解决方案Visual Studio 11 Beta版本，并在同日起在中国地区接受公测下载。下载链接为：http://www.microsoft.com/click/services/Redirect2.ashx?CR_CC=200091720

记得刚成为开发团队一员的时候，Team Leader总是对我提交的代码逐行进行检查，确保没有重大问题后才容许提交进入代码库，在这过程中我的代码中很多缺陷被发现，有的缺陷非常复杂，并且是正常测试用例所覆盖不到的，这意味着一旦在生产环境中运行，可能会带来严重的后果。后来才知道这个过程叫做代码评审，随着开发经验的丰富，也经常对其他人的代码进行评审，这个实践在开发经历中一直得以保持。

代码评审最常用的方式就是文章开头提到的那样，由专人进行逐行查看，有的团队会采用团队集体评审的方式进行，但也都是基于手工操作的模式，其最大的问题就是效率比较低。当我把需要入库的代码交由Team Leader评审的时候，不仅需要把整个工程打包并通过FTP传给他，还要准备一个预评审文档对本次提交的代码进行补充说明（注释经常不能详细说明一个复杂场景），Team Leader也不会立即对我的评审请求进行反馈，他通常在下班前才会做这件事，并且经常要把我叫到他的电脑旁进行询问，这个期间我常常在惴惴不安中度过。

在从Visual Studio 2010升级到Visual Studio 11后发现新的TeamExplorer界面中多出了一个Code Review & Requests功能区，这是Visual Studio 11提供的全新的代码评审功能。

下面我们通过一个简单的场景来验证一下这个功能：

步骤1：开发人员Peter已经完成了一个功能点，在把修改过的代码检入TFS之前发起代码评审流程：

Peter需要选择哪几个评审人对他的代码进行评审，也可以根据某种条件由系统缺省指定；输入本次评审的名称以及描述等信息。

步骤2：评审人William会收到一封邮件，描述了本次评审的发起人，工作内容链接以及其他相关信息

步骤3：评审人William可以直接点击邮件中的链接，系统会自动打开Visual Studio集成开发环境，并定位到Code Review标签页，他可以选择接受(Accept)或者拒绝(Decline)这个代码审查任务：

步骤4：William点击Accept后，开始审查代码

评审人可以点击Code Review中涉及的文件，看到了文件被修改的具体情况

评审人可以对其中的某一行代码添加注释

评审人也可以在对每个代码变更行添加注释后，写一个总的Review注释

完成评审后，评审人可以向评审发起人提交评审结果

步骤5：评审发起人Peter收到评审更新的邮件通知，看到评审人William的评审意见后进行相应修改，然后再将修改后的结果告知评审人。

Peter收到William添加注释的Code Review 通知，直接在邮件中打开链接进入Visual Studio集成开发环境中看到William的注释

Peter点击第一个William的注释，可以看到注释所对应的代码行 - 第4行被高亮显示

Peter点击第二个William的注释，可以看到注释所对应的代码行–第10行被高亮显示

Peter根据William的反馈修改相应代码

Peter修改完成后，回复William的注释，告知意见被采纳

步骤6：评审人William收到评审意见已经被采纳的邮件通知，打开Visual Studio集成开发环境确认Peter已经完成了相应修改，批准通过了本次代码评审工作；最终当所有评审人都批准了本次代码评审工作，评审工作发起人Peter可以结束本次评审。

William收到Peter回复注释的邮件，被告之修改意见被采纳

William批准通过了本次代码评审工作

Peter待所有审批人都完成代码评审后（本例中只有William一个评审人），关闭此次评审请求