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

wangbo1987 发表于 2023-03-17 18:33:49
有这样的大哥是你职场的幸运，好好跟大哥学，把现在的代码好好写，延误了发布有大哥兜着怕什么。然后跟大哥一起花时间把现有的屎山再重构一下，以后如果这个产品很有前途很重要，那么这个组前景会非常好的。

嘿嘿，延误了大哥兜着？那review的时候大哥能不能兜着？只发了一个PR能算2个，就因为大哥执着？

匿名用户 发表于 2023-03-17 15:03:38
等你后面遇到那种表面笑嘻嘻，反手直接approval，从来不认真review code的senior/staff的时候，就该想念这个老哥了

这样的人不好么？反正产品出问题了总是都要我背锅，也不是code quality能解决的。反手approve的起码让我赚个move fast的名声

还有很多比这些更严格的。。。。。。

如果他review comment 给的快 那还好 大方向定了 改 的快点就完了

如果 review comment 给的慢 或者大方向没确定 跟老板聊聊多加点时间 说明难处

这事得有个度，过分了就不是学东西了，那是block项目进度。感觉这种事一般只有老牌大厂里的senior闲的没事干就block别人，彰显自己的话语权。狗这样的人尤其多。如果周围就这么一个人，那就少找他review，绕不开拉个会跟他align清楚，大家都少点back and forth。如果周围很多这种人，而楼主又想move fast，尽早跳槽吧。

建议楼主跳槽去脸书。撸码撸到手抽筋

review有没有用还是蛮容易感觉出来的, 我之前的tech lead就会留很多comment, 有design方面的也有业务方面的, 当时改code改的很心累, 但是学到了很多东西, 他后来升manager就不怎么管code了, 但有什么涉及核心很负责的code我还是会让他帮忙review一下.
. 1point3acres现在我们组有个junior想升senior(我感觉的), 公司promote有个要求是能提供有效的code review, 于是他就开启了“找茬”之路. backend他经验不太多, 基本只能给组里一个更junior的junior comment, 给别人提不出来什么, 就挑前端UI的毛病. 那个前端就是个internal tool, 只有不到200人在用, 我们组主要是做backend的, 前端不是很熟练, 大家都是实现功能为主, 就他code review时总挑style的毛病, 什么没对齐(就一点点), 颜色太浅, layout不好看. style改起来真的很费时间, 但又不太好说因为水平不够改不了, 头疼.

以前组里也有这么一位，刚开始真的挺讨厌的，各种觉得迂腐，但是我嘴皮子还行，不至于说不过别人。但是后来慢慢习惯之后，有一天我给别人提意见的时候才发现自己已经慢慢变成了他

除了实现方式这点，外人没法评价优劣，其它楼主说得这些点，感觉楼主应该庆幸碰到这样的reviewer的，多弄几次以后对个人提高很大

这就是组员完美主义lol 没治