小 心 任 何 二 次 接 手 的 代 码

22 小时 52 分钟前
 MoeMagicMango

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

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

开篇惊喜

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

Interface 乱飞

怪不得会跑路

能跑就行的逻辑

Cookie 生成逻辑

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

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

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

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

硬编码 +++++

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

最后

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

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

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

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

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

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

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

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

© 2021 V2EX