SQLite Web: gitea.db

id

Primary key. INTEGER NOT NULL

repo_id

INTEGER

index

INTEGER

poster_id

INTEGER

original_author

TEXT

original_author_id

INTEGER

name

TEXT

content

## 自动代码审查报告

**分支**: pay-260616
**提交**: `07ec3b59147682933f558d7b84f1de5ec605c784`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-01 20:03:20

---

## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码业务逻辑完整，覆盖了预订、支付回调、退款、消息推送等核心链路，具备较强的工程落地能力。但存在**事务管理不规范、SQL 拼接注入风险、循环内查询（N+1）、魔法数字泛滥**等典型问题。部分支付回调逻辑缺乏安全校验，静态缓存使用方式在特定运行环境下存在隐患。
- **风险等级**：🟠 中高风险（主要源于 SQL 注入隐患、支付回调安全缺失及事务状态机冲突）

> 📌 **框架说明**：根据目录结构（`system/`、`application/models/`）及 `$CI =& get_instance()`、`$this->load->model()` 等特征，判定该项目实际基于 **CodeIgniter 3.x** 框架。以下审查将基于 CI3 最佳实践进行。若实际为其他框架，请补充说明。

---

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_book_order_model.php` `refund_by_notify` 方法 | **SQL 注入风险**：使用字符串拼接构造 `WHERE` 条件与 `UPDATE` 字段（如 `$log_where = '_relation_id="' . $order_data['_id'] . '"...'`），若上游数据未严格过滤，将导致注入。 | 废弃字符串拼接，全面改用 CI 查询构建器（Query Builder）或参数绑定。 | `$this->ahead_pay_log_model->where('_relation_id', $order_data['_id']) ->where('_status', 1) ->where_in('_type', [5, 13]) ->update(['_status' => 4, '_refund_amount' => '_actual_pay', ...]);` |
| 🔴 严重 | `Ahead_book_order_model.php` `check_notify` 方法 | **事务状态机冲突**：CI3 的 `trans_start()` 与 `trans_complete()` 为配对设计，内部已自动处理回滚。在 `try` 块中手动调用 `trans_rollback()` 会破坏 CI 事务状态机，可能导致后续 `trans_complete()` 误提交或抛出异常。 | 改用 `trans_begin()` 显式控制，或移除 `try-catch` 中的手动回滚，依赖 `trans_complete()` 自动回滚机制。 | `$this->db->trans_begin(); try { // 业务逻辑... if (!$success) { $this->db->trans_rollback(); return [...]; } $this->db->trans_commit(); } catch (\Exception $e) { $this->db->trans_rollback(); return [...]; }` |
| 🔴 严重 | `Ahead_book_order_model.php` `check_notify` 方法 | **支付回调缺乏签名校验**：直接处理 `$order_id` 和 `$transaction_id` 并执行退款/入库，未体现支付平台签名验证逻辑，易受伪造回调攻击导致资损。 | 在方法入口处增加支付网关签名校验，校验失败直接返回并记录日志。 | `if (!$this->verify_pay_sign($this->input->post())) { doLog('支付回调签名验证失败', 'pay_callback'); return ['status' => false, 'msg' => '签名错误']; }` |
| 🟠 警告 | `Ahead_book_order_model.php` `get_list` 方法 | **N+1 查询性能瓶颈**：在 `foreach` 循环中调用 `$this->ahead_merchant_model->get_one()`，订单量增大时将导致数据库连接数飙升与响应延迟。 | 提取所有 `merchant_id`，使用 `where_in` 批量查询后构建映射数组。 | `$ids = array_column($order_info, 'merchant_id'); $merchants = $this->ahead_merchant_model->where_in('_id', $ids)->get()->result_array(); $merchant_map = array_column($merchants, '_business_model', '_id'); foreach ($order_info as &$v) { $v['business_model'] = $merchant_map[$v['merchant_id']] ?? '1'; }` |
| 🟠 警告 | `Ahead_shop_book_time_info_model.php` 全局静态属性 | **静态缓存潜在内存泄漏/脏数据**：大量使用 `self::$shop_data`、`self::$date_room_book_time_info` 等静态属性缓存。在 PHP-FPM 下虽随请求重置，但在 CLI 任务、Swoole/Workerman 长连接或单元测试中极易引发内存溢出或数据串扰。 | 改为实例属性（`private $xxx = []`），或统一使用 CI Cache 驱动（Redis/Memcached）管理。 | `private $shop_data = []; // 在方法中通过 $this->shop_data 读写，避免跨请求污染。` |
| 🟠 警告 | `Ahead_book_order_model.php` `send_success_msg` 方法 | **敏感信息日志泄露**：`doLog('支付失败' . json_encode($e->getTrace(), 256), 'book_order')` 会打印完整堆栈，可能暴露服务器路径、数据库结构或内部配置。 | 仅记录异常消息与关键业务标识，堆栈信息应脱敏或仅输出到调试环境。 | `doLog('支付失败: ' . $e->getMessage() . ' | OrderID: ' . ($order_data['_id'] ?? 'unknown'), 'book_order');` |
| 🟡 建议 | 全局 | **魔法数字泛滥**：状态码 `-1, 1, 2, 3, 4, 5`、支付平台 `1, 3, 14`、时间常量 `3600, 86400` 硬编码在业务逻辑中，可读性与可维护性差。 | 提取为类常量或独立配置文件，统一维护。 | `const STATUS_PENDING = -1; const STATUS_PAID = 1; const REFUND_LIMIT_HOURS = 1;` |
| 🟡 建议 | `Ahead_book_order_model.php` 文件顶部 | **CI 实例获取位置不当**：`$CI = &get_instance();` 放在类外部，每次 `include` 该文件都会执行一次，违反 CI 生命周期规范且浪费性能。 | 移至构造函数或具体方法内部按需获取。 | `public function __construct() { parent::__construct(); $this->ci = &get_instance(); }` |
| 🟡 建议 | 全局 | **冗余代码与过时注释**：存在大量注释掉的代码块（如 `send_success_msg` 中的旧版短信逻辑）及带人名的历史注释（`//add by nan 18.1.22`）。 | 使用 Git 管理历史，清理无用注释，保持代码库整洁。 | 直接删除 `/* ... */` 及 `//add by...` 注释块。 |

---

## 3. 总结与行动建议

### 🔑 优先修复项（P0/P1）
1. **修复 SQL 拼接漏洞**：立即将 `refund_by_notify` 中的 `$log_where` 和 `$log_up` 字符串拼接替换为 CI Query Builder 链式调用，彻底阻断注入路径。
2. **规范事务管理**：统一使用 `$this->db->trans_begin()` / `trans_commit()` / `trans_rollback()` 显式控制事务，避免与 CI 自动事务机制冲突导致数据不一致。
3. **补充支付回调验签**：在 `check_notify` 入口增加支付平台签名校验逻辑，确保回调请求来源合法后再执行订单状态变更。

### 🛠 后续重构与优化方向
1. **架构解耦**：当前 Model 承载了过多业务逻辑（支付、退款、短信、微信模板消息、库存扣减）。建议将支付网关交互、消息推送、库存校验抽离至独立的 `Service` 层，Model 仅负责数据持久化。
2. **性能治理**：
   - 消除 `get_list` 等方法的循环查询，全面采用批量查询+内存映射。
   - `Ahead_shop_book_time_info_model` 中的时间区间计算逻辑极其复杂，建议引入 `DateInterval` 或专业时间处理库（如 `nesbot/carbon`），并将高频计算结果缓存至 Redis。
3. **规范与可维护性**：
   - 建立项目级常量文件（如 `application/config/constants.php`）统一管理状态码、支付渠道、时间阈值。
   - 统一错误处理机制，避免混用 `return ['status'=>false]` 与 `throwError()`，建议封装统一的 `ApiResponse` 类。
4. **框架适配提示**：若项目计划升级至 CI4 或迁移至现代框架（如 Laravel/Symfony），当前大量依赖 `$CI =& get_instance()` 和全局静态缓存的代码将面临较大重构成本。建议尽早向依赖注入（DI）与实例化设计模式靠拢。

> ⚠️ **局限性说明**：提供的代码片段在 `create_community_shop_book_order` 与 `get_book_day_time_info` 方法末尾被截断，部分边界条件处理、异常捕获及完整事务链路未能全面评估。建议补充完整代码或提供对应方法的上下文以便进行深度复核。

---
*此 Issue 由代码审查服务自动创建*

TEXT

milestone_id

INTEGER

priority

INTEGER

is_closed

INTEGER

is_pull

INTEGER

num_comments

INTEGER

ref

TEXT

deadline_unix

INTEGER

created_unix

INTEGER

updated_unix

INTEGER

closed_unix

INTEGER

is_locked

INTEGER NOT NULL (default 0

content_version

INTEGER NOT NULL (default 0

time_estimate

INTEGER NOT NULL (default 0

Update row 445 in issue