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 280 from issue
id
280
repo_id
21
index
63
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260519 - 1
content
## 自动代码审查报告 **分支**: pay-260519 **提交**: `527b37c10
## 自动代码审查报告 **分支**: pay-260519 **提交**: `527b37c106b743b4e035cf0fe54af5fd99580d7c` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-05-21 16:51:28 --- ## 1. 审查摘要 - **代码质量评分**：6.5 / 10 分 - **总体评价**：代码实现了复杂的门店时段预订逻辑，涵盖了跨天、团购券、最低时长、清扫时间等多维度业务规则。但存在**静态缓存污染、对象状态意外修改、方法职责过重、框架上下文滥用**等典型问题。整体可维护性较低，边界条件与异常处理不够严谨。 - **风险等级**：🟠 中（存在数据串扰与状态污染隐患，高并发或复杂预订场景下易引发逻辑错乱） > 📌 **注**：代码结构高度符合 `CodeIgniter 3` 规范（如 `get_instance()`、`$this->load->model()`）。若 `phpci` 为内部定制框架，请结合其官方文档调整生命周期与依赖管理。此外，**提交代码在末尾被截断**，以下审查基于已提供部分，未覆盖完整逻辑。 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `get_book_days_info` 方法 | **静态缓存未区分参数**：`self::$book_days_info` 作为全局静态变量，首次赋值后直接返回，忽略后续传入的 `$merchant_id`、`$shop_id`、`$check_date` 等参数，导致多商户/多日期数据严重串扰。 | 移除静态缓存，或改用基于参数哈希的实例级缓存/Redis缓存。 | `// 错误：直接返回静态变量<br>if (!empty(self::$book_days_info)) return self::$book_days_info;<br><br>// 正确：使用参数键值缓存<br>$cache_key = md5(implode('_', func_get_args()));<br>if (isset($this->cache['book_days'][$cache_key])) return $this->cache['book_days'][$cache_key];` | | 🔴 严重 | `get_book_days_info` 方法 | **意外修改实例状态**：`$this->book_days += 1;` 直接修改了类属性。若该方法被多次调用（如 `$add_day` 交替为 true/false），将导致后续所有计算基准错误。 | 使用局部变量进行计算，严禁在查询方法中修改类属性。 | `$calc_days = $add_day ? $this->book_days + 1 : $this->book_days;<br>for ($i = 0; $i < $calc_days; $i++) { ... }` | | 🟠 警告 | `get_book_day_time_info` 方法 | **方法过长且职责混杂**：该方法超 300 行，混合了数据查询、时间切片、团购券校验、营业规则过滤、状态标记等逻辑，违反单一职责原则（SRP），极难测试与维护。 | 拆分为独立私有方法：`_fetch_booking_data()`、`_calculate_time_slots()`、`_apply_voucher_rules()`、`_filter_unavailable_times()`。 | 见下方重构建议 | | 🟠 警告 | 多处（`set_shop_config`、`set_room_info` 等） | **滥用 `get_instance()` 且越权修改控制器上下文**：模型中频繁调用 `get_instance()`，且直接修改 `$CI->operational_scene`、`$CI->package_not_available_time` 等全局变量，破坏 MVC 分层，易引发请求间状态污染。 | 模型应仅负责数据与业务计算，上下文数据通过参数传入或返回值传出。移除冗余 `get_instance()`。 | `// 错误<br>$CI = &get_instance();<br>$CI->operational_scene = $room_operational_scene;<br><br>// 正确<br>return $room_operational_scene; // 由控制器接收并赋值` | | 🟠 警告 | `get_book_day_time_info` 循环内 | **数组分块逻辑存在越界风险**：`array_chunk($v['time_range_arr'], $time_range_arr_count / 2)` 依赖偶数长度假设。若数组长度为奇数，后半段切片可能缺失或错位，导致后续 `array_intersect` 判断失效。 | 使用 `ceil()` 明确分界，或改用索引范围截取。 | `$mid = (int)ceil($count / 2);<br>$first_half = array_slice($arr, 0, $mid);<br>$last_half = array_slice($arr, $mid);` | | 🟡 建议 | `_get_un_book_time` 及多处 | **`json_decode` 未做安全校验**：直接对数据库字段 `_time_info_new` 解码，若数据损坏或非 JSON 格式，将返回 `null`，后续 `foreach` 或数组访问会触发 `Warning`。 | 增加类型校验与默认值回退。 | `$raw = $row['_time_info_new'] ?? '[]';<br>$decoded = json_decode($raw, true);<br>$book_time_info = is_array($decoded) ? $decoded : [];` | | 🟡 建议 | 类属性定义区 | **魔法数字/字符串泛滥**：大量硬编码如 `86400`、`3600`、`'1'`、`'-1'`、`'7'`，降低可读性且易引发维护错误。 | 提取为类常量或配置文件。 | `const SECONDS_PER_DAY = 86400;<br>const STATUS_AVAILABLE = '1';<br>const STATUS_UNAVAILABLE = '-1';` | | 🟡 建议 | 构造函数 `__construct` | **依赖隐式加载且无异常处理**：`$this->load->library('Tuangou');` 未校验加载结果，若库不存在或 Redis 连接失败，后续 `$this->tuangou->...` 将触发致命错误。 | 增加加载校验与降级策略，或改用依赖注入。 | `if (!$this->load->is_loaded('tuangou')) {<br> $this->load->library('Tuangou');<br>}<br>if (method_exists($this->tuangou, 'get_voucher_info_from_redis')) {<br> $this->tuangou->get_voucher_info_from_redis();<br>}` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **修复静态缓存串扰**：立即移除 `self::$book_days_info`、`self::$shop_data` 等静态缓存，或改为基于 `merchant_id + shop_id + date` 的实例级缓存数组。 2. **禁止方法内修改实例属性**：将 `$this->book_days += 1` 改为局部变量计算，确保对象状态纯净。 3. **清理控制器上下文污染**：移除模型中对 `$CI->xxx` 的赋值操作，改为通过方法参数传递或返回值暴露数据。 ### 🛠 后续重构与优化方向 1. **方法拆分与职责收敛**： - 将 `get_book_day_time_info` 拆分为： - `prepareContext($params)`：加载门店配置、营业时间、团购券信息。 - `fetchBookingData($date)`：查询当日/前后日包厢占用数据。 - `generateTimeSlots($business_from, $business_end)`：生成基础时间片。 - `applyFilters($slots, $rules)`：依次应用最低时长、清扫时间、团购券、停用规则等过滤逻辑。 2. **引入现代 PHP 特性**： - 添加类型声明（`declare(strict_types=1);`、参数类型、返回类型）。 - 使用 `DateTimeImmutable` 替代 `strtotime` + 字符串拼接，彻底规避时区与跨天计算隐患。 3. **性能优化**： - 将重复的 `get_shop_setting` 调用合并为批量查询或引入配置缓存层。 - 对 `array_intersect`、`array_merge` 操作的大数组进行预过滤，避免在循环中重复计算。 4. **安全与健壮性**： - 对 `$params` 输入进行严格类型校验（如 `filter_var`、`ctype_digit`）。 - 为 `json_decode`、`array_chunk`、`strtotime` 等易错函数添加防御性编程。 > 💡 **提示**：由于代码在 `_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
1779353488
updated_unix
1779353488
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel