遇上一个对Code Quality 超级执着的组员，快被逼疯了，经验

匿名用户 发表于 2023-03-19 10:38:07
你说的很有道理，基本完全同意。我们事实上也大致是这么做的。

第一点具体化kpi我们有做，product excellence是我们今年的okr之一。

这是什么神仙mentor…啊不，同事…

从读这个帖子开始我都一直默认是senior在带新人，现在才回过味来，这只是组员间互相帮衬吗？甚至楼主的资历比这个所谓的senior还老？

匿名用户 发表于 2023-3-19 19:06
楼主至少认可人大哥的建议是正确的，以前遇到一新来的傻13天天堆屎，还觉得自己写得很好，别人的意见听不进 ...

被喷的senior按照年纪跟工作年限的话算小哥吧 论年纪跟工作年限的话 楼主可能得算“大哥”

人在江湖 身不由己说的就是这个吧

其实我觉得还好，算是好事吧，只能说互相理解一下吧

匿名用户 发表于 2023-03-17 17:15:23
看回复里有人说跟这样的人能学到很多东西的, 我只能说还有点学生气. 怕就怕你这还在学的酣畅淋漓呢, 老板哪天把你放 PIP 了, 因为在外人看来你就是 deliver 慢, PR问题多, 能力不行.

你说得对，我现在喜提PIP了…
老板搞我的理由就是PR问题太多，Deliver 太忙，我应该早换组，不该纠缠

可能单纯觉得你的coding太菜了，会污染codebase，多卡卡你直到自己满意为止。。。代码其实跑起来没问题都可以，什么维护性，readability都是扯淡，你的项目，公司都可能比你的代码先垮掉

匿名用户 发表于 2023-11-25 03:33
你说得对，我现在喜提PIP了…
老板搞我的理由就是PR问题太多，Deliver 太忙，我应该早换组，不该纠缠

pat pat

junior们千万注意了，不管面试的时候你怎么说，工作固然可以是一个用来给你学习提高的场所，但更是一个必须出活的场所。如果你发现你的delivery被各种有的没的东西受影响了，一定要学会怼回去，捍卫自己的delivery。你的代码写得再clean，最后还是会屎的，这是熵增定律，更何况项目的寿命可能还不如你在公司的tenure。

所以千万不要因为Senior在这种细枝末节的地方对你要求高而感到庆幸，无论对方是否出于好心。永远记住，凡是挡我deliver者，杀无赦。

最怕 fast delivery + clean code。只有后者真的好说，是学习机会。

只能换组，这类组员害人不浅

PR review来回太多在🐶家不是好事，reply count和clstats都是可以query的metric，现在都会看的。如果cl发出去没什么comment马上被lgtm是一个好的信号，反之来回好多次是一个不好的信号，manager们都会互相比较这个。

而且我觉得有些人确实是在以code quality为理由block别人。review好的人解释问题解释的清楚，link和code example直接给，总之都是actionable的comment。你知道怎么改或者也可以给出合适的理由push back，双方都是可以平等交流的。没有道理用代码质量为理由让别人交不了代码，就是reviewer没有把unblock别人当作priority，留一大堆不actionable的comment在别人的cl下面讨论代码质量，也不想着怎么用简洁的话解释清楚问题，故意留下一堆comment展现ta的influence。

也不要pua自己说什么提升自我了，提升自我悄悄搞看看别人写的好的代码就好。想改legacy代码和其他愿意搞的人一起搞一个hotlist跟老板提议花时间弄tech debt。但是我觉得有些reviewer并不是这个目的，不然ta也许可以成长为好的reviewer。