非native speaker写的代码总不能让别人满意，很沮丧

英语非母语， 写的代码在逻辑上没有问题，可是总是在命名，注释上不能令人满意，Code review 的人偏偏是个native speaker，我已经非常小心了，用Chatgpt 润色多次，每次还是让人指出以大篇的问题，他说他到这些问题很frastrate, 我也很沮丧。


刚刚又来了几个：
FilterUser(...)      ->    SearchUser()...
shouldPaymentPass  ->   shouldPaymentAccepted
CheckFraudulent()      ->   VerifyFraudulent()
Log.Warn($"Desired items for '{searchPattern}' not exist.")   ->   Log.Warn($"Search pattern '{searchPattern}' not found.")
...


一个PR这些问题一共提了50多次，这人是核心成员， code review 绕不开的，虽然他不是我的boss，他明确地跟我说他看到这些nit problem 他看了很frastrate，我也感觉很沮丧。这是跟这个组不合拍的信号吗？还是我因为我不是native speaker对英文的拿捏不到位...

故意的，有些人就拿pr review作为power.

除了第一个filteruser改成search稍微好点，别的都是hair splitting…

我目前的公司之前有别的组的人review我的pr就这样挑刺，基本都是一些代码风格的问题，我直接说既然你觉得这种风格好为什么不加到lint里？不加就说明不要求，你觉得这种风格好，我还觉得我的风格好呢。

后来有次轮到我做deployment，看到他们一些骚操作直接把pipeline都没过的代码merge到master导致整个deployment fail，我也直接公事公办，把错误链接贴出来，把他们merge的pr贴出来，把作者和approve的人一个个@出来，200多人的大群里直接说你们这个pr把deployment搞崩了，啥时候能修好？

搞了一次后面大半年都没敢随便搞我。有些人真的手里有一点权力就喜欢搞别人，这话不仅仅适用于中国人。

补充内容 (2024-09-10 02:28 +08:00):

我上面说的那个挑我刺儿同时也被我在200多人大群里@出来的的也是个E6“核心成员”


而且全公司我只知道他有权限pipeline不过就直接merge，上个月刚被裁了


大家都是来打工的，谁也不比谁高级多少，生产资料都不在你手里，哪有什么核心，真把自己当资本家了。。

补充内容 (2024-09-10 05:55 +08:00):

这种所谓的code review在我看来既无必要，又非常鸡肋，其实就是给这种attention bitch刷存在感的。因为很多东西完全可以通过eslint, unit test和e2e test来自动化掉。


要不是那次deployment刚好栽我手里，谁知道这浓眉大眼要求严格的E6对待起自己人的pr request的要求简直比同性恋的后门还松，pipeline跑不完就敢直接merge，也是牛逼。

我在美国职场10年+，这种情况其实就是故意找你茬呢。这种人我见多了，故意抓一个点，不停打击你，让你怀疑自己能力。

改呗。小问题而已

这些代码language的问题你可以记录下来,每次套用就行，来来回回就那几个。native也是不是随便说，也要有language的。language就是数据库，去哪个图书馆哪个房间哪个抽屉取哪个文件的东西。（补充一下，有人的工作就是提出意见，你一轮就过，他就没业绩了，这点是priority）

weifeng0715 发表于 2024-9-9 10:19
改呗。小问题而已

可是这些"小"问题会让那个人“沮丧“，在多了就变成”懊恼“，然后就是”讨厌“，然后在老板目前说我写的代码质量有问题。别忘了他是核心成员，说话是有分量的。然而我根本就看不到怎么才能让他满意，因为满地都是小雷。总不能命名一个变量都要问一下他的意思吧。郁闷无解，只能说不合拍了。

shouldPaymentAccepted难道不应该是shouldAcceptPayment，他改的也不咋地啊。这纯粹挑刺儿。

这多好啊 ，
有人给你指出这些问题，
改进太简单了，
就这几个简单的词汇记住就行了，
最难的是口语好吧，
职场往上走，口语不好是没人给你想像这样把地道的写出来的。

匿名用户 发表于 2024-09-09 10:31:45
shouldPaymentAccepted难道不应该是shouldAcceptPayment，他改的也不咋地啊。这纯粹挑刺儿。

一般情况这是个被动
Be accepted更合适

写完code放进ChatGPT refactor 一下不就解决了