SQLite Web: gitea.db

id

Primary key. INTEGER NOT NULL

repo_id

INTEGER

index

INTEGER

poster_id

INTEGER

original_author

TEXT

original_author_id

INTEGER

name

TEXT

content

## 自动代码审查报告

**分支**: 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 public function __construct() { parent::__construct(); $this->load->model('Simple_model'); } ``` |
| 🔴 严重 | `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 $sql .= " AND `shop`.`_name` LIKE ?"; $binds[] = "%{$shop_name}%"; // 执行时： $data = $this->db->query($sql, $binds)->result_array(); ``` |
| 🟠 警告 | `get_package_price_list` / `foreach` 循环 | 使用 `unset($row)` 试图移除数组元素。在 `foreach ($list as &$row)` 中，`unset($row)` 仅销毁引用，**不会从原数组中删除元素**，导致无效数据仍被返回。 | 使用键值遍历或 `array_filter` 安全移除。 | ```php foreach ($list as $key => $row) { if ($book_arrival_time < $order_end_time) { unset($list[$key]); continue; } } ``` |
| 🟠 警告 | `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 protected function convertDiscountRate(int $rate): float { return $rate / 100.0; // 明确将 95 转为 0.95 } ``` |
| 🟠 警告 | `get_package_price_list` / 分类逻辑 | 循环内频繁调用 `array_unshift($result['drink'], $row)`。该操作时间复杂度为 `O(N)`，嵌套在循环中导致整体 `O(N²)`，数据量大时严重拖慢响应。 | 分离推荐与普通数组，循环结束后使用 `array_merge` 合并。 | ```php $drink_rec = []; $drink_norm = []; foreach ($list as $row) { if ($row['package_type'] == 2) { $row['recommend'] > 0 ? $drink_rec[] = $row : $drink_norm[] = $row; } } $result['drink'] = array_merge($drink_rec, $drink_norm); ``` |
| 🟠 警告 | 全局静态变量 | `static public $_shop_id_arr = [];` 等静态缓存。在 PHP-FPM 环境下虽会随请求重置，但若未来迁移至 Swoole/Workerman 等常驻内存环境，将导致**数据跨请求污染**。 | 改用 CI Cache 驱动或请求级上下文存储，避免静态变量。 | ```php $this->load->driver('cache'); $cache_key = "shop_ids_{$city_id}_{$lat}_{$lon}"; $shop_id_arr = $this->cache->get($cache_key) ?: $this->ahead_shop_model->get_id_by_distance(...); ``` |
| 🟡 建议 | 多处方法内部 | 频繁在业务方法中调用 `$this->load->model()`。CI 模型加载有开销，重复调用降低性能且破坏单一职责。 | 统一在 `__construct()` 中加载，或配置 `config/autoload.php` 自动加载。 | ```php public function __construct() { parent::__construct(); $this->load->model(['Ahead_vip_model', 'Ahead_vip_level_model', 'ahead_room_package_model']); } ``` |
| 🟡 建议 | 全局 | 未遵循 PSR-12 规范：缺乏严格类型声明、参数/返回值类型缺失、命名风格不统一（如 `$baseParams` 与 `$params` 混用）、注释含个人署名与日期。 | 添加 `declare(strict_types=1);`，补充类型提示，清理冗余注释，统一命名规范。 | ```php declare(strict_types=1); class Ahead_room_package_infos_model extends Simple_model { public function get_package_price_list(array $baseParams, array $params, int $uid = 0): array { ... } } ``` |

---

## 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 由代码审查服务自动创建*

TEXT

milestone_id

INTEGER

priority

INTEGER

is_closed

INTEGER

is_pull

INTEGER

num_comments

INTEGER

ref

TEXT

deadline_unix

INTEGER

created_unix

INTEGER

updated_unix

INTEGER

closed_unix

INTEGER

is_locked

INTEGER NOT NULL (default 0

content_version

INTEGER NOT NULL (default 0

time_estimate

INTEGER NOT NULL (default 0

Update row 588 in issue