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

匿名用户 发表于 2023-3-19 13:51
不过实际上楼主并不是这种情况 虽然我换组了 但公司code 内部都可见

楼主这位比我说的情况恶劣多了。这么多年了没说什么，现在非要从楼主这边开始提高要求，这是待人严对己宽（如果是我，我会亲自开一个新ticket来重构整个代码，然后从此开始明确要求每一位组员遵守code quality）。以及多删一行代码都不行非要改回去，这是病，得治。而且来回5、6个版本还没过，说明这位Senior表达能力太差，无法一次就把问题和需求提清楚，或者说他自己也没想明白到底什么才是clean code，只是像摘樱桃一样见一个摘一个。

我不知道这位Senior其他技术方面如何，但做人方面，他显然需要再多练练。层主你如果是这位Senior的朋友的话，有机会劝劝他。退一万步讲，我知道这位Senior或许是在忠实执行软件工程师的职业操守，但在这么个toxic的老板下面，他也应该懂得变通，知道自己的执着会给组员带来多大的负面影响。

donnice 发表于 2023-3-18 23:00
楼主这位比我说的情况恶劣多了。这么多年了没说什么，现在非要从楼主这边开始提高要求，这是待人严对己宽 ...

你为什么默认是这么多年。。。 你知道被喷的senior进组多久么 ps你是不是默认楼主是个junior。。。 我告诉你楼主不是junior呢。。。 ps 5、6个版本都还没过的cl在我司真的不少 尤其新人进公司。 “自己的执着会给组员带来多大的负面影响” 真的出bug了负面影响谁背

匿名用户 发表于 2023-3-19 14:02
你为什么默认是这么多年。。。 你知道被喷的senior进组多久么 ps你是不是默认楼主是个junior。。。 我告 ...

测试code本身就是屎山，只能跟着前面的人再堆屎了，然并卵，这位非得要求从现在开始，不能再堆屎了，要从我这里开始开始一点点改变。

至少楼主应该不是第一个或者第二个堆屎的吧，明明是自己对代码质量有要求，偏偏要从楼主这里开始？自己不能发个PR重构？楼主不是Junior那问题就更严重了，这是老Senior在借机打压新Senior？这可就是政治斗争了啊

5、6个版本都还没过的cl在我司真的不少，我估计你司在review的时候可能不在乎PR有几个版本？那虽然问题不那么大，但我个人觉得你司的这个习惯要改改，太墨迹了在这个市场上活不下去的。

donnice 发表于 2023-3-18 23:06
至少楼主应该不是第一个或者第二个堆屎的吧，明明是自己对代码质量有要求，偏偏要从楼主这里开始？自 ...

“这是老Senior在借机打压新Senior” 你想的好多 我看过那个cl了 至少我作为一只老🐶 我觉得是正常的review

匿名用户 发表于 2023-3-19 14:07
“这是老Senior在借机打压新Senior” 你想的好多 我看过那个cl了 至少我作为一只老🐶 我觉得是正 ...

笑死，你都被同化了当然觉得正常。我就问一句话，如果是你觉得要提高代码质量了，却刻意从别人的PR入手而不是自己去做重构，这是为什么？

donnice 发表于 2023-3-18 23:08
笑死，你都被同化了当然觉得正常

反正还是一件事情 如果觉得公司文化不合适 可以跳槽

匿名用户 发表于 2023-3-19 14:09
反正还是一件事情 如果觉得公司文化不合适 可以跳槽

同理，也可以从我做起，从这个组开始，改变公司的文化，或者让对方离职

donnice 发表于 2023-3-18 23:08
笑死，你都被同化了当然觉得正常。我就问一句话，如果是你觉得要提高代码质量了，却刻意从别人的PR入手而 ...

你为什么觉得别人senior平时没有做重构的活 是不是每个新组员写cl正好可以小重构的时候都得组里老组员上

donnice 发表于 2023-3-18 23:10
同理，也可以从我做起，从这个组开始，改变公司的文化，或者让对方离职

我没觉得公司注重code quality有什么不好 当然有人觉得不好可以自己努力看看能不能把公司改成move fast的文化

让我们来看一看GPT-4怎么说:

看起来你在和这位资深组员的代码审查过程中遇到了一些挑战。我理解你的感受。这里有一些建议，希望能帮助你应对这些问题：

和组员沟通：试着和这位组员沟通，表达你的顾虑。告诉他你很尊敬他的专业知识，但希望他能理解你的立场和进度的压力。寻求他的建议，看看是否可以找到一个更好的平衡点。

寻求其他人的意见：如果可能的话，找一些其他同事来review你的代码，这样可以让你从不同的角度收到反馈。这有助于提高代码质量，同时也可以给你一些在和这位资深组员讨论时使用的论据。

学习和成长：虽然这位组员的代码审查可能让你感到沮丧，但这也是一个很好的学习机会。从他的建议中汲取经验，提高自己的技能。这样，你在未来的项目中就能更好地避免类似的问题。

提前准备：在和这位组员讨论之前，仔细阅读他的评论，并准备好你的论据。这样你可以在讨论时更有信心地表达自己的观点。

时间管理：为了不影响你的生产力，你可以尝试设置一个时间限制，比如每天只花一定时间处理代码审查。这样，你可以在完成其他任务的同时，逐步解决审查中的问题。

关注长远目标：要记住，你们的最终目标是为了提高代码质量和团队的整体水平。虽然这个过程可能会让人感到沮丧，但从长远来看，它对你和团队都是有益的。

至于提高吵架和临场反应的能力，我建议：

多练习：多参与讨论和辩论，这样你就能逐渐适应并提高你的临场反应。
阅读和观看辩论：阅读辩论书籍，观看辩论比赛视频，从中学习别人是如何在压力下思考和应对的。
提前准备：在参加讨论之前，提前预测可能出现的问题，并思考你的回应。
深呼吸和保持冷静：在争论中，保持冷静很重要。深呼吸有助于你集中注意力，更好地应对压力。