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 588 from issue
id
588
repo_id
21
index
264
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260616 - 测试
content
## 自动代码审查报告 **分支**: pay-260616 **提交**: `dad66f9f1
## 自动代码审查报告 **分支**: pay-260616 **提交**: `dad66f9f1363ff3183644f193fc203c25afc4c7a` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-08 14:37:09 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：该 Model 承载了较复杂的套餐查询、VIP折扣计算、多端适配（触屏/微信预定）等业务逻辑。整体功能完整，但存在**严重的 SQL 注入风险**、**类外部执行代码破坏框架生命周期**、**多处逻辑隐患与性能瓶颈**。代码风格未遵循 PSR-12，魔法数字与硬编码较多，且存在明显的 DRY 原则违背。 - **风险等级**：🔴 高（主要源于未过滤的 SQL 拼接与框架生命周期冲突） > 📌 **框架说明**：根据目录结构（`system/`、`application/models/`）及 `$this->load->model()`、`$this->db->query()` 等特征，推断项目基于 **CodeIgniter 3.x** 架构。若确为自研 `phpci` 框架，请结合其官方文档对生命周期与模型加载机制进行适配调整。 --- ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | 顶部全局 / 第 4-5 行 | 在类外部直接调用 `get_instance()` 并加载模型。CI 框架在自动加载模型时会解析文件，顶部执行代码会导致重复实例化、全局变量污染或 Fatal Error。 | 移除顶部代码，将依赖加载移至构造函数或按需加载。 | ```php<br>public function __construct()<br>{<br> parent::__construct();<br> $this->load->model('Simple_model');<br>}<br>``` | | 🔴 严重 | `get_book_package_list` / `get_hot_sale_top5` | 原生 SQL 直接拼接用户输入（`$shop_name`、`$param['special_merchant_id']`、`$shop_id`），未使用参数绑定或查询构造器，存在 **SQL 注入** 风险。 | 使用 CI 查询构造器（Query Builder）或 `$this->db->query($sql, $binds)` 进行预处理。 | ```php<br>$sql .= " AND `shop`.`_name` LIKE ?";<br>$binds[] = "%{$shop_name}%";<br>// 执行时：<br>$data = $this->db->query($sql, $binds)->result_array();<br>``` | | 🟠 警告 | `get_package_price_list` / `foreach` 循环 | 使用 `unset($row)` 试图移除数组元素。在 `foreach ($list as &$row)` 中，`unset($row)` 仅销毁引用，**不会从原数组中删除元素**，导致无效数据仍被返回。 | 使用键值遍历或 `array_filter` 安全移除。 | ```php<br>foreach ($list as $key => $row) {<br> if ($book_arrival_time < $order_end_time) {<br> unset($list[$key]);<br> continue;<br> }<br>}<br>``` | | 🟠 警告 | `get_package_price_list` / 折扣计算 | `$goods_discount_rate = $goods_discount_rate / 10; $goods_discount_rate_dis = $goods_discount_rate / 10;` 连续除以 10 逻辑晦涩，且与 `vip_level_1_discount_rate` 计算重复，易引发精度丢失或业务误解。 | 封装折扣转换方法，明确除以 100 的业务意图，避免魔法运算。 | ```php<br>protected function convertDiscountRate(int $rate): float<br>{<br> return $rate / 100.0; // 明确将 95 转为 0.95<br>}<br>``` | | 🟠 警告 | `get_package_price_list` / 分类逻辑 | 循环内频繁调用 `array_unshift($result['drink'], $row)`。该操作时间复杂度为 `O(N)`，嵌套在循环中导致整体 `O(N²)`，数据量大时严重拖慢响应。 | 分离推荐与普通数组，循环结束后使用 `array_merge` 合并。 | ```php<br>$drink_rec = []; $drink_norm = [];<br>foreach ($list as $row) {<br> if ($row['package_type'] == 2) {<br> $row['recommend'] > 0 ? $drink_rec[] = $row : $drink_norm[] = $row;<br> }<br>}<br>$result['drink'] = array_merge($drink_rec, $drink_norm);<br>``` | | 🟠 警告 | 全局静态变量 | `static public $_shop_id_arr = [];` 等静态缓存。在 PHP-FPM 环境下虽会随请求重置，但若未来迁移至 Swoole/Workerman 等常驻内存环境，将导致**数据跨请求污染**。 | 改用 CI Cache 驱动或请求级上下文存储，避免静态变量。 | ```php<br>$this->load->driver('cache');<br>$cache_key = "shop_ids_{$city_id}_{$lat}_{$lon}";<br>$shop_id_arr = $this->cache->get($cache_key) ?: $this->ahead_shop_model->get_id_by_distance(...);<br>``` | | 🟡 建议 | 多处方法内部 | 频繁在业务方法中调用 `$this->load->model()`。CI 模型加载有开销，重复调用降低性能且破坏单一职责。 | 统一在 `__construct()` 中加载，或配置 `config/autoload.php` 自动加载。 | ```php<br>public function __construct()<br>{<br> parent::__construct();<br> $this->load->model(['Ahead_vip_model', 'Ahead_vip_level_model', 'ahead_room_package_model']);<br>}<br>``` | | 🟡 建议 | 全局 | 未遵循 PSR-12 规范：缺乏严格类型声明、参数/返回值类型缺失、命名风格不统一（如 `$baseParams` 与 `$params` 混用）、注释含个人署名与日期。 | 添加 `declare(strict_types=1);`，补充类型提示，清理冗余注释，统一命名规范。 | ```php<br>declare(strict_types=1);<br>class Ahead_room_package_infos_model extends Simple_model<br>{<br> public function get_package_price_list(array $baseParams, array $params, int $uid = 0): array<br> { ... }<br>}<br>``` | --- ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **立即修复 SQL 注入漏洞**：`get_book_package_list` 与 `get_hot_sale_top5` 中的原生 SQL 拼接必须替换为参数绑定或 Query Builder。这是生产环境的高危红线。 2. **移除类外部执行代码**：删除文件顶部的 `$CI = &get_instance();` 及模型加载语句，严格遵循 CI 模型生命周期规范，避免框架初始化异常。 3. **修正数组移除逻辑**：将 `unset($row)` 替换为 `unset($list[$key])`，否则过滤逻辑形同虚设，可能导致前端展示异常套餐。 ### 🛠 后续重构与优化方向 1. **折扣与价格计算抽离**：当前 `get_package_price_list` 与 `get_price_set_detail` 中存在大量重复的 VIP 等级价格计算、折扣率转换逻辑。建议提取为独立的 `PriceCalculator` 服务类或 Trait，实现单一职责与高内聚。 2. **消除 N+1 查询与循环加载**：多处使用 `foreach` 循环内 `load->model()` 或 `get_one()`。可考虑使用 `WHERE IN` 批量查询后在 PHP 层映射，或引入 CI 的 `db->query()` 预加载关联数据。 3. **规范化配置与常量管理**：将 `SCREEN_RENEW_TYPE`、`SCREEN_NEW_TYPE`、折扣除数、业务场景标识（`'3' => 'card_'`）等魔法值统一收敛至配置文件或类常量，提升可维护性。 4. **类型安全与静态分析**：逐步补充 `declare(strict_types=1)`、参数类型声明及 PHPDoc 规范。可引入 `PHPStan` 或 `Psalm` 进行静态扫描，提前拦截类型错误与未定义变量访问（如 `$flavors_data[$fvv[0]]` 未判空）。 > ⚠️ **局限性说明**：提供的代码在 `get_book_package_list_group_by_shop` 方法处被截断，该方法的完整逻辑、分页处理及排序分支未能纳入本次审查。建议补充完整代码后再次进行针对性评估。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780900629
updated_unix
1780900629
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel