记得刚成为开发团队一员的时候,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一个评审人),关闭此次评审请求
网站名称:VS11之代码评审
文章分享:http://www.shufengxianlan.com/qtweb/news43/54943.html
网站建设、网络推广公司-创新互联,是专注品牌与效果的网站制作,网络营销seo公司;服务项目有等
声明:本网站发布的内容(图片、视频和文字)以用户投稿、用户转载内容为主,如果涉及侵权请尽快告知,我们将会在第一时间删除。文章观点不代表本网站立场,如需处理请联系客服。电话:028-86922220;邮箱:631063699@qq.com。内容未经允许不得转载,或转载时需注明来源: 创新互联