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 459 from issue
id
459
repo_id
21
index
172
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260519 - 1
content
## 自动代码审查报告 **分支**: pay-260519 **提交**: `33cf68b8e
## 自动代码审查报告 **分支**: pay-260519 **提交**: `33cf68b8eed08573f0ad52ebc59f46f33e002b6c` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-02 13:42:33 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：该模型承载了极其复杂的预订时间计算与状态判定逻辑，业务覆盖全面。但核心方法过长、职责边界模糊、大量依赖全局 `$CI` 对象传递状态，且存在多处逻辑交叉覆盖风险。代码风格偏向传统 PHP 5/7 早期写法，未遵循现代 PSR 规范，可维护性与性能存在明显瓶颈。 - **风险等级**：🔴 高 > 📌 **框架说明**：根据代码结构（`$CI = &get_instance()`、`system/` 目录、`$this->load->model()` 等），该代码实际基于 **CodeIgniter 3** 架构。以下审查基于 CI3 最佳实践。若确为内部自研 `phpci` 框架，请对照其官方文档调整组件加载与生命周期调用方式。 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `get_book_day_time_info()` 核心循环 | 方法超 300 行，`$v['status']` 在多个 `if` 分支中被反复覆盖修改。条件交叉重叠（如套餐限制、营业时间、不可用时间、跨天逻辑等），极易因执行顺序导致状态误判或漏判。 | **拆分职责**：将校验逻辑抽离为独立方法（如 `checkVoucherTime()`, `checkBusinessHours()`, `checkLockedTime()`），采用**责任链模式**或**策略模式**依次校验，任一失败则提前返回或标记状态，避免嵌套覆盖。 | `foreach ($time_info as &$v) { $v['status'] = $this->validateTimeSlot($v, $constraints); }` | | 🔴 严重 | `__construct()` & `set_shop_config()` | 频繁调用 `$CI = &get_instance()` 并直接读写 `$CI->operational_scene`、`$CI->request_source` 等属性。破坏 MVC 封装，易引发全局状态污染，且在并发或 CLI 环境下可能导致数据串扰。 | 移除对 `$CI` 属性的直接赋值。改为通过方法参数传递上下文，或使用 `$this->ci = &get_instance()` 在构造函数中缓存一次，仅通过 `$this->ci->config->item()` 读取配置。 | `protected $ci; public function __construct() { parent::__construct(); $this->ci = &get_instance(); }` | | 🟠 警告 | `_get_un_book_time()` & `get_book_day_time_info()` | 多次重复调用 `ahead_shop_config_second_model->get_shop_setting()` 获取相同配置项，未做请求级缓存，增加数据库 I/O 压力。 | 使用 CI 缓存驱动或请求级静态数组缓存配置。若配置变更不频繁，建议接入 Redis/文件缓存。 | `protected function getShopConfig($key) { static $cache = []; return $cache[$key] ??= $this->ci->ahead_shop_config_second_model->get_shop_setting(...); }` | | 🟠 警告 | `get_book_day_time_info()` 数据解析 | `json_decode($book_time_info['_time_info_new'], 1)` 未校验 JSON 格式。若数据库字段损坏或包含非法字符，将触发 Warning 并返回 `null`，导致后续 `foreach` 报错。 | 增加 `json_last_error()` 校验，并提供安全降级默认值。 | `$data = json_decode($json, true); if (json_last_error() !== JSON_ERROR_NONE) { $data = []; }` | | 🟠 警告 | 全局时间计算 | 大量使用字符串时间（如 `'202407010800'`）进行 `array_intersect`、`strtotime` 转换。字符串比较易受时区、格式不一致影响，且性能低于整型时间戳。 | **统一使用 Unix 时间戳（int）**进行区间计算与交集判断，仅在最终输出时格式化为字符串。可封装 `TimeRange` 值对象。 | `class TimeRange { public int $start; public int $end; public function intersects(self $other): bool { ... } }` | | 🟡 建议 | 全局常量与魔法值 | 大量硬编码魔法值：`'1'`, `'-1'`, `'2'`, `'3'`, `'4'`, `86400`, `3600`。可读性差，后期维护易出错。 | 定义类常量或枚举（PHP 8.1+），集中管理业务状态与时间单位。 | `const STATUS_AVAILABLE = '1'; const STATUS_UNAVAILABLE = '-1'; const SECONDS_PER_DAY = 86400;` | | 🟡 建议 | 辅助函数依赖 | 依赖大量未命名空间的全局函数（`timeToHour`, `mergeTimeRanges`, `throwError` 等），不符合 PSR-4/12 规范，难以进行单元测试。 | 封装为独立的服务类（如 `BookingTimeCalculator`、`ValidationHelper`），通过依赖注入或静态方法调用，便于 Mock 测试。 | `class TimeUtils { public static function mergeRanges(array $ranges): array { ... } }` | | 🟡 建议 | `set_shop_config()` 与 `set_room_info()` | 两个方法逻辑高度重复（获取场景、加载配置、计算分钟单位），违反 DRY 原则。 | 合并为单一受保护方法，通过参数区分调用场景，或提取为 `SceneConfigLoader` 组件。 | `protected function loadSceneConfig(?int $roomId = null): void { ... }` | ## 3. 总结与行动建议 ### 🚨 优先修复项（P0/P1） 1. **消除 `$v['status']` 状态覆盖风险**：当前 `get_book_day_time_info()` 的 `foreach` 循环是系统核心，状态被多次覆盖极易引发线上客诉（如“明明可订却显示不可订”）。请立即将校验逻辑拆分为独立方法，采用**短路返回**或**状态累加器**模式。 2. **收敛全局 `$CI` 对象滥用**：禁止在 Model 中直接修改 `$CI->xxx` 属性。所有上下文数据（如 `request_source`, `operational_scene`）应通过方法参数传入，或使用 `$this->ci->session->userdata()` 等标准方式获取。 3. **增强 JSON 解析与输入校验**：对 `$params` 增加类型强转与基础校验（如 `ctype_digit($params['merchant_id'])`），对 `json_decode` 增加错误捕获，防止脏数据导致白屏或逻辑中断。 ### 🛠 后续重构与优化方向 1. **架构分层**：当前 Model 承担了 `数据查询`、`业务规则计算`、`状态组装` 三重职责。建议引入 **Service 层**（如 `BookingAvailabilityService`）处理复杂时间计算，Model 仅负责数据持久化与基础查询。 2. **时间计算引擎化**：将 `mergeTimeRanges`、`shiftTimeRange`、`minutesToUnits` 等函数抽象为独立的 `TimeRangeCalculator` 类。内部统一使用 `int` 时间戳运算，彻底告别字符串时间拼接，提升计算精度与性能。 3. **配置缓存策略**：门店配置（`book_minute_unit`, `book_hour_options` 等）变更频率低，建议接入 CI 缓存驱动（`$this->ci->cache->save()`）或 Redis，设置合理 TTL（如 30 分钟），减少高频查询对 DB 的冲击。 4. **单元测试覆盖**：该模块逻辑分支极多，强烈建议使用 PHPUnit 编写核心校验逻辑的单元测试。通过 Mock 数据库与 `$CI` 对象，验证不同场景（跨天、套餐限制、最低时长、清扫时间）下的时间槽输出是否符合预期。 > 💡 **提示**：若需进一步审查 `Simple_model` 基类实现、`Tuangou` 库逻辑或控制器层调用方式，请提供相关代码片段。当前审查仅基于已提交的 Model 文件，部分全局函数与框架生命周期细节可能存在上下文盲区。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780378954
updated_unix
1780378954
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel