被 Code Review 折磨疯的组员

3 天前
 MrRongts

Code Review 经常性把别人的写的都推翻,让人按照他的想法来,这他妈的是什么个心理。 组里都特么都在吐槽,大环境下没人敢说不,太难了。以前担心被裁员,现在期望被裁员拿赔偿走。

5452 次点击
所在节点    程序员
79 条回复
zsc8917zsc
3 天前
既然有 Code Review ,那写之前没定规范吗?
vfs
3 天前
反过来讲: 会不会是自己代码写得不太行?
DiamondYuan
3 天前
ai 时代这些都不是事,让 ai 写就行了。

请按照 xxx 要求修改代码。 修改后保持原有功能不变。
xing7673
3 天前
@vfs 写的都推翻,我是对面的问题更大
S1ahs3r
3 天前
Code Review 已经是眼高手低的伪程序员唯一能发挥的地方了。

见过每次都对命名指手画脚的人么?(都符合命名规范)
MrRongts
3 天前
@vfs 你费劲巴力的写了 2 天, 业务功能也完成了, 功能也测试过了。Code Review 说你这样写不好,有更好的方法,好你在根据他的想法重写,来回 Review, 为了一个他所谓“好” 折腾别人,无非是显自己很优越,我见过组里有人,为这种问题搞了好几周,人都麻了。
cutecore
3 天前
@MrRongts 刚入职会遇到有这种,感觉是看看水平和服从性测试,正式开始开发就没人管了,按时交付就行
bbao
3 天前
@MrRongts 不,可能你真的比较弱,你都说了,费劲吧啦写了 2 天。要从中学习到大家提的建议,或者你有理有据的反驳。都做不到,可能就是,单纯的,比较弱
vfs
3 天前
@MrRongts 打工人是按照时间挣钱的,之前的代码已经挣过钱了,重写的又能重新挣钱 :)
bojackhorseman
3 天前
能跑就行了,功能实现了就行了。搁这写高考满分作文呢,留着家传吗?
fj24911
3 天前
我认为 CodeView 作用有限。
将更多的时间放在前期设计和文档完善收益更大。
单元测试通过了就可以了,对结果负责,过程细节难以掌控。
AI 是个好帮手
MrRongts
3 天前
有次写 js 的时候, 我用了一个 ?? 语法,他问这是什么屌语法改掉。 有次写 rust 的时候, 我使用 HashMap 的时候,

我这样
```
let a = HashMap::new();

if a.contains_key("1") {
a.insert("1", vec![]);
}

a.get("1").unwrap().push("1");
```

然后他跟我说有一个 or_insert, 让我改掉

最早的时候,我经常使用 rust 链式处理方法,他说不好,给我的感觉就是只有他不知道,你就得改
lambdaq
3 天前
你说下次一定啊。

下次写之前让他开个头。
nananqujava
3 天前
现在还有什么弱不弱的, 我用 CC 写, 你咋挑刺?
Huelse
3 天前
个人建议:听劝,他说什么都对,出问题也是推给他负责
red13
3 天前
组员被 Code Review 折磨疯,说明 Leader 不具备管理能力
janus77
3 天前
底层语言奇技淫巧多就会这样。写 java 会有这种苦恼?[狗头]
MrRongts
3 天前
@bbao 很好奇你所谓强到底他妈的是啥。 都特么写一下 CRUD 能特么强到哪里去。 还能写出个花来?,还是发明了个什么设计模式,或者什么牛皮的算法。有些所谓的代码好,都是自我感觉良好罢了。

都是 7 ,8 多年程序员,功能都能实现,保证自己的东西不会出问题,对自己的东西负责。老是把自己想法强加于别人只会适得其反。
jamesxu
3 天前
@MrRongts 这人有点闲得蛋疼,语法糖而已,哪种写法不都行
Romic
3 天前
@vfs #9 有没有可能是双倍的时间挣一分钱,就比如本来晚上 6 点下班,codereview 之后 12 点下班。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://ex.noerr.eu.org/t/1158605

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX