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-260616
**提交**: `230bfa5c0f0903d47edda3edd8f2585a60f34596`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 14:59:24

---

## 1. 审查摘要
- **代码质量评分**：6.0 / 10 分
- **总体评价**：代码实现了社区商家营收明细的查询、过滤与导出功能，业务逻辑基本完整。但存在明显的 SQL 注入风险、N+1 查询性能瓶颈、分页统计逻辑缺陷以及多处不符合现代 PHP/CI 规范的写法。整体可维护性与安全性有待提升。
- **风险等级**：🔴 高

## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 第 138-145 行 | **SQL 注入漏洞**：`$pay_platform_where` 数组通过字符串拼接直接构造 SQL 片段，若 `$params['pay_platform_arr']` 来源不可信，将导致严重注入风险。 | 使用查询构建器的参数绑定机制，或强制类型转换后使用框架提供的 `or_where` / `where_in` 组合。 | ```php // 安全写法示例 $this->db->group_start(); foreach ($params['pay_platform_arr'] as $pp) { $parts = explode('_', $pp); $this->db->or_where('a._pay_platform', (int)$parts[0]); if (!empty($parts[1])) { $this->db->or_where('a._second_pay_platform', (int)$parts[1]); } } $this->db->group_end(); ``` |
| 🔴 严重 | 第 1 行 | **全局作用域执行实例化**：`$CI = &get_instance();` 放在类外部，文件被 `include/require` 时即执行，破坏框架生命周期，易引发内存泄漏或上下文污染。 | 移除全局代码，将模型加载移至类构造函数中。 | ```php public function __construct() { parent::__construct(); $this->load->model('Report_model'); } ``` |
| 🟠 警告 | 第 108-122 行 | **分页统计逻辑缺陷**：`$count` 与 `$sum_data` 仅在 `$params['page'] == '1'` 时计算。翻至第 2 页时变量未定义，虽用 `??` 兜底，但会导致分页总数与总金额显示为 0，破坏业务体验。 | 移除 `if ($params['page'] == '1')` 条件，始终执行统计查询；或引入缓存机制避免重复计算。 | ```php // 始终计算统计值 $count = $this->count($where); $sum_data = $this->get_one($where, 'sum(...) as total_amount'); ``` |
| 🟠 警告 | 第 158-165 行 | **N+1 查询性能瓶颈**：在 `foreach ($data as &$v)` 循环内调用 `$this->ahead_book_order_model->get_one()`，数据量大时将产生大量数据库往返请求。 | 提取所有关联订单 ID，使用批量查询（如 `get_data_by_ids`）后通过数组映射回填。 | ```php $book_ids = array_unique(array_column($data, 'order_id')); $book_orders = $this->ahead_book_order_model->get_data_by_ids($book_ids, '_id,_shop_name,_arrival_time,_end_time', '_id'); // 循环内直接 $v['book_info'] = $book_orders[$book_order_id] ?? [];``` |
| 🟠 警告 | 第 118-120 行 | `json_decode` 未指定关联数组参数，且未处理解析失败情况。若传入非法 JSON，后续逻辑可能静默失败。 | 添加 `true` 参数并配合 `json_last_error()` 或 `JSON_THROW_ON_ERROR` 进行校验。 | ```php $arr = json_decode($params['order_type_arr'], true); if (!is_array($arr)) { $arr = []; } ``` |
| 🟡 建议 | 第 1 行 | **类名不符合 PSR-12 规范**：`Jh_community_shop_revenues_detail_model` 使用下划线命名，现代 PHP 推荐 PascalCase。 | 重命名为 `JhCommunityShopRevenuesDetailModel`，并同步更新自动加载与调用处。 | `class JhCommunityShopRevenuesDetailModel extends Report_model` |
| 🟡 建议 | 第 95-98 行 | **日期参数未校验**：`strtotime($params['start_time'])` 若传入非法字符串将返回 `false`，导致查询条件变为 `0` 或 `-1`。 | 增加日期格式校验与默认值回退机制。 | ```php $start = strtotime($params['start_time'] ?? date('Y-m-d')); $end = strtotime($params['end_time'] ?? date('Y-m-d', strtotime('+1 day'))); ``` |
| 🟡 建议 | 多处 | **模型重复加载**：`$this->load->model()` 在多个方法内重复调用，增加框架解析开销。 | 统一在 `__construct()` 中加载，或使用依赖注入容器管理。 | 见 🔴 严重第 1 行示例 |

> 📌 **框架适配说明**：代码呈现典型的 **CodeIgniter 3** 架构特征（如 `$CI = &get_instance()`、`$this->load->model()`、自定义 `Report_model` 查询构建器）。若 `phpci` 为内部定制框架，请确保 `enforce_con_db()`、`count()`、`select()` 等方法与框架底层 DB 驱动兼容，并查阅官方文档确认查询数组 `$where` 是否支持原生 SQL 片段注入。

## 3. 总结与行动建议

### 🔑 优先修复的关键问题
1. **立即修复 SQL 注入**：第 138-145 行的字符串拼接查询必须改为参数化查询或框架安全构建器，这是最高优先级安全漏洞。
2. **修正分页统计逻辑**：移除 `page == 1` 的条件判断，确保任何分页状态下都能正确返回 `count` 与 `total_amount`。
3. **消除 N+1 查询**：将循环内的单条查询改为批量查询，预计可将该接口响应时间降低 60%~80%（尤其在数据量 > 50 条时）。

### 🛠 后续重构与优化方向
- **架构规范化**：将全局 `$CI` 引用移至构造函数，遵循 PSR-12 命名规范，逐步引入 PHP 8 类型声明（如 `public function get_community_revenues_list(int $merchant_id, array $params, bool $export = false): array`）。
- **查询构建器抽象**：当前 `$where` 数组结构高度定制化，建议封装为独立的 `QueryBuilder` 类或使用框架原生 Active Record，避免手动拼接 `join`、`where`、`like` 导致维护困难。
- **配置与常量分离**：`ORDER_TYPE`、`revenues_pay_platform_arr` 等硬编码数据建议移至 `config/` 目录或数据库字典表，便于运营动态调整。
- **防御性编程**：对 `$params` 增加统一校验层（如使用 `Form_validation` 或自定义 DTO），确保 `start_time`、`end_time`、`shop_id` 等关键字段类型与范围合法后再进入业务逻辑。

> 💡 若需针对 `phpci` 框架的特定查询构建器或缓存机制进行深度适配，请提供 `Report_model` 核心方法签名或框架官方文档链接，以便输出更精准的底层优化方案。

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