团队的Code Review实践

高效地进行 Code Review 一直是我们想要做的事情，如何持续保持高效 Review 也是我们日常开发中所亟需解决的问题。

在疫情爆发之前，团队大多是线下一起办公。大家会聚在一起 Code Review，把讨论出的反馈记录在便利贴并贴在白板上。而当远程、线上的同事越来越多后，我们选择将 Code Review 的反馈记录在 Trello 等电子看板上。

今天我们总结了一种比较高效的 Code Review 方案，并带来了一个比较好用的工具。下面来和大家分享一下我们团队 Code Review 实践的历程。

Code Review的好处

一直以来，人们都说 Code Reivew 作为敏捷开发中重要的一环，实践 Code Review 有很多益处，那么到底 Code Review 有什么益处呢? 到底是因为什么魔力，我们这么笃定每天去做 Code Review?

我们认真地思考了 Code Review 的益处，认为以下几点在整个敏捷开发流程中起到了至关重要的作用。

互相学习，知识共享

我们无法保证项目中每个人都拥有相同的知识储备，而且对于同一个问题，不同的人有不同的解决方案。通过 Code Review 我们可以快速地、有重点地将自己的知识传递给其他人。其他人也可以将代码上下文、知识以及更好的解决方案提供给我们，让我们做出更优的实现。

Code Review 可以让业务上下文在团队中共享。日常工作中，开发人员只会对自己负责的这块业务比较熟悉。通过 Code Review 的方式，我们可以快速了解到其他业务上下文，更好地了解业务知识，让每位同学对系统都有更全面的认识。

统一风格，提高代码质量

团队一般会有统一的代码规范。有些规范可以借助工具扫描来实现统一，工具扫描不到的就要靠 Code Review 来统一规范。

莎士比亚说过：“一千个人眼里有一千个哈姆雷特”，针对同一段代码可能多种实现都是可行的。如果没有统一的风格，不便于后续对代码的维护。通过 Code Review 的方式，我们可以在团队内达成共识，形成统一的代码规范，为后续开发提供标准，让代码看起来是同一个人写的。

尽早暴露问题，降低修复成本

代码开发过程中，我们很容易陷入到自己所预定的 Happy Path 中，反复验证却无法发现问题。通过 Code Review 我们可以借助他人的视角来审查自己的代码。是复制的时候变量名忘记了修改，还是忘记了测试异常情况，或是理解错业务含义等等。代码从写完到 Review 的时间间隔越短，解决问题的成本越低。和测试左移类似，Code Review 也可以将问题左移&#