代码质量改进 TODO 清单

创建日期: 2026-02-08
状态: 进行中

快速参考清单,用于跟踪代码质量改进任务。


🔴 高优先级(1-2周)

支付集成

  • 微信支付 SDK 集成 (private/platform/src/users/payment_service.py:112)

    • 安装 wechatpay-python SDK
    • 实现统一下单接口
    • 实现支付回调处理
    • 添加支付测试
  • 支付宝 SDK 集成 (private/platform/src/users/payment_service.py:138)

    • 安装 alipay-sdk-python
    • 实现统一下单接口
    • 实现支付回调处理
    • 添加支付测试
  • Stripe SDK 集成 (private/platform/src/users/payment_service.py:162)

    • 安装 stripe SDK
    • 实现支付接口
    • 实现 Webhook 处理
    • 添加支付测试

用户认证安全

  • 修复硬编码 user_id (shared/src/api/webhooks.py:418, 426) ✅ 已完成
    • ✅ 创建 get_current_user_id 函数支持多种认证方式(Bearer Token、API Key、Device ID)
    • ✅ 创建 require_user_id 依赖函数要求认证
    • ✅ 更新所有端点使用依赖注入获取真实 user_id
    • ✅ 添加权限验证(用户只能访问自己的 webhook)
    • ✅ 移除所有硬编码的 "user_123"

错误处理统一

  • 创建统一错误响应格式 (shared/src/utils/error_handler.py)

    • 定义 ErrorResponse 模型
    • 创建 APIError 异常类
    • 更新全局异常处理器
  • 改进用户服务错误处理 (private/platform/src/users/router.py)

    • 替换通用 Exception 为具体异常类型
    • 添加错误日志和追踪
    • 统一错误响应格式
  • 改进 Token 计费错误处理 (private/platform/src/users/token_billing.py)

    • 区分 Redis 错误和数据库错误
    • 添加错误追踪
    • 改进错误日志

🟡 中优先级(2-4周)

性能优化

  • 修复 N+1 查询问题 (private/platform/src/users/router.py)

    • 使用 selectinload/joinedload
    • 批量加载关联数据
    • 添加性能测试
  • 优化数据库连接池 (shared/src/storage/database.py)

    • 配置合适的 pool_size
    • 设置连接回收时间
    • 添加连接监控
  • Redis Pipeline 批量操作 (private/platform/src/users/token_billing.py)

    • 使用 Pipeline 批量更新缓存
    • 减少 Redis 往返次数
    • 添加性能测试
  • 启用 API 响应压缩 (shared/src/main.py)

    • 添加 GZipMiddleware
    • 配置压缩阈值
    • 测试压缩效果

缓存优化

  • 结算结果缓存 (shared/src/tasks/settlement_tasks.py:375)

    • 实现 Redis 缓存
    • 设置过期时间
    • 添加缓存失效策略
  • 统计结果缓存 (shared/src/tasks/statistics_tasks.py:164, 394)

    • 实现 Redis 缓存
    • 设置过期时间
    • 添加缓存失效策略

邮件通知完善

  • 从配置获取管理员邮箱 (shared/src/tasks/settlement_tasks.py:131)

    • 添加 ADMIN_EMAIL 配置项
    • 更新代码使用配置
    • 添加配置验证
  • 从数据库获取开发者邮箱 (shared/src/tasks/settlement_tasks.py:247, 251, 285, 289)

    • 查询用户表获取邮箱
    • 处理邮箱不存在的情况
    • 添加错误处理

订阅管理

  • Webhook 订阅状态更新 (shared/src/api/webhook.py:283, 357)

    • 实现订阅状态更新逻辑
    • 发送通知
    • 添加测试
  • 订阅功能实现 (shared/src/api/subscription_enhanced.py:162)

    • 更新数据库
    • 实现订阅逻辑
    • 添加测试

Celery 任务

  • 实现订阅到期扫描 (shared/src/tasks/celery_tasks.py:175)

    • 扫描到期订阅
    • 发送到期通知
    • 处理订阅续费
  • 实现其他 Celery 任务 (shared/src/tasks/celery_tasks.py:231, 280, 406)

    • 实现具体业务逻辑
    • 添加错误处理
    • 添加测试

🟢 低优先级(持续改进)

前端功能

  • 实现编辑功能 (private/platform/src/api_admin/templates/terminals.html:325)
  • 实现对话加载 (private/platform/src/api_admin/templates/dashboard.html:218)
  • 改进通知组件 (private/platform/src/api_admin/templates/base.html:150)

用户信息

  • 获取真实用户姓名 (public/education/backend/api/course_review_api.py:243, 331)
    • 从用户系统查询
    • 处理姓名不存在的情况
    • 添加缓存

统计功能

  • 统计绑定专家的终端数 (private/platform/src/api_admin/terminals.py:157)
    • 实现统计查询
    • 添加索引优化
    • 添加缓存

通知功能

  • 实现邮件/短信通知 (private/platform/src/api_admin/sub_accounts_api.py:2154)
    • 集成邮件服务
    • 集成短信服务
    • 添加通知模板

性能监控

  • 从性能监控系统获取数据 (private/platform/src/feedback/developer_feedback.py:532)
    • 集成性能监控 API
    • 获取性能指标
    • 添加错误处理

📊 进度统计

总体进度

  • 高优先级: 1/6 完成 (17%)
  • 中优先级: 0/12 完成 (0%)
  • 低优先级: 0/6 完成 (0%)
  • 总计: 1/24 完成 (4%)

按类别统计

  • 支付集成: 0/3 (0%)
  • 安全改进: 1/1 (100%) ✅
  • 错误处理: 0/3 (0%)
  • 性能优化: 0/4 (0%)
  • 缓存优化: 0/2 (0%)
  • 功能完善: 0/11 (0%)

🔗 相关文档


最后更新: 2026-02-08