代码审查的主要考虑因素是什么？

Question

代码审查如今非常流行。

如果我编写了有效的代码并且有测试来证明它，为什么我需要额外的“竖起大拇指”呢？
是否只是为了检查我没有犯任何我编写的测试未涵盖的错误？

Answer 1

代码审查的一个目的是确保代码正常工作并且测试反映了这一点。

然而，这样做的好处可能有限，因为许多组织将其视为满足组织或监管要求的“竖起大拇指”练习。

我发现一个好的拉取请求/代码审查系统背后的真正价值是多方面的，远远超出了“代码是否正常工作？”。

可以通过多种方式编写代码来实现相同的目的。
良好的代码审查将根据许多因素检查代码，而不仅仅是“它是否运行”。
这些因素可能包括以下一些：

• 有测试吗？
• 测试质量高吗？
• 是否考虑了安全问题？
• 是否很好地涵盖了负面测试用例？
• 是否处理了错误输入数据的异常？
• 代码是否与任何依赖项隔离？
• 以后代码容易改吗？
• 代码是否足够高效？
• 代码是否易于阅读 - 即使是新手？
• 代码是否干燥，避免复制和粘贴模式？
• 代码是否遵循一般或现有的组织设计模式？
• 是否使用全英文来避免未来对首字母缩写词和符号的认知负担？

在更高的层次上，代码审查还具有以下目的：

• 对团队的其他成员进行有关更改的教育
• 鼓励不同的观点和方法
• 避免与代码所有权相关的“忍者”/“英雄”总线问题
• 向非团队成员（例如 qa 和其他团队）提供可见性