被同事气死了

2022-04-26 19:48:43 +08:00
 liangkang1436
- 变量命名用拼音,不用英语

- 不分离代码,一个 JS 写 3000 多行,一个类 2000 多行

- 拒绝格式化,不改

- 到处复制粘贴,修改代码的方式就是把原来的代码复制一份和之前的逻辑用 if-else 分开,这也是导致他经受过的类代码量陡增的原因



最终导致,他的代码没法调试,一个小 bug ,让我调了两个小时,最终通过打日志才解决。

气得我边骂边捶桌子调式。
9265 次点击
所在节点    职场话题
74 条回复
wd
2022-04-26 19:49:48 +08:00
他的代码为什么不是他自己调试?
fnd
2022-04-26 19:50:11 +08:00
liangkang1436
2022-04-26 19:51:55 +08:00
@wd 因为他去别的项目组了,我留下来殿后
liangkang1436
2022-04-26 19:54:12 +08:00
同一段代码到处复制,愣是不抽象出一个工具类,结果导致逻辑修改的时候到处找,我不止一次跟项目经理说,重构吧,项目经理说没时间,凑活用
liangkang1436
2022-04-26 19:56:52 +08:00
项目最开始的原班人马大部队已经去进行别的项目的开发了,但是现有的项目需要有开发继续维护,我就是那个留下来的人,经过长时间的迭代,代码已经不满足现有的需求,甚至还成为了阻碍,急需一次重构升级,但是没有时间搞,就这么拧巴着,最终,今天晚上爆发了
liangkang1436
2022-04-26 19:58:27 +08:00
算了,摆烂吧,下班回家自己玩自己的项目的时候上点心就行了
sprite82
2022-04-26 20:17:55 +08:00
代码和你有一个能跑就行[doge]
v2eb
2022-04-26 20:20:28 +08:00
我的同事临离职前写了好多 bug.
imzcg2
2022-04-26 20:20:41 +08:00
您确定是重构?不是重头开始?技术债,哪那么容易还,尤其是加需求的时候,让原来开发的人过来都想跑路,更何况你
liangkang1436
2022-04-26 20:21:40 +08:00
@imzcg2 别扎了
arthas2234
2022-04-26 20:30:10 +08:00
他还在项目组?那不是继续留后患?
贴脸输出:代码截图,发邮件,抄送领导
echo1937
2022-04-26 20:32:16 +08:00
@liangkang1436 #5 我司的项目历来如此。
potatowish
2022-04-26 20:35:19 +08:00
去别的项目组继续祸害
ericls
2022-04-26 20:48:37 +08:00
没有人教吗?谁 merge 的? CI 怎么过的 review 怎么过的?

有没有可能你们公司奖励这种你唾弃的代码呢? 至少应该跟团队谈谈 看看大家是怎么想的
InDom
2022-04-26 20:50:44 +08:00
发帖是靠通灵嘛 🐶
liangkang1436
2022-04-26 20:54:17 +08:00
@ericls 功能是没问题的,只是维护起来很麻烦,前期为了项目进度,领导也默许了
ericls
2022-04-26 20:58:21 +08:00
@liangkang1436 吸取经验教训吧 下次用这个为理由 让团队争取给自己更多时间去优化代码结构 理由就是为了以后更好维护
littlewing
2022-04-26 21:24:19 +08:00
能跑吗?能跑就行
github2020
2022-04-26 23:05:10 +08:00
团队负责人不管吗?
FreshOldMan
2022-04-26 23:06:50 +08:00
格式化上 eslint ,代码问题上 code review 啊

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

https://www.v2ex.com/t/849426

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

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

© 2021 V2EX