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
**提交**: `d712d39b4237953f9e9cd9a4623af65a0e6a1259`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-25 15:29:04

---

## 1. 审查摘要
- **代码质量评分**：6.0 / 10 分
- **总体评价**：业务逻辑覆盖较全面，能支撑 AI 预订、多业态（KTV/台球/棋牌/酒馆）套餐推荐与支付流程。但代码存在明显的架构反模式：大量使用公开属性存储中间状态、频繁修改全局 `$CI` 对象、静态缓存未隔离导致跨店数据污染、超长方法违反单一职责原则。性能与可维护性存在较大优化空间。
- **风险等级**：🟠 中高（存在数据隔离缺陷、全局状态污染、潜在 N+1 查询瓶颈）

---

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_merchant_room_type_model.php` L15, L28-40 | **静态缓存未隔离导致跨店数据污染**：`self::$room_type_data` 为扁平数组，首次查询门店A后，后续查询门店B会直接返回门店A的缓存数据。 | 移除静态缓存或改为带唯一键的缓存结构，或交由框架缓存组件管理。 | `if (!isset(self::$room_type_data[$merchant_id . '_' . $shop_id])) { ... }` `return self::$room_type_data[$merchant_id . '_' . $shop_id] ?? [];` |
| 🔴 严重 | `Ahead_ai_book_model.php` `Ahead_billiards_model.php` 文件头部 L1-3 | **类外部使用 `$CI = &get_instance();` 违反框架规范**：在模型文件顶部直接获取实例并加载模型，易引发未初始化错误，且破坏 CI 生命周期。 | 删除文件头部代码。依赖加载应移至 `__construct()` 或具体业务方法内。 | `// 删除顶部代码` `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🟠 警告 | `Ahead_ai_book_model.php` L18-27 | **公开属性滥用存储中间状态**：`$time_eligible_package_data` 等大量公开数组用于累积计算结果，破坏封装性，若实例复用会导致状态残留。 | 改为 `private` 属性，并在方法入口处显式重置，或改为局部变量通过返回值传递。 | `private $time_eligible_package_data = [];` `// 方法开头：$this->time_eligible_package_data = [];` |
| 🟠 警告 | `Ahead_ai_book_model.php` L102, L360 `Ahead_billiards_model.php` L52 | **频繁修改 `$CI` 超全局属性**：如 `$CI->operational_scene = $operational_scene;` 污染全局上下文，极易引发并发请求或后续逻辑的隐蔽冲突。 | 避免修改 `$CI`。应通过方法参数传递业务上下文，或使用独立的 `Context` 对象。 | 移除 `$CI->xxx = ...`，改为参数传递或依赖注入。 |
| 🟠 警告 | `Ahead_ai_book_model.php` L250-260 | **循环逐日查询引发性能瓶颈**：`for ($i = 0; $i < 7; $i++)` 每次循环调用 `_get_date_room_list_info`，触发完整模型加载与 DB 查询，存在严重 N+1 问题。 | 改为批量查询未来 7 天数据，或在 SQL 层使用 `BETWEEN` 一次性拉取，内存中过滤。 | `WHERE book_date BETWEEN ? AND ?` 替代循环查询。 |
| 🟠 警告 | `Ahead_ai_book_model.php` L130-160 | **手动括号匹配解析 JSON 脆弱**：依赖 `strpos` 和计数器提取 JSON，若 AI 返回嵌套字符串或格式微调，极易解析失败或越界。 | 使用正则提取或要求 AI 严格输出 JSON 块，配合 `json_decode` 与异常捕获。 | `preg_match('/book_params=\s*(\{.*\})/s', $str, $m);` `$data = json_decode($m[1], true);` |
| 🟡 建议 | 全局多处 | **重复加载模型/库**：每个方法内部频繁调用 `$this->load->model()`，浪费 I/O 与内存。 | 将高频依赖移至构造函数，或配置框架自动加载。 | `public function __construct() { parent::__construct(); $this->load->model(['xxx','yyy']); }` |
| 🟡 建议 | `Ahead_billiards_model.php` L180-200 | **存在大量注释死代码**：如 `/*if ($result['operational_scene'] == '2')...*/` 影响可读性与静态分析。 | 彻底清理注释掉的代码，历史版本交由 Git 管理。 | 直接删除注释块。 |
| 🟡 建议 | 全局 | **违反 PSR-12 与类型规范**：命名不统一（`minToStr` vs `mintoStr`）、使用 `array()`、缺乏类型声明、方法过长（>300行）。 | 统一驼峰/蛇形命名，使用 `[]`，为 PHP 7.4+ 添加类型提示，拆分超长方法。 | `public function get_page_info(int $uid, int $mid, int $sid, array $params): array` |

---

## 3. 总结与行动建议

### 🔑 优先修复的关键问题
1. **修复静态缓存隔离缺陷**：立即修改 `Ahead_merchant_room_type_model::get_shop_room_type_list` 中的 `self::$room_type_data`，按 `merchant_id_shop_id` 键值隔离，否则会导致严重的跨店数据错乱。
2. **清理全局状态污染**：移除所有 `$CI = &get_instance();` 及 `$CI->xxx = ...` 赋值。业务上下文（如 `operational_scene`、`merchant_id`）应通过方法参数或独立的 `RequestContext` 对象传递。
3. **优化 7 天循环查询**：将 `get_ai_book_recommend` 中的 `for ($i=0; $i<7; $i++)` 替换为单次范围查询（如 `WHERE date >= CURDATE() AND date <= DATE_ADD(CURDATE(), INTERVAL 7 DAY)`），在 PHP 层按日期分组处理，预计可降低 60%+ DB 交互耗时。

### 🛠 后续重构与优化方向
- **遵循单一职责原则 (SRP)**：`get_ai_book_recommend` 方法已超 400 行，建议拆分为：
  - `resolve_booking_time()`：处理时间解析、模糊时间转换、卡券时间校验。
  - `fetch_available_rooms()`：批量获取空闲包厢与套餐。
  - `calculate_recommendation_priority()`：封装推荐权重排序逻辑。
- **引入数据校验层**：当前 `$params` 仅使用 `??` 默认值，建议在入口增加严格校验（如 `book_date` 格式、`book_hour` 范围），可使用框架自带的 `Form_validation` 或自定义 DTO 类。
- **统一异常处理**：`throwError()` 若为自定义函数，建议统一替换为 `throw new \DomainException()` 或 `BusinessException`，便于全局捕获与标准化错误码返回。
- **框架适配说明**：代码结构高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请确认其是否支持依赖注入容器（DIC）或 PSR-11 规范。若支持，建议逐步将 `$this->load->model()` 替换为构造函数注入，以提升可测试性与生命周期可控性。

> 💡 **提示**：`Ahead_billiards_model.php` 末尾代码被截断，若包含支付回调或订单状态机逻辑，请补充完整以便进行事务一致性审查。当前审查基于已提供片段，建议结合完整调用链进行集成测试。

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