从一次回顾会议开始
“要不……我们不做……代码审查了……试试？”还记得当有人抛出这个建议时周围同学的表情，那种表情用两个字加两个标点符号就可以形容：“什么？！”
对了，先介绍一下背景，这是项目一次普通的回顾会议，我们正在讨论的是如何让代码审查更有效率和效果。我们做代码审查的方式比较简单直接，就是每日站会后，大家围在一台开发机周围，逐一轮换讲解昨天所有提交的内容，就像下图中的那样。还有，这是一个已经超过了 7 年的比较大型的项目，代码审查是我们从项目开始就坚持的一个实践，所以当有人提议废除它的时候，这在很多同学心里是想都没想过的事情。
代码审查是一个很好的实践，可以帮助团队里的同学了解其他同学在做什么，可以分享项目的上下文，可以分享技术上的一些小魔法，可以发现很多潜在的代码缺陷，可以提高代码质量，还可以有很多很多好处……
但是，在真正的实施过程中，很多情况下并不像想象的那般美好，经常出现例如有些同学由于跟不上其他人讲解的速度（毕竟不是自己写的）或是没有相关的上下文（例如刚加入项目的新成员），或是由于提交没有被很好的切分和组织，导致整个过程都处于游离状态（就像下图中的我……毫无摆拍痕迹），而代码审查的效果也打了折扣，渐渐的变成了一个流程，一个过场， 一个习惯。
图 1. Code Review
于是团队里就有人站了出来，引导大家去发现背后的问题，也就引来了这样一场激烈的讨论。在讨论中，有些同学坚持在说代码审查还是很有用的，有这样那样的好处，需要保持下去；有些同学则非常实际地指出了执行上的各种困难和问题。讨论异常激烈，直到有人小心翼翼地提出了文章开头的那个建议，一片哗然后大家都陷入沉寂：是啊，不做代码审查了，我们会失去或是得到什么呢？
本文转载自健荐公众号。
评论