小 心 任 何 二 次 接 手 的 代 码

[MoeMagicMango]

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

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

开篇惊喜

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

Interface 乱飞

怪不得会跑路

能跑就行的逻辑

Cookie 生成逻辑

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

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

大哥，这人一多不得打穿数据库啊（

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

硬编码 +++++

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

最后

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

[jkjoke]

加钱，必须加钱

[Elaina]

这就是传说中的防御性编程么🤣🤣🤣

[xmumiffy]

接手价至少得是重写的 2 ～ 3 倍起跳😂

[coolfan]

main1 main2 main3🤣

[JoeDH]

什么项目

[MoeMagicMango]

@JoeDH 一个软件的解析项目，爬虫系

[vasto]

倒很正常，就是新手写的

[MoeMagicMango]

@vasto 但这已经是商业项目，要赚钱的（

[falcon05]

一般吧

[geligaoli]

你是没见过真烂的 python 代码，这些例子都算是上上等的模范代码了。那些代码可怕的程度超乎想象。每个接口的处理都是复制粘贴的一路到底，没有函数的概念。所有的变量基本都是全局的，包括类里面的也是到处直接读写。业务里用的所有内容都放一张表里，字段多的全用拼音首字母加数字，表的索引几十个。到处共享数据还丝毫没有锁的概念。

[zhenjiachen]

session 存 db 是正常操作，每次访问更新 session 的过期时间也是正常操作。你可以看看 spring session 的分布式 session 。不会 Python ，但是这种代码在 Java 里面应该还算正常。

[Maboroshii]

挺简洁的，至少一眼就能看明白。不差了

[Reficul]

我以为啥呢，基操。

年纪大了之后，现在再看这种程度的代码已经毫无波澜了。

[chenqh]

唯一的问题也就就是 check_expire_time 好像是扫表吧,

[maocat]

fastapi+sqlmodel 挺前卫的啊

[xiri]

“#写一个接口，当这个接口接收到请求后 ...”
很明显的面向 AI 编程，写个注释让 AI 帮忙写逻辑🤣

[aycclm]

正常，商业代码这种情况多的很，能 RUN 就行（程序员或者代码）

[ntedshen]

萌新确实不建议摸二手代码，会崩世界观的。。。

这可读性。。。看着挺好的，只有那个遍历 user 的方法即使看着注释也很迷惑。。。
业务写多了手上没屎才是奇怪事，老板要能把维护项目谈成重写项目那属实是老板的能力可嘉，不过不一定会给你第二次机会。。。
看 op 效率（加班情况）怎么样了。。。

[IvanLi127]

这代码还成吧，这才哪到哪。除了代码结构问题，其他关于实现的吐槽都需要有实际瓶颈才有得说，现在这样未必不行，反到 op 有点偏执了。

[IzayakI]

你的代码还不够变态