关于锁，有个奇葩的事

[300]

一到我提 pr 就各种意见，看到我加了锁就想起来这里是多线程/进程读写了

加了还不够，有没有考虑这个锁没生效？会不会死锁？锁的时机对不对？

明明我就是复制粘贴其它文件的代码，别人从来没关心过这个问题也没事

我要是直接只改改名字也就过去了，非得考虑这一下反而给自己找不痛快

review 的时候我都没敢说我是复制粘贴的，不然指不定让我把其它地方全改一遍

代码里一堆烂事，已经没有一开始的热情去优化了，多做多错，能者多劳

[SmithJohn]

有没有类似功能,如果有的话直接别人怎么写你也怎么写呗.
问你你就说参考了哪一段,出于什么考虑改了哪些.
刚进来的就不要表现的太不一样了,不然很容易被针对吧.

[aliveyang]

不用跟人家比，自己对自己负责就行

[exonuclease]

多线程的代码我自己写起来也会很小心 因为这玩意容易写错。。。

[abolast]

会不会是历史遗留问题，之前为了产出不严，现在到了严的时候，也就是看到了就整改。我觉得没问题啊，把自己的排期加嘛，找项目说明原因拉大佬出来刚刚看么

[patrickpu]

有些人是性格问题，不熟的话一开始有攻击性的，以为提 pr 的都是新手全公司就他最牛逼，code review 的时候喜欢用些强语气词增加沟通矛盾

[edisonwong]

换个角度想：如果因为你写的这段线上故障了，人家 review 也是帮了你
如果你杠说：”我都是 copy ，别人以前旧代码本来也是屎山“，那人家跑这么久了不也没崩？反正谁维护谁倒霉哈哈哈

[th00000]

有人愿意认真 review 我的代码，愿意花费自己工作的时间跟我讨论我代码中的问题，我愿意称他一声义父！🙄

[wwd179]

#10 说得比较明白了。

[donaldturinglee]

对 concurrency 来说 review 严格一点其实没有什么坏处。如果是 pr 区别对待的话，那就是职场问题了

[sampeng]

这个我碰到过很多很多了。我觉得是心态问题，不是每个人都热爱代码。做 review 的人我觉得没什么问题，只要不是 review 所谓代码风格其实无所谓。但锁这种东西，你不能说因为别的地方没人管就我这 review 到了你要管我。因为还一个可能，你所谓拷贝的代码可能是 10 年前的，没人敢动。所以不清楚你的实际情况无法评判。实话说，review 如果都是 lz 这种态度，这个 review 的效果是奇差的。code review 在国内还是很难执行下去的。我 review 两次，如果一次不搭理我，我第二次就不管他了。也不再 review 代码，何必起矛盾。