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
**提交**: `31998ab2f5cd19ac60553c26f5a43358634419e8`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-01 19:26:32

---

## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑覆盖较全面（涵盖跨天、套餐、最低时长、清扫时间、停用规则等），但核心方法过于臃肿，存在静态缓存污染风险，且大量混合数据查询、时间计算与状态判定逻辑。代码可维护性与性能有较大优化空间。
- **风险等级**：🟠 中（存在逻辑隐患与性能瓶颈，暂无直接高危安全漏洞）

> 📌 **框架说明**：根据目录结构（`system/`, `application/`）及 `$CI = &get_instance()` 语法，判定该代码基于 **CodeIgniter 3.x** 架构。若 `phpci` 为贵司内部定制框架，请确认以下 CI3 规范是否适用。

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_book_days_info` 开头 | **静态缓存未区分参数**：`if (!empty(self::$book_days_info)) { return self::$book_days_info; }` 会忽略后续传入的 `$check_date` 与 `$add_day`，导致不同请求返回脏数据。 | 将缓存键改为参数组合的哈希值，或移除静态缓存改用实例属性/Redis缓存。 | `private static $book_days_info = []; public function get_book_days_info(...) { $cache_key = md5($merchant_id . '_' . $shop_id . '_' . $check_date . '_' . (int)$add_day); if (isset(self::$book_days_info[$cache_key])) { return self::$book_days_info[$cache_key]; } // ... 计算逻辑 ... self::$book_days_info[$cache_key] = $result; return $result; }` |
| 🔴 严重 | `get_book_day_time_info` | **违反单一职责原则**：该方法超 300 行，混合了数据拉取、时间区间计算、套餐规则校验、状态标记与 UI 提示生成。极易引发逻辑冲突与回归 BUG。 | 将时间计算与规则校验剥离至独立的 `BookingTimeCalculatorService`，Model 仅负责数据存取与基础组装。 | *(见第3节重构建议)* |
| 🟠 警告 | 多处方法内 | **频繁调用 `&get_instance()`**：在 `__construct`、`set_shop_config`、`_get_un_book_time` 等位置重复获取 CI 实例，增加开销且不符合 CI 最佳实践。 | 在 `__construct` 中统一获取并赋值给实例属性，后续直接使用 `$this->ci`。 | `protected $ci; public function __construct() { parent::__construct(); $this->ci = &get_instance(); // ... }` |
| 🟠 警告 | `set_shop_config` & `set_room_info` | **逻辑高度重复**：场景判断 (`billiards_`, `card_`, `tavern_`) 与配置加载代码在两个方法中几乎完全一致。 | 提取为私有方法 `_resolve_shop_config_scene($scene)` 统一调用。 | `private function _resolve_shop_config_scene(string $scene): string { return match($scene) { '2' => 'billiards_', '3' => 'card_', '4' => 'tavern_', default => '' }; }` |
| 🟠 警告 | 类属性定义 | **拼写错误**：`$opreational_scene` 存在拼写错误（应为 `operational`），易导致后续维护或前端对接时字段不一致。 | 全局搜索替换为 `$operational_scene`，并在属性声明处添加类型提示。 | `/** @var string 1:ktv, 2:billiards, 3:card, 4:tavern */ public string $operational_scene = '1';` |
| 🟡 建议 | 全局 | **大量魔法数字与硬编码**：`86400`, `3600`, `'1'`, `'-1'` 等散落在代码中，降低可读性且易引发状态误判。 | 定义为类常量，如 `const STATUS_AVAILABLE = '1'; const STATUS_UNAVAILABLE = '-1'; const SECONDS_PER_DAY = 86400;` | `const STATUS_AVAILABLE = '1'; const STATUS_UNAVAILABLE = '-1'; // 替换所有 '1' / '-1' 状态赋值` |
| 🟡 建议 | `get_book_day_time_info` 循环内 | **循环内高频数组操作**：`array_intersect`, `array_merge`, `array_unique`, `sort` 在 `foreach ($time_info)` 中反复执行，时间复杂度偏高。 | 预计算不可用时间区间树（Interval Tree）或使用位图标记；将重复的 `strtotime`/`date` 计算移至循环外。 | *(建议引入 `DatePeriod` 或区间合并算法预处理 `$un_book_time`，循环内仅做 O(1) 范围比对)* |
| 🟡 建议 | 全局 | **缺乏现代 PHP 类型声明**：未使用 PHP 7.4+/8.x 的类型提示，不利于 IDE 静态分析与团队协作。 | 为方法参数、返回值及类属性添加严格类型声明。 | `public function get_book_days_info(int $merchant_id, int $shop_id, string $check_date = '', bool $add_day = false): array` |

## 3. 总结与行动建议

### 🔑 优先修复的关键问题
1. **修复静态缓存污染**：立即修改 `get_book_days_info` 的缓存键生成逻辑，否则在多条件并发请求下必现数据错乱。
2. **拆分巨型方法**：`get_book_day_time_info` 已超出合理维护边界。建议按职责拆分为：
   - `fetchBookingData()`：负责拉取门店、包厢、历史预订数据。
   - `calculateAvailableSlots()`：负责时间区间计算、规则过滤。
   - `formatSlotStatus()`：负责最终状态标记与 UI 提示组装。
3. **统一 CI 实例获取**：消除冗余的 `&get_instance()` 调用，提升运行效率与代码整洁度。

### 🛠 后续重构与优化方向
1. **引入服务层（Service Layer）**：将复杂的业务规则（套餐时长校验、跨天逻辑、最低开房时长限制）从 Model 中剥离，放入 `app/services/BookingTimeService.php`。Model 仅保留 `get_one`, `insert`, `update` 等数据操作。
2. **时间计算算法优化**：当前使用 `array_intersect` 判断时间交集，在时间片粒度较细（如 5 分钟）时数组体积庞大。建议改用 **区间重叠算法** 或 **时间轴位图**，将复杂度从 `O(N*M)` 降至 `O(N log N)`。
3. **配置集中化管理**：将 `book_time_limit`, `minute_unit`, `book_days` 等硬编码默认值移至配置文件或数据库配置表，通过 `config()` 函数读取，避免代码级修改。
4. **补充输入校验**：在方法入口处对 `$params` 进行严格校验（如 `ctype_digit`, `preg_match('/^\d{8}$/', $date)`），防止非法参数穿透至底层模型引发 SQL 异常或逻辑崩溃。
5. **代码规范对齐**：使用 `php-cs-fixer` 或 `PHP_CodeSniffer` 配置 PSR-12 规则集进行自动化格式化，补充缺失的 `@throws`、`@return` 注释，并统一错误处理机制（建议统一抛出 `InvalidArgumentException` 或自定义 `BookingException`，替代全局 `throwError`）。

> ⚠️ **局限性说明**：提供的代码在 `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 443 in issue