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
**提交**: `e27faf6c59c786fd0d6b4123593926e0cbeb73b4`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-19 19:27:38

---

## 1. 审查摘要
- **代码质量评分**：6.0 / 10 分
- **总体评价**：代码实现了自助台球/包厢业务的核心链路（状态判断、计费、支付路由、消息推送），业务逻辑覆盖较全。但存在严重的架构反模式（滥用 `$CI` 全局实例导致状态污染）、大量魔法值、SQL 拼接隐患以及不符合现代 PHP 规范的问题。代码末尾存在截断，部分自定义函数未提供源码，影响完整评估。
- **风险等级**：🔴 高（主要因全局状态污染、潜在 SQL 注入风险及高耦合架构导致）

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `check_room` / `get_room_package_list` 等多处 | **全局实例状态污染**：在 Model 中直接修改 `$CI->merchant_id`、`$CI->open_room_ignore_time_info`、`$CI->is_scan_open_room` 等属性。在并发请求、CLI 任务或同一请求多次调用时，会导致严重的数据串扰与逻辑错乱。 | 彻底移除对 `$CI` 属性的直接赋值。改为通过方法参数传递、返回值返回，或封装独立的 `RequestContext` / `DTO` 对象管理请求级状态。 | `// ❌ 错误：$CI->shop_id = $this->room_data['_shop_id'];`<br>`// ✅ 正确：将 shop_id 作为参数传入后续方法，或封装为配置数组返回。` |
| 🔴 严重 | `get_room_page_info` 约第 145-150 行 | **SQL 条件直接拼接**：`$where_str[] = 'FIND_IN_SET(' . $week . ', _week_cycle)';` 等使用字符串拼接构造 SQL。虽当前变量源自 `date()`，但违反安全编码规范，易被后续维护者误用导致注入，且不利于查询计划缓存。 | 使用框架查询构造器（Query Builder）或参数绑定（Prepared Statements）。 | `$this->db->where("FIND_IN_SET(?, _week_cycle)", $week);`<br>`$this->db->where("(_start_time <= ? AND _end_time >= ?)", [$hour_time, $hour_time]);` |
| 🟠 警告 | 全局多处 | **魔法值泛滥**：大量硬编码 `'1'`, `'2'`, `'-1'`, `'2333'` 表示业务状态、场景或错误码。可读性差，后期维护极易引发逻辑分支错误。 | 定义常量类或 PHP 8.1+ `Enum` 统一管理。例如 `class OperationalScene { const KTV = '1'; const BILLIARDS = '2'; ... }` | `if ($this->room_data['operational_scene'] === OperationalScene::TAVERN) { ... }` |
| 🟠 警告 | `check_room` 方法 | **实例属性缓存隐患**：`public $room_data = [];` 在方法内缓存数据。若 Model 实例被复用（如循环处理不同桌台或队列任务），将返回陈旧数据。 | 移除 `public` 可见性，改为私有属性，并在每次调用前重置，或改为纯方法局部变量。 | `private $currentRoomData = null;`<br>`// 方法入口: $this->currentRoomData = null;` |
| 🟠 警告 | `get_actual_pay` / `buy_room_package` 等 | **异常处理不规范**：使用全局函数 `throwError()` 而非 PHP 原生 `Exception`。不利于全局异常捕获、统一日志记录、API 标准化响应及事务回滚。 | 替换为 `throw new \InvalidArgumentException($msg, $code);` 或框架标准异常类，在 Controller 层统一 `try-catch` 并格式化输出。 | `throw new \RuntimeException('该包厢已关房，无法续费', 2333);` |
| 🟡 建议 | 全局 | **未遵循 PSR-12 规范**：缺少参数/返回值类型声明，未开启严格模式，方法可见性不一致，PHPDoc 注释不完整。 | 补充 `declare(strict_types=1);`，添加类型提示，统一命名与注释规范。 | `public function check_room_status(int $uid, int $merchant_id, array $params): array` |
| 🟡 建议 | `get_room_package_list` | **方法过长且职责混杂**：单方法超 150 行，混合了时间计算、套餐过滤、数组重组、配置读取等多重逻辑，违反单一职责原则（SRP）。 | 拆分为独立私有方法：`calculateAvailableTimeRange()`, `filterPackagesByConstraints()`, `buildHourOptions()` 等。 | *(重构建议见第 3 节)* |

## 3. 总结与行动建议

### 🔑 优先修复的关键问题
1. **解除 `$CI` 全局状态依赖**：这是当前架构的最大隐患。建议将 `check_room` 返回一个包含 `room_data`、`shop_id`、`merchant_id` 的数组或 DTO，后续方法通过参数接收，彻底切断 Model 与全局实例的隐式耦合。
2. **统一异常与错误处理**：将 `throwError()` 替换为标准 `Exception`，并在入口层（Controller/Router）配置全局异常处理器，确保错误码、堆栈信息不直接暴露给前端，同时支持事务自动回滚。
3. **消除 SQL 拼接隐患**：全面审查 `where` 条件构造逻辑，强制使用参数绑定或框架提供的 `where()` 链式调用，杜绝字符串拼接。

### 🛠 后续重构与优化方向
- **引入常量/枚举管理**：创建 `config/constants.php` 或独立的 `Enums/` 目录，集中管理 `RoomStatus`、`OperationalScene`、`PayPlatform`、`RedirectPage` 等业务标识，替换所有魔法值。
- **拆分臃肿方法**：`get_room_package_list` 和 `get_pay_page_info` 建议按“数据获取 -> 规则计算 -> 视图组装”三层拆分。例如将时间窗口计算、套餐过滤逻辑抽离至独立的 `RoomPackageCalculator` 服务类。
- **性能优化**：
  - `FIND_IN_SET` 在数据量大时会导致全表扫描。建议将 `_week_cycle` 改为 `TINYINT` 位运算字段，或使用关联表/JSON 字段配合索引优化。
  - 提取 `time()`、`date()` 调用为方法首行的局部变量（如 `$now = time();`），避免重复系统调用。
- **框架适配说明**：当前代码结构高度类似 **CodeIgniter 3**。若 `phpci` 为基于 CI 的定制框架，建议遵循其官方文档中关于 `Model` 生命周期与 `$CI` 实例使用的规范。若为自研框架，强烈建议引入依赖注入（DI）容器替代 `get_instance()`，以提升可测试性与架构清晰度。

> ⚠️ **局限性说明**：提供的代码在 `get_valid_room_package_coupon` 方法末尾截断，无法审查该方法的完整逻辑与返回值。此外，`throwError`、`send_mini_content`、`hourToTime` 等全局辅助函数未提供源码，本次审查基于其常规行为假设。建议补充完整代码以便进行更精准的边界条件与异常流分析。

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