这是一个创建于 73 天前的主题，其中的信息可能已经有所发展或是发生改变。
上周接手了个项目，老板说大部分项目都不能工作，本来是以维护的价格来处理接手这一段代码的
没想到坑比我想的还多，这是其中展示通用的一部分，大部分出于保密性就不能透露了
开篇惊喜
我滴妈欸，你说这是 Python 新手练手的代码我都信了，这是家公司的代码？？？
Interface 乱飞
怪不得会跑路
能跑就行的逻辑
Cookie 生成逻辑
给 User 生成 Cookie 用的，不是 用 个 JWT 会要了命吗？这么写干嘛啊
而且最草的一点，Cookie 存在 MYSQL 数据库 里面
大哥，这人一多不得打穿数据库啊（
每个请求都要往数据库请求一次，妈欸
硬编码 +++++
存心不给维护啊，写硬编码写了十几处
最后
已经在重新写这一套玩意了，真的太哈人了
第 1 条附言 · 73 天前
害怕，第一次发的贴子火起来了
大致看了下各位的回复，确实，代码评价之处确实带入了我自己修改的个人恩怨了，而且存在自身经验不足，并且认为“应当这样做才是合理的”，算是我自己经验不足的问题
发这贴子也是一时整的无语了 （实际上是接了三套 两客户界面 + 一 后端）
太多的硬编码改起来血压挺高的，莫名其妙查出来这一块硬编码，有一说一也挺头疼的，大晚上 DEBUG 为此血压高了几次
不过也说得对，能用就行，哎
大致看了下各位的回复，确实，代码评价之处确实带入了我自己修改的个人恩怨了，而且存在自身经验不足，并且认为“应当这样做才是合理的”，算是我自己经验不足的问题
发这贴子也是一时整的无语了 （实际上是接了三套 两客户界面 + 一 后端）
太多的硬编码改起来血压挺高的，莫名其妙查出来这一块硬编码，有一说一也挺头疼的，大晚上 DEBUG 为此血压高了几次
不过也说得对，能用就行，哎
114 条回复 • 2024-12-09 16:02:52 +08:00
 |
101
simo 73 天前
看着就像爬虫，可能各个平台爬虫教程太多了，入行门槛降低
|
 |
102
bler 73 天前
这个代码还行啊，main1,main2 应该是测试代码，我平常也是这样进行测试的，按照标准的测试流程纯属浪费时间。而且最最关键的是你这是爬虫项目，他已经把通用的东西给你提出来，你这还不满足，爬虫项目重复的东西本身就少，费心思封装就是浪费时间，面向对象是规则，那么它就是束缚，后续你发现需求又变了，你设计好的规范很大可能又会被打破，然而你的框架已经成型，你为了新得需求去适配你原来的框架，你会发现不亚于重新写一遍整个项目，所以爬虫项目我的建议就是提一些通用得东西作为工具类就行了。
cookie 存数据库也没问题，最重要的还是看你的实际访问量，没必要非得上 redis 。 硬编码确实有点不太好，这样不知道到底有多少东西需要配置，不够直观，但是看你这基本都是数据库和 api key 的东西，基本不会变，也不是不能接受 代码整体来讲没啥大问题，我都不知道你在抱怨什么，看你这架势估计是 java web 写多了，java 那条条框框确实多 |
 |
103
yulgang 73 天前
他“卖”出去的代码对方果然维护不了
 |
 |
104
elevioux 73 天前
代码写多了、看多了之后，感觉心如止水、能跑就行、继续拉屎。
年轻时候的折腾、重构、代码洁癖，都不及提前下班打游戏重要。 |
 |
105
cabing 73 天前
这个 main main1 main2 。。。。
面向防御性编程了，大学刚毕业也不会这么干。 |
 |
106
Perolong 73 天前
我接手过一个到手都是纯静态 html 的项目，然后原本以为只是改改样式，协助过审就行了，结果需求方这要改那要动,适配手机，加新页面，甚至说出了“大家来看看怎么样改的更好”，欲哭无泪，无奈只能说明情况，宽几天改成 react 不然满足不了那么多变的需求还有 router 要求,只能说干到最后都不是技术问题，是项目管理的问题了
|
 |
107
wnpllrzodiac 72 天前 via Android
没有懒代码，只有懒价位。给你 6 位数，小学生写的代码也连连叫香
|
 |
108
l4ever 72 天前
Interface 乱飞??
人家这里用了 pydantic, 挺好的. |
 |
109
yuruizhe 72 天前
py 代码，动态一时爽，维护火葬场，人来写，鬼来看，吼吼
|
 |
110
yc8332 72 天前
估计就是个简单业务，能跑就行。
|
 |
111
oom 71 天前
虽然...但是...，为什么账号或密码错误返回 201 状态码
|
 |
113
justfortest 16 天前
你是没讲过烂的
|
|
114
justfortest 16 天前
我接受过 go 项目，全是 interface 、magic number 、不明所以的变量名，真 tm 离谱
|
Select Language