如果你还不够优秀，那么请务必遵守规则

【1】

“小朱，你有权限吗？帮我评个代码吧。”

“我！？我没有啊~”

“哎，看来有权限的人都走了。我这个不合，会导致编译不过呀。”

“额……也是，毕竟是周五晚上嘛”

我笑着指指家雷哥，说道“要不你找家雷哥问问吧？”

没料想，一旁的家雷哥一听到我说他的名字，立马转头瞪着我，简直是要用眼神杀死我的节奏啊。

“我已经到额度了，不能再评啦”家雷哥不满地抗议到。

【2】

其实我心里清楚他在顾忌什么？

回退率，回退率，回退率。

哈哈，回退率是个啥？它本质上是个考核指标。

我们公司为了规范程序员编写的代码质量，引入了gerrit对每个人提交的代码进行评审。只有评审通过的代码才会被合入到仓库，否则，即使程序员写的代码再重要，也是无法被采用的。

都说林子大了，什么鸟都有。公司的程序员多了，要评审的代码多了，很多评审人员忙不过来，有时来不及细看代码，就匆匆点了通过。

因此，公司为了进一步提高代码质量，就提出了回退率这一指标，由我们配置管理人员开发脚本，每周对评审人员的代码退回情况进行统计。如果你连续几周回退率为零，那么对不起啦，你不仅会被通报批评，而且还要对你评的代码组织专家进行复评。

家雷哥是评审人员，他所说的到额度了其实就是他本周评审的数量已经快满20个了。如果这20个评审，他一个都没有退回，那根据回退率这个指标，他就会被问责啦。

【3】

“家雷你，要不你退回来，然后让杰哥再提一次呗”我在一旁建议道。

“不行，这样子杰哥又会有问题啦。”小伟哥旁观者，看的非常透彻。

他这一说，我也明白过来了。是的，回退率不仅考核评审人员，还要考核代码提交人员。如果一个程序员提交的代码，被退回率达到5%以上，那他的代码质量就会被质疑啦。

“所以说，我们公司这个规则的制定让人很头疼啊。”小伟哥感慨道。

“可是，我这代码就没有问题啊，干嘛要回退？”等着合代码的杰哥有些坐不住了。

对，大多数程序员的代码功能上都没有问题，有些可能就加一行代码，或者改个注释啥的，但，也要按照这套流程走。提交，评审，合入。

“规则是死的，人是活的，你可以给小伟开权限，让他帮我评嘛”杰哥催促道。

【4】

从我个人的角度来讲，我不只一次吐槽过这个退回率。因为它为我增加了很多工作量，包括写脚本、给他们查为啥没退回、统计要二次评审的chang-id等等。

但此时此刻，我却看到了事情的另一面。

指标的制定是为了考核员工的，那规则其实是用来约束底层员工的，尤其是初级开发人员。很多刚从学校毕业出来的程序员写代码时只看重功能的实现，不去管编码规范、性能影响等。如果这样的代码多了，会导致维护困难，整个系统的性能也会受影响。

增加评审的环节能够让有经验的专家审核一遍，大大减少劣质代码率。

而回退率这个指标，又能起到同时监督评审人员和程序员的作用。这样就不会出现，有些程序员急着上代码，催促评审人员赶快评审，或者评审人碍不过情面，放过那些不够优秀的代码。

【5】

“张sir，张sir应该有权限，你把他电话给我，我问他要账号”小伟哥看杰哥着急，也帮他想办法。

“算啦，你提上来，我帮你评吧，不过你得保证没有问题哦~”我无奈地说道。

是啊，如果给张sir打电话，他不还是看不到代码吗？既然都看不了代码，与其麻烦他，不如我动动手指头啦。

我没有告诉他们，因为我是管理员，我有所有仓库的评审权限。

因为，规则是用来约束底层员工的，我不能打破规则，否则，他们如何变得更优秀呢？

不过，小伟哥是工作10多年的资深程序员啦，有他为代码质量担保，我相信他。

因为，我说过了，规则只是用来约束底层员工的，当你已经足够优秀，那么你可以打破规则去做一些创新，只要你能解决问题。

毕竟，你的首要职责就是要想尽一切办法解决问题。这是职场之道啊！