代码审查(Code Review) -- 全面解读C++test 代码审查(Code Review)功能
本文源自于软件测试人专业博客,任何转载均需注明原址:http://www.mytestlife.com/post/code_review.html
Code Review代码审查简介
Code Review又称代码审查或代码评审,是提高代码质量的重要手段。一般而言,Code Review又分同行评审(Peer Code Review )和多人评审。对于同行评审,即一对一评审,彼此评审或交叉评审。多人评审,一般在固定周期内,比如一周或一个月,参与项目的小组人员,在会议室使用投影仪将代码投射出来,评审代码。
Code Review代码审查过程中暴露的缺陷
1. 同行评审流程不能有效监控,评审者之间需要频繁来回走动评审代码,导致同行评审无法有效进行,长此以往,使得评审者之间私下达成默契,降低同行评审或者干脆放弃同行评审,使得同行评审流于形式;
2. 多人评审,需要将大家集合在一处,如果一个项目在两地合作开发,则此时很难进行多人评审;
C++test的Code Review代码评审流程(注:从Parasoft C++test 9.0开始,Parasoft把Code Review模块独立成一个单独的软件,当然也会保留C++test中的Code Review功能)
1. 三种角色:Author, Reviewer, Monitor;
C++test的代码审查,提供三种角色,除了代码作者,评审者外,还有一个监视者的角色。该角色不实际进行代码评审,但可以监控代码评审的流程,以确保代码评审真正落实。
2. 两种模式:Pre-commit和Post-commit;
Pre-commit是指开发人员在提交代码到源码控制器之前进行的扫描配置,C++test的Code Review模块将自动扫描新编写的代码部分,并提交给Reviewer进行评审,Reviewer的电脑上则可以看到需要评审的代码部分;这种模式一般针对团队中的新手,防止由于其经验不足导致Check in进到源码控制器中的代码影响整个软件的构建;
Post-commit是指开发人员提交代码到源码控制器后,由服务器版本进行集中进行扫描和分发;
3. 两种策略:Restricted 和None Restricted;
Restricted策略下,必须经过Reviwer的同意,评审任务方可终止,此时Author方可提交代码进入源码控制器;
None Restricted策略下,Author可自行决定是否终止评审任务;
以上是对于Parasoft C++test的Code Review功能的说明,有任何疑问,欢迎在软件测试人的C++test专栏展开探讨。
原文链接:http://www.mytestlife.com/post/code_review.html
