Files

T

claude_dev 510f77e6ea fix(backtest): result_id 用 DB 行 id + equity/trades 落 JSON（S1.1+S1.2）

- BacktestResult 加 id；save_result 设 result.id=lastrowid（修 get_result bug）
- runner._on_done 用 result.id（getattr 兜底 FactorReport）
- cta_engine 构建 equity_curve/trades DataFrame；save 传 file_dir
- result_store parquet→JSON（去 pyarrow 依赖，本机/容器都稳）
- 16 tests passed

2026-07-07 06:06:10 +08:00

2.1 KiB

Raw Blame History

Phase 2 代码审查报告

日期: 2026-07-03 审查人: Claude (Main Agent) 审查范围: Phase 2 新增代码

审查文件

文件	行数	状态
`sanguo_web/api/routes/settings.py`	118	✅ 通过
`sanguo_web/api/routes/accounts.py`	98	✅ 通过
`sanguo_web/api/routes/trades.py`	164	✅ 通过
`sanguo_web/static/js/app.js`	1132	✅ 通过
`sanguo_web/templates/index.html`	1085	✅ 通过

审查结果

✅ 通过项

1. 代码质量

✓ 命名规范清晰
✓ 代码结构合理
✓ 注释充分
✓ 类型提示完整

2. 安全性

✓ 依赖注入 (Depends(get_current_user)) 确保认证
✓ 输入验证 (validate_settings)
✓ 错误处理完善 (try/except, HTTPException)
✓ 敏感信息保护（不返回明文密码）

3. 性能

✓ 查询效率合理（使用 get() 避免 KeyError）
✓ 列表推导式使用得当
✓ 数据分页支持 (/latest?limit=50)

4. 一致性

✓ 与项目现有代码风格一致
✓ API 响应格式统一
✓ 错误处理模式一致

观察到的小问题（非阻塞）

1. settings.py

# Line 39-42: 可能的性能问题
for key, value in SETTINGS.items():
    settings_dict[key] = value

建议: 如果配置项很多，可以考虑使用 dict(SETTINGS) 直接复制

2. trades.py

# Line 63: 潜在的类型问题
key=lambda x: x.get("time", datetime.min),

建议: 确保 time 字段类型一致性

3. accounts.py

# Line 85-87: 可能的精度问题
total_balance = sum(acc.get("balance", 0.0) for acc in accounts)

建议: 金融计算建议使用 decimal.Decimal

审查结论

总体评价: ✅ 通过审查

代码质量良好，无明显缺陷。观察到的问题都是优化建议，不影响当前功能。

建议: 可以合并到主分支。

下一步

修复建议的小问题（可选）
运行完整的集成测试
更新文档
标记 Phase 2 为完成

2.1 KiB Raw Blame History Unescape Escape