小 心 任 何 二 次 接 手 的 代 码

111 天前
 MoeMagicMango

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

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

开篇惊喜

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

Interface 乱飞

怪不得会跑路

能跑就行的逻辑

Cookie 生成逻辑

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

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

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

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

硬编码 +++++

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

最后

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

12002 次点击
所在节点    程序员
114 条回复
jkjoke
111 天前
加钱，必须加钱
Elaina
111 天前
这就是传说中的防御性编程么🤣🤣🤣
xmumiffy
111 天前
接手价至少得是重写的 2 ～ 3 倍起跳😂
coolfan
111 天前
main1 main2 main3🤣
JoeDH
111 天前
什么项目
MoeMagicMango
111 天前
@JoeDH 一个软件的解析项目，爬虫系
vasto
111 天前
倒很正常，就是新手写的
MoeMagicMango
111 天前
@vasto 但这已经是商业项目，要赚钱的（
falcon05
111 天前
一般吧
geligaoli
111 天前
你是没见过真烂的 python 代码，这些例子都算是上上等的模范代码了。那些代码可怕的程度超乎想象。每个接口的处理都是复制粘贴的一路到底，没有函数的概念。所有的变量基本都是全局的，包括类里面的也是到处直接读写。业务里用的所有内容都放一张表里，字段多的全用拼音首字母加数字，表的索引几十个。到处共享数据还丝毫没有锁的概念。
zhenjiachen
111 天前
session 存 db 是正常操作，每次访问更新 session 的过期时间也是正常操作。你可以看看 spring session 的分布式 session 。不会 Python ，但是这种代码在 Java 里面应该还算正常。
Maboroshii
111 天前
挺简洁的，至少一眼就能看明白。不差了
Reficul
111 天前
我以为啥呢，基操。

年纪大了之后，现在再看这种程度的代码已经毫无波澜了。
chenqh
111 天前
唯一的问题也就就是 check_expire_time 好像是扫表吧,
maocat
111 天前
fastapi+sqlmodel 挺前卫的啊
xiri
111 天前
“#写一个接口，当这个接口接收到请求后 ...”
很明显的面向 AI 编程，写个注释让 AI 帮忙写逻辑🤣
aycclm
111 天前
正常，商业代码这种情况多的很，能 RUN 就行（程序员或者代码）
ntedshen
111 天前
萌新确实不建议摸二手代码，会崩世界观的。。。

这可读性。。。看着挺好的，只有那个遍历 user 的方法即使看着注释也很迷惑。。。
业务写多了手上没屎才是奇怪事，老板要能把维护项目谈成重写项目那属实是老板的能力可嘉，不过不一定会给你第二次机会。。。
看 op 效率（加班情况）怎么样了。。。
IvanLi127
111 天前
这代码还成吧，这才哪到哪。除了代码结构问题，其他关于实现的吐槽都需要有实际瓶颈才有得说，现在这样未必不行，反到 op 有点偏执了。
IzayakI
111 天前
你的代码还不够变态

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

https://tanronggui.xyz/t/1079893

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

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

© 2021 V2EX