关于代码评审的事

我从来没有这么不理性过。


0.起因

早在之前,团队leader就说过要进行代码评审的事情,项目忙着一直拖到了现在。我也认为这是好事情,从毕业到现在,每次Review都让我受益匪浅。

只是没想过这次会让我感觉不一样。

我们代码评审是两个模块的兄弟共同评审,一条代码一条代码地过。评完一个模块基本花掉了我们一个下午的时间,期间主要是由负责模块的老司机对工作资历较浅的同事的代码进行评审。

能感觉出被评审的体验不太好受。而自从有代码评审这件事,大家不断在强调代码规范的时候,我也一直在注意自己的逻辑的编写清晰、注意要求的MVP架构、花较多的时间重构复杂的代码。评审前,findbugs检查没什么错误,我甚至有一些跃跃欲试想和大家分享。

所以当晚上团队leader一句一句问我为什么这样写的时候,我一下子就炸了

当自己费尽心思去重构代码,分化逻辑,然后被人指指点点的感觉。只是感觉劳动成果并没有得到尊重。我看见之前模块被评审的同事在玩着手机,一言不发或不关心着。

于是会感觉所有的事情也就那样,所有的悲喜,都会这样。沉寂那么久的团队,平淡那么久,大家好像都会希望爆发出一点新奇的东西。 那就如你们所愿,我听见自己说。


1.获取别人认同是很难的事情

当然…我不会去怼自己的上级,那是傻子做的事情。但我还是固执地解释着自己的逻辑,会反驳,然后说很多地方你们并没有说服我。按正常的自己应该会认为这时候听着就好,多说无益,但心里就会有一团气在。

这段时间,老大离职后,leader的管理风格更倾向于面面俱到,又期望每个人可以在各个方面主动做到最好。批评多于认可,教育大于引导,我有些不适应。以前有一直帮老大做着项目跟踪的汇报PPT和会议纪要,而这些任务现在沿留了下来,每周会占据我一天多的时间,没有人认可过这些事情的价值。我为什么要去做它呢?这种想法一直在脑海里。

所以我知道这并不是代码评审的事情,而是团队信任和氛围的事情。

我承认我心态失衡了,获取别人认同是很难的事情,尤其是想要说服对方的时候。


2.对他人抱有最大的善意

“不要再去做与代码无关又得不到承认的事”,我不知道怎么调整这种自私的想法。在第二天其他人剩余代码的评审时候,我也一言不发,我不想在这种条件下去讨论代码的事情。

其实我很好奇其他公司是怎么做代码评审的,我刷着知乎,看见有条消息把我逗乐了。是这样的:
在这里插入图片描述
知乎-代码评审

哈哈哈,平心而论,就代码评审的事情而言,带给我的好处是大于不愉快的。有我不服气的地方,比如:

//我不认为这个有什么错误
Message msg = Message.obtain();
//但leader挑出来希望我改成这个
Message msg = mHandler.obtainMessage();

也有我确实写的不对的地方:

//我把Handler的常量存储在了一个接口里,这是一个不好的实践
interface myInterface{
	public static int WHAT_DO_SOMETHING = 0X001;
	....
}

只是我想,获取别人认同是很难很难的事情,但对于每一个人而言,都有自己要实现的价值。

每个Review至少给出一条正面的评价(取自知乎)

3.资料分享

Android编码规范手册,提取码ebjd

发布了12 篇原创文章 · 获赞 36 · 访问量 4769

猜你喜欢

转载自blog.csdn.net/weixin_42229694/article/details/102892568