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 230 from issue
id
230
repo_id
21
index
40
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260519 - 预订时长组合支付计时价格没有用会员价
content
## 自动代码审查报告 **分支**: pay-260519 **提交**: `9eb182323
## 自动代码审查报告 **分支**: pay-260519 **提交**: `9eb18232353528d3423c023b1e5acd6b231a0380` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-05-20 10:14:29 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：该模型类承载了极其复杂的预订时间计算、套餐校验、门店配置加载及多场景（KTV/台球/棋牌/酒馆）兼容逻辑。整体业务意图清晰，但存在严重的架构与工程化问题：文件级全局实例调用、方法职责严重超载、实例状态被意外修改、硬编码泛滥且缺乏输入校验。代码末尾被截断，存在直接导致运行时致命错误的风险。 - **风险等级**：🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `L1-L4` (文件顶部) | 在类定义外部直接执行 `$CI = &get_instance();` 和 `$CI->load->model()`。若文件在框架引导完成前被 `include/require`，将触发 `Fatal Error`。且破坏了类的封装性。 | 移除文件级代码，将模型加载移至构造函数或依赖框架自动加载机制。 | ```php<br>// ❌ 错误<br>$CI = &get_instance();<br>$CI->load->model('Simple_model');<br><br>// ✅ 正确：在 __construct 中处理<br>public function __construct() {<br> parent::__construct();<br> // 框架通常会自动加载父类模型，无需手动 load<br>}``` | | 🔴 严重 | 文件末尾 (`$next_first_hour_range = reset($next_`) | **代码被意外截断**，逻辑不完整。缺失闭合括号、数组赋值及后续时间计算逻辑，部署后将直接导致 `Parse Error` 或 `Undefined variable`。 | 立即补全截断代码，并确保所有控制流（`if/foreach`）正确闭合。 | *(需根据完整业务逻辑补全)* | | 🟠 警告 | `get_book_days_info()` | `$this->book_days += 1;` 直接修改了实例属性。若同一请求中多次调用该方法，`$book_days` 会持续累加，导致后续日期计算越界或逻辑错乱。 | 使用局部变量计算，避免修改实例状态。 | ```php<br>// ❌ 错误<br>if ($add_day) { $this->book_days += 1; }<br><br>// ✅ 正确<br>$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）。混合了配置读取、时间范围计算、套餐校验、价格查询、状态过滤等逻辑。圈复杂度过高，极难测试与维护。 | 拆分为多个私有方法：`_loadShopConfig()`, `_calculateUnbookableRanges()`, `_applyVoucherRules()`, `_filterAvailableSlots()` 等。 | *(架构级重构，见下方建议)* | | 🟠 警告 | 全局多处 | 大量使用魔法数字（如 `3600`, `86400`, `1`, `2`, `3`, `7`, `1200` 等）和硬编码场景标识。降低可读性，且修改业务规则时需全局搜索。 | 提取为类常量或配置数组。 | ```php<br>const SECONDS_PER_HOUR = 3600;<br>const SECONDS_PER_DAY = 86400;<br>const SCENE_KTV = '1';<br>const SCENE_BILLIARDS = '2';<br>const DEFAULT_BOOK_DAYS = 7;``` | | 🟠 警告 | 静态属性缓存 (`self::$shop_data` 等) | 使用 `static` 属性缓存请求级数据，但未提供清理/重置机制。在 CLI 脚本、长连接或单元测试复用实例时，极易引发脏数据污染或内存泄漏。 | 提供 `resetCache()` 方法，或在请求生命周期结束前清理；或改用依赖注入的缓存服务。 | ```php<br>public static function resetCache(): void {<br> self::$shop_data = [];<br> self::$date_time_info = [];<br> // 清理其他静态缓存...<br>}``` | | 🟡 建议 | 类属性定义 | 所有属性均为 `public`，破坏封装性。外部可直接修改 `$book_room_id`, `$tuangou` 等核心状态，导致不可预知的副作用。 | 改为 `protected` 或 `private`，通过 `setter/getter` 或构造函数注入控制赋值。 | ```php<br>protected $book_room_id = 0;<br>protected $book_time_limit = 3600;<br><br>public function setBookRoomId(int $id): self {<br> $this->book_room_id = $id;<br> return $this;<br>}``` | | 🟡 建议 | 全局辅助函数依赖 | 频繁调用未定义的全局函数（如 `throwError`, `mergeTimeRanges`, `shiftTimeRange`, `timeToHour` 等）。若未正确加载将导致运行时崩溃。 | 确认这些函数属于框架 Helper 或已 `require`。建议改用面向对象的服务类或框架内置工具，便于 Mock 测试。 | *(依赖框架规范，建议统一使用 `$this->helper->xxx()` 或命名空间工具类)* | | 🟡 建议 | `$params` 输入处理 | `$params['date']`, `$params['merchant_id']` 等直接用于业务逻辑与数据库查询，缺乏类型校验与基础过滤。 | 增加基础校验，防止非法参数传入导致 SQL 异常或逻辑越权。 | ```php<br>$date = $params['date'] ?? '';<br>if (!preg_match('/^\d{8}$/', $date)) {<br> throwError('日期格式错误');<br>}``` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **补全截断代码**：文件末尾逻辑中断是最高优先级风险，需立即核对完整源码并修复语法错误。 2. **移除文件级 `$CI` 调用**：将 `get_instance()` 和模型加载移至 `__construct()` 内部，避免框架引导期崩溃。 3. **消除状态副作用**：修复 `get_book_days_info()` 中 `$this->book_days += 1` 的累加逻辑，改用局部变量计算。 4. **输入参数校验**：对 `$params` 中的关键字段（日期、商户ID、门店ID）增加格式与类型校验，防止脏数据穿透。 ### 🛠 后续重构与优化方向 1. **方法拆分与职责分离**： - `get_book_day_time_info()` 建议拆分为：`_resolveTimeConstraints()`、`_applyVoucherFilters()`、`_calculateAvailability()`。 - 将复杂的 `if/else` 状态判断提取为独立的策略类或规则引擎，降低圈复杂度。 2. **架构模式升级**： - 引入 **DTO (Data Transfer Object)** 封装 `$params` 与返回结果，替代松散数组。 - 将硬编码的“场景标识”、“时间单位”、“业务规则”抽离至配置中心或常量类，支持动态下发。 3. **性能与内存优化**： - `array_intersect(...array_values($all_room_book_time))` 在房间数较多时可能触发内存溢出。建议改用迭代器或分批计算。 - 静态缓存需配合请求生命周期管理，或在高频接口中改用 Redis/Memcached 替代内存静态变量。 4. **规范与安全对齐**： - 严格遵循 **PSR-12** 规范（命名空间、方法驼峰、属性可见性、控制流缩进）。 - 若 `phpci` 框架与 CodeIgniter 3 存在差异，请重点核对 `$this->load->model()` 与 `$CI = &get_instance()` 的生命周期差异，必要时改用依赖注入容器（DIC）管理模型实例。 > 💡 **注**：本次审查基于提供的代码片段。若 `phpci` 为高度定制框架，部分服务加载方式（如 `load->model`）可能与标准 CI3 不同，建议结合官方文档确认最佳实践。代码中大量依赖的全局辅助函数（如 `mergeTimeRanges`）未提供实现，审查时默认其已正确加载，实际部署前请确保其存在且经过单元测试覆盖。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779243269
updated_unix
1779243269
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel