求问如何对待吹毛求疵的code reviewer

+1 我也遇到过这样的reviewer。
而且楼上提到的统一的standard根本不存在好吗，就是他在review的时候，想到啥就给你comment上了，完全没有什么统一的style。
其实我感觉做code review还是蛮难的，他能提出一些建议至少说明认真读过代码，并且思考过。
有时候我做review的时候，感觉给出实质上的建议还是比较困难，除非是明显的逻辑错误。
我的解决办法，就是虚心接受，并且努力提升代码质量吧，毕竟也是个学习的机会。
至于影响工作积极性，确实会有，特别是看到那么多comment，非常烦躁，但还是要克服呀。

推荐一个方法，如果你真的有心气。
说白了就是他觉得自己code写的好，别人的都必须按照他的标准来嘛。
他如果在小问题上提意见，要求他写标准化doc，没有提前定好的标准都不算标准，倒也不是不按照他说的来，这种情况他一般都有无数理由说他的注意好。不要跟他争论这个问题。
就要求他也干活，比如给大家写标准化的doc。而且要语气极为nice甚至崇拜的的要求他，一边捧他认真负责，写code标准高，对大家都好，但是其实是要捧他去干这个coding标准化的活，这种活绝对出力不讨好。把他架起来。看他怎么下台。
如果他提的东西真的不错，也愿意标准化，那就照着来，也不是坏事，也算是学到东西了。

以前组里也有这种人（google来的。。。狗家确实要求高），无非就是喝喝咖啡安心聊聊，互相让一步。最后都是不错的朋友

我新人刚入职几个月，我的mentor review 我的PR就很严格，特别是文档性的东西，有时十几个回合。我最初也有点懊恼，到后来我想通了。我想我写的那些烂东西我自己都看不下去，他还看那么仔细，还花了那么多时间精力，我感动啊。后来我轻易不敢犯低级的错误了

wjx1111 发表于 2022-05-28 20:49:35
推荐一个方法，如果你真的有心气。
说白了就是他觉得自己code写的好，别人的都必须按照他的标准来嘛。
他如果在小问题上提意见，要求他写标准化doc，没有提前定好的标准都不算标准，倒也不是不按照他说

我觉得这是楼主的机会，楼主应该drive标准化这个东西，弄个组里的coding style best practice，约组里一起review，达成一致之后就按照统一标准来，以后这种琐事会越来越少，楼主还能扩大impact。

匿名者 发表于 2022-5-28 20:34
review 会block吗， 还是得找一个宽的组队stamp， 但是这种review 还是有建设性的， 尽量学习一下。
如果 ...

这个才是有建设性的回复

感觉这是一个很好的behavior 例子i

+1 我也遇到过这样的TL，经常对coding style吹毛求疵，必须按照他的来。开始觉得没什么人家要求严格正好是自己学习的机会。后来他block了我的一个大的diff，我就商量说这个要求比较着急，而且code repo里面他的style和我的style都有（实际上我写的style占大多数所以我才这么写的），能不能这次我们就这么merge了。结果他非说我没看到repo里面有我写的style，必须按照他说的来。后来和lz的感觉一样，非常影响工作积极性，都不想push代码了。而且感觉已经得罪了他，也只能跑路解决问题。

楼上都这么心胸大度吗？。。

我也有个同事这么吹毛求疵，每次code review评论一堆有的没的。。

因为我写前端，有一次PR里写的react，有些dom上检查Length的逻辑其实必须写成xx.length > 0, 因为如果直接写xx.length，就会直接render一个0，这种他也会说应该改成xx.length, 因为早知道这人喜欢挑刺儿，然后我就改了，然后直接截图说这样不行 ┑(￣Д ￣)┍
还有一次我把业务逻辑写在reducer里，也不知道这个哪里触碰到他了，直接pr上给我说这些得写在action creators里，我直接贴上redux官方文档, 上面很直白地说 We recommend putting as much logic as possible into reducers. 然后虚心请教，is this a team preference to put business logic into action creators? 然后cc一些别的teammates

这种东西吧，常在河边走，哪有不湿鞋的，对付这种装逼犯抓住他几个致命错误怼他一两次就老实了┑(￣Д ￣)┍。另外还要尽量建立自己的merge盟友，平时尽量找盟友。。。

遇到了一模一样的情况，吹毛求疵，也对对方说能不能写一个doc统一一下标准，结果对方以move fast为理由不肯写。这种事情，你一旦反抗了，就是把对方得罪了。我那个大哥一堆堆nits，还不一次给你写完，你改好了又来一次。建议，不要跟大哥扯那么多，你直接就照着他说的改，反抗是没有用的（因为他level比你高，告manager他会说是帮助你learn，他怎么说都有理）……有些人就是喜欢这样弄new hire，真的是帮助还是吹毛求疵其实大家心里都有数。还有就是不要增加太多情绪，告诉自己这就是一份工作，他要改就改吧