Continue Coding Assistant 是你的智…

# users.py 重构方案：抽取公共数据库查询方法

前提说明

我无法访问你提到的具体 GitHub 仓库和 users.py 的完整源码，以下基于 Flask 2.0 + 常见的 SQLAlchemy 写法给出可落地的重构模板。请根据你的实际代码调整字段与模型名。

重构思路

• 识别 users.py 中重复出现的查询模式（如按 ID 查、按字段查、列表查询）
• 统一抽取为一个通用的公共查询方法，减少重复代码
• 保持函数签名向后兼容，避免影响现有路由

公共方法示例（兼容 Flask 2.0 / SQLAlchemy）

```python

from flask import abort

from models import db, User

class UserService:

@staticmethod

def get_one(filters, error_code=404):

"""通用单条查询：filters 为 SQLAlchemy 过滤条件列表"""

result = User.query.filter(*filters).first()

if result is None:

abort(error_code)

return result

@staticmethod

def get_list(filters=None, order_by=None, limit=None):

"""通用列表查询"""

query = User.query

if filters:

query = query.filter(*filters)

if order_by is not None:

query = query.order_by(order_by)

if limit:

query = query.limit(limit)

return query.all()

# 路由中的用法（保持原逻辑不变）

@user_bp.route('/<int:user_id>')

def get_user(user_id):

user = UserService.get_one([User.id == user_id])

return jsonify(user.to_dict())

```

重构要点清单

• 保持路由签名不变：仅替换内部实现，接口行为不变
• 异常处理统一：查询为空时统一 abort(404)，或按需自定义错误处理
• 不要在方法里直接 return jsonify：查询层与视图层解耦，方便复用和测试
• 兼容性确认：Flask 2.0 + SQLAlchemy 1.4/2.x 语法基本一致；若项目用的是原生 SQL（db.engine.execute），需注意 1.4 中该写法已变更
• 补充单元测试：重构前后用 pytest 对比查询结果一致性

建议下一步

如果你把 users.py 的重复代码片段贴出来，我可以给出针对性的精确重构版本。