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
**提交**: `957cbb6a73dfead2972a3f1e02720cb6c81d94bc`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-27 16:18:07

---

## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖全面，能处理多场景（KTV/台球/棋牌/酒馆）、套餐规则、跨天营业等复杂预订需求。但代码存在严重的架构设计缺陷：静态缓存未隔离租户、直接污染全局 `$CI` 实例、核心方法严重违反单一职责原则。时间处理依赖大量字符串拼接与未定义的辅助函数，可维护性与可测试性较低。
- **风险等级**：🔴 高

> 📌 **框架说明**：根据目录结构、`get_instance()`、`$this->load->model()` 等特征，该代码实际基于 **CodeIgniter 3** 架构。若 `phpci` 为内部定制框架，请对照其官方文档调整。以下审查基于 CI3 最佳实践与现代 PHP 规范。

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 静态属性/缓存 | `self::$book_days_info`、`self::$shop_data` 等静态缓存未区分 `merchant_id` 与 `shop_id`。在多租户/多门店并发或长连接环境下，会导致严重的数据串扰与越权访问。 | 为静态缓存增加复合键，或改用 CI Cache/Redis 驱动，并设置合理 TTL。避免在 Model 中使用无状态隔离的静态变量。 | `private static $cache = [];` `$key = md5($merchant_id . '_' . $shop_id . '_' . $date);` `if (isset(self::$cache[$key])) return self::$cache[$key];` `self::$cache[$key] = $result;` |
| 🔴 严重 | 多处 | 直接修改全局 `$CI` 实例属性（如 `$CI->operational_scene`、`$CI->package_not_available_time`、`$CI->change_book_order_id`）。破坏框架请求隔离机制，极易引发并发状态污染与难以追踪的隐蔽 Bug。 | 严禁修改 `$CI` 全局状态。应通过类属性、方法返回值、Session 或 Config 传递上下文数据。 | 移除 `$CI->xxx = ...`，改为 `$this->operational_scene = $scene;` 或作为参数/返回值传递。 |
| 🔴 严重 | `get_book_day_time_info` | 方法超 300 行，混合了配置加载、时间区间计算、多规则校验（套餐/锁定/营业/清扫）、数据格式化。严重违反单一职责原则(SRP)，无法进行单元测试，后期维护成本极高。 | 拆分为独立私有方法，或抽离为 `BookingTimeCalculator` 服务类。按职责分层：配置获取 → 基础时间生成 → 规则过滤 → 结果组装。 | 见下方【重构方向】建议。 |
| 🟠 警告 | 多处 | 大量使用硬编码魔法值（如 `'1'`, `'2'`, `'merchantApp'`, `'-1'`, `'7'`），缺乏语义化常量。业务逻辑变更时极易遗漏或误改。 | 定义类常量或全局配置常量，提升代码可读性与安全性。 | `const SCENE_KTV = '1'; const SCENE_BILLIARDS = '2';` `const STATUS_AVAILABLE = '1'; const STATUS_UNAVAILABLE = '-1';` |
| 🟠 警告 | `_get_un_book_time` 及循环内 | 频繁调用 `get_shop_setting()` 获取配置，未做批量查询或内存缓存。在复杂请求链路中易引发 N+1 查询与数据库性能瓶颈。 | 在方法入口一次性批量获取所需配置，或使用 CI Cache 缓存门店配置项。 | `$keys = ['book_minute_unit', 'book_hour_options', ...];` `$configs = $this->ahead_shop_config_second_model->get_batch_settings($mid, $sid, $keys);` |
| 🟠 警告 | `array_intersect(...)` | `array_intersect(...array_values($all_room_book_time))` 在数组为空或仅含一个元素时，PHP 会抛出 `Warning`，且解包逻辑不符合预期。 | 增加数组长度安全判断，或使用循环计算交集。 | `if (count($all_room_book_time) > 1) { $un_book_time = array_intersect(...array_values($all_room_book_time)); } else { $un_book_time = $all_room_book_time[0] ?? []; }` |
| 🟡 建议 | 全局 | 未遵循 PSR-12 规范：缺少严格类型声明、属性可见性不统一、注释不完整、行过长。部分辅助函数（如 `timeToHour`, `shiftTimeRange`）未提供定义，难以评估其边界行为。 | 补充 `declare(strict_types=1);`，为方法参数/返回值添加类型提示，统一使用 `private/protected` 属性，完善 PHPDoc。 | `public function get_book_days_info(int $merchant_id, int $shop_id, string $check_date = '', bool $add_day = false): array` |
| 🟡 建议 | 时间处理 | 依赖大量自定义辅助函数与字符串拼接处理时间，缺乏时区控制与 `DateTimeImmutable` 对象封装。跨天、24小时制、边界分钟计算易出现逻辑漏洞。 | 引入 `DateTimeImmutable` 或专用时间区间 Value Object 替代字符串操作，提升类型安全与可读性。 | 建议使用 `Carbon` 或原生 `DateTimeImmutable` 进行区间计算，避免 `strtotime` + 字符串拼接。 |

## 3. 总结与行动建议

### 🔑 优先修复的关键问题
1. **隔离静态缓存**：立即为所有 `self::$xxx` 静态属性添加 `merchant_id` + `shop_id` + `date` 的复合键，或彻底移除静态缓存改用 CI Cache/Redis，杜绝多租户数据串扰。
2. **清除全局 `$CI` 污染**：全面排查并移除 `$CI->xxx = ...` 赋值操作。将上下文数据改为通过方法参数传递或封装在类实例属性中。
3. **拆分超长方法**：将 `get_book_day_time_info` 拆分为至少 4 个独立步骤：
   - `loadBookingConfig()`：加载门店与场景配置
   - `generateBaseTimeSlots()`：生成基础时间轴
   - `applyBookingRules()`：应用套餐、锁定、清扫、营业规则过滤
   - `formatTimeSlots()`：组装最终返回结构

### 🛠 后续重构与优化方向
- **引入策略模式**：不同场景（KTV/台球/棋牌/酒馆）的预订规则差异较大，建议定义 `BookingRuleInterface`，按场景注入对应策略类，替代冗长的 `if/elseif` 分支。
- **时间区间对象化**：创建 `TimeRange` 值对象，封装 `start`, `end`, `isOverlap()`, `merge()`, `shift()` 等方法，彻底替代当前散落的字符串与数组操作。
- **完善输入校验**：对 `$params` 数组进行严格类型校验与白名单过滤，避免未定义键导致 `Undefined index` 警告或逻辑跳过。
- **补充单元测试**：核心时间计算逻辑必须覆盖边界用例（如跨天 23:55-00:05、24小时营业、套餐时长不足、并发锁定等），建议引入 `PHPUnit` 建立测试基线。

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