小 心 任 何 二 次 接 手 的 代 码

2024 年 10 月 13 日
 MoeMagicMango

上周接手了个项目,老板说大部分项目都不能工作,本来是以维护的价格来处理接手这一段代码的

没想到坑比我想的还多,这是其中展示通用的一部分,大部分出于保密性就不能透露了

开篇惊喜

我滴妈欸,你说这是 Python 新手练手的代码我都信了,这是家公司的代码???

Interface 乱飞

怪不得会跑路

能跑就行的逻辑

Cookie 生成逻辑

给 User 生成 Cookie 用的,不是 用 个 JWT 会要了命吗?这么写干嘛啊

而且最草的一点,Cookie 存在 MYSQL 数据库 里面

大哥,这人一多不得打穿数据库啊(

每个请求都要往数据库请求一次,妈欸

硬编码 +++++

存心不给维护啊,写硬编码写了十几处

最后

已经在重新写这一套玩意了,真的太哈人了

15873 次点击
所在节点    程序员
114 条回复
HashV2
2024 年 10 月 14 日
@geligaoli 是的 这个已经算好的了,起码能知道是干什么的,我现在接手的无注释、一个方法几百行的代码才是炸裂,我准备梳理完逻辑直接重构了,真改不了一点。
wowawesome
2024 年 10 月 14 日
属于能跑就行
cxe2v
2024 年 10 月 14 日
1. 这个代码帮老板挣到钱了吗?
2. 你写的优雅代码能帮老板挣到更多利润吗?
XLeopard
2024 年 10 月 14 日
看着这种代码就控制不住的想骂
BlackSiao
2024 年 10 月 14 日
@Maboroshii 还真是,我不是科班出身的,大学的时候写的代码和这个比也差不多了
charlie21
2024 年 10 月 14 日
3. 老板挣钱给你买法拉利了吗?
991547436
2024 年 10 月 14 日
ai 写的
Latin
2024 年 10 月 14 日
能跑就行 另外这个代码算很整洁了
0x4b0082
2024 年 10 月 14 日
其实还好 这都算中规中矩的了
xcai007
2024 年 10 月 14 日
也有可能是公司的问题。。。
NO9527
2024 年 10 月 14 日
每天访问量个位数的话,存 db 也就这样了
lwlizhe
2024 年 10 月 14 日
竟然还给你写注释,良心啊~~🐶
Xi
2024 年 10 月 14 日
这是人情世故代码,考验老板,也恶心下一位接手的人。
就算你花大精力重构了代码,要是之后项目再次不能工作,老板很大可能会觉得你的水平和之前的差不多,这就是所谓的吃力不讨好。
Felldeadbird
2024 年 10 月 14 日
问题不大,能跑就好了。
钱给够,多屎我都可以在上面雕花。
yh7gdiaYW
2024 年 10 月 14 日
看起来除了 main1 2 3 4 有点难绷外,其他的还不错了,他甚至还写了注释
querysecret
2024 年 10 月 14 日
不是啥大事,up 应该工作 5 年内,哈哈,大于这个数应该见怪不怪了
cbythe434
2024 年 10 月 14 日
main2 main3 main4....main7 ,main 届 goat
300
2024 年 10 月 14 日
这个好歹有板有眼,知道是干嘛的

以前重构某日企车厂 app 的三手代码。页面接收一堆参数,最后应该显示两个按钮,结果一通处理显示了 4 个,他们的处理方法就是直接加 size = 2 。反正 mock server 的数据是固定的。几十个页面全部都是这种写法,有什么 if else 还是 when 条件的,最后强行设置结果就行了。

从代码看一手的时候是有真正逻辑的,二手的时候可能是修 bug 或者改设计,直接就真正的放飞自我了
utre938722
2024 年 10 月 14 日
现在写代码 本着能用就行了
ColdBird
2024 年 10 月 14 日
看的我都笑死了,真是绷不住了。
我对垃圾代码是真的零容忍,但是很多人都是抱着又不是不能跑的心态写代码的,没啥办法

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

https://study.congcong.us/t/1079893

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

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

© 2021 V2EX