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 326 from issue
id
326
repo_id
21
index
92
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260519 - Merge remote-tracking branch
🔍 代码审查报告：pay-260519 - Merge remote-tracking branch 'pay/pay-260519' into
...
content
## 自动代码审查报告 **分支**: pay-260519 **提交**: `f90ce2136
## 自动代码审查报告 **分支**: pay-260519 **提交**: `f90ce21368ba163b3bba3807fa37d2362192d9e9` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-05-26 14:46:11 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：代码实现了复杂的门店预订时段计算与会员注册/支付逻辑，业务覆盖全面。但存在明显的架构反模式（如“上帝对象”、巨型方法）、硬编码敏感信息、SQL 拼接隐患及状态污染问题。财务计算未考虑浮点精度，部分方法职责过重，可维护性与安全性亟待提升。 - **风险等级**：🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `Ahead_vip_model.php`<br>`check_is_register()` ~185行 | **SQL 注入风险**：直接拼接 `$uid` 和 `$mobile` 到 SQL 条件字符串中，未做转义或参数绑定。 | 使用查询构建器或参数化查询替代字符串拼接。 | `$this->db->where("(_ahead_user_id = ? OR _mobile = ?) AND (_expiry_date = -1 OR _expiry_date > ?)", [$uid, $mobile, $now_time]);` | | 🔴 严重 | `Ahead_vip_model.php`<br>`pay_order()` ~485行 | **SQL 注入与金额精度风险**：直接拼接金额到 UPDATE 语句，且未处理浮点数精度问题，易导致余额计算偏差。 | 使用框架 Query Builder 安全更新；金额统一转为“分”（整数）或使用 `bcmath` 计算。 | `$this->db->set('_account', '_account - ' . (int)($amount * 100), FALSE)->where($vip_where)->update($this->table_name);` | | 🔴 严重 | `Ahead_vip_model.php`<br>第 12 行 | **硬编码敏感信息**：`public $encrypt = "Vs!Fs7VT";` 将加密密钥直接暴露在源码中，违反安全规范。 | 移至配置文件或环境变量，通过配置中心读取。 | `protected $encrypt; public function __construct() { parent::__construct(); $this->encrypt = config_item('vip_encrypt_key'); }` | | 🟠 警告 | `Ahead_shop_book_time_info_model.php`<br>`get_book_days_info()` ~118行 | **状态污染/逻辑缺陷**：`$this->book_days += 1;` 直接修改实例属性。若该方法被多次调用或结合静态缓存，会导致天数累加错误。 | 使用局部变量进行计算，绝不修改实例状态。 | `$days_to_check = $add_day ? $this->book_days + 1 : $this->book_days; for ($i = 0; $i < $days_to_check; $i++) { ... }` | | 🟠 警告 | `Ahead_shop_book_time_info_model.php`<br>`get_book_day_time_info()` ~145行起 | **巨型方法/违反单一职责**：该方法超 300 行，混合了时段生成、套餐校验、门店配置加载、DB 查询、时间交集计算等，极难测试与维护。 | 拆分为多个私有方法，如 `load_unavailable_times()`, `calculate_package_slots()`, `filter_by_business_hours()` 等。 | 见下方重构建议 | | 🟠 警告 | `Ahead_shop_book_time_info_model.php`<br>`set_shop_config()` & `set_room_info()` | **代码重复**：两个方法中加载场景配置、计算 `shop_config_scene`、获取 `minute_unit` 的逻辑高度重复。 | 提取为私有方法 `get_scene_config_prefix()` 和 `load_scene_settings()` 复用。 | `private function get_scene_prefix($scene) { return match($scene) { '2' => 'billiards_', '3' => 'card_', '4' => 'tavern_', default => '' }; }` | | 🟡 建议 | 全局多处 | **魔法数字泛滥**：`1`, `-1`, `86400`, `600`, `3600` 等硬编码散落在业务逻辑中，语义不明。 | 定义类常量或枚举，提升可读性与可维护性。 | `const STATUS_AVAILABLE = 1; const STATUS_UNAVAILABLE = -1; const SECONDS_PER_DAY = 86400; const MINUTE_UNIT_DEFAULT = 30;` | | 🟡 建议 | `Ahead_vip_model.php`<br>`update_acount()` ~340行 | **方法名拼写错误**：`acount` 应为 `account`，影响代码规范性与 IDE 自动补全。 | 重命名方法，并全局搜索替换所有调用点。 | `public function update_account($uid, $cardNo, ...)` | | 🟡 建议 | 全局多处 | **冗余的 `$CI = &get_instance();`**：模型已继承框架基类，频繁获取实例不仅冗余，且在部分框架版本中可能引发性能损耗。 | 直接使用 `$this->load`、`$this->config` 或 `$this->db`。若需访问控制器属性，应通过参数传递或依赖注入。 | 移除 `$CI = &get_instance();`，改用 `$this->load->model('xxx');` 或 `$this->config->item('xxx');` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **立即修复 SQL 注入漏洞**：`check_is_register()` 与 `pay_order()` 中的字符串拼接必须替换为参数化查询或 Query Builder 安全方法。 2. **消除状态污染**：`get_book_days_info()` 中的 `$this->book_days += 1` 必须改为局部变量，否则在高并发或连续请求下会导致预订天数计算错乱。 3. **敏感信息脱敏**：将 `$encrypt` 密钥移至配置文件，禁止硬编码。 4. **财务计算规范化**：所有涉及金额的计算（充值、扣款、折扣）必须使用整数（单位：分）或 `bcmath` 扩展，避免浮点数精度丢失导致账目不平。 ### 🛠 后续重构与优化方向 1. **拆分巨型方法**：`get_book_day_time_info()` 是典型的核心瓶颈。建议按业务阶段拆分： - `load_base_config()`：加载门店、包厢、套餐基础配置。 - `fetch_unavailable_ranges()`：批量获取已预订、锁定、停业、规则停用时间。 - `generate_time_slots()`：基于营业时间和最小单位生成基础时间段。 - `apply_package_rules()`：应用团购券/套餐的可用时间、跨天、星期限制逻辑。 - `filter_and_format()`：执行交集/差集计算，返回最终状态。 2. **优化数据库查询**：当前存在明显的 N+1 查询隐患（如循环内调用 `get_shop_setting`、`get_room_lock_book_time`）。建议改为批量查询（`WHERE IN`）或引入 Redis 缓存热点配置与预订状态。 3. **规范框架适配**：代码高度疑似基于 CodeIgniter 架构。若 `phpci` 为内部定制框架，请确认 `$CI = &get_instance()` 是否为官方推荐用法。建议统一使用 `$this->load` 链式调用，并遵循 PSR-12 规范（如属性可见性、常量定义、类型声明）。 4. **补充类型声明与异常处理**：建议为方法参数和返回值添加 PHP 7.4+ 类型提示（如 `array`, `int`, `bool`），并在关键业务节点（如支付回调、事务提交）增加结构化日志记录，便于生产环境排查。 > ⚠️ **局限性说明**：提供的代码片段在 `Ahead_shop_book_time_info_model.php` 末尾及 `Ahead_vip_model.php` 的 `notify_preorder()` 处被截断，部分依赖的全局辅助函数（如 `mergeTimeRanges`, `shiftTimeRange`, `timeToHour` 等）未提供实现。以上审查基于可见代码逻辑推断，若辅助函数内部存在未处理的边界条件或性能问题，建议一并进行审查。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779777971
updated_unix
1779777971
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel