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

那你就跟他不断强调和productivity的tradeoff，反正他就拿高标准来怼你你也没法说他是错的。

然后等下次他 pr 的时候你就也来要求要求他呗，有时候代码好坏不是绝对的总有利弊，你就盯着弊端讲，他说自己这样写好 maintain 那你就看是否不 readable 或者 performance 有一点影响，这东西也没法量化你就只管展开说弊端吧。

tentenymy 发表于 2023-3-17 12:18
lz站在对方的角度想想，他跟你耗同样时间，去搞一个不是自己的项目，还不得不看新人的code，他不是更痛苦吗 ...

这种人就是吃饱了没事做。。拿着鸡毛当令箭

我觉得这是好事啊，尤其是对于新人来说，组里有这么个人能有效避免屎山代码

我们组的code分成两类，一类是核心的，如果弄不好就会把整个产品弄崩的。另一种是边角的，就是不管谁搞砸了只会影响ta自己的那一小块项目。

不到两年前以前我们组来了两个ng. 开始时两个人我都给分配第二类工作，但review都严格要求。结果一个认真的改一个不认真。慢慢的我就把重要的工作交给那个认真的。他的代码也越写越好。另一个人因为工作不重要我也懒得再严格要求了。把他推给了一个l4带.

认真的一年刚出头就帮他升职了。另一个这么久仍然只能打杂。ta说想做重要项目升值，我直接了当地说你要先build trust. 心里想想升职我估计还得再两年吧。如果年景好我会建议他换组。

补充内容 (2023-03-18 09:40 +08:00):
认真的是印度人，不认真的是中国人。我也很想帮中国人但使不上劲啊。

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

哎我都是急事直接找人review，不急的话，让我改我就改，慢慢来呗。只要对方也是为了产品好，我觉得没啥。怕就怕那种根本就不是为了产品，就是要刷脸的那种同事，在需求部门等着hotfix的时候为这种事情插手让fix delay。。。事后你还要说感谢他主动帮忙。真的不爽这种。

比起那种没要求，也不认真看就直接approve 的我喜欢这种。

我刚毕业的时候自己做一块东西，组里没有人能review 我的代码都说看不懂，让我直接交，导致我代码质量一坨屎，后来自己重构几次都要吐了。
后来跳槽了以后遇到了超级picky的老哥review 我的代码，我真是感恩戴德，代码质量飞速提高。那时候组里没有任何pip人的记录，默认学习周期一年，我在那一年里就把代码发给各种人Review，能把三个人拉进来喷我就绝对不只拉两个人。
不过现在时代变了有pip了，还是要从产出速度上再优化一下这个问题：

大体的框架/实现方法有问题：这个最好是在一开始prototype 里就达成一致的，也就是最好在你还没开始怎么做的时候多拉人进来。
实现的方法有个人偏好：这点我也没有什么好的方法，有的人的偏好在我看来就是胡说八道，把其他语言的偏好拿来强加给另一个语言。你可以让他陈述原因，如果他聊什么可扩展性你就是嗯对有道理留个TODO那到时候再改，我们初期就baby step。
💩上堆💩：💩山的出现每个人都有责任，不能说让你自己一个人来重构。发个邮件给全组，大为赞叹那位老哥发现了这个问题，提出这是个tech debt，希望放进下一次的fix it/ code health的活动里修。这样你这次就可以继续堆💩了。
命名：没什么办法，我超讨厌命名，别人想帮我起名我开心还来不及。
改无关的东西要重新开pr: 这是在这种裁员的时候给你刷pr 数啊朋友。这种pr通常approve超级爽快，改得快交得快，我狂交。

我貌似也有楼主相似的经历。
刚开始一个CR要改五六版才能通过，
一个东西我实现了，但同事常会提出让我用另外一种方法实现。一个unit test的命名也会讨论一番。还有一些小的东西。
刚开始我也特别痛苦，总是在想我都实现了为啥非要用你推荐的方法？为啥你就认为你推荐的方法是最优的？但是人家资历深，我就沉住气一个一个的改。。。
慢慢的，你知道我发现了啥？
我发现通过别人指导之后修改的代码，看着很漂亮很顺眼，拿我之前写的代码一对比，自己之前写的就是屎。。。
过程虽然很痛苦，但我确确实实学到了特别多的东西
神奇的是，我的代码质量也越来越高，现在基本上1版或者2版就能通过，comment也越来越少。。。真的是超开心。
我之后发的代码，如果和那个同事有关，我就尽量发给他让他来帮我看看，因为我感觉有一个特别负责人的人帮忙看代码，是提高写代码能力的一个特别好的方法。

我最近还发现我们组里有个人，他总是能找到一个更好更酷的方法实现同一个东西，有时候为了方便理解，他直接把那个更优的代码写在下面，或者给个example的链接，有时候还会把说明为什么要这么做的链接写出来。。。我真的是，第一次看到他的comment我都惊呆了！然后我就告诉他特别喜欢他的comment，让他以后多多给我提，因为通过他的comment，我可以学到很多新的东西。
总之，我个人感觉能遇见一个认真的reviewer，是很难得，他们也要花很多时间去看一堆屎代码（我感觉自己刚开始写的代码就很屎）。
所以如果对方提出的comment比较make sense，楼主不妨沉住气，以一种学习的状态去对待，可能会好受些。等熬过了这个过程，可能就柳暗花明了～
P.s.:我刚开始对我的reviewer是又爱又恨。。。现在就只剩下爱了，非常感谢他的认真，让我学到了很多。

很希望有一个这样的mentor training我，再说入职半年承担的业务需要估计不会特别复杂，应该不会特别忙碌，如果是我的话会多花点时间思考