小 心 任 何 二 次 接 手 的 代 码

[MoeMagicMango]

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

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

开篇惊喜

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

Interface 乱飞

怪不得会跑路

能跑就行的逻辑

Cookie 生成逻辑

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

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

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

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

硬编码 +++++

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

最后

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

[user100saysth]

草台班子，整个世界都是

[seedhk]

@kpsanmao 兄弟，请问下，私活哪里接比较合适啊

[Tink]

老板在找几个人都搞不了，最后还得找原开发

[Lightbright]

挂我是吧，晚上别睡太死

[cccvno1]

可能屎见多了，看这么短的代码甚至有点眉清目秀😂

[cndenis]

你能轻松看懂说明这代码已经是及格以上的水平了, 可以了.

而且你说这是爬虫, 那代码就是针对性的, 没有复用可能, 硬编码完全没问题.

你不会觉得一个爬微博的虫只要改个域名就能爬抖音吧, 如果不能改的话, 写死微博的域名有啥问题

[llsquaer]

觉得还行啊，楼主看能发个修改过的对比下，学习学习

[wangxiaoer]

看起来的确挺烂。但是 session 存数据库这个不算问题，多台服务器不用考虑负载均衡的模式。

[imnpc]

代码没问题的 这个估计也是找人做的急活 功能都实现了 而且做的爬虫的话 根本不会有大规模用户

什么样的需求 给的什么样的报价 写什么样的代码

[woodfizky]

已经不算很差了，不过确实项目结构还不成形。
起码这个 BaseModel 应该是 pydantic 的 BaseModel 吧，起码知道用显式定义属性的类。

我知道的个别同事都是字典用到天昏地暗。
一个字典几十个键值对，还有嵌套的，不管大部分属性能不能用到，反正就从头用到尾，好几层调用方法都要把这个祖传字典传进去，修改里面的值。
还有所有业务，涉及到数据库的，也不知道怎么配置 ORM 的，反正就一条语句一个事务。

接手的那个同事，接手了几个月，改 bug+优化+改造，头发都白了几百根。

[zgsi]

又不是不能用

[rlds]

看了这代码，比我们那些项目好多了。。

[ugpu]

代码没问题. 挺规范的.
又不是什么大工程 临时爬虫项目而已 以后要改得地方多着呢 这不 OP 就改了吗？.
看项目就几个文件几个代码
我内心已经对这些毫无波澜

[ma46]

如果你觉得这就是屎山, 那你是没见过真正的屎山

[CodeCodeStudy]

还好吧，不算太烂

[superchijinpeng]

代码没啥问题，简洁明了

[harryWebb]

你这个得加钱，你还要重新写，也许这代码运行一段时间公司倒闭了就没了

也很正常

你没遇到过当年我遇到 php 得代码，和 html 混写，啥都混一起，那个是真的雷人

[gimp]

能跑就行

[HywelXu]

刚被一个烂尾项目坑过，说是维护就好，加几个接口而已。
没想到改着改着就等于重新写了一遍 既不讨客户喜，还不赚钱。
再也不要干这种活儿了。

[haneki]

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