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 565 from issue
id
565
repo_id
21
index
251
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260616 - 1
content
## 自动代码审查报告 **分支**: pay-260616 **提交**: `32e0218d7
## 自动代码审查报告 **分支**: pay-260616 **提交**: `32e0218d795d10c3148394c106ba7f9a3472685e` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-05 14:26:19 --- ## 1. 审查摘要 - **代码质量评分**：4/10 分 - **总体评价**：该模型承载了复杂的业务逻辑（套餐查询、VIP折扣、社区模式时间过滤、跨天校验等），但存在大量重复代码与超长方法。核心缺陷在于**严重的 SQL 注入风险**、**数组过滤逻辑失效**以及**时间边界判断脆弱**。未遵循 PSR-12 规范，模型依赖加载方式陈旧，整体可维护性与安全性亟待重构。 - **风险等级**：🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `get_book_package_list` / `get_hot_sale_top5` | **SQL 注入漏洞**：`$shop_name`、`$special_merchant_id`、`$offset`、`$page_size` 等外部参数未经转义直接拼接入原生 SQL，攻击者可构造恶意输入破坏查询或窃取数据。 | 全面替换为查询构造器（Query Builder）或使用 `$this->db->escape()` / 预处理绑定。 | `$this->db->like('shop._name', $shop_name, 'both');`<br>`$this->db->limit($page_size, $offset);` | | 🔴 严重 | `get_package_price_list` / `get_screen_list` | **数组过滤失效**：`foreach ($list as &$row) { unset($row); continue; }` 仅解绑了引用变量 `$row`，**并未从原数组 `$list` 中移除元素**，导致无效套餐仍会返回给前端。 | 使用键值遍历或 `array_filter` 正确移除元素。 | `foreach ($list as $key => $row) { if ($book_arrival_time < $order_end_time) { unset($list[$key]); } }` | | 🟠 警告 | `check_use_package` | **跨天时间判断逻辑脆弱**：`$ltime = $data['_end_time'] - 86400;` 配合 `if ($now_time_str < $data['_start_time'] && $now_time_str > $ltime)` 在夏令时、跨多日或边界分钟时易产生误判，且硬编码 `86400` 缺乏语义。 | 提取为独立的时间区间校验方法，使用 `DateTime` 或标准化秒数比对逻辑，增加单元测试覆盖。 | 建议封装 `isTimeInRange($now, $start, $end, $weekCycle)` 方法，避免散落的 `if/else`。 | | 🟠 警告 | 全局多处 | **模型频繁加载**：每个方法内部重复调用 `$this->load->model()`，增加文件 I/O 与内存开销，且破坏单一职责。 | 移至 `__construct()` 统一加载，或使用框架的依赖注入容器/懒加载机制。 | `public function __construct() { parent::__construct(); $this->load->model(['Ahead_vip_model', 'Ahead_yc_merchant_model', ...]); }` | | 🟠 警告 | `get_package_by_fields` | **索引失效与全表扫描**：`FIND_IN_SET('{$businessDay}',_disabled_date) = ''` 会导致数据库无法使用索引，且字符串直接拼接存在隐患。 | 建议将 `_disabled_date` 拆分为独立关联表或使用 JSON 字段+虚拟列索引；若保留，需使用参数绑定。 | `$this->db->where("FIND_IN_SET(?, _disabled_date) = 0", $businessDay);` | | 🟡 建议 | 文件头部/类定义 | **违反 PSR-12 规范**：类名 `Ahead_room_package_infos_model` 应为大驼峰 `AheadRoomPackageInfosModel`；文件顶部 `$CI = &get_instance();` 属于全局污染。 | 遵循 PSR-12 命名规范，移除文件级 `$CI` 引用，在方法内按需 `$CI = &get_instance();` 或使用框架 DI。 | `class AheadRoomPackageInfosModel extends Simple_model` | | 🟡 建议 | 多处 | **魔法数字与硬编码**：`4`, `2`, `100`, `0.01`, `86400`, `3` 等散落各处，缺乏业务语义，后期维护成本极高。 | 提取为类常量或配置文件，提升可读性与可配置性。 | `const PACKAGE_TYPE_GROUP_BUY = 4; const DISCOUNT_DIVISOR = 10; const SECONDS_PER_DAY = 86400;` | | 🟡 建议 | `get_book_package_detail` | **敏感数据未脱敏**：直接返回 `$shop_data['_manager_mobile']`，若接口未做权限控制，将导致商家手机号泄露。 | 在返回前进行掩码处理，或根据调用方权限动态过滤字段。 | `$mobile = $shop_data['_manager_mobile'] ?? '';`<br>`$room_package_infos_data['shop_info']['manager_mobile'] = substr($mobile, 0, 3) . '****' . substr($mobile, -4);` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **立即修复 SQL 注入**：所有原生 SQL 拼接处必须替换为框架提供的 Query Builder 或预处理语句。这是生产环境的高危漏洞。 2. **修正数组过滤逻辑**：将 `unset($row)` 改为 `unset($list[$key])` 或使用 `array_filter`，否则业务过滤形同虚设。 3. **统一依赖加载**：将分散在各方法中的 `$this->load->model()` 收敛至构造函数，或采用框架推荐的依赖注入方式，降低运行时开销。 ### 🛠 后续重构与优化方向 1. **方法拆分与逻辑复用**： - `get_package_price_list` 与 `get_screen_list` 存在大量重复代码（社区模式时间判断、VIP折扣计算、时间格式化）。建议提取为私有方法：`_applyCommunityTimeFilter()`, `_calculateVipPrice()`, `_formatPackageTime()`。 - 控制单个方法行数在 80 行以内，降低圈复杂度（Cyclomatic Complexity）。 2. **时间处理标准化**： - 废弃 `date('Ymd')`、`strtotime()` 的散乱调用，统一使用 `DateTime` 或框架内置的时间工具类。 - 跨天套餐校验建议采用区间重叠算法：`(startA <= endB) && (endA >= startB)`，避免硬编码 `86400` 带来的边界错误。 3. **缓存机制升级**： - `static public $_shop_id_arr` 仅为请求级内存缓存，无过期策略且易引发并发脏读。建议替换为框架的 `Cache` 组件（如 Redis/Memcached），并设置合理的 TTL。 4. **框架适配说明**： - 代码结构高度类似 **CodeIgniter 3**。若 `phpci` 为 CI3 的定制分支，建议保留 `$this->load->model()` 但移至构造函数；若 `phpci` 已升级至支持 PSR 标准或依赖注入（如 CI4/Laravel 风格），请全面改用构造函数注入或 `model()` 辅助函数。 - 对于不确定的 `phpci` 生命周期钩子或配置加载方式，建议查阅官方文档中关于 `Model` 初始化与 `DB` 驱动绑定的章节。 > ⚠️ **局限性说明**：提供的代码在 `get_book_package_list_group_by_shop` 方法处截断，未能审查完整逻辑。若该方法包含类似的原生 SQL 拼接或复杂分页逻辑，请同步按上述安全与性能规范进行审查。建议补充完整代码或提供对应单元测试用例以便进行更精准的边界验证。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780640780
updated_unix
1780640780
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel