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
**提交**: `b43d085d86dcd31d3597c0311377317a67d79671`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-28 16:59:00

---

> 📌 **框架说明**：根据目录结构 (`system/`, `application/`)、`$CI = &get_instance()` 语法及加载方式，该代码实际基于 **CodeIgniter 3** 框架。以下审查将基于 CI3 架构规范与现代 PHP 最佳实践进行。若 `phpci` 为内部定制分支，请同步核对底层差异。

## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖全面，但模型承担了配置加载、时间计算、团购券校验、UI状态标记等多重职责，严重违反单一职责原则（SRP）。存在静态变量跨请求污染、全局 `$CI` 状态篡改、时间格式混用及大量魔法值等隐患。代码片段在 `_get_un_book_time` 方法中途截断，部分逻辑无法完整评估。
- **风险等级**：🔴 高

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 类级别 / `self::$xxx` 静态属性 | **静态缓存导致跨请求数据污染**：`self::$book_days_info`、`self::$shop_data` 等静态变量在 PHP-FPM 进程生命周期内持久化。若同一 Worker 处理不同 `merchant_id` 的请求，将直接返回脏数据，引发严重业务错乱。 | 移除 `static` 关键字，改为实例级缓存（如 `$this->cache['shop_data']`）或按 `merchant_id`+`shop_id` 组合键缓存。请求结束后自动释放。 | `// 错误：public static $shop_data = [];` `// 正确：protected $cache = [];` `if (!isset($this->cache['shop_data'])) { ... }` |
| 🔴 严重 | `set_shop_config()` / `set_room_info()` | **直接修改全局 `$CI` 属性引发副作用**：`$CI->operational_scene = $room_operational_scene;` 篡改了 CI 超级对象状态。在并发或后续中间件/控制器中极易引发不可预知的逻辑覆盖。 | 避免直接赋值全局对象。应通过方法返回值传递，或使用 CI 配置项 `$this->config->set_item()`，或注入到当前实例属性中。 | `// 错误：$CI->operational_scene = $room_operational_scene;` `// 正确：$this->book_room_operational_scene = $room_operational_scene;` `return $this->book_room_operational_scene;` |
| 🟠 警告 | `get_book_day_time_info()` 循环体 | **状态覆盖与优先级混乱**：连续多个 `if ($v['status'] == '1')` 判断，后序条件可能覆盖前序已标记的 `status` 或 `notice_type`，且缺乏明确的优先级控制与短路机制。 | 引入状态优先级常量，使用 `continue` 提前终止已判定为不可用的时间点；或抽离为独立的状态机/规则链。 | `foreach ($time_info as $k => &$v) {` ` if ($v['status'] !== '1') continue; // 已不可用则跳过` ` // 按优先级依次判断...` `}` |
| 🟠 警告 | 多处时间处理逻辑 | **时间格式混用与重复计算**：`Ymd`、`YmdHi`、`H:i`、时间戳、自定义字符串混用，且 `strtotime($date)`、`date('Ymd')` 在循环/分支中重复调用，易引发跨天/时区边界 Bug。 | 统一使用 `DateTimeImmutable` 或纯时间戳进行计算，仅在最终输出时格式化。缓存基础时间戳变量。 | `$date_ts = strtotime($date);` `$today_ts = strtotime(date('Y-m-d'));` `// 后续全部基于 $date_ts 进行加减运算` |
| 🟠 警告 | `__construct()` 及多处 | **重复加载模型与冗余 `$CI` 获取**：`$CI = &get_instance()` 在文件顶部、构造函数、多个方法中重复声明；`$this->load->model()` 在方法内频繁调用。虽 CI3 有加载缓存，但严重影响可读性与执行效率。 | 在构造函数中统一加载依赖模型与库；移除冗余的 `$CI` 声明，直接使用 `$this->load` 或 `$this->ci`（若父类提供）。 | `public function __construct() {` ` parent::__construct();` ` $this->load->model(['ahead_shop_model', 'ahead_shop_config_second_model']);` ` $this->load->library('Tuangou');` `}` |
| 🟡 建议 | 类属性与方法定义 | **违反 PSR-12 与单一职责原则**：属性命名混用 `snake_case` 与 `camelCase`；大量 `public` 属性暴露内部状态；模型同时处理配置、时间计算、团购校验、UI 状态标记。 | 遵循 PSR-12：方法/属性统一使用 `camelCase`，内部属性设为 `protected`。按职责拆分为 `ShopConfigService`、`BookingTimeCalculator`、`VoucherRuleValidator`。 | `// 属性规范` `protected $defaultHours = [];` `protected $bookTimeLimit = 3600;` `// 方法规范` `public function getBookDaysInfo(...)` |
| 🟡 建议 | 全文件 | **魔法值与硬编码散落**：`'1'`、`'2'`、`'3'`、`'merchantApp'`、`'-1'` 等状态码与场景标识直接硬编码，维护成本极高。 | 定义类常量或枚举类（PHP 8.1+），集中管理业务字典。 | `const SCENE_KTV = '1';` `const SCENE_BILLIARDS = '2';` `const STATUS_AVAILABLE = '1';` `const STATUS_UNAVAILABLE = '-1';` |

## 3. 总结与行动建议

### 🚨 优先修复的关键问题
1. **清除静态状态污染**：立即将 `self::$xxx` 静态属性改为实例属性或请求级缓存，防止多商户并发请求下的数据串扰。
2. **隔离全局 `$CI` 修改**：停止直接赋值 `$CI->xxx`，改为通过方法返回值、配置项或依赖注入传递上下文，确保请求隔离性。
3. **统一时间计算基线**：将所有时间操作收敛为时间戳或 `DateTime` 对象，消除字符串格式混用带来的隐式转换风险。

### 🛠 后续重构与优化方向
- **架构拆分（SRP）**：当前模型已演变为“上帝类”。建议将业务逻辑拆分为：
  - `ShopConfigLoader`：负责读取门店配置、场景映射。
  - `BookingTimeEngine`：专注时间片计算、冲突检测、跨天逻辑。
  - `VoucherRuleChecker`：封装团购券可用时间、星期、时长限制等规则。
- **规则链模式替代嵌套 `if`**：将 `get_book_day_time_info` 中的状态判定改为责任链或策略模式。每个规则返回 `['status' => '1', 'reason' => '']`，主流程按优先级合并结果，提升可测试性与扩展性。
- **输入校验与异常处理**：对 `$params` 中的 `date`、`merchant_id`、`shop_id` 增加类型与格式校验。将 `throwError()` 替换为标准 `throw new \InvalidArgumentException()` 或 CI3 的 `show_error()`，避免敏感路径泄露。
- **性能优化**：`array_intersect`、`mergeTimeRanges`、`shiftTimeRanges` 等数组操作在循环中频繁执行，建议将时间片计算移至数据库层（如使用 MySQL 的 `INTERVAL` 或生成临时表），或引入位图/区间树算法降低时间复杂度。

> ⚠️ **局限性说明**：提供的代码在 `_get_un_book_time` 方法中途截断，后续逻辑（如次日时间处理、最终返回结构）无法完整审查。建议补充完整文件后，可进一步评估边界条件处理与内存泄漏风险。

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