小 心 任 何 二 次 接 手 的 代 码

350 天前
 MoeMagicMango

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

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

开篇惊喜

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

Interface 乱飞

怪不得会跑路

能跑就行的逻辑

Cookie 生成逻辑

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

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

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

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

硬编码 +++++

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

最后

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

14443 次点击
所在节点    程序员
114 条回复
user100saysth
349 天前
草台班子，整个世界都是
seedhk
349 天前
@kpsanmao 兄弟，请问下，私活哪里接比较合适啊
Tink
349 天前
老板在找几个人都搞不了，最后还得找原开发
Lightbright
349 天前
挂我是吧，晚上别睡太死
cccvno1
349 天前
可能屎见多了，看这么短的代码甚至有点眉清目秀😂
cndenis
349 天前
你能轻松看懂说明这代码已经是及格以上的水平了, 可以了.

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

你不会觉得一个爬微博的虫只要改个域名就能爬抖音吧, 如果不能改的话, 写死微博的域名有啥问题
llsquaer
349 天前
觉得还行啊，楼主看能发个修改过的对比下，学习学习
wangxiaoer
349 天前
看起来的确挺烂。但是 session 存数据库这个不算问题，多台服务器不用考虑负载均衡的模式。
imnpc
349 天前
代码没问题的 这个估计也是找人做的急活 功能都实现了 而且做的爬虫的话 根本不会有大规模用户

什么样的需求 给的什么样的报价 写什么样的代码
woodfizky
349 天前
已经不算很差了，不过确实项目结构还不成形。
起码这个 BaseModel 应该是 pydantic 的 BaseModel 吧，起码知道用显式定义属性的类。

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

接手的那个同事，接手了几个月，改 bug+优化+改造，头发都白了几百根。
zgsi
349 天前
又不是不能用
rlds
349 天前
看了这代码，比我们那些项目好多了。。
ugpu
349 天前
代码没问题. 挺规范的.
又不是什么大工程 临时爬虫项目而已 以后要改得地方多着呢 这不 OP 就改了吗？.
看项目就几个文件几个代码
我内心已经对这些毫无波澜
ma46
349 天前
如果你觉得这就是屎山, 那你是没见过真正的屎山
CodeCodeStudy
349 天前
还好吧，不算太烂
superchijinpeng
349 天前
代码没啥问题，简洁明了
harryWebb
349 天前
你这个得加钱，你还要重新写，也许这代码运行一段时间公司倒闭了就没了

也很正常

你没遇到过当年我遇到 php 得代码，和 html 混写，啥都混一起，那个是真的雷人
gimp
349 天前
能跑就行
HywelXu
349 天前
刚被一个烂尾项目坑过，说是维护就好，加几个接口而已。
没想到改着改着就等于重新写了一遍 既不讨客户喜，还不赚钱。
再也不要干这种活儿了。
haneki
349 天前
代码和人有一个能跑就行。

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

https://ex.noerr.eu.org/t/1079893

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

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

© 2021 V2EX