作为程序员，在我们的日常 Coding 中，或多或少会在代码中隐留一些 Bug，但这些 Bug 看似无伤大雅，那是否还有代码审查的必要性？
作者 | Sophie Alpert
译者 | 梁蕊
责编 | 屠敏
出品 | CSDN（ID：CSDNNews）
以下为译文：
最近一个朋友问我，为什么做代码审查很有价值。大多数硅谷的科技公司都会对每一个变更进行代码审查，至少有两组人盯着它看。在我之前的一项工作中，我们做了一段时间的代码审查（很少），然后一个来自 Google 的新员工加入了我们，鼓励我们评审所有的代码——我们做到了。这是一个伟大的决定。
如果你正确的进行代码审查，它不应让你感到繁琐。你和你的审查者不是对手，你们在一起合作开发出最好的软件。（重要的是不要把反馈当成是在针对你——即使你的代码需要修改，但这并不意味着你有问题。得到反馈是很正常的，这能帮助你成长！）
有些公司制定了复杂的规则，规定对于每一段代码需要多少人审查，对于谁负责每一段代码有严格的“所有者”。我从来没有觉得这样做是必要的。我更喜欢一个更加简单的系统，其中唯一的规则是每段代码必须由一个人检查。在实践中，您仍然会向负责维护您所更改的特定代码的人发送审查，但是最好不要有硬性要求。
下面是我认为代码审查之所以有价值的最重要的原因。有很多东西！
代码本身。代码审查中最明显的价值通常体现在“捕捉 Bug”中。或者，如果你看的再深入一点，找出作者不知道的最佳实践或潜规则的缺漏，在这些情况下，审查者可以通过响应实际的具体代码来帮助改进代码。
宏观层面的知识共享。当你审查别人的代码时，你会倾向于学习新的技术，这可以让你在日后受益——反之亦然，如果有人在审查你的代码时提出了更好的方法。如果你可以学以致用，你将成长为一名工程师。
微观层面的知识共享。或者，通过增加熟悉给定代码段的人员数量来减少“巴士系数”。
方向共享。与此相关的是，代码审查迫使你与团队成员交流你正在做的事情，这有助于确保你不会在几天或几周内走向错误的方向，因为这会给他们一个机会进行回推。
交流练习。无论是在团队内部还是团队之外，清晰的沟通都是在工作中取得成功的最重要的技能之一。代码审查为你提供了一个清晰地练习写作的机会，包括在描述更改的目的时以及在对更改进行反馈时。运气好的话，当下次你需要写一些“真正重要”的东西时，你会准备的更充分。
历史记录。根据我的经验，当人们知道有人在阅读他们的信息时，他们倾向于写更好的提交信息。这在以后回顾旧的更改时通常很有用。
讨论的问题。当你试图就要进行的更改达成一致意见时，有时候很难用言语描述并在细节上达成一致，比如说，一个特定的算法。通过一段代码进行交流可以更精确，因为代码往往是明确的。
团队凝聚力。定期进行代码审查时，工作感觉更像是一个团队一起工作，而不是每个人在各自的领域中工作。
阅读练习。练习阅读别人的代码可以帮助你记住如何让自己的代码可读性更强（从而更易于维护）。这会带来更好的代码！
如果要我选的话，理由 2、5 和 6 对我来说可能是最有价值的。那你怎么看呢？
原文：https://sophiebits.com/2018/12/25/why-review-code.html
本文为 CSDN 翻译，如需转载，请注明来源出处。
热 文 推 荐
☞Java JDK 收费，Android 也坐不住了，程序员们该咋办？
☞华为 2018 手机销量破 2 亿台；全国首个 5G 地铁站开通；iPhone 7/8 下架 | 极客头条
print_r('点个好看吧！');
var_dump('点个好看吧！');
NSLog(@"点个好看吧！");
System.out.println("点个好看吧！");
console.log("点个好看吧！");
print("点个好看吧！");
printf("点个好看吧！\n");
cout << "点个好看吧！" << endl;
Console.WriteLine("点个好看吧！");
fmt.Println("点个好看吧！");
Response.Write("点个好看吧！");
alert("点个好看吧！")
echo "点个好看吧！"
点击“阅读原文”，打开 CSDN App 阅读更贴心！