代码Review，Review些什么？如何Review？

本文作者从个人面试经历出发，指出许多公司执行的代码Review存在的问题，并提出了正确的Review方法和目的。作者认为，大多数公司对代码Review的理解不足，导致Review流于形式且效果有限。甚至有些公司认为Review的目的仅是为了寻找Bug，这是不正确的。

作者强调代码Review的目的有三个主要方面：

• 保障代码的可阅读性：确保其他人能够理解代码逻辑，避免代码孤岛的产生。可阅读性涉及命名、代码格式、方法长度等多个维度。
• 检查设计原则：评估代码是否违反了如SOLID原则、迪米特原则等公认的设计原则，以保证代码逻辑清晰且易于修改。
• 评估单元测试：检验代码是否编写了合适的单元测试，并确保测试代码本身的可阅读性。
• 性能考量：从性能角度检查代码中是否存在影响性能的问题，并确保代码满足产品的性能要求。

对于如何进行有效的代码Review，作者提出以下建议：

• 避免通过长时间、大规模的集体Review，因为这样会导致精神压力大且效率低下。
• 建议每完成一小块代码后就进行Review，以Feature为单位，控制代码行数在100行左右。
• Review时应让编写人讲述问题和编写思路，然后逐步过代码，有问题即时讨论修改。
• 鼓励团队内交叉Review，避免仅由Leader单方面进行Review，以促进团队成员的相互学习和成长。
• 对于较大的团队，可以进行随机的事后Review，以促进技术交流和编程技巧的提高。

总结来说，作者认为正确的代码Review应关注代码的可读性、设计原则遵循、单元测试的有效性和性能要求，同时采取分块、交叉和随机事后Review的方式，以实现真正有效的代码质量提升。