【技术研究】 CodeReview初识

简介: 【技术研究】 CodeReview初识

一、codeview是什么?

    codeview是代码评审,不是代码分析。代码评审也称代码复查,是指通过阅读代码来检查源代码与编码标准的符合性以及代码质量的活动。


二、为什么要使用codeview?(好处)


1、提高代码质量

2、在项目的早期发现缺陷,将损失降至最低

3、评审的过程也是重新梳理思路的过程,双方都加深了对系统的理解

4、促进团队沟通、促进知识共享、共同提高

5、交叉评审——代码走查:团队成员互相检查代码


三、什么时候进行codereview


可以选择在下班前半小时,对当天改动的模块进行评审。开发进行到某一阶段时,对共性问题进行总结,对

好的做法进行提炼和推广


四、如何进行codereview

参与者:包括项目组全体成员,其它组的开发组长也应尽量参加。


  结对方式:可以是任意两个组员,或开发组长分别与每个组员结对进行。


  代码作者讲解如何以及为何这样实现、评审者提出问题和建议。


  注意: 每次解决的问题要记录到SVN注释或JIRA


     每次评审不要贪多,当一次评审超过400行代码时,能发现缺陷数显著降低——事倍功半


  会审:以项目为单位,召开专门的代码评审会议。


  比如,我们在集体开发过程中,写完自己的代码后,检查没有错误后,提交到SVN上。代码虽然没有错误,但是可能有其他问题:代码冗余,逻辑错误,没有写日志、异常处理,或不符合项目的格式要求等,我们需要再次检查代码处理这些问题。代码量多了之后,检查工作会变得很复杂。


  所以,我们研究代码审查工具来解决这些问题。过程如下:


  本地修改代码,提交SVN,设置审核者,编写修改注释;


  审核者会收到审核请求邮件,审核者在Phabricator中会看到Audit中有待处理审核,审核者进行审核;


  如果审核者认为代码需要修改的话,codewriter就不应该把代码提交到SVN,而是应该修改之后再次执行arc diff命令,直到所有的审核者都认为代码OK了之后才应该使用arccommit,提交到SVN上。所有的diff都会在Phabricator被记录,而且可以使用arc获取回来,后续再研究。


  因为审查者在每次组员提交代码后进行评审,所以每次评审的任务量变少了,更容易发现问题。


其他介绍,请见百度:https://baike.baidu.com/item/%E4%BB%A3%E7%A0%81%E8%AF%84%E5%AE%A1/8960487?fr=aladdin













相关文章
|
29天前
交付成果 提高IT领导力的七大窍门
交付成果 提高IT领导力的七大窍门
|
2月前
|
敏捷开发 安全 Devops
DevSecOps:软件安全的新思维
DevOps作为一种敏捷开发模式,已经被越来越多的企业所采用。但随着互联网的蓬勃发展,网络安全问题日益突出,传统的软件开发方式已经无法满足安全需求。因此,我们需要引入DevSecOps的概念,将安全与开发流程相结合,以实现全面的软件安全保护。
31 3
|
9月前
|
缓存 网络协议 中间件
软件研发的道德情操
亚当斯密对我的启发,我称之为“软件研发的道德情操”。
132910 18
CMMI之技术预研
CMMI之技术预研
443 0
|
供应链 监控 架构师
软件测试工程师 路在何方?如何发展?职业规划是怎样的?
软件测试工程师 路在何方?如何发展?职业规划是怎样的?
175 2
软件测试工程师 路在何方?如何发展?职业规划是怎样的?
|
算法 测试技术 Python
热饭的测开成果盘点第九期:白盒自动化平台热饭的测开成果盘点第九期:白盒自动化平台
本期介绍的是一个技术含量很变态的工具-白盒自动化测试。何为白盒测试?其实就是测试具体代码,有五种方式叫做五种逻辑覆盖率,比如路径覆盖/语句覆盖等。
热饭的测开成果盘点第九期:白盒自动化平台热饭的测开成果盘点第九期:白盒自动化平台
|
测试技术
【软件测试基础理论】身为测试主管,你必须知道的事情!(质量铁三角和CMM)
【软件测试基础理论】身为测试主管,你必须知道的事情!(质量铁三角和CMM)
|
监控
CMMI落地中PQA实施的苦恼
CMMI一直强调组织愿景,组织战略,一切目标的制定,活动的裁剪都是围绕着“战略”二字展开。因此不同角色的定位和工作内容也由高层的战略指导方向而定,那么QA能做到什么样,老大的理解、定位、投入是很关键的。
CMMI落地中PQA实施的苦恼
|
设计模式 自然语言处理 运维
阿里研究员:软件测试中的18个难题
对于软件测试来说,怎么样才算测够了?如何评价测试的有效性?那么多测试用例,以后怎么删?在软件测试中会遇到非常多的问题,阿里研究员郑子颖分享了18个他总结出的难题以及相关看法,希望对同学们有所启发。
3670 0
阿里研究员:软件测试中的18个难题
一张图读懂科技部《关于促进新型研发机构发展的指导意见》
近日,科技部印发《关于促进新型研发机构发展的指导意见》,其中提出,鼓励设立科技类民办非企业单位(社会服务机构)性质的新型研发机构;鼓励地方通过中央引导地方科技发展专项资金,支持新型研发机构建设运行;地方政府可采用创新券等支持方式,推动企业向新型研发机构购买研发创新服务等。