代码质量改进 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