|
|
|
很多中小型公司，因为没有代码审查，所以很容易出现“不可替代”的大牛程序员。
很多所谓的大牛，能解决问题，但是很少有人看得懂他的代码。如果试图分析过这些代码，会发现各种随心所欲，比如：全局变量一大推、函数过于随意、代码逻辑混乱、函数嵌套和递归到处可见、链式调用非常多、没有任何注释等等。
可由于代码有效但难以理解，所以就很容易被捧为“大牛”，加上多年的工作经验，能快速分析并解决问题，各种救场，很快就成为了团队最能解决问题的人。正应了一句：不管黑猫白猫，能抓耗子就是好猫。可这样的“好猫”对团队的长期发展并非好事，抱怨声会越来越重，但其上司也只能是既爱又恨。
让其他人分摊大牛的工作，同时为了避免“撂挑子”，还要适当地给大牛奖励，例如加薪、安排新的业务、提升职位等等。同时把控好新接手人的代码习惯和质量。
引入代码扫描工具，集中代码管理，使用工具对代码的编写规范、潜在漏洞进行全面的检查。
引入代码评审机制。通过多人的代码互审，来避免某个人的代码成为项目中的黑盒。
技术层面和协同层面说起来很简单，但如果不借助工具，基本上就变成一句空话，没法执行，而且如果工具使用麻烦，也会对执行效率造成很大的影响。
下面我们以 Gitee 企业版为例，介绍如何通过工具检查代码规范，进行代码审核。
新建一次代码扫描，选择要扫描的分支、路径以及语言（可多选）
代码审核
在企业仓库的 Pull Request 中，可以清晰地看到一次PR涉及的文件和改动
管理员可以对本次提交的代码进行评论
代码审查通过后，即可对这次提交进行合并的操作
代码审查是一件需要长期严格贯彻的研发管理制度，所有人都会拥护这个制度，但如果在执行层面过于繁琐，会让这个制度沦为形式主义。工具只是辅助，制度才是核心，只有充分利用工具带来的便利性，避免制度成为程序员日常工作的负担，才能真正的提升团队的代码质量，从而保障公司业务的稳定发展。
我是 Gitee 的李家宁，如果你觉得这篇文章对你有价值，-nina)一起探讨（请备注详细信息）。如果你对 Gitee 如何助力代码审查感兴趣，也可以联系我索取优惠码。 |
|
发表于 2021-5-21 20:00:17
