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
**提交**: `da5f379382fb71fa01d0340a7f3ec62b05f240ae`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 11:14:37

---

## 1. 审查摘要
- **代码质量评分**：5.5/10
- **总体评价**：代码实现了复杂的包厢预订时间计算逻辑，但存在严重的状态管理缺陷、方法职责过重、静态缓存污染风险及性能隐患。整体可维护性较低，在并发或多租户场景下极易引发数据错乱。
- **风险等级**：🔴 高
> **注**：`phpci` 通常指 PHP 持续集成服务器，而非 PHP 开发框架。从代码特征（`$CI = &get_instance()`、`$this->load->model()`、`load->library()`）判断，实际使用的是 **CodeIgniter 3**。以下审查基于 CI3 架构与 PHP 最佳实践。若为自研框架，请对照其生命周期规范调整。
> **局限性说明**：代码在 `_get_un_book_time` 方法末尾截断，未包含完整逻辑。以下审查基于已提供部分，若截断处包含关键事务提交或状态回滚逻辑，请补充后复核。

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_book_days_info` | **静态缓存未区分参数**：`self::$book_days_info` 首次赋值后全局生效。不同商户/门店/日期请求会直接复用脏数据，导致严重业务错乱。 | 移除静态缓存或改用带参数哈希的内存缓存键。若需缓存，应使用 `private static $cache = []` 并以参数组合作为 Key。 | `private static $book_days_cache = []; $cache_key = md5(serialize([$merchant_id, $shop_id, $check_date, $add_day])); if (isset(self::$book_days_cache[$cache_key])) return self::$book_days_cache[$cache_key]; // ... 计算逻辑 ... self::$book_days_cache[$cache_key] = $result;` |
| 🔴 严重 | `get_book_days_info` | **实例状态被意外修改**：`$this->book_days += 1;` 直接修改对象属性。多次调用会导致天数无限累加，破坏后续所有依赖该属性的逻辑。 | 使用局部变量替代属性修改，保持对象状态不可变性。 | `$calc_days = $this->book_days + ($add_day ? 1 : 0); for ($i = 0; $i < $calc_days; $i++) { ... }` |
| 🟠 警告 | `get_book_day_time_info` | **方法职责过重（God Method）**：单方法超 300 行，混合了数据查询、时间区间计算、业务规则过滤、UI 状态标记。违反单一职责原则，极难单元测试。 | 拆分为独立私有方法或抽离为 `BookingTimeCalculator` 服务类：`fetchRoomData()`、`calculateAvailableSlots()`、`applyVoucherRules()`、`formatTimeSlots()`。 | （建议按数据流拆分，保持单方法 < 80 行） |
| 🟠 警告 | 全局/多处 | **安全隐患：输入未校验**：`$params` 数组直接传入模型，缺乏类型与边界检查（如日期格式、ID 合法性）。恶意构造参数可能导致时间计算溢出或越权查询。 | 在 Controller 层使用 CI 表单验证或 PHP 8 类型声明；模型入口处增加基础校验。 | `if (!preg_match('/^\d{8}$/', $params['date'] ?? '')) { throwError('日期格式错误'); } $params['merchant_id'] = filter_var($params['merchant_id'], FILTER_VALIDATE_INT);` |
| 🟠 警告 | `_get_un_book_time` | **性能瓶颈：参数展开求交集**：`array_intersect(...array_values($all_room_book_time))` 使用 `...` 展开，当包厢数较多时可能触发 `ArgumentCountError` 或内存飙升。 | 改用迭代方式求交集，或使用 `array_reduce`。 | `$intersection = array_shift($all_room_book_time); foreach ($all_room_book_time as $arr) { $intersection = array_intersect($intersection, $arr); }` |
| 🟠 警告 | 构造函数/多处 | **框架开销：重复获取实例与加载**：多处重复 `$CI = &get_instance();` 及重复 `load->model()`。CI 虽会去重，但增加解析开销且破坏代码整洁度。 | 在构造函数中统一赋值 `$this->ci = &get_instance();`，利用 CI 的自动加载机制。 | `public function __construct() { parent::__construct(); $this->ci = &get_instance(); $this->load->model('Simple_model'); // 其他初始化... }` |
| 🟡 建议 | 类属性定义 | **封装性不足**：大量 `public` 属性暴露内部状态，外部可直接篡改业务规则（如 `$book_time_limit`、`$ignore_tuangou_time_limit`）。 | 改为 `protected` 或 `private`，通过 `getter/setter` 控制访问，并在 setter 中增加合法性校验。 | `protected $book_time_limit = 3600; public function setBookTimeLimit(int $seconds): void { $this->book_time_limit = max(300, $seconds); }` |
| 🟡 建议 | 全局函数依赖 | **隐式依赖全局辅助函数**：大量使用 `timeToHour`、`mergeTimeRanges`、`shiftTimeRange` 等未声明的全局函数，降低可测试性与 IDE 静态分析能力。 | 将时间计算逻辑封装为独立的 `TimeRangeHelper` 类或 Service，通过依赖注入或静态方法调用。 | `class TimeRangeHelper { public static function merge(array $ranges): array { ... } }` |

## 3. 总结与行动建议

### 🔑 优先修复的关键问题
1. **修复静态缓存污染与状态突变**：立即移除 `self::$book_days_info` 的无差别缓存，并删除 `$this->book_days += 1` 的副作用代码。这是当前导致多租户数据串扰的核心隐患。
2. **拆分巨型方法**：将 `get_book_day_time_info` 按数据流向拆分为 `获取基础数据` → `计算不可用区间` → `应用业务规则（团购/营业/锁定）` → `格式化输出` 四个独立步骤，提升可读性与可测试性。
3. **输入参数强校验**：在模型入口或 Controller 层对 `$params['date']`、`$params['merchant_id']`、`$params['shop_id']` 进行严格类型与格式校验，防止非法数据流入核心计算逻辑。

### 🛠 后续重构与优化方向
- **架构升级建议**：若项目允许，建议逐步迁移至 **CodeIgniter 4** 或现代 PHP 框架（Laravel/Symfony）。CI4 原生支持依赖注入、命名空间、类型声明与更安全的 Query Builder，可大幅减少 `$CI = &get_instance()` 的滥用。
- **时间计算服务化**：当前时间区间合并、偏移、交集判断逻辑高度耦合。建议抽离为独立的 `BookingTimeService`，使用值对象（Value Object）如 `TimeSlot`、`DateRange` 替代裸数组，提升类型安全。
- **缓存策略优化**：静态数组缓存仅适用于单次请求生命周期。若需跨请求复用，应改用 CI Cache 驱动（Redis/Memcached）并设置合理的 TTL；若仅用于单次请求内复用，请使用 `private static $cache = []` 并严格绑定参数哈希。
- **性能监控**：在 `_get_un_book_time` 中涉及多次 `json_decode` 与数据库查询。建议对 `_time_info_new` 字段建立合理索引，或考虑将高频查询的预订状态同步至 Redis 位图/集合中，降低 MySQL 压力。

> 如需针对截断部分或特定业务场景（如团购券跨天逻辑、包厢锁定机制）进行深度逻辑推演，请提供完整代码片段。

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