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

## 自动代码审查报告

**分支**: pc-260519
**提交**: `7a9c58a8ac9316ae11f2bc5f93e7160f6c6401a4`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-19 15:48:13

---

## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑链路基本完整，覆盖了套餐设置、列表查询、同步更新及团购券绑定等核心场景。但代码中存在**多处高危 SQL 注入漏洞**、**严重的 N+1 查询与循环内资源加载**、**事务与异常处理不规范**等问题。整体可维护性与安全性亟待提升。
- **风险等级**：🔴 高

> 📌 **框架说明**：代码结构高度符合 `CodeIgniter 3` 规范（如 `$CI = &get_instance()`、`$this->load->model()`、`$this->db->trans_start()` 等）。若 `phpci` 为内部定制框架，以下修复方案基于 CI3 标准 Query Builder 语法，核心安全与性能原则通用，具体底层 API 请以官方文档为准。

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_room_package_infos_model.php` `set_package_price` 方法内 | `$this->load->model('');` 传入空字符串，将触发框架致命错误或 Warning。 | 移除该行，或传入正确的模型名称。模型应在类顶部或方法开头按需加载。 | `// 直接删除此行` |
| 🔴 严重 | `Ahead_room_package_infos_model.php` `get_package_price_list` 方法内 | 使用 `addslashes()` 拼接 `LIKE` 条件，无法防御 SQL 注入且绕过框架自动转义机制。 | 使用框架查询构建器的 `like()` 方法或参数绑定。 | `$this->db->like('ahead_room_package._name', $params['package_name']);` |
| 🔴 严重 | `Ahead_room_package_infos_model.php` `mult_set_room_package_service_charge_rate` | 原生 SQL 直接拼接 `$merchantId`、`$goods_types`、`$shop_id`，存在严重 SQL 注入风险。 | 改用 Query Builder 或 `$this->db->escape()` 处理所有外部变量。 | `$this->db->set('_service_charge_rate', $serviceChargeRate)->where($where)->update($this->table_name);` |
| 🔴 严重 | `Ahead_room_package_infos_model.php` `batch_update` 方法内 | WHERE 条件使用字符串拼接 `$merchant_id`、`$shop_id`、`$package_id_arr`，存在注入风险。 | 使用数组条件或查询构建器，避免手动拼接 SQL。 | `$this->db->where('_merchant_id', $merchant_id)->where('_shop_id', $shop_id)->where_in('_package_id', $package_id_arr)->update($this->table_name, $update);` |
| 🟠 警告 | `Ahead_room_package_infos_model.php` `set_package_price` 方法内 | `$packageInfo` 可能为 `null`，直接访问 `$packageInfo['_type']` 会触发 PHP Warning。 | 增加空值校验后再访问数组键。 | `if (!empty($packageInfo) && $packageInfo['_type'] == self::PACKAGE_GROUP_TYPE) { ... }` |
| 🟠 警告 | `Ahead_room_package_infos_model.php` `set_package_price` 方法内 | `$startTimeStr` 默认值为 `0`，`explode(":", "0")` 后访问索引 `[1]` 导致 `Undefined offset`。 | 增加时间格式校验或安全访问逻辑。 | `if (strpos($startTimeStr, ':') !== false) { $parts = explode(':', $startTimeStr); ... }` |
| 🟠 警告 | `Ahead_room_package_infos_model.php` `get_package_price_list` 方法内 | `foreach` 循环内部调用 `$this->load->model()`，每次迭代重复加载，严重消耗性能。 | 将模型加载移至循环外部或类初始化阶段。 | `// 移至方法开头：$this->load->model(['ahead_room_package_goods_model', 'ahead_merchant_goods_model']);` |
| 🟠 警告 | `Ahead_room_package_infos_model.php` `get_package_price_list` 方法内 | 循环内执行 `$this->ahead_room_package_goods_model->select()`，典型 N+1 查询问题。 | 收集所有 `package_id` 后批量查询，再通过内存映射关联数据。 | `$ids = array_column($list['rows'], 'package_id'); $goods = $this->ahead_room_package_goods_model->where_in('_package_id', $ids)->get()->result_array();` |
| 🟠 警告 | `Ahead_room_package_infos_model.php` `update_with_link` 方法内 | `$this->ahead_room_package_infos_model->update_v2()` 疑似笔误，应为调用当前实例方法。 | 修正为 `$this->update_v2()`。 | `$this->update_v2($updateData, ['_shop_id' => $shop_id, '_link_id' => $link_id]);` |
| 🟡 建议 | `Ahead_room_package_infos_model.php` `get_package_price_list` 方法内 | `timeToHour()` 结果立即被 `hourTimeTurn()` 覆盖，产生冗余计算。 | 删除第一行转换调用。 | `// 删除 $row['start_time'] = timeToHour($row['start_time']);` |
| 🟡 建议 | `Ahead_room_package_infos_model.php` `get_price_set_detail` 方法内 | 使用 `@json_decode()` 抑制错误，掩盖 JSON 格式异常，不利于排查。 | 移除 `@`，使用 `json_last_error()` 校验。 | `$groups = json_decode($priceSetInfo['groups'], true); if (json_last_error() !== JSON_ERROR_NONE) { $groups = []; }` |
| 🟡 建议 | `Ahead_shop_group_buying_coupon_model.php` `sync_list` 方法内 | 循环内 `$this->get_one()` 判断记录是否存在，N+1 查询拖慢同步性能。 | 批量查询已存在的 `_deal_group_id`，构建映射数组后判断。 | `$exist_map = array_column($this->where_in('_deal_group_id', $group_ids)->get()->result_array(), '_id', '_deal_group_id');` |
| 🟡 建议 | 全局 | 缩进、大括号位置、命名风格未严格遵循 PSR-12；部分魔法数字未提取为常量。 | 使用 `PHP-CS-Fixer` 或 IDE 自动格式化；提取硬编码值。 | 统一 4 空格缩进，方法名采用 `camelCase`，类名 `PascalCase`。 |

## 3. 总结与行动建议

### 🚨 优先修复的关键问题
1. **彻底消除 SQL 注入风险**：`mult_set_room_package_service_charge_rate` 与 `batch_update` 中的原生 SQL 拼接必须替换为 Query Builder 或预处理语句。`addslashes` 绝不能用于防注入。
2. **解决 N+1 查询与循环内资源加载**：将 `get_package_price_list` 和 `sync_list` 中的循环内 `load->model()` 与 `select()` 提取为批量查询。数据量稍大时即可导致接口超时或数据库连接池耗尽。
3. **规范事务与异常处理**：当前 `try...catch` 中手动 `trans_rollback()` 后直接 `return FALSE`，若 `throwError` 内部为 `exit()` 或 `die()`，事务将无法回滚。建议统一使用 CI3 的 `$this->db->trans_start(); ... $this->db->trans_complete();` 机制，依赖框架自动回滚，或在 `catch` 中抛出标准 `Exception` 交由上层统一处理。

### 🛠 后续重构与优化方向
- **架构分层优化**：Model 层应仅负责数据存取，时间格式化（`timeToHour`/`hourTimeTurn`）、权限过滤（`priv_shop_ids`）、JSON 解析等逻辑建议下沉至 Service 层或 Helper 中，保持 Model 纯净。
- **常量与配置集中管理**：如 `PACKAGE_GROUP_TYPE = 4`、`STATUS_TYPE_ARR`、平台映射等硬编码值，建议统一收敛至 `config/` 目录或枚举类中，便于后续扩展与维护。
- **引入静态分析工具**：建议接入 `PHPStan` 或 `Psalm` 进行类型检查，配合 `PHP_CodeSniffer` 强制 PSR-12 规范，可在 CI/CD 流水线中拦截低级语法与规范问题。
- **框架适配确认**：若 `phpci` 对 Query Builder 或事务 API 有定制封装，请核对 `$this->db->where_in()`、`$this->db->like()` 等方法的参数顺序是否与 CI3 一致，必要时查阅框架源码或官方文档进行微调。

---
*此 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 203 in issue