网络科技

    今日:198| 主题:246161
收藏本版
互联网、科技极客的综合动态。

[其他] 让代码审查扮演更好的角色

[复制链接]
女人能输不能哭 发表于 2016-10-5 18:58:09
192 10

立即注册CoLaBug.com会员,免费获得投稿人的专业资料,享用更多功能,玩转个人品牌!

您需要 登录 才可以下载或查看,没有帐号?立即注册

x
代码审查(Code Review)是很多大公司里面都有的一个流程。它指的是一个人编码,另有几个人负责审查,并提出修改意见。代码审查在大多数情况下对公司整体的工程质量是有提高的,但是如果使用不当的话,很可能反倒会降低工程质量。代码审查究竟在一个组织里面是有正面效应或者是负面效应取决于很多因素,而我认为其中最重要的是代码审查在开发过程中扮演的角色。
  
让代码审查扮演更好的角色-1 (角色,风格,工程,接口,开发)
  首先,我们先看看在代码审查中所需要找出的问题类型。它们可以是:
  
       
  • 语法及代码风格问题:一般有静态检查工具可以解决,但难免有疏漏。   
  • 效率问题:需要有一定经验的人来辨别低效的部分。   
  • 命名问题:这其实是一个很经常出现也很重要的问题。对于一个人来讲说得通的命名不见得对于团队而言说得通,所以很多时候较难的命名要由团队通过代码审查协同解决。   
  • 设计问题:小到接口的设计,大到服务间通信的协议,都属于设计问题,根据情况可以由小部分人或者整个团队解决。设计问题是代码审查中最常见的问题。  
  对于前三种问题,相对来讲都很好解决。其中相对棘手的莫过效率问题,但实际上基本上知道效率问题的人都知道优化方案。然而,如果一个审查的人突然提出一个很合理的设计问题,需要你重新修改源代码,你会发现你需要花大量地时间重新编写。
  例如,在编写一个JavaScript库packageA的时候,你提交了代码审查。有人可能会提醒你:packageA是用于桌面端网站的库,相对应的还有一个移动端的库packageB。为了保持工程上的一致性,建议把packageA改成盒packageB一样的API。一致性一直以来是一个让人无法反驳的设计追求,所以你只好把辛辛苦苦自己设计好的API全部重改…
  所以,若你的代码里面被提出存在设计问题,消耗的工程时间会增加。而工程时间对公司来讲就是金钱。
  造成存在需要大改的设计问题的原因其实无非三个:
  
       
  • 设计能力不足   
  • 对开发的系统不熟悉,缺乏上下文(Context)   
  • 过晚提交代码审查  
  前两个原因都很直白,但是第三个原因有点匪夷所思。什么叫做过晚提交代码审查?
  我想是代码审查英文单词中的”Review”给予人的误导,很多人是在代码几乎完成或者已经完成后才提交代码审查的。就好像在做一盘菜,做到最后一步的时候才想起来要尝一小口看看味道对不对,结果发现没加盐。
  在最后一步进行代码审查,还会因为审查者一下子接收太多信息,而造成他可能无法发现一些应该发现的问题。

让代码审查扮演更好的角色-2 (角色,风格,工程,接口,开发)
  显然“审查”扮演的角色在这里出现了问题,它不应该是传统意义上的到最后一步进行把关,而应该是贯穿整个编码过程的一个辅助过程。用比较老式的软件工程“土话”说,它应该是一个Umbrella Activity(雨伞活动),全程保护编码过程的质量。
  现在,我的代码审查流程是这样的:首先完成一个基本的设计,加上基本的注释,达到一个完成度——最可能出现大设计问题的完成度。接着commit,并推入到代码审查中,邀请其他人来审查。这基本上就是对他们说,“看,这是我写的,很简单,可能烂得跟一坨屎一样,麻烦你们帮我看看有没有什么大问题”。
12下一页
友荐云推荐




上一篇:谷歌发新机却不得华为助力,代工权投向HTC,遭网友狂喷
下一篇:PPI 过千,夏普超清显示屏与 VR 最配?
酷辣虫提示酷辣虫禁止发表任何与中华人民共和国法律有抵触的内容!所有内容由用户发布,并不代表酷辣虫的观点,酷辣虫无法对用户发布内容真实性提供任何的保证,请自行验证并承担风险与后果。如您有版权、违规等问题,请通过"联系我们"或"违规举报"告知我们处理。

sskankanniu 发表于 2016-10-5 19:36:45
呵呵。。。
回复 支持 反对

使用道具 举报

gsrhp 发表于 2016-10-5 19:40:19
酷辣虫人气好旺!
回复 支持 反对

使用道具 举报

大强长u 发表于 2016-10-5 20:00:32
不错 支持下
回复 支持 反对

使用道具 举报

liluo1991 发表于 2016-10-5 20:06:00
这么经典的话只有女人能输不能哭能想到!
回复 支持 反对

使用道具 举报

wenxinliuxy 发表于 2016-10-5 20:09:57
走过路过,千万不要错过!
回复 支持 反对

使用道具 举报

聶磊 发表于 2016-10-5 20:10:10
我就是来看帖子的,楼主英明啊!
回复 支持 反对

使用道具 举报

王江民 发表于 2016-10-6 01:05:48
我消极对待减肥,能不能取消我胖子的资格啊
回复 支持 反对

使用道具 举报

林飞 发表于 2016-10-6 04:15:39
看完了这么强的文章,我想说点什么,但是又不知道说什么好,想来想去还是想想就好了
回复 支持 反对

使用道具 举报

新柔 发表于 2016-10-7 04:58:58
楼下的不要小看我,我可不是吃素的。
回复 支持 反对

使用道具 举报

*滑动验证:
您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

我要投稿

推荐阅读

扫码访问 @iTTTTT瑞翔 的微博
回页顶回复上一篇下一篇回列表手机版
手机版/CoLaBug.com ( 粤ICP备05003221号 | 文网文[2010]257号 )|网站地图 酷辣虫

© 2001-2016 Comsenz Inc. Design: Dean. DiscuzFans.

返回顶部 返回列表