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-260519
**提交**: `a1f205967928b1d283141eb90b17072f62877ee3`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-21 15:56:15

---

## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码具备完整的业务闭环（支付回调、退款、下单、消息推送），但存在明显的架构臃肿问题。核心方法（如 `refund_by_notify`）承担职责过多，混合了支付网关交互、数据库事务、日志记录与第三方通知。部分逻辑存在并发安全隐患与 SQL 拼接风险，且未充分利用现代 PHP 特性与框架最佳实践。
- **风险等级**：🔴 高（存在 SQL 注入隐患、库存超卖风险、事务状态不一致可能）

> 📌 **框架说明**：根据目录结构、`$CI = &get_instance()`、`$this->load->model()` 及 `$this->db->trans_start()` 等特征，判定为 **CodeIgniter 3.x** 架构。若为内部定制框架 `phpci`，请对照官方文档调整组件调用方式。
> ⚠️ **局限性说明**：末尾 `create_community_shop_book_order` 方法代码被截断，本次审查仅基于已提供片段进行静态分析。

---

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `refund_by_notify` ~L145 | **SQL 注入风险**：使用字符串拼接构造 WHERE 条件 `$log_where = '_relation_id="' . $order_data['_id'] . '" ...'`，若传入数据未严格过滤，将导致注入。 | 废弃字符串拼接，统一使用 CI 查询构造器或参数绑定。 | `$this->db->where('_relation_id', $order_data['_id'])<br>->where('_status', 1)<br>->where_in('_type', [5, 13])<br>->update('pay_log', $update_data);` |
| 🔴 严重 | `check_notify` ~L50-80 | **并发超卖风险**：`check_remain_sale_num` 校验库存后直接执行后续扣减，未加行级锁或分布式锁，高并发下极易超卖。 | 引入数据库行级锁 (`SELECT ... FOR UPDATE`) 或 Redis 原子操作保障库存扣减一致性。 | `$this->db->where('_id', $id)->get(..., NULL, NULL, TRUE); // CI3 支持 lock('FOR UPDATE')` |
| 🔴 严重 | 文件顶部 L7-8 | **全局实例化隐患**：`$CI = &get_instance();` 在类外部执行。若文件被提前 `include` 或 CLI 环境加载，将触发致命错误。 | 移除顶部代码。Model 内部应直接使用 `$this->load` 或 `$this->config`。 | 删除顶部两行，内部改用 `$this->load->model()` 或 `$this->config->item()` |
| 🟠 警告 | `check_notify` & `refund_by_notify` | **事务控制不规范**：`try-catch` 中手动 `trans_rollback()` 与 CI 自动回滚机制混用，且 `trans_complete()` 仅在 `else` 分支调用，易导致事务状态残留或重复回滚报错。 | 依赖 CI 的 `trans_start()` / `trans_complete()` 自动机制，移除显式 `trans_rollback()`；失败时直接 `return`，CI 会在脚本结束时自动回滚。 | 移除 `try` 内的 `$this->db->trans_rollback();`，统一在方法末尾调用 `$this->db->trans_complete();` |
| 🟠 警告 | `get_list` ~L200 | **循环查询性能损耗**：虽做了内存缓存 `$merchant_business_model`，但首次仍为多次 `get_one` 查询。数据量大时拖慢响应。 | 提取所有 `merchant_id`，使用 `where_in` 批量查询后映射。 | `$ids = array_column($order_info, 'merchant_id');<br>$merchants = $this->ahead_merchant_model->where_in('_id', $ids)->get()->result_array();` |
| 🟠 警告 | 全局多处 | **重复加载模型/配置**：方法内部频繁 `$this->load->model()`，降低可读性且违反依赖注入原则。CI 虽会缓存，但增加解析开销。 | 在 `__construct()` 中集中加载高频模型，或采用 Service 层解耦。 | `public function __construct() { parent::__construct(); $this->load->model(['ahead_merchant_model', 'ahead_vip_model']); }` |
| 🟡 建议 | 全局 | **命名规范与 PSR-12 冲突**：方法名使用 `snake_case`，数组混用 `array()` 与 `[]`，缺少 PHP 7+ 类型声明。 | 遵循 PSR-12，统一使用 `camelCase`，添加类型提示，清理历史注释代码。 | `public function checkNotify(string $orderId, string $transactionId = ''): array` |
| 🟡 建议 | `check_notify` ~L115 | **异常日志泄露敏感信息**：`json_encode($e->getTrace(), 256)` 记录完整堆栈，可能暴露服务器路径、配置或用户隐私。 | 仅记录异常消息、行号及脱敏后的业务主键。 | `doLog('支付失败: ' . $e->getMessage() . ' | Order: ' . $order_id, 'book_order');` |
| 🟡 建议 | `refund_by_notify` | **单一职责违背 (SRP)**：退款逻辑超 200 行，混合支付网关、DB 更新、财务流水、消息推送。维护成本极高。 | 拆分为独立 Service：`RefundService`、`PaymentGatewayService`、`NotificationService`。Model 仅负责数据持久化。 | 将退款核心流程抽离至 `app/services/RefundService.php`，Model 仅暴露 `updateStatus()` 等方法。 |

---

## 3. 总结与行动建议

### 🔑 优先修复项（P0/P1）
1. **修复 SQL 注入漏洞**：立即替换 `refund_by_notify` 中的字符串拼接 WHERE 条件，全面改用 CI Query Builder 或预处理语句。
2. **解决库存超卖问题**：在 `check_notify` 的库存校验环节引入 `SELECT ... FOR UPDATE` 行级锁，或迁移至 Redis `DECR` + Lua 脚本实现原子扣减。
3. **规范事务生命周期**：清理 `try-catch` 中冗余的 `trans_rollback()`，严格遵循 `trans_start() -> 业务逻辑 -> trans_complete()` 模式，避免事务状态污染。

### 🛠 后续重构与优化方向
1. **架构分层解耦**：当前 Model 承担了 Controller 和 Service 的职责。建议引入 Service 层处理复杂业务流（如支付回调、退款、消息推送），Model 仅保留 CRUD 与基础查询。
2. **统一编码规范**：
   - 启用 `declare(strict_types=1);`
   - 方法名统一转为 `camelCase`（如 `checkNotify`、`refundByNotify`）
   - 清理 `//add by nan 18.1.22` 等历史注释及大段注释代码，必要时提交至 Git 历史保留。
3. **性能与可观测性提升**：
   - 将循环内的单条查询改为批量 `WHERE IN` 查询。
   - 关键业务节点（支付回调、退款发起）增加结构化日志（如 JSON 格式），便于 ELK 等日志系统检索与告警。
   - 对 `send_success_msg` 中的微信模板消息与短信发送改为**异步队列**处理，避免阻塞主支付回调流程。

> 💡 **提示**：若 `phpci` 为内部定制框架且与 CI3 行为存在差异（如事务自动回滚机制、查询构造器语法），请以官方文档为准调整上述建议。代码末尾截断部分建议补充完整后再次提交审查。

---
*此 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 277 in issue