• john_zhang
    2019-09-20
    我们推行过一段时间代码审查,因为三五个人,所以采用的是团体审查,每天半个小时左右,可惜后来开发进度赶,慢慢就没做了,现在开发同事总是以进度赶为由,不太认同代码审查,怎么破?

    作者回复: 从点赞的数量来看,大家对这个问题是比较感兴趣的。我建议以下几个办法:

    1. 团队统一思想,代码审查是有效工作的一部分,应该计算到工作量里面
    2. 减少团体审查,选择更多使用工具进行1对1的审查。前者很难做到效率高。应该只是针对一些重点的提交采用这种方式。
    3. 培训团队,统一认识,让大家了解到代码审查的长期收益,让大家不能不能只看到当前的开发进度这个短期收益,还要考虑代码可维护性以及后续添加新功能的速度这些长期收益。甚至可以坚持推动代码审查这个操作作为团队的制度。

    后面的答疑文章我会作进一步的讨论。

     1
     12
  • Just for fun
    2019-11-18
    老师您好,根据文中的建议,一方面要求审查人要及时进行审查,另一方面又要求提交人进行频繁的原子性提交,这样就会导致审查人的工作会被频繁打断,这样不会影响审查人的工作效率吗?我们团队也是要求开发人员小步快跑,但是这样之后发现审查人的工作又被频繁打断,工作效率又降低了很多

    作者回复: 解决的办法:
    1. 让开发者代码在被审查的时候不被阻塞,还可以做其他的事情。比如我在第26,27篇文章中描述的使用git方式。
    2. 审查者抽出一个时间段集中审查。

    
     1
  • -W.LI-
    2019-10-03
    老师好!公司的代码规范是用的阿里巴巴开发手册。也和leader沟通过怎么提升代码质量。leader建议多阅读开源项目源码。现状就是忙于业务代码,需求变动,并没太多时间阅读源码,也不知从何入手。以下几个现象比较严重。
    1.为了实现业务老写出一个很长的方法,又找不到合适的角度去拆分这个方法。
    2.无法平衡效率和可读性。
    3.好用的工具库,jdk新特性掌握太少。

    作者回复: 我在对一个比较长的方法进行拆分的时候。通常会先把。这个方法的,所做的操作。按照所做的工作按照步骤进行注释
    ==函数==
    foo() {
      ooo;
      ppp;
      qqq;
      rrr;
    }

    ==1. 把程序分成极大部分,添加注释==
    foo() {
      // Do something A
      ooo;
      ppp;

      // Do something B
      qqq;

      // Do seomthing C
      rrr;
    }

    ==2. 把几个步骤提取出来,从而把foo()缩短==
    DoSomethingA() {
      ooo;
      ppp;
    }

    DoSomethingB() {
      qqq;
    }

    DoSomethingC() {
      rrr;
    }

    foo() {
      DoSomethingA();
      DoSomethingB();
      DoSomethingC();
    }

    这个里面关键在于第一步进行步骤的拆分。仔细对foo的工作思考,一定可以拆小的。

    
     1
  • Donald
    2020-02-04
    你好,从目前互联网发展的形势来看,现在越来越的公司的测试人力越来越少,开发测试比越来越高,但是开发对测试的要求却丝毫没有降低,同样要求测试需要对质量保障负责。所以,我想问一下,有什么好的方案解决这个问题吗?
    
    
  • 不是云不飘
    2019-10-15
    规范和审查一直由项目赶着上线没有实施,导致现在代码的坏味道越来越多,也没有时间去改,感觉一直恶性循环。

    作者回复: 推荐听一下这篇:
    14 | 质量与速度的均衡:让“唯快不破”快得更持久
    https://time.geekbang.org/column/article/138916

    
    
  • 二狗
    2019-09-30
    一个功能怎么拆成多个原子提交,关于原子性提交不是很理解 (日常都是一个功能,一个bug一次提交)

    作者回复: 这是因为通常一个功能可能会很大,如果一整个功能作为一个提交的话,提交可能很大,就像文章中漫画里的巨大星球一样 😀

    做法是把功能差分成几个“子功能”,分别提交。比如说一个功能是提供一个API。如果这个API比较大,我们可以把它拆分成(数据模型+API业务)两个部分。如果这样下来还是很大,还可以继续拆小。比如API业务可以分成(refactor+添加新业务)两个部分。

    总之,是把每一个提交都做成能够独立完成一些任务,但是有不太大。

    
    
  • Geek_1988
    2019-09-27
    葛老师,如果太追求代码的原子性,产生很多提交,会不会导致git太庞大而影响运行呢?

    作者回复: 很棒的思考!Git仓库对非常大的代码仓的确会有性能问题。不过追求原子性多产生的提交的量级应该不会太大,一般来说不会对Git的性能造成关键影响。而且每个提交变小了,也会抵消一部分多提交导致的性能下降。

    
    
  • Alick
    2019-09-22
    葛老师,能否介绍下 Facebook 代码评审的绩效考评方法?

    基于评审意见数,觉得纬度单一;基于问题严重性,有易起纷争的担忧;

    采取何种代码评审绩效评价方式,能够起到正向积极导向效果?

    作者回复: 评审意见数只能是一个简单的参考。

    在Facebook,对代码评审工作的衡量,主要是通过同事间的主观反馈来获得的。比如我在文中举的例子,我的主管给我的两个反馈,一次是审查不积极,一次是审查太注意细节,都是其他同时反馈给他的。

    
    
我们在线,来聊聊吧