遇上一个对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一下.
现在我们组有个junior想升senior(我感觉的), 公司promote有个要求是能提供有效的code review, 于是他就开启了“找茬”之路. backend他经验不太多, 基本只能给组里一个更junior的junior comment, 给别人提不出来什么, 就挑前端UI的毛病. 那个前端就是个internal tool, 只有不到200人在用, 我们组主要是做backend的, 前端不是很熟练, 大家都是实现功能为主, 就他code review时总挑style的毛病, 什么没对齐(就一点点), 颜色太浅, layout不好看. style改起来真的很费时间, 但又不太好说因为水平不够改不了, 头疼.

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

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

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