被已离职同事遗留的项目气晕

[LandCruiser]

我打工的小作坊主打一个短平快，项目制。需求，代码一直在变化。主要服务银行，政府等老式客户。 这大哥遗留的代码，vue2.7setup,TS+自己封装的一塌糊涂的 element-ui,不知道当初怎么跟服务端协商的，请求参数加密，登录前端校验 token 过期时间，各种离谱操作。工期又紧，晕了

[wsseo]

是这样的

[a4854857]

都不知道几手代码了.是这样的

[laogui]

是这样的，你后面的人也会被你的代码气晕😂

[guanzhangzhang]

世界都是草台班子组成，我搞东西都留 readme 和折腾博客还有代码注释

[Lirika]

传说中的屎上雕花，你让他回来看他都不一定看得懂

[qazwsxkevin]

你好，我就是离职的那个，你也会说“跟服务端协商”，“工期又紧”，“各种离谱操作“了，我很欣慰。

[imboring]

@guanzhangzhang 那也没用的，经过若干年后，经手的人多了，后来的人不更新 readme 和注释，再后来的人也是没办法，甚至会因为老的 readme 被误导。目前这种职场环境，这种情况无解。

[cybort]

要不为啥离职 O⁠_⁠o

[doommm]

短平快，需求代码又一直变，应该也缺乏文档，只能靠口口相传，变成什么样都不奇怪。不过这个技术栈我觉得没什么问题

[guanzhangzhang]

@imboring 我自己做事是这样，其他人就不管了，尽量让以后接我手的人不难受😎

[imboring]

@guanzhangzhang 是的，我也是这样，不然浑身难受。

[Jame00001]

只要接手的所有人水平还行，bug 都是有迹可循的。怕就怕某一个是个半吊子，直接把 bug 当基座往上堆业务。还有一堆破釜沉舟式代码，完全不可扩展。

[Rrrrrr]

别人离职了，你还没放过

[ivvei]

你碰过了，以后这个代码就属于是你留下的了

[kneo]

我看不出来“请求参数加密”有什么离谱的。

[chenshaxiong]

@imboring 老板难受，老板痒呗

[murmur]

@kneo 今天刚处理一个类似的需求，有个客户没事就找人漏扫，昨天要求我们 api 地址和 webpack 加载的模块名在生成的 bundle 不能用明文

[Baymaxbowen]

各种客户急着要，今天就要上，A 客户要这样，B 客户要那样，C 客户都不要

[zhumengyang]

（又不是不能用

[LandCruiser]

@qazwsxkevin 工期紧还搞 ts 各种封装？完全是反向操作啊

[LandCruiser]

@cybort 全部代码他自己写的，不用 ts ，不过度封装工期就不紧，我基本都是 6 点下班

[yogogo]

这叫不可替代

[estk]

@LandCruiser #21
什么年代了还有不用 ts 的纯 js 项目？

[LandCruiser]

@estk 注意看我交代的背景，小作坊，需求经常变化。不用 TS 到点就下班，工作很轻松。这个大哥遗留的项目是 TS 的，原本 5 秒写完的发送一个请求代码，在他这里边直接变成一个 service ，复杂谈不上，就是一堆看起来很严谨，但是没用的代码。。。

[kneo]

@LandCruiser 变成 service 和用 ts 有啥关系？你说的 ts 是 typescript ？

[yidinghe]

不一定是他一个人留下来的，我这边也有一个前端项目，历经多个版本最后形成了结合 jQuery 和 VUE 的形态，也就是部分页面是纯静态的，部分页面是要编译的……

[qloog]

那机会不就来了😝，给它弄的优雅点

[kneo]

再多说一句，从你的描述上来看，你的前同事看起来是一位非常优秀的同事。看这架势，他辛辛苦苦盖了间房子，他的后辈找不着钥匙恐怕要直接把房子给拆了。

[ColdBird]

人员交替快的地方都 n 手代码了，有的 vue 文件动不动 3 、5K 行，然后到处都是复制粘贴的代码，习以为常

[MorJS]

看描述前面大哥技术确实没有问题

倒是你的想法 "原本 5 秒写完的发送一个请求代码", 有点⚠️

[lyb11232345688]

人家自己都干不下去了 你还接着干

[chenalex]

除了"登录前端校验 token 过期时间", 其他没看出有什么问题

[92Developer]

多拉些你的屎，到时辞了你还得把你招回来。

[zzNucker]

你把你写的代码和他写的代码都发给 ChatGPT ，让公正的 GPT 评价一下

[angrylid]

你是想说，很多请求本来可以直接在 methos 里面 fetch 短快平，但是他非要抽离出去自己搓了一套类似 rtk query 的 endpoints 或者 spring controller 的玩意？

不过这跟 ts 有什么关系，你不想标注类型，tsconfig 调宽松到处 any 不就退化成 js 了吗？

[chinaguaiu]

@guanzhangzhang #4 写注释还可以，不会被轻易删除。但是 readme 和博客真的没人看，也容易弄丢，工期紧的时候也没有太多时间另外去整理思路写 readme

[legendBro]

你猜为什么他离职

[zephyru]

加 ts 封装（或者任何抽象吧）基本考虑的是维护方便，不想管直接 any 走天下也没什么区别...我觉得不至于套一层 ts 封装把工期拉长多少...

[guanzhangzhang]

@chinaguaiu gitlab 上的 README 还不看吗，那看啥，oa 上留的聊天记录吗🙄

[darson]

大作坊其实都差不多的

[JefferyWang]

你后面的人也会被你的代码气晕

[wdg8106]

一句工期紧就能说明所有的问题了，后面的人接手你的代码，估计也是一样的吐槽
这不是你或他的问题，是公司的问题，要抓住主要矛盾呀
再就是工期紧跟你有啥关系，正确评估出来工期，反馈给 leader 就行，该延期就延期呗（有加班费另说）

[Yasuo666]

他当时的处境跟你一样，当然你的下一个接班人也会跟你有同样的遭遇。

[DAMNYOU]

纠正一下。这个现在是你的烂摊子。不是他 哈哈

[godwinma]

@qazwsxkevin #6 哈哈，这。。。是本尊来回复了吗？世界这么小。

[propc]

@qazwsxkevin +10086 优秀，应该如此~~~

[wanqiangcrack]

看来喜欢套壳这点，在哪都是一样的。 将来别人看你的代码也会是一样的感受，什么 chatgpt 替代开发，做梦哟。chatgpt 恐怕自己的屎山也不小。

[otakustay]

别在意，你的下一个接手者也会被你气晕的，哪怕这项目是你从 0 开始搞起来的

[ZGame]

@LandCruiser #24 我怎么感觉是你菜...

[an0nym0u5u5er]

如果你接手的代码就一两个人开发的，那么其实还好，只要掌握一下他的思路，跟跟代码，也就 OK 了
几年前我修过一个考试系统，这个系统的基础框架也是作者自己开发的，没用市面上的框架，但是其实大同小异。
最恶心的是在 IDE 里面一个 git blame 就能看到 10 个人改过的那种，5 年以上的项目，我也经历过，还被 TMD 老人 diss 能力不行，几乎没文档，几年一直迭代，shit in shit ！

[yuntun]

这才是核心技术人员(赞赏

[RogerL]

听着咋像是我写的？不过我没干过校验 token 过期的事情

[blackmirror]

你就说能不能用就是了，能用还要什么自行车

[daodao]

别人都离职了，你还不放过自己，还要在屎上雕花。赶紧甩膀子别干了~

[taoey]

是这样的，建议重构（估计老板不会给你时间）

[yifangtongxing28]

码农这份钱难挣就体现在，老板只管加需求，从不考虑你的代码是不是有问题

当然，人和代码，总有一个能跑就行了

[bojackhorseman]

啊，怎么感觉是在说我哈哈

[joyhub2140]

这真是祖传代码了，严重影响生活质量的玩意儿。

接了这个真是倒大霉，能力挽狂澜的都不是普通人，要么甩手开摆，要么继续往上堆屎，直到屎山摇摇欲坠。

[chinaguaiu]

@guanzhangzhang #39 🤣微信聊天记录+口口相传

[991547436]

再拉一坨呗，还能咋办

[wupher]

我也碰到过类似情况。

一开始是沉迷封装，啥都要搞自己的一套，但本质上还是 Spring ，可能显示我为公司封装了框架？？

后面就碰到分项目，其它人直接 copy / plaster ，碰上不支持开始各种魔改，或者要求他更新。

最后，这东西变成了防御性编程 :-)

屎上雕花这个算最准确的了。

[yusheng88]

只听你的描述：
1 、 你没了解前辈的需求背景，业务逻辑 [看不懂代码||看不惯代码风格] ，就吐槽代码为“屎山”
2 、 你的前辈使用了 TS 和封装，你觉得繁琐，评价为“屎”

给我的感觉，你的前辈在做工程化的项目，你在“短平快”地糊屎。
你觉得你比你前辈优秀，可以贴代码||方案出来，看看别人是否赞同你

[luckyjack]

未经他人苦，莫喷他人屎（狗头

[chill777]

小公司，就自己重构，反正参与项目的人没几个。

[jefferyJQ]

@LandCruiser #24 你的意思是他前端写的代码分层了，你觉得分层没用？🤣

[justsosososo]

是你 ts 不熟吧，哈哈哈哈

[nikola11]

没有完美的代码 能跑比啥都强

[aikilan]

我上次喷菜鸡前同事写的屎山，直到我放出代码，才开始有人同情我，你不如也放出来糊糊大家的眼睛。

[alexwrong]

你离职后你写的代码 也是屎山

[needpp]

@qazwsxkevin 哈哈哈，终于知道你为何离职了

[salmon5]

ta 只是技术不行，不是人品不行（没在你工位上拉屎），所以可以理解。

[isnullstring]

短平快，那就没什么好说的，不爽自己重新整

[migu]

不是，OP 哪里贴了代码，咋不行啊，一行代码都没看到，你们咋就觉得是那位同事的问题了(😵

[dz5362]

@qazwsxkevin #6 哈哈哈，有趣

[ThinkCat]

代码和人，有一个能跑就行

[sleepybear1113]

我很欣慰的，离职 2 年路上遇到了前同事，说只有我的代码是注释完整的，可读性好很多，至少不算是特别屎山

[lwldcr]

看我上个帖子

[flyff1991]

@laogui 后面的人是被他前任

[flyff1991]

@ivvei 不存在的 提交记录在那，就算屎上拉屎，也不能说前面的屎是他拉的

[wusheng0]

吐槽 TS 没看懂，配置一下全部 any ，不就和 js 一样了。

后来的人：被已离职同事遗留的项目气晕，TS 都用 any ，什么水平

[leegradyllljjjj]

堆屎者恒被人堆之，此为无间堆屎地狱

[FireKey]

这种情况让 ai 去读代码真的很好用

[ruoxie]

其它不知道，但是吐槽 ts 就有点。。。19 年开始用 ts ，不用 ts 影响我下班，特别是接手不是 ts 的项目

[Daming]

1. OP 大概率不熟悉 setup
2. OP 大概率不熟悉 TS
3. 参数加密没毛病

[TimPeake]

看到项目里有人用 jsonSchem 封装 element ui 表格 ，我就想上去给个大逼兜子😡😡😡

[Jinnn]

都短平快了，还会有什么项目质量

[xiangyuecn]

手艺人😜

[behindeye]

我也接触过屎山，但我知道及时我看这些代码很不顺眼，很不爽，我也不会随便去重构，虽然它烂，但是它能运行就说明此时此刻的状态是稳定的，也别手贱跟领导说要重构，就在这基础上修修改改，我曾经吃过亏，觉得要大刀阔斧去优化代码，殊不知任何一个魔法值的变量都是有意义的，包括随处散落的硬编码字符串，你理解不了这块逻辑，不代表这块就是没有用

[ShuWei]

我经常被自己几个月前的代码气到不行

[BraveChi]

💩山代码的主观原因是：缺少设计模式和缺少代码规范和审查。客观原因是需求变动混乱(不是快，而是乱或者开发人员不理解需求从而修改多次)，设计分析不足出现问题重复改动。后人几乎不要想着纠正过来，除非拿出两三倍的工作量，但是现实不实际。

[murmur]

@estk 太有了，我们的新项目第一件事就是把 tslint 删掉，然后告诉大家全按 js 语法写，不要跳类型体操

[kneo]

@chenalex 前段校验 token 过期时间单独拿出来说也不是什么离谱的事情。很多系统都会有 token 过期自动重新申请的功能。前段以前校验可以增加平滑度。前段校验也不等于后端就不校验。

[haoswil]

@guanzhangzhang 馆长好，没想到看到熟人，在这里逛都是职业焦虑的

[jjwjiang]

@Daming 4. 前端校验 token 时限也没问题
总感觉还是 OP 自己的水平问题

[LandCruiser]

@kneo 你的描述不准确，我这个前同事是技术优秀，但不懂得因地制宜，把简单问题复杂化了。也不存在找不着钥匙的问题，活儿我都干完了。拆房子是真的，交不起物业费没必要住别墅。随便搭个草房子就可以了。早点下班回家才是真的，其他都是扯淡。

[LandCruiser]

@jjwjiang 有问题，前端校验 token 过期与否还是需要调一个接口获取服务器时间（或者什么标识）。这不是脱了裤子放屁吗？

[mHKy6969]

政府有什么，街道退休人员登记系统都是发包。12345 政务 2023 年全年投诉登记表 10M ，12345 政务除微信小程序，网页版没法登录，广东省建委投诉平台，20 年前可直接投诉今天关闭了。政府只有一地鸡毛。

[LandCruiser]

@Daming 运行在银行内网的项目，请传授传授参数加密的作用，学习下。

[LandCruiser]

下沉了，争论也没啥意义

[sinx003]

也可能这哥们接手的时候 也是这样.受不了了就走了