最近，笔者在Twitter上看到这样一句话：
可悲的是，对于很多学生、自由职业者以及机构来说，代码审查似乎相当陌生。
很明显，代码审查的重要性并不为每个人所熟知。你可以说我很天真，但是笔者确实认为所有的IT公司都离不开该过程。显然实际并非如此，真是让我大吃一惊。
在本文中，笔者想给出关于代码审查的想法，以及为什么我认为这是代码迁移过程中非常重要的组成部分，怎样进行审查等。如果你目前不进行代码审查，或者想要做得更好，希望本文能有助于你！
什么是代码审查？
我们生活在维基百科的时代，所以开始之前，先引用一下其中关于代码审查的定义：
代码审查是计算机源代码的系统性检验（有时被称为同行评审）。其目的在于找到开发初期所忽略的错误，从而提高软件的整体质量。审查的形式多种多样，如结对编程，非正式走查，正式检查等。
顾名思义，代码审查就是审查一些代码，以确保其能够正常工作，并尽可能改善其性能。
代码审查的方法
正如维基百科中的定义，代码审查有多种方法。然而，目前太多的代码都存在于GitHub上，代码审查也就经常伴随着所谓的“pull request”出现。
Pull request是一个请求，使用分布式版本控制系统（Git、SVN、Mercurial等）对代码库作出修改。它通过“牵引”原代码、写入更改，然后提交请求以便将更改合并。
得益于GitHub友好的用户界面，这个过程变得非常简单高效，GitHub也概括了大部分Git知识需求。
为什么代码审查非常重要
那么，既然我们可以不经过任何审查与监督，直接进行代码迁移，为什么代码审查还这么重要呢？毕竟，我们都能胜任该工作。
从理论上说是这样。但在实践中，有很多原因可以表明代码审查的重要性。让我们来看看其中的几个。
降低风险
这可能是最重要的原因。有专人复核我们的工作并不是无关痛痒的，这能降低被忽视的错误所带来的风险。毕竟即使再好的开发人员也有可能一时失察。
并且，确保没有忘记任何事情总是有必要的。举例来说，前端开发中经常会忽略适当的键盘导航，屏幕阅读器的可用性，适应国际化的灵活性，以及友好的非JavaScript行为等问题，在这里仅列出这四项。
显著提高代码质量
清楚点说，这不是单纯的代码标准和代码检查（至少不全是），而是使代码更高效。
在一个团队里，每个人都有自己的背景和特长，而团队始终需要进步。因此总有人可能提出更聪明的解决方案，更合适的设计模式，或者能降低复杂性或提高性能的方法。
使每个人都得到提高
通过合作，每个人都可以相互学习并取得进步。提交代码者很有可能从该工作中得到反馈，并意识到可能存在的问题和需要改进的部分；而审查者也可以通过阅读他人代码学到新的东西，并找出适用于他们自己的工作方案。
有助于熟悉项目
当一个团队在做一个项目时，想要每个开发人员致力于应用的每个部分，这是极不可能的。有时候，会出现这种情况：在某一段时间，一个开发人员正为项目的大部分模块辛苦地工作，而另一个人则完全在做别的东西。
因此，代码审查有助于人们了解其他人所写，但以后可能会需要自己来维护的那部分代码。它促进了代码库知识在团队中的传播，也有可能加快未来的发展。
怎样适当地进行代码审查
再次强调，有固定的代码审查过程非常有用，非常重要。不管用什么方法，每个团队创造的代码都应该进行代码审查。
话虽这么说，但进行有意义的代码审查并不像看上去那么简单明了。不过，别担心，即使做得不好也不会有什么坏处，就是浪费点时间。
最近，我的团队回顾了之前进行的代码审查。当我们意识到12个开发人员中，只有3个在做代码审查时，我们就明白有些地方出了问题。
为了改变这种状况，我们的一位 Scrum 专家组织了一次回顾分析，以确定还可能改进的空间，以及我们将怎样改变。
提前规划
代码审查做得不够，为了自圆其说，最常用的借口就是，它需要时间——其他人不能或不愿意在这上面花费时间。
我必须说，笔者并不太理解这种说法，因为我的观点是：如果一个同事直接来找我，让我帮他的忙，我就不会说“我没有时间，也不感兴趣”。反而，我会抽空来帮忙，可能不是现在，是一个小时之后——但是显然，我会花时间帮助他们。为什么呢？ 因为：
