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

看楼主对自己代码的 confidence 和 ownership.

我在 write in a new program laungage, touch a new code base 会找这种人 review. 因为 knowledge transfer 会让我以后跟有效.

在处理我应经很熟悉的项目或者我本身就是 owner 的项目我一般找第一轮就给 LGTM (even with many comments). 自己的 codebase 我喜欢自己折腾 refactoring, 尝试不同的 patterns.

在自己 review code 时, 我发现个神奇现象, 开始是看不出问题, 之后是看出一堆问题, 再之后同样的代码又觉得没什么问题了. 当读代码能力提高后, 发现不需要计较细枝末节那些差别.

对于这种"严格"要求代码的人怎么看? 工作避免陷入对人本身的评价, 这类人既不是"好心人", 也不是 nickpicker. 觉得对自己有帮助的时候就用, 觉得浪费时间就绕着走。

我前司，几乎所有PR都是LGTM；然后，QA时候，很多打回头的ticket；hot fix也是家常便饭

当然，锱铢必较也确实烦

匿名用户 发表于 2023-03-17 15:28:03
这个人听起来不是对code quality要求高，是非常nitpicking，这种人就算很senior水平很高合作起来也是很头疼的。从这种人身上其实可以学到很多正面的知识，只是忍一段时间学完之后建议换

这种人其实是非常nit的那种货，有时候为了刷存在感，leadership加一堆comments，本身代码符合coding style也会吹毛求疵。和这种人合作经常拖进度

追求Code Quality我觉得是OK的。楼主你应该多花时间跟他吵，项目进不进度的就别管了。

yuuc99445 发表于 2023-03-17 23:49:48
这种人其实是非常nit的那种货，有时候为了刷存在感，leadership加一堆comments，本身代码符合coding style也会吹毛求疵。和这种人合作经常拖进度

这个楼里都在说“这种人不好吗”让我大不懂。都被pua惯了吧，认真review的跟lz这种nitpicking的有本质差别。

匿名用户 发表于 2023-3-18 00:01
这个楼里都在说“这种人不好吗”让我大不懂。都被pua惯了吧，认真review的跟lz这种nitpicking的有本质差别 ...

这个帖子看下来我都懵了
以前的组天天撕变量名，撕Optional如何好，撕不让写for循环必须写streaming，Enum怎么能写出花儿，恨不得互相拿clean code和effective java互相抽大嘴巴，还有code repot根本没有test你交个CR修bug丫突然insist on highest standards非得让你先把test framework弄好不然不给你过的
合着我还得感谢他们

给新人review pr真的很痛苦。

写少了会被manager问我是不是没给人家仔细看，是不是context没给人家讲清楚，新人代码你得多费些心，测试要多写一些，变量名、格式都注意一些很重要的，我们的产品虽然以前做得快一些，tech debt多了点，但是从现在开始都是要改进的。作为组里的老同志了，你要好好把关，云云。

写多了又怕新人压力大。所以每次send之后都在chat上给人解释，大多都是小comments啦，整体都很ok，你这水平很高，姿势很帅，代码稍微改一下就好，有没写清楚的随时来问我，咱公司就是这样子喜欢抠代码质量，有时候comments多一点不要在意很正常，大家当年也是被这样折腾过来的，云云。

所以我大部分时候宁愿自己写，也不想review别人的代码。


都是打工的，互相体谅一下。

对senior来说，最轻松的方式就是自己干，这点小破活一个下午就干了，但无奈老板要发展，新人要培养，得分出活给新人低效低质地干
当他看到周围一圈人都在堆屎时，最绝望的应该是他，估计等市场回暖就立马转组或跳槽了

请问我看很多人多次提到的lgtm和CR是什么意思，一直不懂这缩写的含义

看楼主描述，不客气的说水平远没到senior。