code review 和同事吵架，老板说我态度有问题

写review comment除了单纯的要求改正，很多时候也是想交流讨论和互相学习，平时大家都习惯了，讨论归讨论，不带什么个人情绪。如果碰上反应大的，下次我就尽量少些不写算了。还有的人不想回答的comment就略过, 找其他人给approve，那我也就当看不见。除非严重得需要挡住的PR, 也就这么回事了。

如果是代码上互相帮助要看别人能力怎么样 可能我说这话有点ego，但是我觉得技术不过关的人有的时候只会帮倒忙 写程序其实感觉还是follow强者比较有用
Roat1965 发表于 2023-08-16 00:23

看你这段 感觉你看不上你同事 觉得自己才是强者 别人…… 所以才会觉得别人故意找茬 这个吧 如果楼主已经混到公司里技术大牛 老板非常依赖你 那就算了 如果刚工作没多久 就算很牛 也可以态度上稍微柔和一点

看你这段 感觉你看不上你同事 觉得自己才是强者 别人…… 所以才会觉得别人故意找茬 这个吧 如果楼主已经混到公司里技术大牛 老板非常依赖你 那就算了 如果刚工作没多久 就算很牛 也可以态度上稍微柔和一点
dimplemm 发表于 2023-08-16 01:40

你说得很对。团队里也有关系好的人对我代码提过建议，但是对方能力很强，所以很开心对方给我提建议，我感觉对方是真的在帮我（对方和我私下说上次他写xxx别人说他了，所以他建议我不要这么写）。code review上的这个同事本来水平就那样，我平时都不搭理他，所以他一提建议，我心里就在章子怡os你懂不懂代码。我也不是什么大牛，更多的是觉得自己比较牛。就是黄晓明的那种我不要你觉得，我要我觉得。
然后确实有点毛病就是别人给我错误的信息我会很烦？好像有点厌蠢？ （生活中也是这样，关系好的我不会表现出来）

Code review 不都是在git上写review comments吗 怎么吵起来的...

Code review 不都是在git上写review comments吗 怎么吵起来的...
evelyn417 发表于 2023-08-16 02:29

我们是live meeting

你的领导，还有这贴里这么多同行都告诉你了，你同事提的建议是合理的，至少是不同方面的视角。这完全不是太阳从西边出来这种非黑即白的常识问题。而且这事本身是非常小的事情，就算措辞沟通有问题，也不值得一般职场人有这么大的反应。 建议你可以去看看therapist, 你这个思维模式，加上情绪控制，在职场上是非常大的问题，毕竟你也不是啥天才，多几次因为这些小事被report, 特别是情绪不稳定，现在这个环境下是比工作能力不行还糟糕的事情
noodlewo 发表于 2023-08-16 01:13

看了楼主一系列回帖，不知道在家里面和另外一半也这样相处吗？

回复 14楼Sleepy3824的帖子
这些话听完根本不知道怎么implement啊。能举例子吗？

我刚开始看code review comments也觉得很火大对方是挑刺，后来就觉得只是需要politely defend 我的idea，而且code review时候提出来，比自己project做完给更大规模的人做presentation 提出好多了，可以提前准备这些问题的答案。
有些问题可能有人觉得理所当然，但是如何把理所当然的道理很好的讲出来，也是需要花功夫的，希望可以开解到楼主

作为reviewer，总要说两句，不然看了半天，点个approve，要么显示自己水平不行，要么态度不行。 这种改个一两行的事情，不值得较真。如果要改的太多，挑着改就是了。 不值得浪费时间

也不是, 也有母老虎呀,或是abuse女的的男的 家里说一不二 re 前面说结婚没离婚的

回复 1楼Roat1965的帖子
lz 听着有点强硬 不过老板的帽子扣得也太大了

也是我学会的美国社会职场教训，“raise voice”好似不得了的错处。

Splice怎么用callback / lambda做过滤楼主教我一下？

这是中国人的通病吧。一激动嗓门就大了。切记要记住讲话语气要平和。就事论事，不要带个人感情。再火大的情况也要冷静。跟人argue之前先顺一下别人的毛，I see what you are saying, but....

文化问题。国内剧集里面的女强人经常训员工像训孙子，以示她的优秀和成功。如果放在美国早被人告到hr了。在这里吵架要语气温油客气词用尽
zoelee 发表于 2023-08-16 01:21

在這裡只要講話語氣稍稍為急了一點，其他與會人員就開始準備爆米花吃瓜了，根本不會有大聲指責的橋段，級別越高說什麼話都是在三思量，不太有可能出現暴躁老爹，除非是像馬斯克那種一言堂的類型

我老板觉得我yell了，但我只是说话比较急，纯讨论code，并没有骂人／也没有指责别人 我只是反复问filter比splice好在哪里？ 感觉是文化性格差异吧，我老板说话确实是不带情绪的，他比较喜欢别人quiet一点
Roat1965 发表于 2023-08-15 17:45

“我只是反复问filter比splice好在哪里？”
的确有点咄咄逼人 聪明人会问到底程序哪儿有问题请指出？ （AKA, 没问题你叫个JB，我用汇编语言写都不关你屌事）

Review 的挑一些有的没的，这个有多重要吗。领导肯定都想下面人一团和气干活的，不替自己说话的领导也不用想着长期跟着了

对，polite and firm.
we can be totally mean assholes and bitches without raising our voice.
需要甩锅的时候把话藏在between the line，让领导自己发现 需要背后插刀的时候把埋汰藏在赞扬里 需要坚持意见的时候拉大旗做虎皮 需要撕的时候把弹药递出去，让别人出头

Sleepy3824 发表于 2023-08-15 18:02

总结的太牛了，我要记下了！！！

我们老板在乎态度，但是upper management更在乎代码的code quality。 但你说的对，确实我老板特别在乎是不是态度／礼貌这些
Roat1965 发表于 2023-08-15 19:38

你在职场上很拎不清啊
你老板直接管你，决定你的promotion, raise, 甚至去留 upper management 和你没啥关系，code quality 是你老板要担心的问题，not yours
多读点怎么effective communicate 的书吧

虽然楼主ID写着1965，我觉得楼主像是1996年生人

我刚开始看code review comments也觉得很火大对方是挑刺，后来就觉得只是需要politely defend 我的idea，而且code review时候提出来，比自己project做完给更大规模的人做presentation 提出好多了，可以提前准备这些问题的答案。
有些问题可能有人觉得理所当然，但是如何把理所当然的道理很好的讲出来，也是需要花功夫的，希望可以开解到楼主
nanno 发表于 2023-08-16 06:17

我们code review和别家公司不一样的是： live meeting，所以不是comment approve pr那个process 基本没人挑错，就是走个过场 但是我的态度确实有问题，一点就着 我觉得不管这个同事是不是故意找茬，我都不该当面质问他，就像最赞说的，有理也变成没理了

“我只是反复问filter比splice好在哪里？”
的确有点咄咄逼人 聪明人会问到底程序哪儿有问题请指出？ （AKA, 没问题你叫个JB，我用汇编语言写都不关你屌事）
Silverwing 发表于 2023-08-16 09:45

是的

你在职场上很拎不清啊
你老板直接管你，决定你的promotion, raise, 甚至去留 upper management 和你没啥关系，code quality 是你老板要担心的问题，not yours
多读点怎么effective communicate 的书吧
lanlan_ocean 发表于 2023-08-16 10:20

确实。但是upper management是我老板的老板，实权更多的在他手上（老板的老板比较care code quality，我老板比较care说话态度／礼貌，感觉每个人性格不一样）但是平时我都是和老板沟通，所以还是棘手

确实。但是upper management是我老板的老板，实权更多的在他手上（老板的老板比较care code quality，我老板比较care说话态度／礼貌，感觉每个人性格不一样）但是平时我都是和老板沟通，所以还是棘手
Roat1965 发表于 2023-08-16 11:17

这么说吧，目前upper　management跟你没关系，你这种沟通方式都不需要考虑其他了，长期下去没人愿意和你合作，你自以为自己技术再强也没用的。

吃饱撑的。换我我直接无视，老板要怎么弄我就怎么弄。

首先，你没必要对于reviewer的意见那么defensive。然后你可以仔细想一下对方为什么会这么建议。你可以从学术技术的角度先理解对方的意图。比如你可以说，谢谢你的建议，这个filter的好处是xxxx。那么我不采用的原因是xxxx。你怎么看呢？
LZ，你要知道，对方只是reviewer。你check in自己code的时候是不需要写你的reviewer是谁的。也就是这段code最终是由你负责的。如果好，是你的好，坏也是你的坏。reviewer是来帮助你的（也是帮公司哈）。所以你不要一副很对抗的态度。如果你真的觉得对方的建议很不合理，比如要求改动的地方太多，跟你的change irrelevant（我遇到过），你可以直接说，你的建议很好，但是out of scope，需要改动的地方很多，还需要做extra testing。我们可以把你的这个建议列入improvement project，那么这次改动可不可以不include。
沟通的方法很多种，对抗型是最不effective的一种。你心里要清楚，reviewer是来help你的，你要把他当成自己的asset而不是liability. 人家花时间，花心思才会有建议给你。你得好好appreciate。如果不认真，随便approve就好啦。这段code真出问题，你认为是你的责任多，还是他的责任多？

回复 124楼扶苏的帖子
谢谢mm建议，我们没有指定reviewer code review就是大家看一下写的code，也没有approve process 10个code review有8个都是没问题的 至于对方指出这种不是问题的问题是不是在真的帮我，那就见仁见智了。 但是我同意之前说的无论对方是不是帮我，我都应该keep calm，平心静气地撕逼

回复 1楼Roat1965的帖子
这不是space的问题，而是你想干什么，你需要不修改原array的话就用filter，如果你要in place改的话就splice。这两个做的事都不一样

回复 1楼Roat1965的帖子
这不是space的问题，而是你想干什么，你需要不修改原array的话就用filter，如果你要in place改的话就splice。这两个做的事都不一样
Finallz 发表于 2023-08-16 12:55

删改的是一个copy of array，所以用splice／filter没区别吧

在职场上你raise your voice 你就输了八分了
Sleepy3824 发表于 2023-08-15 17:59

对，永远要镇静礼貌，尤其是女生。

对，polite and firm.
we can be totally mean assholes and bitches without raising our voice.
需要甩锅的时候把话藏在between the line，让领导自己发现 需要背后插刀的时候把埋汰藏在赞扬里 需要坚持意见的时候拉大旗做虎皮 需要撕的时候把弹药递出去，让别人出头

Sleepy3824 发表于 2023-08-15 18:02

我的职场环境比较好，这么多年没跟人撕过，导致我都不会吵架了。这个回复真是精辟，赶紧mark学起来。

对，永远要镇静礼貌，尤其是女生。
健康平安 发表于 2023-08-16 13:08

为什么尤其是女生？性别歧视？

感觉就是很难接受别人给我false的information 也不知道是不是病 别人说的是错的（譬如太阳从西边升起）我就不想听下去
Roat1965 发表于 2023-08-16 00:57

我们面试里一堆behavior question就是为了筛掉你这种类型的。技术差不多就行了，换谁都能做，更重要的是个性和合作能力。说实话，你老板真够意思了

对，polite and firm.
we can be totally mean assholes and bitches without raising our voice.
需要甩锅的时候把话藏在between the line，让领导自己发现 需要背后插刀的时候把埋汰藏在赞扬里 需要坚持意见的时候拉大旗做虎皮 需要撕的时候把弹药递出去，让别人出头

Sleepy3824 发表于 2023-08-15 18:02

美国政府不就是这么做的吗？啊哈哈，文化呀文化。

回复 124楼扶苏的帖子
谢谢mm建议，我们没有指定reviewer code review就是大家看一下写的code，也没有approve process 10个code review有8个都是没问题的 至于对方指出这种不是问题的问题是不是在真的帮我，那就见仁见智了。 但是我同意之前说的无论对方是不是帮我，我都应该keep calm，平心静气地撕逼
Roat1965 发表于 2023-08-16 11:41

还这么想真的脑子拎不清，你得好好转变思维才会改变自己态度

你说得很对。团队里也有关系好的人对我代码提过建议，但是对方能力很强，所以很开心对方给我提建议，我感觉对方是真的在帮我（对方和我私下说上次他写xxx别人说他了，所以他建议我不要这么写）。code review上的这个同事本来水平就那样，我平时都不搭理他，所以他一提建议，我心里就在章子怡os你懂不懂代码。我也不是什么大牛，更多的是觉得自己比较牛。就是黄晓明的那种我不要你觉得，我要我觉得。
然后确实有点毛病就是别人给我错误的信息我会很烦？好像有点厌蠢？ （生活中也是这样，关系好的我不会表现出来）
Roat1965 发表于 2023-08-16 02:00

只能说你太年轻太幼稚了还是太傻了，做人表面上就得夹着尾巴说好听的被批评也和颜悦色

确实。但是upper management是我老板的老板，实权更多的在他手上（老板的老板比较care code quality，我老板比较care说话态度／礼貌，感觉每个人性格不一样）但是平时我都是和老板沟通，所以还是棘手
Roat1965 发表于 2023-08-16 11:17

还是拎不清
upper management 在乎什么跟你毫无关系, out of your pay band.

还是拎不清
upper management 在乎什么跟你毫无关系, out of your pay band.


lanlan_ocean 发表于 2023-08-16 16:03

upper management 和我们挺close的（他经常参加我们的standup） 他和我的直属上司都有权利决定我的future

upper management 和我们挺close的（他经常参加我们的standup） 他和我的直属上司都有权利决定我的future
Roat1965 发表于 2023-08-16 16:09

说这话的人，一般是搞不定自己的老板， 只能幻想着自己能搞定老板的老板。。