zoukankan      html  css  js  c++  java
  • Code Review

    Code Review是目前常用的代码质量保证方法,在这种实践中,写code的程序员要向同伴们展示他的源代码,接受大家的审查,并且解释他的设计思路。

    这是一种很好的软件工程实践,也是对软件缺陷预防的一个重要手段。从质量角度,由非作者阅读代码,可以及时发现被作者遗漏过去的问题;从维护角度,可以使更多人掌握代码的实际含义,今后的代码维护也变得容易;从团队建设的角度,可以使team的成员相互学习程序设计思想、方法和技巧;从成本的角度,虽然在review的时候会有较高的cost,但由于是在代码编写阶段进行的,对bug进行预防性的活动,能有效降低集成测试阶段的bug fix的工作量。

    但是,没有银弹。

    code review本身来说,整个流程非常清晰,逻辑也非常简单,但在执行的过程中,往往不能顺利的取得预期的效果。在某些时候,甚至会蜕变成掩盖问题,甚至产生更多新问题的陷阱。和所有的prcoess一样,code review也要面对具体执行过程中,执行力不足的困难。

    困难之一,是进行code review的一个重要条件,即参加review的人员的编码和对系统的了解情况,必须相差不大,处在一个水平线上。在XP Practices中,有一个核心实践pair programming,就要求相互结对的程序员的水平要基本相似,其原因和理由是和review中的情况相似的。如果水平相差过大,就变成了training而不是review了。所谓的审查会议,也只能空耗时间而难有建树。

    困难之二,是如何有效保证review的时间,避免因时间问题使其变成走过场,流于形式。考虑到review开始前的准备、一起进行的review会议、即会议后对结果的回顾和检查,每个参加者对每百行代码的时间成本,至少在1小时以上,而一般review的参与者都在三个人以上,从而每百行代码的时间成本,要接近到一个staff day。对于一个两万行核心代码的小型项目来说,code review要占用到八到十个人月的时间成本,对整个项目的schedule来说,是一个影响很大的task项。是否能够在项目初期,就能考虑并保留出这段时间,在项目中期,能顶着schedule的压力,坚持按计划完成review,在项目后期,能有效的分析和回顾review的结果,预防bug的发生,即需要来自上层管理者的肯定,也需要整个team共同的认识。在一个匆忙赶进度,以时间而不是质量为第一目标的开发氛围中,code review非常容易变成一个无效的空壳,成为只为数据服务的形象工程。

    困难之三,是如何真正的把握review的目的,以良好的态度来完成。review的参与者可以有两种态度,相互学习肯定对方能力的态度,和相互纠错证明自己能力的态度。在一个真正的开发团队中,大家有着共同的目标,是以第一种心态来进行review工作的;而在大多数缺乏认同感的开发团队中,心态往往就是第二种。从执行和结果上来看,两者都会产同类似的review记录,找到同样多的bug,但将一个松散的团队放在review的聚光灯下,只会引起更多的内耗。而项目的管理者,对review也可以有两种态度,针对review的数据并带领团队改善的态度,和针对review参与者并作为对其绩效考核的态度。在以是否在review过程中找到bug来作为判断参与者绩效和能力的氛围中,也容易时review的参与者开始惧怕面对错误而使review的效果降低。

    软件质量不能仅仅依靠测试来保证,必须从项目开始的第一天,就逐步的构建出来。code review是对代码的回顾,是跳出局部的检查,是对bug的预防。克劳士比所说,第一次就要把事情做对!code review就是在“质量预防”的思想下进行的软件工程实践。

  • 相关阅读:
    win7+Apache 设置域名指向本地文件夹
    JavaScript 函数式编程
    JS防抖动
    13 个最佳 JavaScript 数据网格库
    js笔试-接收get请求参数
    这10道javascript笔试题你都会么
    60行JavaScript代码俄罗斯方块
    先少谈点人工智能好吗?
    gulp+webpack构建配置
    Gulp和webpack的区别,是一种工具吗?
  • 原文地址:https://www.cnblogs.com/shaohz2014/p/3690814.html
Copyright © 2011-2022 走看看