这是一个创建于 336 天前的主题，其中的信息可能已经有所发展或是发生改变。
话说团队最近开始了 codereview 就是一堆人集中开会看代码，中途出去了一下处理问题，没在。后面一同事 A 转告我说某同事 B （接手我负责的模块并负责改重构该模块的） codereview 时说我之前的代码有点恶心。 之前的代码垃圾我也承认，只是我不太接受这种形式,的确是刚毕业那时还是太年轻。，包括自己看到比较久之前写的代码也觉得有点可笑，方便也就改写法。
20 条回复
 |
1
RedisMasterNode 336 天前  1
CR 对事不对人，代码写得不好是常事，接受和修正，然后下次能够写出更好得代码才是目的。
另外为什么要 Review 旧代码呢，CR 形式感觉还是关注新代码改动、在 Pull/Merge Request 时进行体验更好。 |
 |
2
rxmt 336 天前
有一些自己的看法吧：刚毕业那会儿，我因进度慢被组长 diss 过，但是最后证明我做的比较周到，很合理，组长在例会公开表示歉意了。
我觉得主要看团队氛围，还有团队是不是以做事情为导向的，如果团队在进步而且都在做事情，那么该批评批评，该去改正就改正。如果是个讲政治的团队，可能就需要想想怎么怼回去了。 |
 |
3
luckyrayyy 336 天前 1
害，不用太敏感，谁没写过狗屎代码。现在觉得得意的代码过几年再看可能也是一坨屎。大多数情况下 cr 的时候大家还是对事不对人的，被批评的时候认错挨打改正就完了，你觉得面子上挂不住就多解释几句原因。
|
 |
4
Martens 336 天前
没情商啊，有什么问题指出来，而不说恶心
|
 |
5
jaycelhz 336 天前
嗯嗯 然后呢
|
 |
6
InDom 336 天前 1
承认自己写的就是一坨狗屎，就不会在乎别人说你写的代码屎了。
|
 |
7
samun OP @RedisMasterNode 我也没搞明白 关键是那人是完全重构了我之前的 如果他接手维护这样说我完全接受
|
|
9
samun OP @luckyrayyy 主要我当时也不在,他这种方式不太能接受
|
|
12
RedisMasterNode 336 天前
@samun 安啦。这些事情咧，如果 CR 建议对你有收获那就学，如果人家只是说代码不好，也没说怎么变得更好，那就忘记就是了，影响不了你工作内容、绩效、奖金分毫，你以后也不一定和人家共事。
|
 |
13
pluschen 336 天前
多留意下 A
|
 |
14
franklinray 336 天前
你这同事就是情商低。有一百种方式能合理表达这段代码不好，非要用这种最得罪人的方式。
|
 |
15
yifangtongxing28 336 天前
谁写的代码不恶心？
这点事就玻璃心了 |
 |
16
ZZ74 336 天前
任何人看别人的代码都觉得会恶心。你看自己以前写的代码也会觉得恶心
|
|
19
samun OP @yifangtongxing28 可能是吧 这个事情应该是他对我大概 18 年写的代码重构 然后拿这个来对比他现在的设计高明之处。
|
|
20
samun OP @RedisMasterNode 嗯 感谢老哥建议
|
Select Language