sqlite-web 0.7.2
gitea.db
issue
Create
Query
access
access_token
action
action_artifact
action_run
action_run_index
action_run_job
action_runner
action_runner_token
action_schedule
action_schedule_spec
action_task
action_task_output
action_task_step
action_tasks_version
action_variable
app_state
attachment
auth_token
badge
branch
collaboration
comment
commit_status
commit_status_index
commit_status_summary
commit_sync_log
commit_sync_status
dbfs_data
dbfs_meta
deploy_key
email_address
email_hash
external_login_user
follow
gpg_key
gpg_key_import
hook_task
issue
issue_assignees
issue_content_history
issue_dependency
issue_index
issue_label
issue_pin
issue_user
issue_watch
label
language_stat
lfs_lock
lfs_meta_object
login_source
milestone
mirror
notice
notification
oauth2_application
oauth2_authorization_code
oauth2_grant
org_user
package
package_blob
package_blob_upload
package_cleanup_rule
package_file
package_property
package_version
project
project_board
project_issue
protected_branch
protected_tag
public_key
pull_auto_merge
pull_request
push_mirror
reaction
release
renamed_branch
repo_archiver
repo_hidden_file
repo_indexer_status
repo_license
repo_redirect
repo_topic
repo_transfer
repo_unit
repository
review
review_state
secret
session
sqlite_sequence
star
stopwatch
system_setting
task
team
team_invite
team_repo
team_unit
team_user
topic
tracked_time
two_factor
upload
user
user_badge
user_blocking
user_open_id
user_redirect
user_setting
version
watch
webauthn_credential
webhook
Toggle helper tables
Structure
Content
Query
Insert
Drop
Import
Export
Delete row 453 from issue
id
453
repo_id
21
index
166
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260616 - 1
content
## 自动代码审查报告 **分支**: pay-260616 **提交**: `a06788a8f
## 自动代码审查报告 **分支**: pay-260616 **提交**: `a06788a8fe4f2bc3ce849b73dd1f28c0be458120` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-02 10:55:36 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：该模型承载了极其复杂的门店预订时间计算逻辑，业务覆盖全面（含跨天、套餐、团购券、清扫时间、包厢锁定等）。但代码存在**严重的静态缓存缺陷**、**实例状态污染**、**时间计算脆弱性**以及**方法过长违反单一职责原则**等问题。整体可维护性与健壮性亟待提升。 - **风险等级**：🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `get_book_days_info` 方法开头 | **静态缓存未区分参数**：`self::$book_days_info` 作为静态缓存，未考虑 `$merchant_id`、`$shop_id`、`$check_date`、`$add_day` 的差异，会导致不同门店/日期请求返回脏数据。<br>**实例属性污染**：`$this->book_days += 1;` 直接修改类属性，后续调用将基于错误的基础天数计算。 | 1. 移除静态缓存或改用参数哈希作为 Key。<br>2. 禁止在方法内修改 `$this->book_days`，改用局部变量 `$days = $this->book_days + ($add_day ? 1 : 0);`。 | `private static $cache = [];<br>$key = md5(implode('_', func_get_args()));<br>if (isset(self::$cache[$key])) return self::$cache[$key];` | | 🔴 严重 | `get_book_day_time_info` 中 `array_intersect(...array_values($all_room_book_time))` | **PHP 8+ 兼容性崩溃**：`array_intersect` 至少需要传入两个数组。当 `$room_type_room_ids` 数量为 1 时，解包操作会抛出 `ValueError` 或 `Warning`，导致接口 500。 | 增加数组数量判断，为 1 时直接赋值，大于 1 时才使用解包求交集。 | `if (count($all_room_book_time) === 1) {<br> $un_book_time = reset($all_room_book_time) ?? [];<br>} else {<br> $un_book_time = array_intersect(...array_values($all_room_book_time));<br>}` | | 🟠 警告 | 全局多处 `strtotime`/`date` 及 `86400` 硬编码 | **跨天时间计算隐患**：大量使用 `+86400`、`-86400` 处理跨天逻辑，未考虑夏令时(DST)、闰秒或时区切换，极易在边界时间（如 23:55-00:05）产生计算偏差。 | 废弃手动秒数偏移，全面改用 `DateTime` 或 `Carbon` 进行时间运算，利用 `modify()` 和 `diff()` 保证准确性。 | `$end = (new DateTime($date))->modify('+1 day')->setTime(0,0);<br>$diff = $start->diff($end)->getTimestamp();` | | 🟠 警告 | `json_decode($book_time_info['_time_info_new'], 1)` | **未校验 JSON 有效性**：若数据库字段损坏或为空，`json_decode` 返回 `null`，后续 `['room_book_time']` 访问将触发 `TypeError` 或 `Warning`。 | 增加 JSON 解析错误处理，或使用 PHP 7.3+ 的 `JSON_THROW_ON_ERROR` 标志。 | `$data = json_decode($json, true, 512, JSON_THROW_ON_ERROR);<br>// 或兼容写法：<br>$data = json_decode($json, true);<br>if (json_last_error() !== JSON_ERROR_NONE) { $data = []; }` | | 🟠 警告 | `set_shop_config` / `_get_un_book_time` 中多次调用 `get_shop_setting` | **重复查询性能损耗**：同一方法内多次调用 `get_shop_setting` 获取不同配置，增加数据库/Redis 往返开销。 | 改为批量获取配置，或在模型层引入配置缓存层（如 `get_shop_settings_batch`）。 | `$keys = ['book_minute_unit', 'book_hour_options', 'book_max_days'];<br>$configs = $this->ahead_shop_config_second_model->get_shop_settings_batch($merchant_id, $shop_id, $keys);` | | 🟡 建议 | 整个文件 | **缺乏类型声明与封装**：大量 `public` 属性暴露内部状态，方法参数无类型提示，违反 PSR-12 及现代 PHP 规范。 | 为属性添加 `private/protected` 修饰符并提供 Getter/Setter；为方法参数和返回值添加 PHP 7.4+ 类型声明。 | `private int $book_time_limit = 3600;<br>public function getBookDaysInfo(int $merchantId, int $shopId, string $checkDate = '', bool $addDay = false): array` | | 🟡 建议 | 多处硬编码 `'1'`, `'-1'`, `'7'`, `'2'` 等 | **魔法值泛滥**：状态码、场景码、业务规则硬编码在逻辑中，可读性差且极易引发维护错误。 | 提取为类常量或枚举（PHP 8.1+），统一状态管理。 | `const SCENE_KTV = '1';<br>const SCENE_BILLIARDS = '2';<br>const STATUS_AVAILABLE = '1';<br>const STATUS_UNAVAILABLE = '-1';` | | 🟡 建议 | `get_book_day_time_info` 方法（超 300 行） | **违反单一职责原则 (SRP)**：该方法同时负责：日期校验、营业时间计算、包厢锁定查询、套餐时间过滤、不可用时间合并、状态标记等，圈复杂度过高。 | 拆分为多个私有方法，如 `calculateBusinessHours()`, `mergeUnavailableTimes()`, `applyPackageRules()`, `markTimeSlotStatus()`。 | 见下方重构建议 | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **修复静态缓存与属性污染**：立即移除 `self::$book_days_info` 的无参缓存逻辑，并将 `$this->book_days += 1` 改为局部变量计算。此问题会导致多租户/多日期请求数据串扰，属于线上高危缺陷。 2. **修复 `array_intersect` 解包崩溃**：在 PHP 8 环境下，单数组解包会直接中断执行。需增加 `count()` 判断分支。 3. **增强 JSON 解析容错**：所有 `json_decode` 调用必须增加 `json_last_error()` 校验或 `try-catch`，防止脏数据引发连锁崩溃。 ### 🛠 后续重构与优化方向 1. **时间计算现代化**：全面替换 `strtotime` + `86400` 的原始计算方式。建议引入 `nesbot/carbon` 库，利用 `Carbon::parse()->addHours()->isBetween()` 等语义化 API，彻底解决跨天、夏令时边界问题。 2. **方法拆分与职责收敛**：`get_book_day_time_info` 已演变为“上帝方法”。建议按业务阶段拆分为： - `resolveBaseTimeSlots()`：生成基础时间段 - `applyBusinessRules()`：应用营业/清扫/最低时长限制 - `applyPackageVoucherRules()`：应用团购券/套餐规则 - `mergeAndMarkStatus()`：合并不可用区间并标记最终状态 3. **依赖注入替代全局获取**：当前大量使用 `$CI = &get_instance()` 和 `$this->load->model()`。若 `phpci` 框架支持 DI 容器，建议通过构造函数注入核心依赖（如 `ConfigService`, `RoomLockService`, `VoucherService`），提升可测试性。 4. **补充单元测试**：该模型逻辑分支极多，强烈建议针对 `get_book_days_info` 和 `_get_un_book_time` 编写 PHPUnit 用例，覆盖：跨天、空数据、单包厢、多包厢交集、套餐时长不足、静态缓存隔离等场景。 > 📌 **框架适配说明**：当前代码大量使用 `get_instance()`、`$this->load->model()` 等典型 **CodeIgniter 3** 模式。若您实际使用的是内部定制版 `phpci` 框架，请确认其是否兼容 CI3 生命周期。若 `phpci` 已转向现代 DI/Service 架构，建议逐步将 `load->` 调用替换为构造函数注入，以符合现代 PHP 框架最佳实践。 > > ⚠️ **局限性提示**：您提供的代码在 `_get_un_book_time` 方法末尾被截断，未能完整审查该方法的后续逻辑（如次日时间计算、最终返回结构）。建议补充完整代码以便进行闭环审查。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780368936
updated_unix
1780368936
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel