很多中小型公司,因为没有代码审查,所以很容易呈现“不可代替”的大牛程序员。
很多所谓的大牛,能解决问题,然而很少有人看得懂他的代码。如果试图剖析过这些代码,会发现各种得心应手,比方:全局变量一大推、函数过于随便、代码逻辑凌乱、函数嵌套和递归到处可见、链式调用十分多、没有任何正文等等。
可因为代码无效但难以了解,所以就很容易被捧为“大牛”,加上多年的工作教训,能疾速剖析并解决问题,各种救场,很快就成为了团队最能解决问题的人。正应了一句:不论黑猫白猫,能抓耗子就是好猫。可这样的“好猫”对团队的长期倒退并非坏事,抱怨声会越来越重,但其下属也只能是既爱又恨。
对这样的状况,也只能缓缓的解决,次要能够从如下几方面动手:
治理层面
让其他人摊派大牛的工作,同时为了防止“撂挑子”,还要适当的给大牛处分,例如加薪、安顿新的业务、晋升职位等等。同时把控好新接手人的代码习惯和品质。
技术层面
引入代码扫描工具,集中代码治理,应用工具对代码的编写标准、潜在破绽进行全面的查看。
协同层面
引入代码评审机制。通过多人的代码互审,来防止某个人的代码成为我的项目中的黑盒。
技术层面和协同层面说起来很简略,但如果不借助工具,基本上就变成一句空话,没法执行,而且如果工具应用麻烦,也会对执行效率造成很大的影响。
上面咱们以 Gitee 企业版为例,介绍如何通过工具查看代码标准,进行代码审核。
代码扫描
在须要进行代码扫描的企业仓库中抉择「Gitee Scan」
新建一次代码扫描,抉择要扫描的分支、门路以及语言(可多选)
扫描实现后点击「查看」即可进入扫描报告
扫描报告曾经将问题精准定位至代码行
代码审核
在企业仓库的 Pull Request 中,能够清晰的看到一次 PR 波及的文件和改变
管理员能够对本次提交的代码进行评论
代码审查通过后,即可对这次提交进行合并的操作
代码审查是一件须要长期严格贯彻的研发管理制度,所有人都会拥戴这个制度,但如果在执行层面过于繁琐,会让这个制度沦为形式主义。工具只是辅助,制度才是外围,只有充分利用工具带来的便利性,防止制度成为程序员日常工作的累赘,能力真正的晋升团队的代码品质,从而保障公司业务的稳固倒退。
我是 Gitee 的李家宁,如果你感觉这篇文章对你有价值,欢送扫描二维码增加我的企业微信一起探讨。如果你对 Gitee 如何助力代码审查感兴趣,也能够分割我索取优惠码。