原文： why review code
作者：Siphie Alpert
一个朋友最近问我为什么做代码审查是有价值的。 至少，大多数硅谷科技公司会对程序每一个变化进行审查，每个变化至少两双眼睛的关注。 我们曾做了一段时间的代码审查（但是很少） ，然后一个来自谷歌的新员工加入了之后，鼓励我们审查我们所有的代码——我们确实这样做了。现在看来这是一个伟大的决定。
如果你使用正确的审查代码的方式，那就不会觉得工作量很大。 你和你的审查者并不是对立的； 你们一起工作，一起产出最好的产品。 (重要的是，不要把反馈私下处理——即使你的代码需要更改，因为这并不意味着你有问题。 得到反馈是很正常的，这样才能帮助你成长)
一些公司对于每一段代码需要多少人来审查有非常复杂的规则，每一段代码都有严格的“负责人”。 我倒觉得并没有这个必要的； 我更喜欢一个简单的机制，唯一的规则是每一段代码都必须至少由一个人来审查。 而在实际操作上，你仍然会将反馈发送给维护你审查的这段代码的负责人，但是我认为不做硬性要求也很好。
我为什么认为代码审查是有巨大价值的？在这里罗列一些主要理由：
-
代码本身。代码评审最明显的价值通常被定义为“发现bug”。 如果你能看得更远，还能发现被开发者忽略的最佳实践，或者潜在规则的案例。在这些案例中，评审者可以向开发者反馈，以提高实际的具体代码水平。
-
宏观层面的知识分享。 当你检查别人的代码时，你会倾向于学习新的技术，这些技术以后会对你有益，反之亦然，会有人在检查你的代码时提出一个更好的方法。如果你能把你学到的东西用在以后的工作中，你就能成长为一名优秀的工程师。
-
Micro-level knowledge sharing. Or, mitigating the “bus factor” by increasing the number of people who are familiar with any given piece of code.（这句并不是很理解）
- 技术方向分享。 代码审查迫使你与团队成员交流你正在做的事情，这能确保你不会持续几天或几周的时间走在错误的方向上，他们会马上给你反馈。
-
沟通练习。 无论是在团队内部还是团队之外，清晰无误的沟通能力，是在工作中取得成功的最重要的技能之一。 代码审查让你有机会练习表达能力，一是在描述变更的目的时候，还有在向队友提供反馈时。 运气好的话，下次你需要写一些“真正重要”的东西时候，你会准备得更充分。
-
历史记录。 根据我的经验，当人们知道有人在读信息时，他们往往会写得更好。 这在以后回顾过去的变化时非常有用。
- 利于讨论。 当你们试图作出一些程序上的改变，想要达成一致意见，有时候很难口头描述某个特定算法的细节。 那么通过一段代码进行信息的传递就可以更加精确，因为代码通常是毫不含糊的。
8)团队凝聚力。 当你进行常规的代码审查时，就会感觉是一个团队在一起工作，而不是每个人都在"各司其职"。
9)阅读练习。 练习阅读他人的代码可以提高你的认知：如何使你自己的代码更具可读性（从而更易于维护）。 这会对你的代码质量产生长远的积极影响。
如果让我选的话，2、5、6可能是最有价值的。
网友评论