很多中小型公司，因为没有代码审查，所以很容易出现“不可替代”的大牛程序员。

很多所谓的大牛，能解决问题，但是很少有人看得懂他的代码。如果试图分析过这些代码，会发现各种随心所欲，比如：全局变量一大推、函数过于随意、代码逻辑混乱、函数嵌套和递归到处可见、链式调用非常多、没有任何注释等等。

可由于代码有效但难以理解，所以就很容易被捧为“大牛”，加上多年的工作经验，能快速分析并解决问题，各种救场，很快就成为了团队最能解决问题的人。正应了一句：不管黑猫白猫，能抓耗子就是好猫。可这样的“好猫”对团队的长期发展并非好事，抱怨声会越来越重，但其上司也只能是既爱又恨。

对这样的情况，也只能慢慢的解决，主要可以从如下几方面入手：

管理层面

让其他人分摊大牛的工作，同时为了避免“撂挑子”，还要适当的给大牛奖励，例如加薪、安排新的业务、提升职位等等。同时把控好新接手人的代码习惯和质量。

技术层面

引入代码扫描工具，集中代码管理，使用工具对代码的编写规范、潜在漏洞进行全面的检查。

协同层面

引入代码评审机制。通过多人的代码互审，来避免某个人的代码成为项目中的黑盒。

技术层面和协同层面说起来很简单，但如果不借助工具，基本上就变成一句空话，没法执行，而且如果工具使用麻烦，也会对执行效率造成很大的影响。

下面我们以 Gitee 企业版为例，介绍如何通过工具检查代码规范，进行代码审核。

代码扫描

在需要进行代码扫描的企业仓库中选择「Gitee Scan」

新建一次代码扫描，选择要扫描的分支、路径以及语言（可多选）

扫描完成后点击「查看」即可进入扫描报告

扫描报告已经将问题精准定位至代码行

代码审核

在企业仓库的 Pull Request 中，可以清晰的看到一次PR涉及的文件和改动

管理员可以对本次提交的代码进行评论

代码审查通过后，即可对这次提交进行合并的操作

代码审查是一件需要长期严格贯彻的研发管理制度，所有人都会拥护这个制度，但如果在执行层面过于繁琐，会让这个制度沦为形式主义。工具只是辅助，制度才是核心，只有充分利用工具带来的便利性，避免制度成为程序员日常工作的负担，才能真正的提升团队的代码质量，从而保障公司业务的稳定发展。

我是 Gitee 的李家宁，如果你觉得这篇文章对你有价值，欢迎扫描二维码添加我的企业微信一起探讨。如果你对 Gitee 如何助力代码审查感兴趣，也可以联系我索取优惠码。