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

你不要想着去反驳对方，而且你把他惹毛了，你也不怕被列入pip名单？

shaw007 发表于 2023-03-18 01:30:17
请问我看很多人多次提到的lgtm和CR是什么意思，一直不懂这缩写的含义

Looks good to me, code review

我觉得这是很好的学习机会，尤其工作之后，其实很少有人愿意这样时间分析你的代码。

一个只会点approve点人是对你没有任何帮助的。

你不觉得是你自己代码写的太菜么？
你读屎山代码骂人的时候想过怎么来的吗？

知情人士表示 兄弟 你的心态要磨练下 这senior算对你认真负责了 他给你写那么多也是希望你好 他写那么多还费他自己时间 人senior确实很强 你要知道你上面的两个老板那是怎样的sb 你现在这心态怎么扛 口才是可以锻炼下 不过不是对senior 是对你上面的两个老板 fyi：https://www.1point3acres.com/bbs/thread-964110-1-1.html

补充内容 (2023-03-19 01:25 +8:00):
ps 人senior不管你的话 code review下手松的话  你可以看看你老板是个什么反应

补充内容 (2023-03-19 01:27 +8:00):
“最近要求尽快出东西” 你之后就会知道这不会是一时的要求 除非你上面两个老板被一亩三分地喷完转性了

fxrcode 发表于 2023-3-17 12:52
是的 很烦 换组吧

楼主是可以考虑换组 不过不应该是因为senior 估计楼主还没来得及见识他的老板跟skip的真面目 https://www.1point3acres.com/bbs/thread-964110-3-1.html

匿名用户 发表于 2023-3-17 19:59
这事得有个度，过分了就不是学东西了，那是block项目进度。感觉这种事一般只有老牌大厂里的senior闲的没事 ...

知情人士表示 这个组干活负责输出的senior们才不会闲的没事干

看这个帖子下，一堆说reviewer大题小作的我才是一脸懵逼，也难怪现在公司里各种代码都是屎山。

比如说变量名nitpicking： 当然有意义，有意义的变量名让别人直接看变量名就知道对应变量的意义。最简单的一个例子，假如一个变量叫latency表示某个通信的时延，当一个显然的问题是这个latency到底是什么为单位，是秒还是毫秒，latency_ms 就显然更self explanatory，不用以后写代码的人去看变量在哪里用了来猜测unit。

这还是最小的方面，还有其他各种地方，数都数不尽，真是看了这么多赞同才理解为啥代码里哪里都是屎山😂

希望楼主珍惜这样的好mentor。他虽然比较较真，但都是合情合理的。像他这样的工程师才是大家该学习的。相信楼主有朝一日也会当上mentor/manager，眼光要看的远

以前也遇到过一个非常年纪大的组员，对代码整洁度要求非常高。当时五六次review给我整崩溃了差点就要砸键盘不干了，但是还是咬咬牙忍下来了按他说的改了。我有点儿庆幸他这种人的存在使得当时那个组的代码是合理且看得懂的，并且非常clean。在职业初期遇到这种不算是坏事。我后来去了两个组代码是真的一个比一个shit，完全不按章法就是乱来… test case也没有 注释没有，取名乱取好几个差不多的，前人新开一个坑就开一个新的folder比如这里明明有metrics怎么又来一个也不知道跟另一个有啥区别… 只能看出client用的不一样但是没说哪一个是被deprecated的。这种时候不问老人根本看不懂… 你换组遇到这种组你就懂了，你现在的苦就是在造福后人。
当然经过魔鬼训练还有一个好处，那就是换公司换组交的第一份pr总是有大佬坐镇review的他们一下就看出来你写的水平了会夸奖你，然后之后你的pr他们就放心让别人review了。
说回你现在… 他要是review的快你改的快其实还行没block几天，你在接活的时候就懂把自己的这部分时间给算进去。以及别跟他吵了，老老实实快速按他说的做然后你就会收获一个大佬给你的棒棒的review，对你perf也没坏处。忍一时风平浪静，退一步海阔天空，兄dei～

补充内容 (2023-03-19 02:07 +08:00):
另外我补充一点啊在我现在的公司，给别人review如果都说nit这种是根本不能算leadership的。老板跟你写promo doc的时候特别强调了这种link就不要放进来了。