AntiPattern: Batch Code Review

所谓批量code review是指一段时间, 通常是一个迭代或几周之后, 团队坐在一起集中code review. 这种方式有几个问题:

  • 有一些通过code review能发现的bug因此而潜伏下来, 有可能被发布, 就算运气好发布前通过各种测试被发现, 依然带来一些overhead像发bug, 定位问题, 修bug等额外的工作.
  • 有一些通过code review能共享出来的知识因此而很长时间之后才被共享, 而在此期间, 团队其他人由于不了解这些知识, 有可能犯已经犯过的错误, 或走一些弯路, 造成效率的降低
  • 有一些通过code review能得到改善的设计因此而被推迟, 造成技术债的累计, 提高维护成本
  • 有一些故事已淡忘…
  • 如果是通过版本控制系统的diff来做code review, 则按顺序review时很多中间版本都已经过时, 不得不费力解释最新版本是什么样子, 给团队带来理解上的困难, 甚至造成错误的印象
  • code review meeting将持续很长时间, 每个人都很疲惫

然而最关键的问题是: 批量code review得出来的Action, 很难得到执行, 因为已经时过境迁, 团队手头有新的高优先级的任务, 很难回过头去改已经标记为"Done"的任务. 而Action得不到执行, code review的意义已经消失了一半.

Daily code review is a better solution.

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 1
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 1
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值