|
450
|
22
|
51
|
1
|
|
0
|
🔍 代码审查报告：app-260519 - 1
|
## 自动代码审查报告
**分支**: app-260519
**提交**: `900258a12 ## 自动代码审查报告
**分支**: app-260519
**提交**: `900258a1291f0558e08644cef69b1313517a8ea8`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 10:14:39
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑覆盖较完整，能处理复杂的预订、退款、开房及多支付渠道场景。但代码存在明显的“上帝类”倾向，方法冗长、职责混杂；事务边界管理不严谨，存在 SQL 拼接隐患与 N+1 查询性能瓶颈；魔法数字泛滥，可维护性与扩展性较弱。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 文件顶部 (1-3行) | 在类外部直接执行 `$CI = &get_instance();` 并加载模型。每次 `include/require` 该文件都会执行，易导致重复加载、上下文污染或内存泄漏。 | 移至类的 `__construct()` 构造函数中，或使用框架的自动加载机制。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🔴 严重 | `refund_by_notify` 约第108行 | `$log_where` 使用字符串拼接构造 SQL 条件：`'_relation_id="' . $order_data['_id'] . '" and ...'`。若 `_id` 未严格校验，存在 SQL 注入风险，且违背框架查询构建器规范。 | 使用框架提供的查询构建器或参数绑定，彻底杜绝字符串拼接 SQL。 | `$this->db->where('_relation_id', $order_data['_id'])->where('_status', 1)->where_in('_type', [5, 13]);` |
| 🔴 严重 | `invalid_book` / `openRoomByCommunityOrder` / `notifyCommunityBookOrder` | **事务嵌套与手动回滚冲突**。CI 架构下事务通过计数器管理，在已开启事务的方法中调用含 `trans_start/rollback` 的子方法，或手动调用 `trans_rollback()`，极易破坏事务计数器，导致数据不一致或提前提交。 | 统一事务边界。仅在顶层业务方法开启事务，子方法仅返回状态或抛出异常，不直接操作事务生命周期。 | 顶层使用 `try { $this->db->trans_start(); ... $this->db->trans_complete(); } catch(\Exception $e) { $this->db->trans_rollback(); throw $e; }` |
| 🟠 警告 | `get_list` / `get_detail` 循环体 | **N+1 查询性能瓶颈**。在 `foreach` 中循环调用 `$this->ahead_yc_order_model->get_one()` 获取关联数据，数据量增大时将引发严重数据库压力。 | 收集所有关联 ID，使用 `where_in` 批量查询，或在主查询中使用 `JOIN` 一次性拉取，再在内存中映射。 | `$ids = array_column($order_info, 'relation_order_id'); $orders = $this->ahead_yc_order_model->where_in('_id', $ids)->get()->result_array();` |
| 🟠 警告 | 全局多处 | **魔法数字泛滥**。大量使用 `1,2,3,4,5,8,9,14,17` 等硬编码表示状态、支付渠道、业务模式，可读性差且极易因版本迭代引发逻辑错乱。 | 提取为类常量或独立配置类，统一命名管理。 | `const STATUS_PAID = 1; const PAY_PLATFORM_WX = 1; const BUSINESS_MODEL_COMMUNITY = 2;` |
| 🟡 建议 | `refund_by_notify` / `openRoomByCommunityOrder` | **违反单一职责原则 (SRP)**。单个方法超过 150 行，混合了状态更新、外部 API 调用、日志记录、消息推送、打印机控制等逻辑，难以测试与维护。 | 按业务域拆分。将退款、订单创建、通知推送、硬件交互等抽取为独立 Service 类或私有方法。 | 创建 `RefundService::process()`, `OrderNotifyService::send()`, `PrinterService::print()` 等。 |
| 🟡 建议 | 全局多处 | **松散比较与类型不严谨**。如 `$is_refund == '1'`、`$invalid_type != '4'`，PHP 松散比较易引发隐式类型转换漏洞。 | 统一使用严格比较 `===`，并在方法入口进行类型断言或强制转换。 | `if ((int)$is_refund === 1)` 或 `if ($invalid_type !== '4')` |
| 🟡 建议 | 全局多处 | **模型重复加载**。多处重复调用 `$this->load->model(...)`，增加框架解析开销。 | 高频依赖模型在构造函数中统一加载，或配置自动加载。 | `public function __construct() { parent::__construct(); $this->load->model(['model_a', 'model_b']); }` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复 SQL 拼接隐患**：立即将 `refund_by_notify` 中的 `$log_where` 字符串拼接替换为查询构建器或参数绑定，杜绝注入风险。
2. **重构事务管理**：梳理 `invalid_book`、`openRoomByCommunityOrder` 及其调用链的事务边界。移除子方法中的 `trans_start/rollback`，统一在顶层使用 `try-catch` 包裹事务，确保异常时能正确回滚。
3. **消除 N+1 查询**：对 `get_list` 和 `get_detail` 中的循环查询进行批量优化，使用 `JOIN` 或 `where_in` 批量获取关联数据，降低数据库 IO 压力。
### 🛠 后续重构与优化方向
1. **架构拆分**：当前 Model 承担了过多 Controller/Service 的职责。建议引入 Service 层处理复杂业务流（如退款、开房、通知），Model 层仅负责数据持久化与基础查询。
2. **常量/枚举化**：建立全局状态字典或配置类，替换所有魔法数字。可结合 PHP 8.1+ 的 `enum` 特性（若环境支持）提升类型安全。
3. **统一异常处理**：当前混用 `throwError()` 与 `try-catch`。建议统一使用标准 `Exception` 或框架内置异常类，配合全局异常处理器返回标准化错误响应。
4. **代码规范对齐**：严格遵循 PSR-12，统一方法命名（如 `setTablename` 与 `set_table_name` 混用）、缩进与注释规范。
> 📌 **框架适配说明**：代码呈现典型的 CodeIgniter 架构特征（如 `get_instance()`、`$this->load->model()`、`$this->db->trans_start()`）。若 `phpci` 为内部定制框架，其事务计数器机制或模型加载策略可能与原生 CI 存在差异，请务必查阅 `phpci` 官方文档中关于 **事务嵌套** 与 **模型生命周期** 的说明，并据此调整上述建议。
>
> ⚠️ **局限性提示**：提供的代码在 `_add_order_data` 方法末尾截断，未能完整审查该辅助方法及后续逻辑。建议补充完整代码以便进行全链路数据流与边界条件验证。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780366479
|
1780366479
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
460
|
22
|
52
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `b81fc07a2 ## 自动代码审查报告
**分支**: app-260616
**提交**: `b81fc07a2830943f9142c042f75df2aace449b54`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 13:45:48
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了门店包厢时间预订的核心业务逻辑，但存在明显的架构设计缺陷。方法职责过重、时间处理逻辑脆弱、频繁加载模型与重复的 JSON 序列化操作导致性能与维护成本较高。部分边界条件与异常处理缺失，存在潜在的运行时报错风险。
- **风险等级**：🟠 中（存在逻辑隐患、性能瓶颈及不规范用法，需优先重构核心方法）
> ⚠️ **局限性说明**：提供的代码在 `get_shop_book_info_v2` 方法中截断，本次审查仅基于可见部分。若截断处包含关键逻辑（如数据库事务、最终返回结构），请补充后二次审查。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 文件顶部 / 全局作用域 | `$CI = &get_instance();` 定义在类外部，且 `$CI->load->model('Simple_model');` 写法错误。会导致作用域污染、内存泄漏，且在 CLI 或单元测试中直接报错。 | 移除全局 `$CI` 声明。模型加载应统一在 `__construct()` 中完成，或直接继承基类。 | `// 删除顶部代码<br>class Ahead_shop_book_time_info_model extends Simple_model {<br> public function __construct() {<br> parent::__construct();<br> $this->load->model('ahead_family_servers_model');<br> // 其他模型按需加载<br> }<br>}` |
| 🔴 严重 | `_update_un_book_time` 方法内 | `array_intersect(...array_values($all_room_book_time))` 当 `$all_room_book_time` 为空数组时，解包后 `array_intersect()` 接收 0 个参数，PHP 8.0+ 会抛出 `ArgumentCountError`。 | 增加数组元素数量判断，或使用 `array_reduce` 安全求交集。 | `if (count($all_room_book_time) === 0) {<br> $time_info['un_book_time'] = [];<br>} elseif (count($all_room_book_time) === 1) {<br> $time_info['un_book_time'] = array_values($all_room_book_time)[0];<br>} else {<br> $time_info['un_book_time'] = array_intersect(...array_values($all_room_book_time));<br>}` |
| 🟠 警告 | `update_day_book_info` / 时间循环 | 时间步长循环 `for ($i = $start; $i <= $end; $i += $this->min_minute_unit_time)` 未对齐时间单位。若 `$start` 不是 300 秒的整数倍，会导致时间段错位或遗漏，且跨天时 `strtotime($end_date . ' 00:00:00')` 依赖服务器时区。 | 使用 `DateTime` 与 `DateInterval` 生成时间片，或强制对齐起始时间。避免硬编码时区依赖。 | `$start = new DateTime('@' . $book_order['_arrival_time']);<br>$end = new DateTime('@' . $book_order['_end_time']);<br>$interval = new DateInterval('PT5M');<br>while ($start <= $end) {<br> $book_time_arr[] = $start->format('H:i');<br> $start->add($interval);<br>}` |
| 🟠 警告 | `_update_un_book_time` / JSON解析 | `json_decode($data['_time_info'], true)` 未校验返回值。若数据库字段损坏或为空字符串，将返回 `null`，后续 `isset($time_info['room_book_time'])` 会触发 `Warning` 或逻辑异常。 | 增加类型校验与默认值兜底。 | `$time_info = json_decode($data['_time_info'], true);<br>if (!is_array($time_info)) {<br> $time_info = ['room_book_time' => [], 'un_book_time' => []];<br>}` |
| 🟠 警告 | `get_shop_book_info` 方法 | 方法长度超 250 行，混合了数据查询、时间计算、状态过滤、清扫时间叠加、跨天合并等逻辑。违反单一职责原则（SRP），极难测试与维护。 | 拆分为独立方法：`fetchRoomList()`, `calculateBusinessSlots()`, `mergeBookingData()`, `applyCleaningBuffer()`。 | *(架构重构建议，见第3部分)* |
| 🟡 建议 | 多处 | 硬编码魔法数字与字符串（如 `300`, `256`, `'CYD'`, `'ZD'`, `'00:00'`, `'23:55'`）。降低可读性且不利于后续配置化。 | 提取为类常量或配置文件。`256` 应替换为 `JSON_THROW_ON_ERROR`。 | `const MIN_UNIT_SECONDS = 300;<br>const ORDER_TYPE_BOOK = 'CYD';<br>const ORDER_TYPE_BILL = 'ZD';<br>// 使用 json_encode($data, JSON_THROW_ON_ERROR)` |
| 🟡 建议 | 全局/方法内 | 频繁使用 `$this->load->model()` 在业务方法内部动态加载。CI/类CI框架中，模型加载有开销，且破坏依赖可见性。 | 统一在构造函数加载，或使用依赖注入（DI）容器。若 `phpci` 支持懒加载，请确认其性能损耗。 | `// 构造函数中预加载<br>$this->load->model(['ahead_family_servers_model', 'ahead_book_order_model', 'ahead_bill_model']);` |
| 🟡 建议 | `update_day_time_info_by_bill` | 直接访问 `$CI->add_order_by_refund` 全局属性。该属性未定义且易被外部篡改，破坏封装性。 | 通过方法参数传入状态标识，或从 Session/Request 对象中安全获取。 | `public function update_day_time_info_by_bill($bill_data, $type, $is_refund = false) {<br> // 使用 $is_refund 替代 $CI->add_order_by_refund<br>}` |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **移除全局 `$CI` 实例化**：立即将 `$CI = &get_instance();` 从文件顶部移除，改为在类方法内按需使用 `$this->load` 或 `$this->db`。
2. **修复 `array_intersect` 解包崩溃风险**：在 `_update_un_book_time` 中增加数组判空逻辑，防止 PHP 8+ 环境直接 Fatal Error。
3. **JSON 解析安全加固**：所有 `json_decode` 调用必须配合 `is_array()` 校验，并设置合理的默认结构，避免脏数据导致后续逻辑断裂。
### 🛠 后续重构与优化方向
1. **时间处理现代化**：
- 废弃 `date()` / `strtotime()` 字符串拼接，全面迁移至 `DateTimeImmutable` 与 `DatePeriod`。
- 将时间片生成逻辑抽离为独立的 `TimeSlotGenerator` 工具类，支持时区、步长、边界对齐配置。
2. **方法职责拆分（SRP）**：
- `get_shop_book_info` 建议拆分为：
- `getBusinessTimeConfig()`：获取营业时间与跨天标识
- `fetchRawBookingData()`：批量查询预订单与账单（避免 N+1）
- `calculateAvailability()`：计算可预订状态与清扫缓冲
- 使用 DTO（数据传输对象）替代深层嵌套数组，提升类型安全。
3. **性能优化**：
- **批量查询**：`_update_un_book_time` 中多次 `get_one` 可合并为 `where_in` 批量查询。
- **缓存策略**：门店营业时间、清扫时间、包厢列表等低频变动数据应引入 Redis/文件缓存，避免每次请求重复查库。
- **JSON 字段优化**：若 `_time_info` 数据量持续增长，建议评估是否拆分为独立的 `room_time_slots` 关联表，利用数据库索引替代 PHP 内存数组运算。
4. **框架适配提示**：
- 若 `phpci` 为内部定制框架，请确认其是否支持 **Repository 模式** 或 **Service 层**。当前 Model 承担了过多业务逻辑，建议将时间计算、状态流转移至 `ShopBookingService`，Model 仅负责数据持久化。
- 代码中大量使用 `do_log()`、`roundTime()`、`hourToTime()` 等未定义函数，请确保这些 Helper 已正确加载且无全局状态污染。
> 💡 **下一步建议**：建议先编写针对 `update_day_book_info` 与 `_update_un_book_time` 的单元测试，覆盖同天、跨天、边界时间、脏 JSON 等场景。通过测试用例驱动重构，可大幅降低线上回归风险。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780379148
|
1780379148
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
461
|
22
|
53
|
1
|
|
0
|
🔍 代码审查报告：app-260519 - 1
|
## 自动代码审查报告
**分支**: app-260519
**提交**: `31456ab28 ## 自动代码审查报告
**分支**: app-260519
**提交**: `31456ab2853fc7a9a5abd0f0496e6cb8f06c4ee4`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 13:46:58
---
## 1. 审查摘要
- **代码质量评分**：5/10 分
- **总体评价**：代码实现了门店包厢时间预订的核心业务逻辑，功能完整但架构耦合度较高。时间计算、JSON 序列化与数据库交互混杂在单一 Model 中，违反单一职责原则。存在明显的框架误用、潜在的性能瓶颈及容错缺失，在并发或数据异常场景下易引发系统级问题。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 文件顶部 / 全局 | `$CI = &get_instance();` 在类外部直接调用，严重违反框架生命周期。在 CLI 环境、单元测试或异步任务中会直接抛出 `Fatal Error`，且造成全局状态污染。 | 移除文件顶部的 `$CI` 引用。依赖注入或模型加载应在类构造函数中完成，或交由框架自动加载器管理。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🔴 严重 | `_update_un_book_time` / `update_day_book_info` | `json_decode($data['_time_info'], true)` 未做异常容错。若数据库字段被意外修改或存储非法 JSON，将返回 `null`，后续数组操作会触发 `Warning/Fatal Error` 导致流程中断。 | 增加 JSON 解析校验与降级策略，确保始终返回结构化数组。 | `$decoded = json_decode($data['_time_info'], true);<br>$time_info = is_array($decoded) ? $decoded : ['room_book_time' => [], 'un_book_time' => []];` |
| 🟠 警告 | `_update_un_book_time` / `get_shop_book_info` | 方法内部频繁调用 `$this->load->model()`。每次执行都会重新实例化模型类，造成不必要的内存分配与性能损耗。 | 将常用模型移至类属性，在 `__construct()` 中统一加载，或使用框架的依赖注入容器。 | `protected $bill_model; public function __construct() { $this->bill_model = $this->load->model('ahead_bill_model', '', true); }` |
| 🟠 警告 | `get_shop_book_info` / `v2` | 存在大量重复逻辑（跨天处理、订单状态映射、时间格式化），单方法超 300 行。且隐式存在 N+1 查询风险（循环内调用 `get_one` 或关联查询）。 | 抽取公共逻辑至独立 Service 类（如 `BookingTimeCalculator`）。使用 `WHERE IN` 批量查询替代循环单查，构建内存 Map 进行关联。 | `$ids = array_column($orders, '_id');<br>$map = $this->model->get_data_by_ids($ids, '*', '_id');` |
| 🟠 警告 | 多处时间处理逻辑 | 大量使用 `date()`、`strtotime()` 及硬编码 `86400`、`'00:00'`。未处理时区问题，且 `for` 循环按固定秒数步进，若起始时间戳未对齐 `min_minute_unit_time`，可能导致死循环或时间漏算。 | 统一使用 `DateTime` 对象处理时间，明确业务时区。将步进逻辑改为 `while` 循环配合 `modify()` 或严格对齐算法。 | `$dt = new DateTime('@' . $start, new DateTimeZone('Asia/Shanghai'));<br>while ($dt < $end) { $times[] = $dt->format('H:i'); $dt->modify('+5 minutes'); }` |
| 🟡 建议 | 全局多处 | 魔法数字/字符串泛滥（如 `1`, `2`, `'CYD'`, `'ZD'`, `256`, `'-1'`）。降低可读性，后续业务规则变更时极易遗漏修改点。 | 在类顶部定义语义化常量，替换所有硬编码值。`json_encode` 的 `256` 应替换为 `JSON_UNESCAPED_SLASHES` 等命名常量。 | `const TYPE_ADD = 1; const TYPE_DEL = 2; const PREFIX_BOOK = 'CYD'; const JSON_OPTS = JSON_UNESCAPED_UNICODE \| JSON_UNESCAPED_SLASHES;` |
| 🟡 建议 | `_update_un_book_time` | `array_intersect(...array_values($all_room_book_time))` 使用展开运算符 `...`。当房间数较多或时间段数组较大时，可能超出 PHP 函数参数数量限制（默认 10000+）或引发内存溢出。 | 改用循环逐步求交集，或使用 `array_reduce` 安全计算。 | `$common = array_shift($all_room_book_time);<br>foreach ($all_room_book_time as $arr) { $common = array_intersect($common, $arr); }` |
| 🟡 建议 | `update_day_book_info` | `@return true` 类型声明不规范。PHPDoc 应使用 `@return bool`。且方法直接返回 `true` 掩盖了底层数据库操作失败的可能（如 `insert`/`update` 返回 `false`）。 | 修正 PHPDoc 类型。增加数据库操作结果校验，失败时抛出异常或返回 `false`，便于上层捕获。 | `if (!$this->insert($where)) { throw new RuntimeException('Insert failed'); } return true;` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **移除全局 `$CI = &get_instance();`**：立即将其迁移至类构造函数或依赖注入容器中，避免在 CLI/测试/异步场景下引发致命崩溃。
2. **加固 JSON 解析容错**：所有 `json_decode` 处必须增加 `is_array()` 校验并提供默认空结构，防止脏数据导致服务雪崩。
3. **消除隐式 N+1 查询**：在 `get_shop_book_info` 及相关方法中，将循环内的单条查询改为批量 `WHERE IN` 查询，并在内存中构建索引映射，预计可降低 60%~80% 的数据库交互耗时。
### 🛠 后续重构与优化方向
1. **架构分层（SRP 拆分）**：当前 Model 承担了数据持久化、复杂时间计算、JSON 组装、看板数据格式化等多重职责。建议拆分为：
- `Ahead_shop_book_time_info_model`：仅负责 CRUD 与基础数据映射。
- `BookingTimeService`：处理时间步进、跨天切割、可用时间交集计算。
- `BookingDashboardFormatter`：负责看板数据组装、状态映射与前端格式转换。
2. **时间处理标准化**：废弃裸 `strtotime`/`date` 混用模式。引入 `Carbon` 或原生 `DateTimeImmutable`，统一时区配置（如 `Asia/Shanghai`），并封装 `TimeSlotGenerator` 工具类处理 `5分钟` 步进对齐逻辑。
3. **框架适配说明**：代码呈现典型的 **CodeIgniter 3** 架构特征。若 `phpci` 为内部定制框架，请查阅其官方文档确认：
- 是否支持构造函数依赖注入（推荐替代 `$this->load->model()`）。
- 模型基类 `Simple_model` 是否已封装事务处理（当前多处更新未包裹事务，跨天/跨表更新存在数据不一致风险）。
4. **引入事务保护**：`update_day_book_info` 与 `update_day_time_info_by_bill` 涉及多表/多日期状态联动，建议包裹在 `$this->db->trans_start()` / `$this->db->trans_complete()` 中，确保原子性。
> 💡 **提示**：以上建议代码均符合 PHP 7.4+ / 8.x 语法规范。重构前建议补充单元测试覆盖边界时间（如 `23:55`、跨天 `00:00`、夏令时切换）及异常 JSON 场景，以验证逻辑健壮性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780379218
|
1780379218
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
467
|
22
|
54
|
1
|
|
0
|
🔍 代码审查报告：app-260519 - 订单详情返回用户的会员卡号
|
## 自动代码审查报告
**分支**: app-260519
**提交**: `7136a48c5 ## 自动代码审查报告
**分支**: app-260519
**提交**: `7136a48c5358a221cdd324dfa2658ccab6831642`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 15:48:11
---
## 1. 审查摘要
- **代码质量评分**：4.5 / 10 分
- **总体评价**：该控制器承载了路由分发、身份鉴权、多业务逻辑处理及响应格式化，属于典型的“上帝控制器”。代码中存在敏感信息硬编码、明文密码比对、绕过框架输入过滤等高危安全隐患；架构上严重违背单一职责原则，维护成本极高。整体符合早期 CI3 遗留系统的常见写法，但距离现代 PHP 安全与工程规范差距较大。
- **风险等级**：🔴 高
> 📌 **框架说明**：根据目录结构（`system/`、`application/`）、常量 `BASEPATH` 及 `$this->load->` 等语法，该代码实际运行于 **CodeIgniter 3.x** 框架。下文审查将基于 CI3 最佳实践与 PSR-12 规范进行。若 `phpci` 为内部定制框架，核心安全与架构建议依然适用。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 约 L340-L346 | **敏感配置硬编码**：科大讯飞 TTS 的 `APPID`、`APISecret`、`APIKey` 直接暴露在源码中，极易通过版本库泄露。 | 移至独立配置文件（如 `config/xfyun.php`）或服务器环境变量，通过 `$this->config->item()` 读取。 | `// config/xfyun.php<br>$config['xfyun'] = [<br> 'APPID' => getenv('XFYUN_APPID'),<br> 'APISecret' => getenv('XFYUN_SECRET'),<br> 'APIKey' => getenv('XFYUN_KEY')<br>];` |
| 🔴 严重 | 约 L280 | **明文密码存储与比对**：`$data['_discount_pwd'] != $_old_password` 疑似明文比对，违反密码安全基线。 | 数据库必须存储 `password_hash()` 结果，验证时使用 `password_verify()`。 | `if (!password_verify($_old_password, $data['_discount_pwd'])) {<br> $this->error_response('旧密码不正确');<br>}` |
| 🔴 严重 | L45, L18 | **绕过框架输入过滤**：直接使用 `$_POST` 与 `php://input`，未使用 CI 的 `$this->input` 类，丧失 XSS 过滤与全局过滤机制。 | 统一使用 `$this->input->raw_input_stream` 或 `$this->input->post()`，并开启全局 XSS 过滤。 | `$raw = $this->input->raw_input_stream;<br>$this->stream = json_decode($raw, true) ?: [];` |
| 🟠 警告 | L1-L100+ | **上帝控制器/路由耦合**：`__construct` 与 `index()` 混合了鉴权、配置加载、业务路由、数据组装，代码超千行，难以测试与维护。 | 将鉴权逻辑抽离至 `BaseController` 或 CI Hook；将 `switch` 分支拆分为独立方法；业务逻辑下沉至 Service/Model。 | 使用 CI 路由配置映射 `function` 到独立方法，或采用命令模式分发。 |
| 🟠 警告 | 多处 `error_response` | **破坏框架生命周期**：错误处理疑似直接 `exit/die`，导致 CI 的 `post_controller`、Session 写入、日志记录等钩子无法执行。 | 改为抛出标准异常，或使用 CI 的 `show_error()`；若必须返回 JSON，应在统一输出层处理。 | `throw new \RuntimeException('token参数错误', 401);`<br>配合全局异常处理器统一输出 JSON。 |
| 🟠 警告 | L105, L115, L200+ | **重复加载模型/配置**：在 `switch` 各分支中频繁 `load->model()` 与 `load->config()`，增加 I/O 开销。 | 在 `__construct` 中统一加载高频模型；低频模型按需加载但需避免重复调用。 | `$this->load->model(['ahead_yc_merchant_model', 'ahead_yc_shop_model']);` |
| 🟡 建议 | L12, L14 | **拼写与命名不一致**：`AplicationController` 拼写错误；`$current_vresion` 拼写错误；属性命名混合驼峰与下划线。 | 修正拼写，统一使用 `camelCase` 命名类属性，遵循 PSR-12。 | `class MerchantAppServer extends ApplicationController`<br>`public $currentVersion = '2.0';` |
| 🟡 建议 | L118, L300+ | **魔法数字/硬编码泛滥**：如 `23142`、`600`、`533`、`10` 等无业务语义的常量散落各处。 | 提取为类常量或配置文件，添加明确注释。 | `const MERCHANT_CAMBODIA_ID = 23142;`<br>`const PAY_TYPE_MEMBER = 10;` |
| 🟡 建议 | 全文 | **PSR-12 规范缺失**：缩进不统一、超长行未换行、缺少类型声明、注释风格混杂。 | 使用 `PHP-CS-Fixer` 自动格式化；添加 `declare(strict_types=1);` 及参数类型提示。 | 配置 `.php-cs-fixer.php` 并执行 `php-cs-fixer fix`。 |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题（P0/P1）
1. **立即移除硬编码密钥**：将 `xfyun_tts_config` 迁移至环境变量或加密配置，并执行密钥轮换。
2. **修复密码验证逻辑**：全面排查 `_discount_pwd` 等密码字段，实施 `password_hash`/`password_verify` 迁移脚本。
3. **统一输入获取方式**：替换所有 `$_POST`/`php://input` 为 `$this->input->raw_input_stream`，确保全局 XSS 过滤生效。
4. **规范错误响应机制**：避免在控制器中直接 `exit`，改用异常机制或统一 JSON 响应中间件，保障 CI 生命周期完整。
### 🛠 后续重构与优化方向
1. **架构拆分（SRP 原则）**：
- 将 `index()` 中的 `switch` 逻辑改造为 **路由分发器** 或 **命令模式**。每个 `function` 对应一个独立的 Service 类方法。
- 鉴权、Token 解析、权限校验抽离至 `BaseController::initAuth()` 或 CI `pre_controller` Hook。
2. **性能优化**：
- 合并重复的模型加载与配置读取。
- 针对 `array_merge` 频繁操作大数组的场景，改用引用传递或按需构建。
- 对高频查询（如门店配置、权限列表）引入 CI Cache 或 Redis 缓存。
3. **工程规范落地**：
- 引入 `PHP-CS-Fixer` 与 `PHPStan` 进行静态检查与格式化。
- 为关键业务方法补充单元测试（PHPUnit），覆盖鉴权失败、参数异常、权限越权等边界条件。
- 若项目允许，建议逐步向 CI4 或现代 PHP 框架迁移，利用其内置的依赖注入、路由分组、中间件等特性降低耦合。
> ⚠️ **局限性说明**：由于提供的代码在 `case "1001"` 处截断，未能完整审查结账取消逻辑、数据库事务处理及并发控制部分。建议补充完整文件后，针对 `bill_frequency`、`open_room_log` 状态机流转进行二次专项审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780386491
|
1780386491
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
468
|
22
|
55
|
1
|
|
0
|
🔍 代码审查报告：app-260519 - 1
|
## 自动代码审查报告
**分支**: app-260519
**提交**: `cba73c16a ## 自动代码审查报告
**分支**: app-260519
**提交**: `cba73c16a1126fa6360fe68b6349e638ce336128`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 15:51:41
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了商家端核心业务路由与鉴权逻辑，但存在严重的安全硬编码、密码明文比对隐患、控制器职责过重（God Class）及大量魔法数字。整体架构偏向过程式堆砌，缺乏面向对象封装与配置化管理，可维护性与扩展性较差。
- **风险等级**：🔴 高
> 📌 **框架说明**：代码结构、加载方式（`$this->load->`、`$this->db->`、`defined('BASEPATH')`）高度契合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，部分生命周期与组件行为请以官方文档为准。本审查基于 CI3 最佳实践进行。
> ⚠️ **局限性说明**：代码在 `case "1001"` 处被截断，后续逻辑无法评估。以下审查基于已提供片段。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `~330行` | **敏感信息硬编码**：科大讯飞 TTS 的 `APISecret`、`APIKey` 直接写在控制器中，极易随代码库泄露。 | 移至环境变量或受保护的配置文件（如 `application/config/secrets.php`），通过 `getenv()` 或 `$this->config->item()` 读取。 | `// config/secrets.php<br>'xfyun' => [<br> 'app_id' => getenv('XFYUN_APPID'),<br> 'api_key' => getenv('XFYUN_APIKEY'),<br> 'api_secret' => getenv('XFYUN_APISECRET')<br>]<br>// 控制器中<br>$config = $this->config->item('xfyun');` |
| 🔴 严重 | `~480行` | **密码明文比对**：`$data['_discount_pwd'] != $_old_password` 暗示密码可能以明文或弱哈希存储，违反安全基线。 | 数据库必须存储 `password_hash()` 生成的哈希值，验证时使用 `password_verify()`。 | `if (!password_verify($_old_password, $data['_discount_pwd'])) {<br> $this->error_response('旧密码不正确');<br>}<br>$new_hash = password_hash($_new_password1, PASSWORD_DEFAULT);<br>$this->ahead_yc_merchant_user_model->update(['_discount_pwd' => $new_hash], ['_id' => $uid]);` |
| 🔴 严重 | `全局` | **响应方法未终止执行**：`$this->error_response()` / `$this->success_response()` 调用后未 `exit` 或 `return`，导致后续代码继续执行，可能引发重复响应或逻辑越权。 | 确保响应方法内部调用 `exit()`，或在调用处使用 `return $this->error_response(...)`。 | `// 推荐写法<br>return $this->error_response("参数错误");<br>// 或在 error_response 方法末尾添加<br>exit(json_encode($response));` |
| 🟠 警告 | `~35行` | **CORS 策略过于宽松**：`Access-Control-Allow-Origin: *` 允许任意域名跨域请求商家接口，易被恶意站点利用。 | 根据实际业务限制来源域名，或使用动态白名单校验 `Origin` 头。 | `$origin = $_SERVER['HTTP_ORIGIN'] ?? '';<br>$allowed = ['https://merchant.yourdomain.com'];<br>if (in_array($origin, $allowed)) {<br> header("Access-Control-Allow-Origin: $origin");<br>}` |
| 🟠 警告 | `__construct` | **条件跳过初始化导致状态不一致**：`if ($this->router->fetch_method() !== "uploadPic")` 跳过了数据库、配置、模型的加载。若 `uploadPic` 依赖这些组件将直接报错。 | 将公共初始化移至 `__construct` 顶部，仅将鉴权逻辑放入条件判断，或为 `uploadPic` 单独创建控制器。 | `parent::__construct();<br>$this->load->database('default');<br>$this->config->load('config_talent');<br>// 鉴权逻辑独立<br>if ($this->router->fetch_method() !== 'uploadPic') {<br> $this->_check_auth();<br>}` |
| 🟠 警告 | `index()` | **巨型路由方法违反单一职责**：`index()` 方法超千行，包含登录、个人中心、打印机、支付配置等数十个业务分支，难以测试与维护。 | 按业务模块拆分为独立控制器（如 `AuthController`, `ProfileController`, `PrinterController`），或使用框架路由映射。 | `// 路由配置示例<br>$route['merchant/(:any)'] = 'MerchantApp/$1';<br>// 拆分后<br>class AuthController extends AplicationController { public function login() { ... } }` |
| 🟠 警告 | `~380,410行` | **冗余获取 CI 实例**：在控制器内部使用 `$CI = &get_instance();` 赋值属性，控制器本身已是实例，此操作多余且易引发作用域混淆。 | 直接使用 `$this` 访问属性或方法。 | `// 移除<br>$CI = &get_instance();<br>$CI->token = $result['token'];<br>// 改为<br>$this->token = $result['token'];` |
| 🟡 建议 | `全局` | **大量魔法数字与硬编码 ID**：如 `'0005'`, `694`, `849`, `533` 等散落在代码中，业务规则变更时需全局搜索替换。 | 提取为类常量或独立配置文件（如 `config/menu_ids.php`）。 | `const FUNC_LOGIN = '0005';<br>const MENU_FILTER_IDS = [694, 849, 824, 1250];<br>if (in_array($v['id'], self::MENU_FILTER_IDS)) { ... }` |
| 🟡 建议 | `类属性` | **属性全部声明为 `public`**：破坏封装性，外部可直接修改内部状态，增加不可预知的副作用风险。 | 改为 `protected` 或 `private`，通过 getter/setter 或构造函数初始化。 | `protected $stream;<br>protected $expire_sms_time = 600;<br>protected $current_version = "2.0";` |
| 🟡 建议 | `~35行` | **拼写错误**：基类名 `AplicationController` 缺少字母 `p`，若为框架核心类名不一致将导致致命错误。 | 修正为 `ApplicationController`（需确认框架实际基类名称）。 | `class MerchantAppServer extends ApplicationController` |
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **移除硬编码密钥**：立即将 `xfyun_tts_config` 中的 `APISecret` 和 `APIKey` 迁移至环境变量或服务器安全配置中，并检查 Git 历史是否已泄露。
2. **修复密码验证逻辑**：全面排查 `_discount_pwd` 字段存储方式，强制实施 `password_hash()` / `password_verify()` 标准流程。
3. **阻断响应后代码执行**：全局检查 `error_response()` 与 `success_response()` 实现，确保输出 JSON 后立即 `exit`，或统一改为 `return` 模式。
### 🛠 重构与优化方向
1. **控制器瘦身（SRP 原则）**：当前 `MerchantAppServer` 承担了路由分发、鉴权、登录、个人中心、打印机管理、支付配置等职责。建议按业务域拆分为 4~5 个独立控制器，`index()` 仅保留路由分发或废弃改用框架原生路由。
2. **配置与常量集中化**：将菜单过滤 ID、权限 ID、支付平台映射、API 版本号等提取至 `application/config/merchant_rules.php`，避免业务逻辑与配置耦合。
3. **性能优化**：
- `case '0005'` 中存在大量串行 DB 查询（`get_admin_menu`, `get_one`, `get_modules` 等）。建议对高频读取的配置类数据引入 Redis 缓存（框架已提供 `Cache` 组件）。
- 菜单过滤逻辑使用 `foreach` + `unset` + `array_values` 效率较低，可改用 `array_filter` 或预定义白名单映射。
4. **规范与可维护性**：
- 统一属性可见性为 `protected`。
- 清理过期注释（如 `//edit by nan 17.10.25`），改用 Git 提交记录追溯变更。
- 补充输入校验：`json_decode` 后应检查 `json_last_error()`，防止畸形 JSON 导致后续数组访问报错。
> 💡 **后续建议**：在重构前，建议为当前核心接口补充 PHPUnit 单元测试或集成测试，确保拆分与优化过程中业务逻辑不发生退化。若需针对 `case "1001"` 截断部分进行深度审查，请提供完整代码片段。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780386701
|
1780386701
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
469
|
22
|
56
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `67f40bda4 ## 自动代码审查报告
**分支**: app-260616
**提交**: `67f40bda4fe54e9ed5e8f551606cdea0973b268b`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 15:52:23
---
## 1. 审查摘要
- **代码质量评分**：4.5 / 10 分
- **总体评价**：该控制器实现了商家端 App 的核心路由与业务分发功能，业务覆盖较广。但存在明显的架构反模式（巨型 `switch` 充当路由器、构造函数职责过重）、严重的安全隐患（硬编码密钥、CORS 通配符、疑似明文密码比对）以及大量重复逻辑。代码可维护性、安全性与扩展性均处于较低水平，亟需重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | ~380-385 | **硬编码第三方服务密钥**：科大讯飞 TTS 的 `APPID`、`APISecret`、`APIKey` 直接写死在代码中，极易随版本库泄露，导致服务被盗刷或数据泄露。 | 将敏感凭证移至 `application/config/` 独立配置文件或服务器环境变量中，通过 `$this->config->item()` 读取。 | `'xfyun_tts_config' => $this->config->item('xfyun_tts')` |
| 🔴 严重 | ~100-105 | **CORS 通配符配置**：`Access-Control-Allow-Origin: *` 允许任意域名跨域请求，结合 Token 鉴权机制，极易遭受 CSRF 攻击或敏感接口被恶意调用。 | 限制为可信域名白名单，或通过配置项动态输出，禁止使用 `*`。 | `header("Access-Control-Allow-Origin: " . $this->config->item('allowed_origins'));` |
| 🔴 严重 | ~450-455 | **密码校验与存储不规范**：`if ($data['_discount_pwd'] != $_old_password)` 疑似直接比对明文密码；且更新时未进行哈希处理，违反现代密码安全规范。 | 使用 `password_hash()` 存储，`password_verify()` 校验。若历史数据为明文，需规划平滑迁移方案。 | `if (!password_verify($_old_password, $data['_discount_pwd'])) { $this->error_response('旧密码不正确'); }`<br>`$this->ahead_yc_merchant_user_model->update(['_discount_pwd' => password_hash($_new_password1, PASSWORD_DEFAULT)], ['_id' => $uid]);` |
| 🟠 警告 | ~15-20 | **构造函数职责过重**：`__construct` 中混合了请求解析、CORS 设置、鉴权、权限校验、配置加载、日志记录等，严重违反单一职责原则（SRP），且影响单元测试。 | 将鉴权、日志、配置初始化抽离至中间件或基类控制器（如 `BaseApiController`）；构造函数仅保留核心依赖注入。 | 提取 `AuthMiddleware::handle()`、`ApiLogger::log()` |
| 🟠 警告 | ~200-350 | **巨型 Switch 充当路由器**：`index()` 方法使用超长 `switch` 分发 30+ 个接口，导致方法膨胀、难以维护，且每次请求都会加载大量未使用的 Model。 | 采用 CI3 原生路由配置，或改为动态方法调用（如 `$this->{$method}()`），将每个 `case` 拆分为独立方法。 | `public function func_0005() { /* 登录逻辑 */ }`<br>`$method = 'func_' . $request['function']; if (method_exists($this, $method)) { return $this->$method(); }` |
| 🟠 警告 | ~250-260 | **重复的权限校验与菜单过滤**：登录接口中多次调用 `check_priv`，且硬编码菜单 ID（Magic Numbers），业务规则与代码强耦合。 | 将菜单过滤规则配置化（如 `config/menus.php`）；提取独立的 `MenuService` 或 `PermissionService` 处理组装逻辑。 | `$this->menu_service->filterByPriv($menus, $user_type, $role_id);` |
| 🟠 警告 | ~500-530 | **支付方式过滤逻辑冗余**：`00066` 接口中针对 `api_type` 的过滤存在大量重复的 `if` 与 `array_diff`，违反 DRY 原则。 | 使用映射数组统一管理排除规则，通过循环批量处理。 | `$exclude_map = ['bill_pay' => [10], 'vip_recharge' => [3,7,8...]];`<br>`$exclude = $exclude_map[$api_type] ?? [];`<br>`$data['paytype'] = array_values(array_diff($data['paytype'], $exclude));` |
| 🟡 建议 | ~10 | **基类拼写错误**：`extends AplicationController` 疑似拼写错误，可能导致类继承失败或 IDE 无法识别。 | 修正为正确的基类名称（如 `CI_Controller` 或 `ApplicationController`）。 | `class MerchantAppServer extends CI_Controller` |
| 🟡 建议 | ~30-40 | **JSON 解析缺乏容错**：`json_decode($this->stream, true)` 未校验解析结果，若客户端传入非法 JSON 会导致后续逻辑崩溃或返回 500 错误。 | 增加 `json_last_error()` 校验，并统一使用 `php://input` 读取请求体。 | `$raw = file_get_contents('php://input');`<br>`$data = json_decode($raw, true);`<br>`if (json_last_error() !== JSON_ERROR_NONE) { $this->error_response('请求数据格式错误'); }` |
| 🟡 建议 | 全文 | **代码规范与可读性**：缩进不一致、长行未换行、变量命名混杂（如 `$this->stream` 与 `$this->get_stream`）、缺少类型声明与注释，不符合 PSR-12。 | 使用 `PHP-CS-Fixer` 或 IDE 自动格式化；统一使用驼峰/蛇形命名；为复杂逻辑添加 PHPDoc。 | 略（建议接入 CI/CD 静态检查流水线） |
> 📝 **注**：代码在 `case "1001":` 处被截断，无法评估该分支的完整逻辑与潜在风险。若该分支涉及订单状态变更或资金操作，请补充完整代码以便进行深度审查。
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即移除硬编码密钥**：将 `xfyun_tts_config` 等敏感信息迁移至配置文件或 `.env`，并检查 Git 历史是否已泄露。
2. **修复密码安全逻辑**：全面排查 `_discount_pwd` 及其他密码字段的存储与校验方式，强制引入 `password_hash`/`password_verify`。
3. **收敛 CORS 策略**：将 `Access-Control-Allow-Origin: *` 替换为业务域名白名单，避免接口被恶意跨域调用。
4. **增加输入校验容错**：对 `json_decode`、`file_get_contents('php://input')` 增加异常捕获与格式校验，防止恶意请求导致服务崩溃。
### 🛠 后续重构与优化方向
1. **架构解耦（核心）**：
- 废弃 `index()` 巨型 `switch` 路由，改用 **CI3 原生路由** 或 **命令模式/动态方法分发**。
- 将鉴权、日志、配置加载等横切关注点抽离至 **中间件** 或 **基类控制器**，保持业务方法纯净。
2. **服务层拆分**：
- 将菜单组装、权限校验、支付方式过滤等复杂逻辑下沉至独立的 `Service` 类，控制器仅负责参数接收、调用服务、返回响应。
- 消除硬编码的 Magic Numbers（如菜单 ID、支付平台 ID），统一收敛至配置中心或数据库字典表。
3. **性能与规范提升**：
- 模型加载统一移至构造函数或按需懒加载，避免在 `switch` 分支中重复调用 `$this->load->model()`。
- 引入 PSR-12 代码规范检查工具（如 `php-cs-fixer`），统一命名风格，补充关键方法的 PHPDoc 注释。
- 针对高频接口（如登录、个人中心）考虑引入 Redis 缓存菜单/配置数据，降低 DB 查询压力。
> 💡 **框架适配提示**：当前代码特征与 **CodeIgniter 3** 高度吻合。若 `phpci` 为内部定制框架，请确认其是否支持中间件机制与动态路由。若不支持，建议优先采用 `BaseController` 继承模式与 `call_user_func_array` 实现轻量级分发，逐步向现代 MVC 架构演进。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780386743
|
1780386743
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
478
|
22
|
57
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 自助报表
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `6c1575fa3 ## 自动代码审查报告
**分支**: app-260616
**提交**: `6c1575fa3fcb52898b64e380e2fc77cf85718a86`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 18:50:14
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑覆盖较全，报表统计与多维度聚合功能实现完整。但存在明显的 **SQL 注入隐患**、**N+1 查询性能瓶颈** 以及 **面向对象设计不规范** 问题。部分查询构造器使用方式偏离现代 PHP 安全规范，需优先修复。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_songs_sales_pay_log_model.php`<br>约第 268 行 (`refund` 方法) | **SQL 注入风险**：`$refund_reason` 与 `$CI->admin_data` 直接字符串拼接进 `UPDATE` 语句，未做转义或参数绑定。若输入含双引号或特殊字符将导致语法错误或注入。 | 使用框架查询构造器的参数绑定机制，或调用内置的 `escape()` 方法。避免手动拼接 SQL。 | ```php<br>// 推荐写法<br>$this->db->set('_refund_reason', $refund_reason)<br> ->set('_refund_admin_id', $CI->admin_data['_id'] ?? 0)<br> ->set('_refund_admin_name', $CI->admin_data['_name'] ?? '')<br> ->where($where)<br> ->update($this->table_name);<br>``` |
| 🔴 严重 | `Ahead_songs_sales_pay_log_model.php`<br>约第 200 行<br>`Jh_community_shop_revenues_detail_model.php`<br>约第 150 行 | **SQL 注入/语法错误**：`$pay_platform_where` 循环中直接拼接 `$params['pay_platform_arr']`，且**未添加引号**。若值为字符串将引发 SQL 语法报错；若未做类型过滤则存在注入风险。 | 强制类型转换 `(int)`，或改用框架提供的 `or_where()` 链式调用，由底层驱动处理转义。 | ```php<br>// 推荐写法<br$this->db->group_start();<br>foreach ($params['pay_platform_arr'] as $p) {<br> $parts = explode('_', $p);<br> $this->db->or_where('a._pay_platform', (int)$parts[0]);<br> if (!empty($parts[1])) {<br> $this->db->or_where('a._second_pay_platform', (int)$parts[1]);<br> }<br>}<br>$this->db->group_end();<br>``` |
| 🟠 警告 | `Ahead_songs_sales_pay_log_model.php`<br>约第 115 行 (`get_shop_incomes_statement_trend`) | **N+1 查询性能瓶颈**：在 `foreach ($result as &$v)` 循环中逐段执行 `$this->select()`。若查询跨度为 30 天，将产生 30+ 次独立 SQL 请求，严重拖慢接口响应。 | 改为**单次查询**拉取全量时间段数据，利用 `GROUP BY` 时间表达式，在 PHP 层按 Key 映射填充；或引入 Redis 缓存趋势数据。 | ```php<br>// 优化思路：一次性查询<br>$where['_consume_time >='] = $min_start;<br>$where['_consume_time <'] = $max_end;<br>$all_data = $this->select($where, $fields, $group_by);<br>// 在 PHP 中按 create_hour/date 建立索引映射<br>$mapped = array_column($all_data, 'amount', $group_by);<br>``` |
| 🟠 警告 | 两文件顶部 | **反模式：全局 `$CI` 实例化**：在类外部声明 `$CI = &get_instance();` 违反 OOP 原则。在文件被 `include` 时即执行，易在 CLI/多进程环境下引发状态污染或内存泄漏。 | 移除文件顶部全局声明。在类内部通过 `$this->load->model()` 按需加载，或在 `__construct()` 中统一初始化。 | 删除顶部 `$CI = &get_instance();`，方法内直接使用 `$this->load->model('xxx');` |
| 🟠 警告 | `Jh_community_shop_revenues_detail_model.php`<br>多个统计方法 | **重复聚合逻辑**：`get_community_revenues_data`、`get_community_revenues_sum_data` 中计算 `wx_amount`、`tuangou_amount` 等逻辑高度重复，违反 DRY 原则。 | 抽取私有方法 `private function aggregate_revenue_stats(array $data): array` 统一处理收入/退款分类汇总。 | ```php<br>private function aggregate_revenue_stats($data) {<br> $stats = ['wx'=>0, 'tuangou'=>0, 'other'=>0];<br> foreach($data as $v) { /* 统一分类累加 */ }<br> return $stats;<br>}<br>``` |
| 🟡 建议 | 全局多处 | **魔法数字/字符串泛滥**：大量使用 `'1'`, `'4'`, `'8'`, `'9'` 表示状态与支付渠道，缺乏语义化，后期维护成本高。 | 定义类常量或 PHP 8.1+ 枚举（Enum）。如 `const STATUS_PAID = 1; const PAY_WECHAT = '1';`。 | ```php<br>class Ahead_songs_sales_pay_log_model {<br> const PAY_WECHAT = '1';<br> const PAY_VIP = '3';<br> // 替换代码中的硬编码<br>}<br>``` |
| 🟡 建议 | `Jh_community_shop_revenues_detail_model.php`<br>约第 240 行 (`get_date_type_info`) | **静态缓存潜在风险**：`self::$date_type_info` 在 PHP-FPM 下安全，但若项目未来迁移至 Swoole/Workerman 等常驻进程，将导致跨请求数据污染。 | 改为实例属性 `$this->date_type_info`，或增加请求级重置逻辑。若确定仅运行于 FPM 可保留。 | `private $date_type_info = [];` 替代 `public static $date_type_info = [];` |
| 🟡 建议 | `Ahead_songs_sales_pay_log_model.php`<br>约第 140 行 | **频繁加载模型**：同一方法内多次调用 `$this->load->model()`。虽 CI 有内部缓存，但影响代码整洁度与可读性。 | 将高频依赖模型移至构造函数或父类中预加载，或使用依赖注入容器管理。 | 在 `__construct()` 中集中加载 `ahead_shop_group_model` 等。 |
## 3. 总结与行动建议
### 🔑 优先修复项（P0/P1）
1. **彻底修复 SQL 拼接漏洞**：立即替换 `refund()` 方法中的字符串拼接，以及所有 `$pay_platform_where` 的 `implode(' or ', ...)` 逻辑。务必使用框架提供的参数绑定或查询构造器链式方法。
2. **消除 N+1 查询**：重构 `get_shop_incomes_statement_trend()`，将循环内的数据库查询改为**单次范围查询 + PHP 内存映射**。预计可降低 70% 以上的数据库 I/O 开销。
3. **移除全局 `$CI` 声明**：将文件顶部的 `get_instance()` 调用清理，改为标准的模型内部加载机制，避免潜在的状态泄漏。
### 🛠 后续重构与优化方向
- **统一查询构造器规范**：确认 `phpci` 框架底层 `DB_driver` 是否支持 `?` 占位符或命名参数绑定。若不支持，建议封装一层安全查询辅助类，强制拦截未转义的输入。
- **业务常量集中管理**：将支付渠道、订单状态、时间类型等硬编码提取至独立的 `Config` 类或 `Enum` 文件中，提升代码自解释能力。
- **报表计算逻辑抽象**：当前多个模型中存在相似的 `type=1(收入)` 与 `type=2(退款)` 分类汇总逻辑。建议抽象为 `RevenueCalculator` 服务类，通过策略模式或简单工厂处理不同维度的统计，降低模型耦合度。
- **异常处理规范化**：代码中使用了全局函数 `throwError()`。建议逐步替换为 PHP 原生 `throw new \InvalidArgumentException()` 或自定义业务异常类，配合全局异常处理器统一返回格式。
> 💡 **框架适配提示**：代码呈现典型的 CodeIgniter 3 架构特征（如 `get_instance()`、`load->model()`、`select()` 等）。若 `phpci` 为内部定制框架，请重点核对 `system/database/DB_driver.php` 中 `select()`、`up()` 等方法的底层实现是否默认开启参数绑定。若未开启，上述安全修复必须手动调用 `$this->db->escape()` 或 `(int)` 强转。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780397414
|
1780397414
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
479
|
22
|
58
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `6de5af6e3 ## 自动代码审查报告
**分支**: app-260616
**提交**: `6de5af6e35f472ac5142d043b227de4326e9fda9`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 18:54:48
---
## 1. 审查摘要
- **代码质量评分**：6.0 / 10 分
- **总体评价**：代码业务逻辑完整，覆盖了营业额统计、趋势分析、详情查询及流水记录等核心场景。但存在**严重的 SQL 注入隐患**、**循环内查询（N+1）性能瓶颈**以及**框架上下文滥用**问题。代码风格偏向传统 CI3 写法，缺乏现代 PHP 类型声明与常量管理，可维护性与安全性需重点提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `refund()` 方法 | **SQL 注入漏洞**：`$refund_reason` 与 `$order_data['_id']` 直接拼接至 SQL 字符串，未做任何转义或预处理。攻击者可构造恶意输入破坏数据或提权。 | 废弃字符串拼接，改用框架查询构造器（Query Builder）或预处理语句。若 `Simple_model::up()` 不支持绑定参数，需重写该方法或改用原生 `$this->db->update()`。 | ```php<br>// 安全写法<br>$this->db->set('_status', 4)<br> ->set('_refund_time', time())<br> ->set('_refund_amount', '_actual_pay', FALSE)<br> ->set('_other_pay_refund_amount', '_other_pay_amount', FALSE)<br> ->set('_refund_reason', $refund_reason)<br> ->set('_refund_admin_id', $CI->admin_data['_id'] ?? 0)<br> ->set('_refund_admin_name', $CI->admin_data['_name'] ?? '')<br> ->where('_merchant_id', $order_data['_merchant_id'])<br> ->where('_shop_id', $order_data['_shop_id'])<br> ->where('_order_id', $order_data['_id'])<br> ->where('_type', $type)<br> ->where('_status', 1)<br> ->update($this->table_name);<br>``` |
| 🔴 严重 | `get_shop_incomes_statement_detail()` | **SQL 注入漏洞**：`$pay_platform_where` 数组通过字符串拼接构造 `OR` 条件，用户传入的 `$params['pay_platform_arr']` 未过滤直接入 SQL。 | 使用查询构造器的分组条件语法替代手动拼接，确保参数自动转义。 | ```php<br>$this->db->group_start();<br>foreach ($params['pay_platform_arr'] as $pp) {<br> $arr = explode('_', $pp);<br> $this->db->or_where('a._pay_platform', $arr[0]);<br> if (!empty($arr[1])) {<br> $this->db->where('a._second_pay_platform', $arr[1]);<br> }<br>}<br>$this->db->group_end();<br>``` |
| 🟠 警告 | `get_shop_incomes_statement_trend()` | **N+1 查询性能瓶颈**：在 `foreach ($result as &$v)` 循环中反复调用 `$this->select()`。若 `$result` 有 30 个时间段，将触发 30 次独立 DB 查询，高并发下极易拖垮数据库。 | 改为**单次查询**拉取全量数据，在 PHP 层按时间维度聚合；或使用 `IN` 条件一次性查询后分组。 | 建议将时间范围合并为单次查询，返回后使用 `array_reduce` 或循环按 `create_hour/date/month` 映射填充 `$result`。 |
| 🟠 警告 | 文件顶部第 2 行 | **框架生命周期违规**：`$CI = &get_instance();` 在类定义外全局执行。文件被 `include` 时即触发，可能引发上下文污染、内存泄漏或单元测试失败。 | 移除顶部代码，将依赖加载移至 `__construct()` 或具体业务方法中。 | ```php<br>public function __construct()<br>{<br> parent::__construct();<br> // 按需加载模型或保持延迟加载<br>}<br>``` |
| 🟠 警告 | `get_shop_incomes_statement_detail()` | **模型状态污染**：使用 `$this->set_table_name($table_name . ' a')` 修改了模型内部表名状态，虽在末尾恢复，但在并发请求或链式调用中易引发别名冲突。 | 避免修改全局模型状态，建议实例化临时查询对象或使用框架提供的 `from()`/`join()` 链式语法。 | 使用 `$this->db->from($this->table_name . ' a')` 替代全局状态修改。 |
| 🟠 警告 | `add_data()` 方法 | **未初始化变量**：`$second_pay_platform` 仅在 `if` 分支中赋值，末尾依赖 `?? 0` 兜底。在 PHP 严格模式或旧版本中会触发 `Notice`，且逻辑分支遗漏时易产生脏数据。 | 方法开头显式初始化：`$second_pay_platform = 0;`，并确保所有分支均有明确赋值。 | `$second_pay_platform = 0; // 初始化` |
| 🟡 建议 | 全局多处 | **魔法数字/字符串泛滥**：状态值 `1, 4`、平台值 `8, 9, 17~28` 等硬编码散落各处，业务变更时需全局搜索替换，极易遗漏。 | 提取为类常量（`const`），集中管理业务字典。 | ```php<br>const STATUS_PAID = 1;<br>const STATUS_REFUNDED = 4;<br>const PAY_PLATFORM_GROUP = '8';<br>``` |
| 🟡 建议 | 类名/方法名 | **不符合 PSR-12 规范**：类名与方法名使用下划线分隔（如 `Ahead_songs_sales_pay_log_model`、`get_shop_incomes_statement_sum`）。 | 逐步重构为 `StudlyCaps` 类名与 `camelCase` 方法名。若受历史包袱限制，至少保持项目内统一。 | `class AheadSongsSalesPayLogModel extends Simple_model` |
| 🟡 建议 | `get_shop_incomes_statement_trend()` | **非标准异常处理**：使用自定义 `throwError()` 函数中断流程，不利于全局异常捕获与日志追踪。 | 改用标准 PHP 异常或框架异常类。 | `throw new \InvalidArgumentException('营业额走势查询时间错误');` |
## 3. 总结与行动建议
### 🔑 优先修复项（P0）
1. **彻底修复 SQL 注入**：`refund()` 与 `get_shop_incomes_statement_detail()` 中的字符串拼接必须立即替换为查询构造器或参数绑定。这是当前最高危的安全漏洞。
2. **消除 N+1 查询**：重构 `get_shop_incomes_statement_trend()`，将循环查询改为单次批量查询 + PHP 内存聚合。可预期性能提升 5~10 倍。
3. **清理全局上下文**：移除文件顶部的 `$CI = &get_instance();`，遵循框架依赖注入或构造函数初始化规范。
### 🛠 后续重构与优化方向
1. **引入类型声明与常量管理**：
- 为所有方法参数与返回值添加 PHP 7.4+/8.0+ 类型提示（如 `array`, `int`, `string`）。
- 将 `$type_arr`、`$incomes_pay_platform_arr` 等硬编码数组迁移至配置类或数据库字典表，提升可配置性。
2. **统一数据格式化策略**：
- `change_number_format()` 被高频调用，建议在数据库层使用 `ROUND()` 或 `FORMAT()`，或在返回前统一批量处理，减少函数调用开销。
3. **框架适配说明**：
- 代码特征高度匹配 **CodeIgniter 3** 架构。若 `phpci` 为内部定制框架，请确认 `Simple_model::up()`、`$this->select()` 是否底层已实现 PDO 预处理。若未实现，上述 SQL 注入修复方案必须落地。
- 建议查阅 `phpci` 官方文档中关于 `Query Builder` 与 `Model 生命周期` 的章节，确保状态隔离与事务安全。
> 💡 **专家提示**：当前代码业务耦合度较高，建议后续将“数据聚合逻辑”与“数据查询逻辑”拆分。可引入 `Repository` 模式或 `Service` 层处理复杂统计，保持 Model 仅负责基础 CRUD，以提升代码的可测试性与扩展性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780397688
|
1780397688
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
480
|
22
|
59
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `16e0ec6e3 ## 自动代码审查报告
**分支**: app-260616
**提交**: `16e0ec6e3e25170a5a84b90be2523bb5f302c81b`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 19:04:29
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码整体业务逻辑清晰，覆盖了多种营收场景的日志记录与报表查询。但存在**静态缓存未区分门店导致的数据错乱**、**手动拼接 SQL 条件引发注入风险**、**循环内执行数据库查询**等严重问题。同时，模型加载分散、重复代码较多，未充分利用框架特性，可维护性与性能有较大优化空间。
- **风险等级**：🔴 高
> 📌 **框架说明**：根据目录结构（`system/`, `application/models/`）及 `$CI = &get_instance()`、`$this->load->model()` 等语法特征，判定为 **CodeIgniter 3.x** 架构。若 `phpci` 为内部定制框架，请结合其底层实现酌情调整框架适配建议。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_date_type_info` (~L405) | **静态缓存未区分 `$shop_id`**。`self::$date_type_info` 仅在首次调用时初始化，后续传入不同门店 ID 时仍返回首个门店的时间范围，导致报表数据严重错乱。 | 移除静态缓存，或改为以 `$shop_id` 为键的多维缓存数组；建议直接使用 CI Cache 驱动或按需计算。 | `if (!isset(self::$date_type_info[$shop_id])) { /* 初始化逻辑 */ } return self::$date_type_info[$shop_id];` |
| 🔴 严重 | `get_community_revenues_sum_data` / `detail_data` (~L235, L335) | **SQL 注入风险**。手动拼接字符串 `'(a._pay_platform=' . $pay_platform . ' ...)'` 并传入查询条件，绕过框架参数绑定与转义机制。若 `$params` 来自用户输入，极易被注入。 | 使用 CI 查询构造器的分组条件方法，或确保输入严格白名单过滤。 | `$this->db->group_start()->where('a._pay_platform', $p1)->where('a._second_pay_platform', $p2)->group_end();` |
| 🟠 警告 | `get_community_revenues_trend` (~L385) | **循环内执行 DB 查询**。在 `foreach ($result as &$v)` 中反复调用 `$this->select()`，若时间维度较多或并发高，将引发严重的 N+1 查询性能瓶颈。 | 改为单次范围查询拉取所有数据，在 PHP 层按时间键聚合映射；或使用 `WHERE IN` / 范围条件一次性获取。 | 一次性查询 `_business_date` 在起止范围内的数据，使用 `array_reduce` 或哈希表在内存中分组累加。 |
| 🟠 警告 | 多处方法内部 (`_add_log`, `get_room_info_by_order` 等) | **频繁动态加载模型**。在业务方法中反复调用 `$this->load->model()`，增加文件 I/O 与内存开销，违反 CI 最佳实践。 | 将依赖模型统一移至 `__construct()` 中加载，或通过 CI `autoload.php` 配置预加载。 | `public function __construct() { parent::__construct(); $this->load->model(['ahead_yc_merchant_model', 'ahead_family_servers_model']); }` |
| 🟠 警告 | `get_community_revenues_data` vs `sum_data` (~L160 vs L200) | **高度重复的聚合逻辑**。两个方法 80% 以上的数据计算与格式化代码完全一致，违反 DRY 原则，后期维护成本高。 | 提取私有聚合方法 `private function format_revenue_stats(array $data): array`，统一处理计算与格式化。 | 将 `foreach` 累加与 `change_number_format` 逻辑抽离，主方法仅负责构建 `$where` 并调用聚合器。 |
| 🟡 建议 | 全局类定义 (~L10) | **类名不符合 PSR-12 规范**。当前为蛇形命名 `Jh_community_shop_revenues_detail_model`，应使用大驼峰（PascalCase）。 | 重命名为 `JhCommunityShopRevenuesDetailModel`，并同步更新所有引用处。 | `class JhCommunityShopRevenuesDetailModel extends Report_model` |
| 🟡 建议 | `add_by_vip_recharge_refund` 等 (~L480) | **循环逐条插入**。在 `foreach` 中调用 `_add_log` 执行单条 `INSERT`，数据量大时数据库交互频繁，性能低下。 | 收集完整数据数组后，使用 `$this->db->insert_batch()` 批量写入。 | `$this->db->insert_batch($this->table_name, $batch_insert_data);` |
| 🟡 建议 | 全局 PHPDoc | **类型声明不准确**。多处 `@return true` 不符合 PHPDoc 规范，应使用 `bool`；魔法数字（如 `1,2,8,9`）硬编码降低可读性。 | 修正注释类型；将业务状态码定义为类常量（如 `const TYPE_INCOME = 1;`）。 | `/** @return bool */` <br> `const PAY_PLATFORM_WX = '1';` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复静态缓存门店隔离问题**：立即调整 `get_date_type_info` 的缓存策略，避免跨门店数据污染。这是当前最可能导致线上报表数据错误的逻辑缺陷。
2. **消除 SQL 注入隐患**：全面审查 `$where` 数组中手动拼接字符串的逻辑，替换为 CI 查询构造器的 `group_start()` / `group_end()` 或 `or_where()` 链式调用，确保参数自动转义。
3. **优化趋势图查询性能**：将 `get_community_revenues_trend` 中的循环查询重构为**单次范围查询 + 内存聚合**。可大幅降低数据库连接压力与响应延迟。
### 🛠 后续重构与优化方向
- **统一模型加载策略**：在 `__construct()` 中集中加载所有依赖模型，或使用 CI 的自动加载机制。避免在高频调用的业务方法中重复 `load->model()`。
- **抽离公共聚合逻辑**：`get_community_revenues_data` 与 `get_community_revenues_sum_data` 的统计逻辑高度重合，建议封装为独立的私有方法，通过参数控制返回结构，提升代码复用率。
- **引入批量操作与事务控制**：对于 `_add_log` 相关的写入场景，若存在批量调用，应改用 `insert_batch()`。同时，建议在 `enforce_con_db()` 切换连接时包裹 `try...finally` 块，确保异常发生时数据库连接状态能正确恢复，防止连接泄漏。
- **规范化与常量管理**：逐步将硬编码的订单类型、支付平台标识提取为类常量，并统一类名与方法命名风格。修正 PHPDoc 类型声明，为后续接入静态分析工具（如 PHPStan/Psalm）打下基础。
> 💡 **提示**：若 `enforce_con_db()` 为自定义多数据源切换方法，请确保其底层已处理连接池复用与异常回滚。建议在核心写入路径添加基础的事务包裹（`$this->db->trans_start()` / `trans_complete()`）以保障财务数据一致性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780398269
|
1780398269
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
487
|
22
|
60
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `c431e22e0 ## 自动代码审查报告
**分支**: app-260616
**提交**: `c431e22e0891b2a87b3d9a00fc82dbe5fd538abc`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-03 10:17:10
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码实现了社区门店营收明细、统计、趋势及列表查询的核心业务逻辑，结构相对完整。但存在**静态缓存未隔离导致的数据串扰**、**未定义变量引用**、**原始 SQL 拼接注入风险**等严重问题。同时，模型加载分散、魔法数字泛滥、全局变量滥用等设计缺陷影响了可维护性与执行效率。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_date_type_info` (约第 430 行) | `self::$date_type_info` 使用静态属性缓存，但未按 `$shop_id` 隔离。首次调用后，后续不同门店的请求将直接返回缓存数据，导致**时间范围计算错误与数据串扰**。 | 改为按 `$shop_id` 键值缓存，或移除静态缓存改为实例属性。若需全局缓存，应结合 Redis/Memcached 并设置合理过期时间。 | `private static $date_type_info = [];`<br>`$cache_key = $shop_id;`<br>`if (!isset(self::$date_type_info[$cache_key])) { ... }`<br>`return self::$date_type_info[$cache_key];` |
| 🔴 严重 | `add_by_order_refund` (约第 240 行) | `$order_extension` 仅在 `$pay_platform == 8` 时定义，但在 `$insert` 数组中无条件使用 `$order_extension['_group_buying_type']`。非团购退款时将触发 `Undefined variable` 警告。 | 提前初始化变量，或调整作用域确保变量始终存在。 | `$order_extension = [];`<br>`if ($pay_platform == 8) {`<br>` $this->load->model('ahead_yc_order_extension_model');`<br>` $order_extension = $this->ahead_yc_order_extension_model->get_one(...);`<br>`}` |
| 🔴 严重 | `get_community_revenues_sum_data` / `detail_data` (约第 330/480 行) | `$where['where'][] = ['(' . implode(' or ', $pay_platform_where) . ')'];` 直接拼接用户传入的 `$params['pay_platform_arr']` 生成 SQL 片段，**未进行参数绑定或转义**，存在 SQL 注入风险。 | 使用框架查询构造器的 `group_start()` / `or_where()` / `group_end()` 安全构建条件，或确保输入经过严格白名单过滤。 | `$this->db->group_start();`<br>`foreach ($params['pay_platform_arr'] as $pp) {`<br>` $arr = explode('_', $pp);`<br>` $this->db->or_where('a._pay_platform', $arr[0]);`<br>` if (!empty($arr[1])) $this->db->or_where('a._second_pay_platform', $arr[1]);`<br>`}`<br>`$this->db->group_end();` |
| 🟠 警告 | 文件顶部 (第 2 行) | `$CI = &get_instance();` 在类外部全局执行。文件被 `include/require` 时即触发，若框架尚未初始化完成将导致致命错误，且违反框架生命周期规范。 | 移除全局调用，将模型加载移至类构造函数中。 | `public function __construct() {`<br>` parent::__construct();`<br>` $this->load->model('Report_model');`<br>`}` |
| 🟠 警告 | `_add_log` / `get_room_info_by_order` | 频繁在业务方法内部调用 `$this->load->model()`。CI 框架虽支持重复加载，但每次调用仍有开销。高并发写入时易成为性能瓶颈。 | 将依赖模型统一在构造函数中加载，或使用依赖注入（若框架支持）。 | `protected $ahead_yc_merchant_model;`<br>`public function __construct() {`<br>` parent::__construct();`<br>` $this->load->model('ahead_yc_merchant_model');`<br>` $this->ahead_yc_merchant_model = $this->ahead_yc_merchant_model;`<br>`}` |
| 🟠 警告 | `get_community_revenues_detail_data` (约第 530 行) | `in_array($v['pay_platform'], ['8', '9'])` 未开启严格模式。PHP 弱类型比较可能导致 `'8' == 8` 误判，且业务中 `_pay_platform` 类型不统一（字符串/整型混用）。 | 统一数据类型，并强制使用严格比较 `in_array($val, $arr, true)`。 | `in_array((string)$v['pay_platform'], ['8', '9'], true)` |
| 🟡 建议 | 全局多处 | 类名 `Jh_community_shop_revenues_detail_model` 使用蛇形命名，违反 PSR-12 规范。PHPDoc 中 `@return true` 写法不规范。 | 类名改为大驼峰 `JhCommunityShopRevenuesDetailModel`；返回值统一标注为 `@return bool`。 | `class JhCommunityShopRevenuesDetailModel extends Report_model`<br>`/** @return bool */` |
| 🟡 建议 | 全局多处 | 业务逻辑中大量使用魔法数字（如 `1`, `2`, `8`, `9`, `17` 等），可读性差且后期维护极易出错。 | 提取为类常量或枚举类，集中管理业务状态。 | `const TYPE_INCOME = 1;`<br>`const TYPE_REFUND = 2;`<br>`const PAY_WX = '1';`<br>`const PAY_TUANGOU = '8';` |
| 🟡 建议 | `get_community_revenues_trend` | 循环遍历 `$result` 并反复调用 `$this->select()` 查询数据库。若时间跨度大，将产生大量 DB 交互，且 `$where` 数组被重复覆盖，逻辑脆弱。 | 优化为单次 SQL 聚合查询（如使用 `CASE WHEN` 或 `GROUP BY` 时间维度），或在应用层合并数据。 | 建议重构为单条 SQL 按时间维度 `GROUP BY`，减少循环查询次数。 |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复静态缓存串扰**：立即修改 `get_date_type_info` 中的 `self::$date_type_info` 缓存策略，按 `shop_id` 隔离或移除静态缓存，否则将导致多门店报表数据严重错乱。
2. **消除未定义变量风险**：在 `add_by_order_refund` 中初始化 `$order_extension`，避免生产环境触发 `Notice/Warning` 导致日志污染或逻辑中断。
3. **封堵 SQL 注入漏洞**：重构 `$where['where'][]` 的字符串拼接逻辑，全面改用框架提供的参数绑定或查询构造器安全方法（如 `group_start()` / `or_where()`）。
### 🛠 后续重构与优化方向
1. **架构与生命周期规范**：
- 移除文件顶部的全局 `$CI = &get_instance();`，严格遵循框架的类初始化流程。
- 将频繁使用的模型（如 `ahead_yc_merchant_model`、`ahead_user_model`）收敛至构造函数加载，降低运行时开销。
2. **代码可维护性提升**：
- 建立业务常量字典（如 `ORDER_TYPE`、`PAY_PLATFORM`、`TYPE_INCOME/REFUND`），替换散落的魔法数字。
- 统一数据类型处理（如金额统一为 `float` 或 `string`，平台 ID 统一为 `string`），避免 PHP 弱类型比较带来的隐性 Bug。
3. **性能与查询优化**：
- `get_community_revenues_trend` 中的循环查询可优化为单次 `GROUP BY` 聚合查询，利用数据库计算能力替代 PHP 循环。
- 评估 `$this->enforce_con_db()` 的切换成本。若为读写分离或分库逻辑，建议封装为事务或批量操作，减少连接切换频率。
4. **框架适配说明**：
- 代码特征高度符合 **CodeIgniter 3/4** 架构。若 `phpci` 为内部定制框架，请核对 `select()`、`insert()`、`enforce_con_db()` 等底层方法的契约是否支持参数绑定与链式调用。建议查阅官方文档确认查询构造器的安全用法。
> 💡 **提示**：建议在合并前补充单元测试（尤其是 `get_date_type_info` 缓存隔离、`add_by_order_refund` 分支覆盖、以及 `$where` 条件构建的边界测试），并开启 `error_reporting(E_ALL)` 进行全量回归验证。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780453030
|
1780453030
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
488
|
22
|
61
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `94f1af278 ## 自动代码审查报告
**分支**: app-260616
**提交**: `94f1af2781fb88f4aa18866c5d0f6cffc6e416b9`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-03 10:26:07
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10
- **总体评价**：代码整体业务逻辑完整，覆盖了多种营收场景的日志记录与报表查询。但存在**静态缓存污染、SQL注入风险、高频N+1查询**等核心缺陷，且代码复用率低、框架规范遵循度不足。在并发写入或复杂查询场景下易引发数据丢失、性能瓶颈及安全漏洞。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_date_type_info` 方法 | 使用 `public static $date_type_info` 缓存时间配置，但未以 `$shop_id` 为键。同一请求内切换不同门店时，会错误返回首次加载的门店数据，导致报表时间范围错乱。 | 改为以 `$shop_id` 为键的二维数组缓存，或移除静态缓存改为实例属性。 | `if (!isset(self::$date_type_info[$shop_id])) { self::$date_type_info[$shop_id] = [...]; }` |
| 🔴 严重 | `_add_log` / `ORDER_TYPE` 常量 | `sales_type_map` 中存在 `11 => 13` 映射，但 `ORDER_TYPE` 中 `'13'` 已被注释。导致 `_add_log` 中 `isset(self::ORDER_TYPE[$insert['_order_type']])` 校验失败，**静默丢弃该类型日志**，造成财务数据丢失。 | 取消注释 `ORDER_TYPE['13']`，或在映射前增加类型白名单校验。 | `const ORDER_TYPE = [ ..., '13' => 'App-计时开房', ... ];` |
| 🔴 严重 | `get_community_revenues_sum_data` / `detail_data` | `$params['pay_platform_arr']` 直接通过字符串拼接构造 SQL 条件：`'a._pay_platform=' . $pay_platform`。若参数来自前端未过滤，将导致 **SQL 注入漏洞**。 | 强制类型转换 `(int)`，或改用框架查询构造器的参数绑定机制。 | `$pay_platform = (int)($pay_platform_arr[0] ?? 0);`<br>`$this->db->where('a._pay_platform', $pay_platform);` |
| 🟠 警告 | `_add_log` 方法 | 每次写入营收明细时，均调用 `get_room_info_by_order` 查询数据库获取包厢信息。在订单高频写入场景下，会引发严重的 **N+1 查询性能瓶颈**。 | 将包厢信息作为参数传入，或改为批量查询/Redis缓存。避免在写入链路中同步查库。 | `private function _add_log($insert, $order_data, $room_info = [])`<br>`if (empty($room_info)) { $room_info = $this->get_room_info_by_order($order_data); }` |
| 🟠 警告 | 全局 `$CI = &get_instance();` | 文件顶部直接调用 `get_instance()` 并赋值给全局变量 `$CI`，且在 `_add_log` 中读取 `$CI->guotong_refund_id`。破坏 MVC 分层，且全局状态在 CLI/异步任务中极易引发致命错误。 | 移除全局 `$CI`。国通退款单号应通过方法参数传入，或存储于 Session/Config 中。 | `private function _add_log($insert, $order_data, $guotong_refund_id = null)`<br>`if ($insert['_type'] == 2 && !empty($guotong_refund_id)) { ... }` |
| 🟠 警告 | `get_community_revenues_*` 系列方法 | 多个查询方法存在大量重复代码（社区商家校验、表名切换、时间解析、DB连接切换）。违反 DRY 原则，后期维护成本高。 | 提取为私有方法 `prepare_query_context($merchant_id, $params)` 统一处理前置逻辑。 | `private function prepare_query_context($merchant_id, $params) { ... return $where; }` |
| 🟠 警告 | `get_community_revenues_trend` | 循环内直接修改 `$where` 数组：`$where['_business_date >='] = $v['start_date'];`。若后续迭代中其他条件变化，会携带上一次迭代的日期条件，导致查询结果污染。 | 每次循环前克隆基础 `$where` 数组，确保条件隔离。 | `$query_where = $base_where;`<br>`$query_where['_business_date >='] = $v['start_date'];` |
| 🟡 建议 | 全文件 / 类定义 | 类名 `Jh_community_shop_revenues_detail_model` 使用蛇形命名，不符合 PSR-12 规范；DocBlock 中 `@return true` 类型标注错误；魔法数字（如 `1,2,8,9`）散落各处。 | 类名改为 `JhCommunityShopRevenuesDetailModel`；修正 `@return bool`；将支付类型、订单类型提取为类常量。 | `class JhCommunityShopRevenuesDetailModel extends Report_model`<br>`const PAY_PLATFORM_WX = 1;` |
| 🟡 建议 | 多处方法 | 频繁在业务方法内部调用 `$this->load->model()`。在 CI 架构中，模型加载应在构造函数或自动加载器中完成，重复加载增加 I/O 开销。 | 将依赖模型移至 `__construct()` 中统一加载，或使用框架的依赖注入容器。 | `public function __construct() { parent::__construct(); $this->load->model('ahead_yc_merchant_model'); ... }` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复数据丢失漏洞**：立即恢复 `ORDER_TYPE` 中键 `'13'` 的定义，或修正 `sales_type_map` 映射关系，确保所有订单类型均能正常落库。
2. **消除 SQL 注入风险**：对所有来自 `$params` 的数组参数进行严格类型校验（`(int)` 或 `filter_var`），禁止直接字符串拼接 SQL 片段。建议全面迁移至框架提供的 Query Builder 参数绑定语法。
3. **解决静态缓存污染**：将 `self::$date_type_info` 改为按 `$shop_id` 隔离的缓存结构，或改为实例级属性，避免多租户/多门店场景下的数据串扰。
### 🛠 后续重构与优化方向
- **架构解耦**：移除全局 `$CI` 依赖，将外部状态（如 `guotong_refund_id`）通过方法参数或上下文对象传递，提升代码的可测试性与 CLI 兼容性。
- **性能调优**：
- 将 `_add_log` 中的包厢信息查询改为**异步写入**或**批量预加载**，避免阻塞主交易链路。
- 报表聚合逻辑（如 `wx_income_amount` 等）可考虑下沉至数据库层，使用 `SUM(CASE WHEN _type=1 THEN _amount ELSE -_amount END)` 替代 PHP 循环累加，减少内存占用与网络传输。
- **规范与可维护性**：
- 统一遵循 PSR-12 命名规范，提取魔法数字为语义化常量。
- 补充输入参数校验（如 `start_time`、`page` 等），使用框架的 `Validation` 组件或自定义守卫方法，避免 `Undefined index` 报错。
- 若 `phpci` 为内部定制框架，建议查阅官方文档确认 `enforce_con_db()` 的连接池管理机制，确保高并发下连接释放正确，避免连接泄漏。
> 💡 **注**：代码结构高度契合 CodeIgniter 3 规范。若 `phpci` 为基于 CI 的二次开发框架，上述建议均适用。涉及框架底层组件（如 `enforce_con_db`、`throwError`）的用法，请以贵司内部技术文档为准。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780453567
|
1780453567
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
489
|
22
|
62
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `d682eb31c ## 自动代码审查报告
**分支**: app-260616
**提交**: `d682eb31c6980fa06ec90a57de0d6c9016cbfe22`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-03 10:29:27
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：该模型实现了社区商家营收明细的日志记录、多维度统计与趋势查询功能，业务逻辑覆盖较全。但存在**全局实例化滥用、静态缓存未隔离、SQL 拼接隐患、大量重复聚合逻辑**等核心问题。代码结构偏向过程式，未充分利用面向对象特性与框架生命周期，可维护性与安全性有较大提升空间。
- **风险等级**：🔴 高（存在数据串扰与潜在 SQL 注入风险）
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 文件顶部 (1-3行) | `$CI = &get_instance();` 在类外部直接执行。文件被 `include/require` 时即触发，若此时框架未完全初始化会导致致命错误，且破坏单例模式。 | 移除顶部代码。在模型内部通过 `$this->load->model()` 或 `$this->ci` 访问 CI 实例。 | `// 删除顶部两行，改为在方法内按需加载或使用构造函数` |
| 🔴 严重 | `get_date_type_info` | `public static $date_type_info = [];` 作为静态缓存未区分 `$shop_id`。同一请求内查询不同门店时，会返回首个门店的缓存数据，导致**严重业务数据错乱**。 | 使用 `$shop_id` 作为缓存键，或改为实例属性。 | `private static $date_cache = [];`<br>`if (!isset(self::$date_cache[$shop_id])) { ... }`<br>`return self::$date_cache[$shop_id];` |
| 🔴 严重 | `get_community_revenues_sum_data`<br>`get_community_revenues_detail_data` | 使用字符串拼接构造复杂 `WHERE` 条件：`'(a._pay_platform=' . $pay_platform . ' ...'`。若 `$params` 来源未严格校验，将引发 **SQL 注入**。 | 严禁手动拼接 SQL。使用框架查询构建器的 `group_start()`/`group_end()` 或 `where_in` 参数绑定。 | `$this->db->group_start();`<br>`foreach (...) { $this->db->or_where(...); }`<br>`$this->db->group_end();` |
| 🟠 警告 | 多个方法内部 | `$this->load->model()` 在每个业务方法中重复调用。CI 框架中重复加载会增加文件 I/O 与内存开销，且不符合依赖注入规范。 | 统一移至 `__construct()` 中加载，或配置自动加载。 | `public function __construct() { parent::__construct(); $this->load->model(['ahead_yc_merchant_model', 'ahead_yc_order_model', ...]); }` |
| 🟠 警告 | `get_community_revenues_data`<br>`get_community_revenues_sum_data` | 营收分类聚合逻辑（`wx_income_amount` 等计算）在两个方法中**完全重复**。后续新增支付类型需修改多处，极易遗漏导致数据不一致。 | 抽取为私有方法 `_aggregate_revenues(array $data): array`，统一处理。 | 见下方重构示例 |
| 🟠 警告 | `_add_log` | `isset(self::ORDER_TYPE[$insert['_order_type']])` 中常量键为字符串，若传入整型在 PHP 8+ 严格类型场景下可能匹配失败。 | 统一类型转换或使用 `array_key_exists`。 | `isset(self::ORDER_TYPE[(string)$insert['_order_type']])` |
| 🟡 建议 | 类定义/属性 | 类名 `Jh_community_shop_revenues_detail_model` 与方法名不符合 PSR-12 规范（应为 `StudlyCaps`/`camelCase`）。属性声明为 `public` 破坏封装性。 | 重命名类与方法，属性改为 `protected`。 | `class JhCommunityShopRevenuesDetailModel extends Report_model`<br>`protected $main_table_name = ...;` |
| 🟡 建议 | 多处 | 魔法数字/字符串硬编码（如 `'1'`, `'8'`, `86400`, `3600`），降低可读性与可配置性。 | 提取为类常量或业务配置项。 | `const PAY_PLATFORM_WECHAT = '1';`<br>`const SECONDS_PER_DAY = 86400;` |
| 🟡 建议 | PHPDoc 注释 | 多处 `@return true` 不符合 PHPDoc 标准，IDE 无法正确推断类型。 | 统一改为 `@return bool`。 | `@return bool` |
## 3. 总结与行动建议
### 🔑 优先修复项（P0/P1）
1. **移除全局 `$CI` 实例化**：立即删除文件顶部的 `$CI = &get_instance();`，避免框架初始化时序冲突。
2. **修复静态缓存串扰**：将 `get_date_type_info` 的静态缓存改为按 `$shop_id` 隔离，否则多门店并发请求必现数据污染。
3. **消除 SQL 拼接风险**：将 `pay_platform_where` 的字符串拼接替换为框架安全的参数绑定或 `or_where` 链式调用。
4. **统一模型加载**：将分散的 `$this->load->model()` 收敛至构造函数，降低运行时开销。
### 🛠 重构与优化方向
1. **聚合逻辑抽象**：
```php
private function _aggregate_revenues(array $data): array {
$totals = ['wx' => 0, 'tuangou_pay' => 0, 'tuangou' => 0, 'other' => 0];
foreach ($data as $v) {
$key = $v['type'] == 1 ? 'income' : 'refund';
$platform = $v['pay_platform'] == 1 ? 'wx' : ($v['pay_platform'] == 8 ? 'tuangou_pay' : ($v['pay_platform'] == 9 ? 'tuangou' : 'other'));
$totals[$platform][$key] += $v['amount'];
}
// 统一格式化并返回结果数组
return array_map(fn($v) => change_number_format($v['income'] - $v['refund']), $totals);
}
```
2. **数据库连接切换优化**：频繁调用 `enforce_con_db()` 会引发连接池抖动。建议在 `Report_model` 基类中实现**读写分离/分库路由的懒加载机制**，或在事务外层统一控制连接切换。
3. **框架适配说明**：代码中大量使用 `enforce_con_db()`、`set_table_v2()`、`check_table_exist()` 等自定义方法。若 `phpci` 框架未提供官方文档，建议补充内部注释说明其底层实现（如是否基于 CI 的 `DB::instance()` 或自定义分表中间件），以便后续维护者理解生命周期。
4. **异常处理规范化**：将 `throwError('...')` 替换为标准 `throw new \InvalidArgumentException('...')` 或框架内置异常类，便于全局错误拦截与日志追踪。
> 💡 **提示**：若代码片段仅展示模型层，建议同步审查调用该模型的 Controller/Service 层，确保 `$params` 输入经过严格过滤（如 `intval()`、`filter_var()`），从源头阻断越权与注入风险。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780453767
|
1780453767
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
490
|
22
|
63
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `9757cffc5 ## 自动代码审查报告
**分支**: app-260616
**提交**: `9757cffc5a8cc8098f078dfe0b0ccf5cabca1dfb`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-03 10:34:54
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：该模型业务逻辑覆盖较全，能够支撑社区门店营收统计的核心场景。但存在**高危 SQL 注入漏洞**、**静态缓存设计缺陷**、**财务计算精度隐患**及多处不符合现代 PHP 编码规范的问题。代码结构高度契合 CodeIgniter 3 风格，若 `phpci` 为定制框架，需重点核对底层查询构造器与生命周期是否兼容。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_community_revenues_sum_data`<br>`get_community_revenues_detail_data` | **SQL 注入风险**：手动拼接 `pay_platform_where` 字符串，若 `$params['pay_platform_arr']` 来自外部输入，将直接导致 SQL 注入。 | 严禁字符串拼接 SQL 条件。应使用框架提供的参数绑定或查询构造器方法（如 `where_in`、`or_where`）。 | `$this->db->group_start();`<br>`foreach (...) { $this->db->or_where(...); }`<br>`$this->db->group_end();` |
| 🔴 严重 | `get_date_type_info` | **静态缓存污染**：`self::$date_type_info` 为静态变量，首次加载后缓存了特定 `$shop_id` 的营业时间。若同请求处理多商户或定时任务，将返回错误的时间范围。 | 移除静态属性，改为实例属性缓存，或按 `$shop_id` 作为键值隔离缓存。 | `protected $date_type_cache = [];`<br>`if (empty($this->date_type_cache[$shop_id])) { ... }` |
| 🟠 警告 | 全局金额计算 | **财务精度丢失**：使用 `+`/`-` 直接对浮点数进行加减，PHP 浮点运算存在精度误差（如 `0.1+0.2 != 0.3`），可能导致对账不平。 | 财务计算必须使用 `bcmath` 扩展或统一转换为“分”（整数）运算，最后再格式化输出。 | `bcsub($income, $refund, 2)`<br>`bcmul($amount, 100, 0)` |
| 🟠 警告 | 文件顶部 | **框架实例滥用**：`$CI = &get_instance();` 在类文件顶部全局执行，不符合 CI/现代框架规范，易引发作用域污染与内存泄漏。 | 模型已继承基类，应直接使用 `$this`。如需访问核心实例，应在构造函数中赋值。 | `public function __construct() { parent::__construct(); }` |
| 🟠 警告 | `get_community_revenues_trend` | **循环内查询**：在 `foreach ($result as &$v)` 中执行 `$this->select()`，虽仅循环 2 次，但违背批量查询最佳实践，增加 DB 连接开销。 | 提取所有时间范围，使用单次 `IN` 查询或 `GROUP BY` 批量获取，在 PHP 层进行数据映射。 | 见下方优化建议 |
| 🟡 建议 | 类/方法命名 | **违反 PSR-12**：类名 `Jh_community_shop_revenues_detail_model` 与方法名使用蛇形命名，不符合 PHP 规范。 | 类名改为大驼峰（PascalCase），方法名改为小驼峰（camelCase）。 | `class JhCommunityShopRevenuesDetailModel`<br>`public function addByBookOrderRefund()` |
| 🟡 建议 | 多处方法 | **魔法数字泛滥**：大量硬编码 `1, 2, 8, 9, 17` 等代表业务状态，可读性差且易遗漏维护。 | 提取为类常量，配合类型提示提升可维护性。 | `const TYPE_INCOME = 1;`<br>`const PLATFORM_WX = 1;` |
| 🟡 建议 | 模型加载 | **重复加载模型**：各方法内频繁调用 `$this->load->model()`，增加框架 I/O 开销。 | 统一在构造函数中加载，或使用依赖注入容器管理。 | `__construct() { $this->load->model('ahead_yc_merchant_model'); }` |
## 3. 总结与行动建议
### 🔑 优先修复项（P0/P1）
1. **修复 SQL 注入**：立即重构 `get_community_revenues_sum_data` 与 `detail_data` 中的 `pay_platform_arr` 条件拼接逻辑。若 `phpci` 查询构造器支持，请使用参数绑定：
```php
// 安全写法示例
$this->db->group_start();
foreach ($params['pay_platform_arr'] as $platform) {
$parts = explode('_', $platform);
if (!empty($parts[1])) {
$this->db->or_where(['a._pay_platform' => $parts[0], 'a._second_pay_platform' => $parts[1]]);
} else {
$this->db->or_where('a._pay_platform', $parts[0]);
}
}
$this->db->group_end();
```
2. **解决静态缓存错乱**：将 `public static $date_type_info = [];` 改为实例属性 `protected $date_type_info = [];`，并在 `get_date_type_info` 中按 `$shop_id` 隔离缓存。
3. **财务计算精度**：全局替换 `+`/`-` 金额计算为 `bcmath` 函数，例如：
```php
$result['wx_amount'] = change_number_format(bcsub($wx_income_amount, $wx_refund_amount, 2));
```
### 🛠 后续重构方向
- **规范命名与注释**：全面遵循 PSR-12 规范，修正类名、方法名及 `@return` 类型（`true` → `bool`）。补充关键业务逻辑的注释，特别是 `sales_type_map` 与 `revenues_pay_platform_arr` 的映射关系。
- **查询性能优化**：将 `get_community_revenues_trend` 中的循环查询改为单次聚合查询。例如：
```php
// 优化思路：一次性查出所有时间段数据，PHP 层按 key 映射
$where['where_in'][] = ['_business_date', array_merge($start_dates, $end_dates)];
$data = $this->select($where, $fields);
// 使用 array_column 或哈希表快速匹配
```
- **异常处理标准化**：将全局函数 `throwError()` 替换为 PHP 标准异常或框架异常类（如 `throw new \RuntimeException('营业收入走势查询时间错误');`），便于统一捕获与日志记录。
- **框架适配确认**：代码中大量使用 `$this->set_table_v2()`、`$this->enforce_con_db()` 等自定义方法。请确保这些方法在 `Report_model` 基类中已正确处理事务、读写分离与表名动态切换。若 `phpci` 官方文档有推荐的动态表名或分库分表方案，建议对齐官方最佳实践。
> 💡 **局限性说明**：本次审查仅基于提供的单文件代码。若 `Report_model` 基类、全局辅助函数（如 `get_business_date`）或控制器层存在未暴露的上下文逻辑，部分边界条件（如事务回滚、并发写入）需结合完整调用链进一步验证。建议补充单元测试覆盖核心统计方法。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780454094
|
1780454094
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
491
|
22
|
64
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `872263090 ## 自动代码审查报告
**分支**: app-260616
**提交**: `8722630906c7db3c89ec06b6fa901971033f71c0`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-03 10:39:02
---
## 1. 审查摘要
- **代码质量评分**：5/10
- **总体评价**：代码整体业务逻辑完整，模型层对营收数据的聚合与趋势查询结构清晰。但 Helper 文件中存在大量历史遗留代码，包含**硬编码敏感凭证、PHP 8 不兼容语法、非原子性分布式锁实现**等严重问题。模型层存在 N+1 查询隐患与重复加载模型的性能损耗。整体需进行安全加固与现代 PHP 语法重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `common_helper.php`<br>`alioss_deleteObject` / `alioss_addObject` | **硬编码阿里云 OSS AccessKey/SecretKey**。密钥直接暴露在源码中，极易导致云存储数据泄露或被恶意盗刷。 | 将凭证迁移至配置文件或环境变量（如 `.env`），通过框架配置加载。 | `$param = config_item('aliyun_oss');`<br>`$obj = new Alioss($param);` |
| 🔴 严重 | `common_helper.php`<br>`doBatchFromTable` | **数组遍历赋值逻辑错误**。`$data` 是值拷贝，修改 `$data[$k]` 不会影响原数组 `$list`，导致函数始终返回未处理的原数据。 | 直接修改 `$list` 对应键值。 | `foreach ($list as $k => $data) { $list[$k] = doSingleFromTable($data); }` |
| 🔴 严重 | `common_helper.php`<br>`doRedisLockRelease` | **分布式锁释放存在竞态条件**。`GET` 校验与 `DEL` 删除非原子操作，若锁在 `GET` 后刚好过期，可能误删其他进程持有的锁。 | 使用 Lua 脚本保证 `GET` 与 `DEL` 的原子性。 | `lua: if redis.call("get",KEYS[1]) == ARGV[1] then return redis.call("del",KEYS[1]) else return 0 end` |
| 🔴 严重 | `common_helper.php`<br>`decodeUnicode` / `getFirstCharter` | **PHP 8 不兼容语法**。`create_function()` 已在 PHP 8.0 移除；字符串/数组花括号访问 `$str{0}` 已在 PHP 7.4 废弃、8.0 移除。 | 替换为匿名函数与方括号语法。 | `preg_replace_callback(..., fn($m) => mb_convert_encoding(...), $str);`<br>`$fchar = ord($str[0]);` |
| 🟠 警告 | `Jh_community_shop_revenues_detail_model.php`<br>`get_community_revenues_trend` | **N+1 查询性能瓶颈**。在 `foreach ($result as &$v)` 循环中执行 `$this->select()`，若日期跨度大将产生大量 DB 查询。 | 改为单次查询获取全量数据，在 PHP 层按日期分组聚合；或使用 `WHERE IN` 批量拉取。 | `$all_data = $this->select($batch_where, $fields);`<br>`$grouped = array_column($all_data, null, $group_by);` |
| 🟠 警告 | `common_helper.php`<br>`curlRequest` / `asyncRequest` | **禁用 SSL 证书验证**。`CURLOPT_SSL_VERIFYPEER, false` 会暴露于中间人攻击（MITM）风险。 | 启用验证并配置 CA 证书路径，或至少在生产环境强制开启。 | `curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, true);`<br>`curl_setopt($curl, CURLOPT_CAINFO, '/path/to/cacert.pem');` |
| 🟠 警告 | `common_helper.php`<br>`passport_encrypt` / `decrypt` | **弱加密算法**。使用 `srand()` + `XOR` 异或加密，不具备现代密码学安全性，且 `srand()` 已被弃用。 | 替换为 `openssl_encrypt()` / `openssl_decrypt()` (AES-256-CBC)。 | `openssl_encrypt($str, 'AES-256-CBC', $key, 0, $iv);` |
| 🟠 警告 | `Jh_community_shop_revenues_detail_model.php`<br>多处方法 | **重复加载模型**。每个方法内部频繁调用 `$this->load->model()`，增加框架 I/O 开销。 | 在 `__construct()` 中统一加载，或使用 CI 自动加载配置。 | `public function __construct() { parent::__construct(); $this->load->model(['ahead_yc_merchant_model', 'ahead_family_servers_model']); }` |
| 🟡 建议 | `common_helper.php`<br>`checkDateFormate` | **日期校验逻辑不严谨**。`$str !== date($formate, strtotime($str))` 会因时区转换或格式差异误判合法日期。 | 使用 `DateTime::createFromFormat()` 进行严格校验。 | `$dt = DateTime::createFromFormat($formate, $str); return $dt && $dt->format($formate) === $str;` |
| 🟡 建议 | `common_helper.php`<br>`rs_hmset` | **函数重复定义**。`rs_hmset` 被完整定义了两次，触发 `Cannot redeclare function` 致命错误。 | 删除重复的函数块。 | *(直接删除第二个 `if (!function_exists('rs_hmset'))` 块)* |
| 🟡 建议 | `Jh_community_shop_revenues_detail_model.php`<br>`get_community_revenues_sum_data` | **手动拼接 SQL 条件存在注入隐患**。`$pay_platform_where[] = '(a._pay_platform=' . $pay_platform . '...'` 未使用查询构建器。 | 使用框架 Query Builder 的 `where()` 或 `or_where()` 链式调用。 | `$this->db->group_start(); $this->db->where('a._pay_platform', $p1); $this->db->where('a._second_pay_platform', $p2); $this->db->group_end();` |
> 📝 **局限性说明**：文件末尾的 `import()` 函数代码被截断，无法评估其完整实现逻辑。若该函数用于动态加载类/文件，需注意路径遍历漏洞与重复包含问题。
## 3. 总结与行动建议
### 🔑 优先修复项（P0/P1）
1. **立即移除硬编码凭证**：将 OSS `accessId`/`accessKey` 迁移至 `application/config/` 或环境变量，并通过 `config_item()` 读取。
2. **修复 PHP 8 兼容性**：全局替换 `$str{0}` 为 `$str[0]`，移除 `create_function()` 改用匿名函数，确保代码可在 PHP 8.1+ 环境稳定运行。
3. **修复分布式锁竞态**：将 `doRedisLockRelease` 改为 Lua 脚本执行，避免高并发下误删锁导致业务超卖或数据错乱。
4. **修正数组处理 Bug**：修复 `doBatchFromTable` 的赋值逻辑，确保数据转换生效。
### 🛠 后续重构与优化方向
1. **性能架构优化**：
- 解决 `get_community_revenues_trend` 的 N+1 查询问题，采用**单次批量查询 + PHP 内存分组**策略。
- `get_aliyun_redis_conn()` 每次调用新建连接，建议引入 **Redis 连接池** 或使用框架内置的 `Cache` 驱动（如 `CI_Cache`）复用连接。
2. **安全与规范升级**：
- 废弃 `passport_encrypt` 系列函数，全面迁移至 `openssl` 或框架内置的 `Encryption` 库。
- 开启 cURL 的 SSL 验证，生产环境务必配置可信 CA 证书。
- 遵循 PSR-12 规范：补充类型声明（`string`, `int`, `array`, `bool`）、统一命名风格（建议全量采用 `camelCase` 或 `snake_case`）、移除冗余注释。
3. **框架适配建议**：
- 代码呈现典型的 **CodeIgniter 3** 架构特征。若 `phpci` 为 CI 的定制分支，请确认 `enforce_con_db()` 切换数据库连接的生命周期管理是否符合框架规范，避免连接泄漏。
- 模型中的静态属性 `self::$date_type_info` 在 CLI 长驻进程（如 Swoole/Workerman）中会导致状态污染，建议改为实例属性或注入配置服务。
- `defined('BASEPATH') OR exit(...)` 应置于 Helper 文件首行，防止直接访问。
建议按 **安全修复 → 语法兼容 → 性能重构 → 规范统一** 的顺序推进迭代。如需针对特定函数提供完整重构代码或单元测试用例，可提供具体业务场景进一步细化。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780454342
|
1780454342
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
492
|
22
|
65
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `75a9d9baa ## 自动代码审查报告
**分支**: app-260616
**提交**: `75a9d9baa49a930ee4e1d99ccd63a2d417ef0915`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-03 10:40:12
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10
- **总体评价**：该模型覆盖了社区商家营收明细的核心业务场景，数据聚合与趋势计算逻辑较为完整。但存在**静态缓存误用导致的数据污染**、**全局实例化反模式**、**SQL 拼接安全隐患**以及**多处重复代码与硬编码**。财务相关写入操作缺乏事务保护，整体架构可维护性与健壮性有待提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 第 2-4 行 | 文件顶部直接调用 `get_instance()` 并加载模型。该代码在文件被 `include/require` 时即执行，无论类是否被实例化，造成不必要的性能损耗与全局状态污染。 | 移除顶部代码，将依赖模型加载移至构造函数中，遵循面向对象初始化规范。 | `public function __construct() { parent::__construct(); $this->load->model('Report_model'); }` |
| 🔴 严重 | `get_date_type_info` 方法 | 使用 `public static $date_type_info` 缓存时间配置，但缓存逻辑依赖实例方法 `$this->load->model()` 且未区分 `$shop_id`。首次调用后，后续不同门店的请求将返回错误缓存，导致报表时间范围错乱。 | 移除静态属性，改为实例属性缓存或使用框架缓存驱动（如 Redis/CI Cache），并以 `$shop_id` 作为缓存键。 | `$cache_key = 'date_type_' . $shop_id; if (!$this->cache->get($cache_key)) { ... }` |
| 🟠 警告 | `get_community_revenues_sum_data` & `detail_data` | 手动拼接 SQL 条件字符串 `'(a._pay_platform=' . $pay_platform . ...)'` 并传入 `where` 数组。若 `$params` 未经严格类型校验，将绕过查询构建器的转义机制，存在 SQL 注入风险。 | 使用查询构建器的 `group_start()` / `group_end()` 安全拼接，或强制转换为整型后使用参数绑定。 | `$this->db->group_start()->where('a._pay_platform', (int)$p1)->where('a._second_pay_platform', (int)$p2)->group_end();` |
| 🟠 警告 | `get_community_revenues_trend` 方法 | 在 `foreach ($result as &$v)` 循环中执行 `$this->select()`，形成循环查库。虽当前仅循环 2 次，但违背批量查询原则，且未使用 `try...finally` 保证 `enforce_con_db(2)` 必定执行，异常时可能残留连接状态。 | 提取时间范围后使用 `IN` 或范围查询一次性拉取数据，在 PHP 层分组聚合；数据库切换操作需包裹 `try...finally`。 | `try { $this->enforce_con_db(); $data = $this->select(...); } finally { $this->enforce_con_db(2); }` |
| 🟠 警告 | 所有 `add_by_*` 方法 | 财务数据写入直接调用 `$this->insert()`，未包裹数据库事务。若 `_add_log` 内部或后续业务逻辑抛出异常，可能导致营收明细与主订单状态不一致。 | 使用框架事务机制包裹关键写入流程，失败时自动回滚。 | `if ($this->db->trans_begin()) { $this->insert($data); $this->db->trans_commit(); } else { $this->db->trans_rollback(); }` |
| 🟡 建议 | 全局多处 | 大量硬编码魔法数字（如 `1`/`2` 表示收支类型，`8`/`9` 表示支付渠道），可读性差且后期维护易遗漏。 | 定义类常量统一管理业务状态，或使用 PHP 8.1+ `enum`。 | `const TYPE_INCOME = 1; const TYPE_REFUND = 2; const PAY_WECHAT = 1;` |
| 🟡 建议 | `get_community_revenues_data` & `sum_data` | 两个方法的 `foreach` 聚合逻辑完全重复，违反 DRY 原则。 | 提取为私有方法 `aggregate_revenue_data($data)`，统一处理金额计算与格式化。 | `private function aggregate_revenue_data(array $data): array { ... }` |
| 🟡 建议 | 类名与方法名 | 命名不符合 PSR-12 规范（如 `Jh_community_shop_revenues_detail_model`、`_add_log`）。 | 类名改为 `PascalCase`，方法名改为 `camelCase`，移除前导下划线。 | `class JhCommunityShopRevenuesDetailModel extends Report_model`<br>`private function addLog(...)` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复静态缓存污染**：立即移除 `public static $date_type_info` 的跨实例共享逻辑，改为基于 `$shop_id` 的实例缓存或框架缓存，否则多门店并发请求将产生严重数据错乱。
2. **消除 SQL 拼接隐患**：将 `pay_platform_where` 的手动字符串拼接替换为查询构建器的安全条件方法（如 `group_start()`/`group_end()` 或 `or_where()`），并对 `$params` 输入进行严格类型过滤。
3. **补充事务保护**：所有涉及 `_amount` 写入的 `add_by_*` 方法必须包裹数据库事务，确保财务明细与业务主表的数据原子性。
### 🛠 后续重构与优化方向
- **架构规范化**：将文件顶部的 `get_instance()` 调用彻底移除，依赖注入或构造函数加载模型。若 `phpci` 支持依赖注入容器，建议逐步替换 `$this->load->model()`。
- **查询性能优化**：`get_community_revenues_trend` 中的循环查库可优化为单次范围查询 + PHP 数组映射。建议在 `_business_date`、`_pay_platform`、`_type` 字段建立联合索引，加速 `GROUP BY` 聚合。
- **代码可维护性提升**：
- 提取重复的聚合逻辑与社区商家校验逻辑至 `Report_model` 基类。
- 使用常量/枚举替代魔法数字，配合 IDE 提示降低误用率。
- 统一遵循 PSR-12 命名与注释规范，补充 `@throws`、`@return` 类型声明。
- **框架适配提示**：代码呈现典型的 CodeIgniter 3/4 架构特征。若 `phpci` 为内部定制框架，请确认 `enforce_con_db()`、`select()` 等底层方法是否已内置防注入与连接池管理机制。建议查阅 `phpci` 官方文档中关于 **Query Builder 安全绑定** 与 **多数据源切换** 的最佳实践。
> 💡 **审查局限性说明**：本次审查基于提供的单一模型文件。若 `Report_model` 基类或 `select()` 底层实现已内置事务/防注入逻辑，部分风险等级可相应下调。建议结合完整调用链与数据库索引结构进行二次验证。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780454412
|
1780454412
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
566
|
22
|
66
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 续费小程序码
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `a8873702e ## 自动代码审查报告
**分支**: app-260616
**提交**: `a8873702e780376c358ba618e5f45319c71e05af`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-05 16:02:05
---
## 1. 审查摘要
- **代码质量评分**：待评估 (0-10 分)
- **总体评价**：当前输入仅包含项目目录结构，**未提供具体的“变更文件内容”代码片段**。该目录结构与 CodeIgniter 3 高度相似，若 `phpci` 为基于 CI 的定制框架或内部分支，请补充实际变更代码。收到后将立即从逻辑、安全、性能、规范及框架适配 5 个维度进行深度静态分析。
- **风险等级**：待评估 (高/中/低)
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| ⚠️ 信息 | 全局 | **未提供变更代码**：当前输入仅包含项目目录结构，缺少具体的 PHP 代码片段。无法执行逻辑正确性、安全性、性能瓶颈、PSR-12 规范及框架生命周期调用的审查。 | 请提供具体的变更文件内容（建议包含完整类/方法或关键业务逻辑），并标注对应文件路径。收到后将按标准格式输出详细审查报告。 | 无 |
## 3. 总结与行动建议
- **优先补充项**：请粘贴实际变更的 PHP 代码。为提升审查精度，建议附带：
1. 变更所属模块（如 `controllers/`, `models/`, `helpers/`）
2. 涉及的业务场景与数据流向
3. 已知的边界条件或异常处理预期
- **后续重构与优化方向**：
1. **框架适配确认**：目录结构呈现典型的 MVC 架构（类似 CI3）。若 `phpci` 为自研或定制版本，请确认其是否沿用原框架的 `Loader`、`Query Builder`、`Session` 驱动及钩子机制。对于不确定的框架特定用法，我将标注说明并建议查阅 `phpci` 官方文档。
2. **安全基线建议**：在代码提交前，建议统一落实：
- 数据库操作强制使用参数绑定或 Query Builder，杜绝字符串拼接 SQL。
- 输出到视图的数据统一经过 `htmlspecialchars()` 或框架内置 XSS 过滤。
- 敏感操作（如支付、权限变更）需校验 CSRF Token 与 Session 状态。
3. **规范与性能基线**：建议启用 `declare(strict_types=1);`，遵循 PSR-12 命名与缩进规范；避免在循环内执行数据库查询或文件 I/O；优先使用框架提供的缓存驱动（如 `Cache_redis`/`Cache_memcached`）替代重复计算。
请随时补充变更代码，我将立即为您生成完整的深度审查报告。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780646525
|
1780646525
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
567
|
22
|
67
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 续费小程序码
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `697aa9f19 ## 自动代码审查报告
**分支**: app-260616
**提交**: `697aa9f1918d12c5fb5d69b56908c15e2658e2b1`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-05 16:03:21
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：该模型文件承载了大量核心业务逻辑（包厢状态流转、权限过滤、计费差价计算等），整体结构具备一定分层意识。但存在多处**高危 SQL 注入风险**、**Token 解析缺乏安全校验**、**返回值类型严重不一致**以及**全局函数/变量滥用**等问题。代码规范与 PSR-12 存在较大偏差，且末尾代码被截断，部分逻辑无法完整评估。
- **风险等级**：🔴 高（存在直接拼接 SQL 的注入点、未校验的 Token 解析、逻辑缺陷可能导致资损或越权）
> ⚠️ **局限性说明**：您提供的代码在 `get_room_detail` 方法末尾（`$priv_where['_role_i`）被截断。本次审查仅基于已提供的完整代码片段，未覆盖截断后的逻辑。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `list_rooms_info`<br>`list_nearby_family_servers_info`<br>`get_nearby_ids`<br>`get_family_server_ids_by_shop_ids` | **SQL 注入漏洞**：多处使用 `$addsql` 或 `$shop_ids` 直接拼接 SQL 字符串，未做任何过滤或参数绑定。攻击者可构造恶意参数执行任意 SQL。 | 废弃字符串拼接，全面改用 CI 查询构建器（Query Builder）或参数绑定 `$this->db->query($sql, $binds)`。 | ```php<br>// 修复前<br>$sql = "... WHERE _shop_id in " . $shop_ids;<br><br>// 修复后<br>$ids = explode(',', $shop_ids);<br>$this->db->where_in('_shop_id', $ids);<br>$result = $this->db->get('ahead_family_servers')->result_array();<br>``` |
| 🔴 严重 | `get_count_status_room`<br>`get_count_room_by_shop` | **Token 解析无校验且忽略签名**：`explode("_", $token)` 未检查数组长度，越界会触发 Warning；`$sign` 参数完全未参与验签，存在伪造身份越权风险。 | 增加格式校验、长度检查，并调用框架签名验证函数。 | ```php<br>$parts = explode('_', trim($request['token']));<br>if (count($parts) !== 4) {<br> return ['status' => -1, 'msg' => 'Token格式错误'];<br>}<br>[$merchant_id, $uid, $utype, $sign] = $parts;<br>// 必须调用签名验证逻辑<br>if (!$this->verify_token_sign($merchant_id, $uid, $utype, $sign)) {<br> return ['status' => -2, 'msg' => '签名验证失败'];<br>}<br>``` |
| 🔴 严重 | `get_is_valid_family_server_id` | **逻辑缺陷：参数未生效**：方法接收 `$id` 和 `$shop_id`，但 SQL 查询中完全未使用这两个条件，导致返回结果与预期不符。 | 将参数加入 WHERE 条件。 | ```php<br>$sql = "SELECT count(1) as num FROM ahead_family_servers WHERE _family_server_id=? AND _merchant_id=? AND _shop_id=?";<br>$result['data'] = $this->db->query($sql, [$family_server_id, $id, $shop_id])->row_array();<br>``` |
| 🟠 警告 | 全局多处 | **返回值类型不一致**：部分方法返回 `array('status'=>1)`，部分返回 `bool`，部分返回 `string`（如 `last_query()`）。调用方难以统一处理，易引发 `Undefined index` 或类型错误。 | 统一模型层返回规范，建议始终返回结构化数组：`['success' => bool, 'data' => mixed, 'msg' => string]`。 | ```php<br>// 统一规范示例<br>public function add_room(...): array {<br> if ($exists) {<br> $this->update(...);<br> return ['success' => true, 'msg' => '更新成功'];<br> }<br> $res = $this->insert(...);<br> return $res ? ['success' => true, 'msg' => '新增成功'] : ['success' => false, 'msg' => '数据库写入失败'];<br>}<br>``` |
| 🟠 警告 | `get_room_detail` (约 L950) | **字符串拼接含双逗号**：`$functions .= "32,2,5,6,12,18,,20,24,58";` 中 `18,,20` 存在连续逗号，下游解析时会产生空元素或报错。 | 使用数组管理功能码，最后统一 `implode(',', $functions)`。 | ```php<br>$functions = [32, 2, 5, 6, 12, 18, 20, 24, 58];<br>// 后续动态添加<br>$functions[] = 41;<br>$func_str = implode(',', array_unique($functions));<br>``` |
| 🟠 警告 | 文件顶部 & 方法内 | **`$CI =& get_instance()` 滥用**：在类外部调用一次，内部方法又重复调用。违反 CI 框架设计，增加内存开销且破坏封装性。 | 移除文件顶部调用。在 CI 模型中，`$this` 已继承自 `CI_Model`，可直接使用 `$this->load`、`$this->db` 等。 | ```php<br>// 删除文件顶部的 $CI =& get_instance();<br>// 方法内直接使用 $this->load->helper('common'); 即可<br>``` |
| 🟡 建议 | 类属性定义 (L15-L30) | **属性可见性过高**：`$tableName`、`$redis_key`、`$room_info` 等均为 `public`，外部可随意篡改，破坏对象状态。 | 改为 `protected` 或 `private`，通过 Getter/Setter 或受保护方法访问。 | `protected $tableName = 'ahead_family_servers';` |
| 🟡 建议 | 全局函数调用 | **强依赖全局函数**：大量使用 `get_aliyun_redis_conn()`、`smallChangeFormat()`、`preMinute()` 等全局函数。不利于单元测试、依赖注入及框架迁移。 | 将全局函数封装为 Helper 类或服务类，通过 `$this->load->helper()` 或 DI 容器注入。 | 建议逐步重构为 `RedisService::getInstance()->hGet(...)` |
| 🟡 建议 | 代码规范 | **短 `if` 语句未使用大括号**：如 `if ($room_area) $where['_room_area'] = $room_area;` 违反 PSR-12 规范，易引发后续维护时的逻辑错误。 | 严格遵循 PSR-12，所有控制结构必须使用大括号。 | ```php<br>if ($room_area) {<br> $where['_room_area'] = $room_area;<br>}<br>``` |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题（P0/P1）
1. **彻底修复 SQL 注入**：立即替换 `list_rooms_info`、`list_nearby_family_servers_info`、`get_nearby_ids` 等 4 个方法中的字符串拼接 SQL。这是最高危的安全漏洞，必须优先处理。
2. **统一返回值契约**：制定团队级模型返回规范（如 `['code' => int, 'data' => mixed, 'message' => string]`），并在 `update_room_status`、`add_room` 等方法中强制对齐，避免调用层崩溃。
3. **修复 Token 解析逻辑**：补充 `explode` 后的 `count()` 校验，并接入签名验证机制。当前逻辑等同于“裸奔”，极易被伪造请求绕过权限。
### 🛠 后续重构与优化方向
1. **状态机与魔法数字治理**：代码中大量使用 `-1, 0, 1, 2, 3` 及 `32,2,5...` 等硬编码。建议提取为类常量或枚举（PHP 8.1+），例如：
```php
const STATUS_FREE = 0;
const STATUS_CONSUMING = 1;
const STATUS_REPAIRING = 2;
```
2. **Redis 连接生命周期管理**：当前 `get_aliyun_redis_conn()` 在多处被调用且部分未显式 `close()`。建议封装为单例或连接池，或在 CI 的 `__destruct()` 中统一释放，防止连接泄漏。
3. **解耦全局函数**：将 `smallChangeFormat`、`getNewSmallChange` 等计费相关函数抽离至独立的 `PriceCalculator` 服务类，便于后续进行单元测试和价格策略扩展。
4. **框架适配说明**：基于代码特征，`phpci` 高度兼容 CodeIgniter 3。若该框架有自定义的 `DB_driver` 或 `Session` 实现，请确保 `$this->db->query()` 的底层预处理机制未被覆盖。建议查阅 `phpci` 官方文档中关于 **Query Builder 安全过滤** 与 **模型自动加载** 的章节。
> 💡 **下一步建议**：修复上述 P0 问题后，建议引入静态分析工具（如 `PHPStan` 或 `Psalm`）配置 CI 流水线，自动拦截类型不一致与未定义变量问题。如需对截断的 `get_room_detail` 方法或特定计费逻辑进行深度审查，请提供完整代码片段。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780646601
|
1780646601
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
568
|
22
|
68
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 用户续费二维码
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `387674004 ## 自动代码审查报告
**分支**: app-260616
**提交**: `387674004b33a862e7a9cd60a048c980faab1d6a`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-05 16:10:58
---
## 1. 审查摘要
- **代码质量评分**：4/10 分
- **总体评价**：代码实现了较为复杂的包厢状态管理、计费转房及权限控制逻辑，但存在多处严重的安全隐患（SQL注入）、架构反模式（全局实例滥用、职责混杂）及类型安全缺陷。整体技术债务较重，需优先修复安全漏洞并重构核心业务流。
- **风险等级**：🔴 高
> 📌 **框架说明**：代码中大量使用 `$CI =& get_instance()`、`$this->load->model()`、`$this->db` 链式调用，语法特征高度符合 **CodeIgniter 3** 架构。若 `phpci` 为贵司内部定制或衍生框架，请以官方文档为准，以下建议基于通用 PHP 规范及 CI 生态最佳实践。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `list_rooms_info`<br>`list_nearby_family_servers_info`<br>`get_nearby_ids`<br>`get_family_server_ids_by_shop_ids` | **SQL 注入漏洞**：多处直接使用字符串拼接构造 SQL 条件（如 `WHERE " . $addsql`、`in " . $shop_ids`），未进行参数绑定或过滤。若参数来自前端或外部接口，将导致数据库被恶意篡改或拖库。 | 全面废弃原生 SQL 拼接，改用 CI Query Builder 或参数化查询。禁止将外部输入直接拼接入 SQL 字符串。 | `$this->db->select('_family_server_id')->where_in('_shop_id', explode(',', $shop_ids))->get('ahead_family_servers')->result_array();` |
| 🔴 严重 | 文件头部 (第 8-9 行) | **全局实例滥用**：在类外部声明 `$CI =& get_instance();` 并加载 Model。CI 框架中 Model 已继承 `$this->load`，此写法会破坏框架生命周期，易引发内存泄漏或上下文污染。 | 删除文件头部的 `$CI` 声明。在类内部直接使用 `$this->load->model()` 或 `$this->db`。若需在非 CI 类中调用，应在方法内部按需获取。 | `// 删除文件开头的：<br>$CI =& get_instance();<br>$CI->load->model('Simple_model');` |
| 🔴 严重 | `get_count_status_room`<br>`get_count_room_by_shop` | **Token 解析越权风险**：`list($merchant_id, $uid, $utype, $sign) = explode("_", $token);` 未校验 Token 分段数量。若格式不符，PHP 8+ 会抛出 Warning，且变量错位将导致商户/用户身份混淆，引发严重越权。 | 增加结构校验，失败时直接拦截。建议使用 `sscanf` 或严格数组解构。 | `$parts = explode('_', $token);<br>if (count($parts) !== 4) return ['status' => -1, 'msg' => 'Token格式非法'];<br>[$merchant_id, $uid, $utype, $sign] = $parts;` |
| 🟠 警告 | `update_status`<br>`update_room_status` | **缺乏事务保护**：状态更新涉及 DB 写入、操作日志记录、Redis/MQTT 通知。若 DB 成功但后续环节失败，将导致“房态与日志/缓存不一致”。 | 使用 `$this->db->trans_start()` 包裹核心逻辑，并校验 `$this->db->trans_status()`。失败时自动回滚。 | `$this->db->trans_start();<br>$re = $this->update(...);<br>$this->load->model(...)->change_room_status(...);<br>$this->db->trans_complete();<br>return $this->db->trans_status() ? ['status'=>1] : ['status'=>-5];` |
| 🟠 警告 | `update_status` 第 78 行 | **松散比较导致逻辑漏洞**：`if ($pre_status == $status)` 使用 `==`。在 PHP 中 `0 == ""` 为 `true`，若传入空字符串或 `0` 将错误拦截合法状态变更。 | 统一使用严格比较 `===`，并在入口处强制类型转换。 | `if ((int)$pre_status === (int)$status) { return ['status' => -3, 'msg' => '当前状态与要修改的状态一致']; }` |
| 🟠 警告 | `update_room_status` 第 108 行 | **死代码/资源浪费**：`$redis_conn = get_aliyun_redis_conn();` 赋值后全程未调用任何 Redis 方法，白白占用连接池资源。 | 移除未使用的变量，或补充实际的 Redis 状态同步逻辑。 | `// 删除 $redis_conn = get_aliyun_redis_conn();` |
| 🟡 建议 | 全局多处 | **魔术数字与类型混杂**：状态值 `-1,0,1,2,3` 硬编码且类型不统一（字符串/整型混用），`in_array` 未开启严格模式，降低可读性且易引发隐式类型转换 Bug。 | 定义类常量，统一输入类型，`in_array` 启用第三个参数 `true`。 | `const STATUS_FREE = 0; const STATUS_CONSUMING = 1; ...<br>if (!in_array((int)$status, self::$room_status, true))` |
| 🟡 建议 | `get_change_room_info` 第 268 行 | **错误抑制符滥用**：`$openRoomFlags = @json_decode($openRoomFlags, true);` 使用 `@` 掩盖解析错误，导致后续逻辑在非法 JSON 时静默失败。 | 移除 `@`，使用 `json_last_error()` 校验，或封装安全解析方法。 | `$openRoomFlags = json_decode($openRoomFlags, true);<br>if (json_last_error() !== JSON_ERROR_NONE) { $openRoomFlags = []; }` |
| 🟡 建议 | `get_room_detail` 等 | **单一职责违背**：`get_room_detail` 超 300 行，混合了权限校验、价格计算、状态映射、视图数据组装。Model 层不应承载复杂业务编排。 | 将计费逻辑、权限过滤、状态机映射抽离至独立的 `RoomService` 或 `BillingCalculator` 类。Model 仅负责 CRUD。 | (架构级重构建议) |
| 🟡 建议 | 文件末尾 | **代码截断**：代码在 `$priv_where['_role_i` 处突然中断，无法审查 `get_room_detail` 的完整权限校验与功能列表生成逻辑。 | 请补充完整代码以便进行闭环审查。 | N/A |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **彻底消除 SQL 注入风险**：立即替换所有字符串拼接的 SQL 语句。使用 `$this->db->where()`、`$this->db->where_in()` 或 `$this->db->query($sql, $bindings)`。
2. **修复 Token 解析与状态比较漏洞**：增加 Token 分段数量校验；将 `==` 替换为 `===` 并统一类型转换，防止越权与状态机错乱。
3. **引入数据库事务**：为 `update_status`、`update_room_status` 等涉及多表/多系统（DB+Redis+MQTT）的状态变更操作包裹事务，确保数据最终一致性。
### 🛠 后续重构与优化方向
1. **架构分层**：当前 Model 承担了过多业务逻辑（计费、权限、状态机、MQTT通知）。建议遵循 **MVC + Service 层** 架构：
- `Model`：仅负责数据持久化与基础查询。
- `Service`：处理转房差价计算、状态流转校验、权限过滤。
- `Helper/Library`：封装 Redis 操作、MQTT 推送、金额格式化（如 `smallChangeFormat`）。
2. **规范框架用法**：
- 移除全局 `$CI =& get_instance()`，依赖 CI 内置的 `$this` 上下文。
- 将频繁调用的 `$this->load->model()` 移至构造函数或采用懒加载，避免重复初始化开销。
- 公开属性 (`public $tableName` 等) 改为 `protected`，遵循封装原则。
3. **类型安全与常量管理**：
- 使用 `const` 或 PHP 8.1+ `enum` 统一管理房态、开房类型、操作码。
- 统一 API 返回结构（如始终返回 `['code' => int, 'msg' => string, 'data' => mixed]`），避免混用 `status`、`success`、布尔值。
4. **性能优化**：
- `get_nearby_ids` 中的字符串拼接可替换为 `implode(',', array_column(...))` 或直接使用 `where_in`。
- 对高频查询（如包厢列表、状态统计）增加 Redis 缓存层，并设置合理的过期策略与缓存击穿防护。
> ⚠️ **局限性说明**：由于提供的代码在 `get_room_detail` 方法末尾被截断，权限校验逻辑、功能按钮动态生成及后续业务流未能完整审查。建议补充完整文件后，可进一步针对权限越权、并发状态冲突及缓存一致性进行深度审计。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780647059
|
1780647059
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
569
|
22
|
69
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 1
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `6f7666fbf ## 自动代码审查报告
**分支**: app-260616
**提交**: `6f7666fbfcd3c63692feac764fa0a0fc7251b2f9`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-05 16:19:57
---
## 1. 审查摘要
- **代码质量评分**：暂无法评分（待提供具体变更代码）
- **总体评价**：当前仅提供了项目目录结构，未包含具体的 `变更文件内容`。该结构呈现典型的 PHP MVC 框架特征（高度类似 CodeIgniter 3.x 架构）。在缺乏实际代码的情况下，无法进行逻辑、安全、性能及规范维度的深度审查。
- **风险等级**：待评估（需结合具体变更内容判定）
## 2. 问题详情
*注：由于未提供实际变更代码，下表仅基于您提供的目录结构，列出此类框架中**高频出现的典型隐患**作为审查预检参考。实际审查将在您提交代码后精准定位到具体文件与行号。*
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `system/helpers/security_helper.php` / 动态 | 若自定义安全过滤函数未严格处理输出，或依赖过时的 `xss_clean()` 正则，极易引发 XSS 漏洞或 ReDoS 拒绝服务。 | 优先使用 PHP 原生 `htmlspecialchars($str, ENT_QUOTES \| ENT_SUBSTITUTE, 'UTF-8')` 进行输出转义；输入校验使用 `filter_var()` 或框架验证器。 | `echo htmlspecialchars($user_input, ENT_QUOTES \| ENT_SUBSTITUTE, 'UTF-8');` |
| 🔴 严重 | `system/database/DB_driver.php` / 动态 | 原生 SQL 拼接未使用预处理（Prepared Statements）或查询构建器，存在 SQL 注入风险。 | 全面改用参数化查询或框架查询构建器，禁止直接拼接用户输入。 | `$this->db->query('SELECT * FROM users WHERE id = ?', [$id]);` |
| 🟠 警告 | `system/libraries/Cache/Cache.php` / 动态 | 缓存驱动未配置合理的 TTL，或缺少缓存穿透/雪崩防护机制，高并发下易导致数据库负载骤增。 | 为缓存键设置随机过期时间，使用 `Cache::remember()` 模式，并对空结果进行短 TTL 缓存防穿透。 | `if ($data = $this->cache->get('key')) return $data; ... $this->cache->save('key', $data, rand(300, 3600));` |
| 🟠 警告 | `system/libraries/Session/Session.php` / 动态 | Session 驱动未强制绑定 IP/User-Agent 校验，或 Cookie 未设置 `HttpOnly`/`Secure` 标志，易遭会话劫持。 | 在配置中启用 `sess_match_ip`、`sess_match_useragent`，并设置 `cookie_httponly = TRUE`、`cookie_secure = TRUE`（HTTPS 环境）。 | `config.php: $config['cookie_httponly'] = TRUE; $config['cookie_secure'] = TRUE;` |
| 🟡 建议 | `system/helpers/array_helper.php` / 动态 | 数组操作函数未使用 PHP 7+ 类型声明与严格模式，缺乏对非数组参数的防御性编程。 | 增加 `declare(strict_types=1);`，使用 `is_array()` 或类型提示，避免隐式类型转换导致的 Notice/Warning。 | `function array_get(array $array, string $key, $default = null) { ... }` |
| 🟡 建议 | `system/libraries/Upload.php` / 动态 | 文件上传未校验 MIME 类型与扩展名白名单，或未限制上传目录执行权限，存在 WebShell 风险。 | 严格校验 `$_FILES['file']['type']` 与 `finfo_open()`，重命名文件，并将上传目录置于 Web 根目录外或禁用 PHP 执行。 | `if (!in_array(pathinfo($file, PATHINFO_EXTENSION), ['jpg','png'])) { throw new Exception('Invalid type'); }` |
## 3. 总结与行动建议
- **优先修复的关键问题**：请补充具体的 `变更文件内容`（建议提供完整函数/类或 Git Diff）。收到后我将立即针对逻辑闭环、SQL/XSS 防护、内存管理及 PSR-12 规范进行逐行审查。
- **后续重构/优化方向**：
1. **安全基线加固**：确保所有数据库交互强制使用参数化查询；所有用户输入输出经过严格过滤/转义；敏感操作（如登录、支付、数据导出）需校验 CSRF Token 并实施权限拦截。
2. **性能与架构优化**：针对 `system/libraries/Cache/` 与 `system/database/` 模块，建议引入连接池配置、懒加载机制及 N+1 查询检测。对高频调用的 Helper 函数可考虑静态化或编译缓存。
3. **框架适配说明**：您提及的 `phpci` 框架在公开生态中较少见，当前目录结构与 **CodeIgniter 3.x** 高度一致。若确为内部定制框架，请确认其生命周期钩子（如 `pre_controller`、`post_system`）、自动加载机制及路由规则是否与原生 CI 存在差异。如有差异，建议查阅 `phpci` 官方文档或提供核心加载器代码，以便精准对齐框架最佳实践。
4. **规范升级路径**：建议逐步向 PSR-12 靠拢，统一命名空间、引入 PHP 7.4+ 类型声明（Typed Properties/Return Types），并使用 `try-catch` 替代全局错误抑制符 `@`，提升代码可维护性与 IDE 静态分析友好度。
> 📥 **下一步**：请粘贴具体的变更代码片段（或提供文件路径+关键逻辑），我将基于上述维度为您输出完整、可落地的深度审查报告。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780647598
|
1780647598
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
571
|
22
|
70
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 续费二维码加门店ID
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `61c327705 ## 自动代码审查报告
**分支**: app-260616
**提交**: `61c327705c8a45322199027037057e2e0e167776`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-05 16:25:19
---
## 1. 审查摘要
- **代码质量评分**：`N/A`（待提供具体变更代码）
- **总体评价**：当前仅提供了项目目录结构，未包含具体的变更代码片段或 Diff 内容。该目录结构高度符合 **CodeIgniter 3** 的核心架构（Helpers/Libraries/Database Drivers）。在缺乏实际代码的情况下，无法进行逻辑、安全及性能维度的深度评估。
- **风险等级**：`未知`（需结合具体变更内容判定）
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `N/A` | **未提供变更代码**：无法定位具体文件、行号及业务逻辑，审查流程无法启动。 | 请补充具体的代码变更内容（建议提供 Git Diff 或完整文件片段），并标注涉及的业务场景与调用链路。 | 无 |
## 3. 总结与行动建议
- **优先补充**：请提供本次提交的具体代码变更内容。为提升审查效率，建议包含：
1. 修改/新增的 PHP 文件路径及完整代码（或 `git diff` 输出）。
2. 代码所处的业务上下文（如：Controller 调用、Model 查询、Helper 扩展等）。
3. 是否涉及敏感操作（如：文件上传、支付回调、权限校验、外部 API 调用）。
- **后续重构与优化方向（基于提供的目录结构）**：
由于项目结构呈现典型的 CI3/类 CI 架构，在收到代码后，我将严格围绕您要求的 5 个维度展开审查，重点关注：
- **🔒 安全性**：`DB_driver.php` 及查询构造器是否强制使用预处理/参数绑定；`security_helper.php` 与 `form_helper.php` 是否严格处理 XSS/CSRF Token；`Session` 驱动是否启用 `cookie_httponly` 与加密存储；`Upload.php` 是否校验 MIME 类型与文件头而非仅依赖扩展名。
- **⚡ 性能优化**：`Cache` 驱动（Redis/Memcached）的键名设计与序列化策略；`DB_result.php` 是否避免在 `foreach` 中执行未缓存的同步查询；`Image_lib.php` 等重型库是否采用懒加载或队列异步处理。
- **📐 代码规范**：确保新增/修改代码严格遵循 **PSR-12**（类型声明、严格模式、缩进、命名空间）；Helper 函数避免全局污染，优先采用静态方法或依赖注入容器管理。
- **🧩 框架适配**：若 `phpci` 为 CI 的定制分支或内部框架，请确认其生命周期钩子（Hooks）、自动加载机制（Autoloader）与路由分发逻辑的差异。对于不确定的框架特定用法，我将明确标注并建议查阅 `phpci` 官方文档。
请补充代码内容，我将立即为您输出完整的深度审查报告。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780647919
|
1780647919
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
587
|
22
|
71
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 自助商家可以退兑换的订单
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `7579aebe0 ## 自动代码审查报告
**分支**: app-260616
**提交**: `7579aebe01b6a39b8b0d1220156c118c82a79eff`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-08 14:02:04
---
## 1. 审查摘要
- **代码质量评分**：4 / 10 分
- **总体评价**：该模型类承载了大量订单查询、统计报表、分页及退款逻辑，功能覆盖较全。但代码存在**严重的 SQL 注入隐患**、**敏感信息硬编码**、**多处 SQL 语法错误**及**重复的分页实现**。整体架构偏向早期 PHP 开发习惯，未遵循现代 PHP 规范与 MVC 分层原则，可维护性与安全性较低。
- **风险等级**：🔴 高
> 📌 **框架说明**：代码结构、`$this->db` 调用方式及目录特征高度符合 **CodeIgniter 3** 框架。若 `phpci` 为贵司内部定制框架，请结合其官方文档对 Query Builder 语法进行适配。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 全局多处 | **SQL 注入风险**：大量使用 `$addsql` 字符串直接拼接至 `WHERE` 或 `JOIN` 条件中，未做参数绑定或白名单过滤。若 `$addsql` 来自用户输入，将导致严重注入。 | 废弃字符串拼接，全面改用框架 Query Builder 或预处理语句。若必须动态拼接，需严格校验键名白名单。 | `$this->db->where($safe_conditions)->get('ahead_yc_order');` |
| 🔴 严重 | 第 14 行 / `refund()` | **敏感信息硬编码**：`public $encrypt` 及退款方法中的 MD5 盐值（`'1441600902'` 等）直接写死在代码中，且属性为 `public`，极易泄露。 | 移至 `application/config/` 或 `.env` 环境变量，属性改为 `protected`，构造函数中读取。 | `protected $encrypt;`<br>`$this->encrypt = config_item('order_encrypt_key');` |
| 🔴 严重 | `get_new_add_people_trend_data` | **SQL 语法错误**：子查询 `NOT IN (select DISTINCT(_ahead_user_id))` **缺失 `FROM` 表名**，执行时将直接抛出数据库语法错误。 | 补全子查询表名及关联条件，确保逻辑闭环。 | `NOT IN (SELECT _ahead_user_id FROM ahead_yc_order WHERE _merchant_id = ?)` |
| 🔴 严重 | `cjy_consumption_trend_data` | **Query Builder 键名错误**：`$where['_merchant_id='] = $merchant_id;` 不符合 CI 规范，键名中的 `=` 会导致生成的 SQL 异常或条件失效。 | 移除键名中的操作符，使用标准数组键值对。 | `$where['_merchant_id'] = $merchant_id;` |
| 🟠 警告 | 多个统计方法 | **模型层越权处理 JSON**：多处使用 `json_encode()` 包装查询结果返回。模型应仅负责数据获取，序列化应交由 Controller 或 API 响应层。 | 移除模型中的 `json_encode()`，直接返回数组。在控制器层统一处理响应格式。 | `return $result['data']; // 移除 json_encode` |
| 🟠 警告 | `get_order_manage_by_page` | **强制索引滥用**：通过 `strpos($addsql, '_shop_id')` 判断后硬编码 `force index(_merchant_id)`。索引策略应由 DBA 通过执行计划决定，硬编码易导致全表扫描或索引失效。 | 移除 PHP 层的 `force index`，依赖数据库优化器。若确需优化，应在数据库层建立复合索引 `(merchant_id, shop_id, timestamp)`。 | 删除 `force index` 拼接逻辑 |
| 🟠 警告 | `set_page_info` / `get_page_info` / `cjy_get_page_info` | **分页逻辑严重重复**：三处独立实现分页计算、状态判断，代码冗余且易产生边界条件不一致。 | 抽取为独立 `PaginationService` 或 CI Library，统一调用。 | `PaginationHelper::calc($total, $page, $pageSize)` |
| 🟡 建议 | 文件顶部 | **模型加载方式错误**：`$CI =& get_instance(); $CI->load->model('Simple_model');` 在类外部执行，违反框架生命周期，且每次文件解析都会执行。 | 移除顶部代码。若需加载父类依赖，应在 `__construct()` 中处理或配置自动加载。 | 删除顶部两行代码 |
| 🟡 建议 | 全局 | **代码规范与命名**：变量带 `$_` 前缀属旧式风格；方法过长（如 `get_business` 超 60 行）违反单一职责；缺乏 PHP 7+ 类型声明。 | 遵循 PSR-12，移除 `$_` 前缀，添加类型提示，拆分大方法。 | `public function getOrderInfo(int $merchantId, string $orderId = ''): array` |
---
## 3. 总结与行动建议
### 🚨 优先修复的关键问题（P0）
1. **修复 SQL 注入漏洞**：立即审查所有传入 `$addsql` 的调用链。若无法短期内全面重构，至少对 `$addsql` 进行严格的正则白名单过滤（仅允许 `字段名 IN (...)` 或 `字段名 = ?` 格式），并强制使用 `$this->db->query($sql, $bindings)` 绑定参数。
2. **修正致命 SQL 语法错误**：修复 `get_new_add_people_trend_data` 中缺失 `FROM` 的子查询，以及 `cjy_consumption_trend_data` 中错误的 Query Builder 数组键名。
3. **移除硬编码密钥**：将 `$encrypt`、退款 MD5 盐值、`PAY_BASE_URL` 等全部迁移至配置文件或环境变量，杜绝生产环境密钥泄露风险。
### 🛠 后续重构与优化方向（P1/P2）
1. **统一数据访问层 (DAL)**：
- 废弃原生 SQL 字符串拼接，全面迁移至框架 Query Builder。
- 将 `json_encode()` 从 Model 剥离，确保 Model 仅返回 `array` 或 `object`。
2. **抽象通用组件**：
- 将重复的分页计算逻辑封装为 `PaginationTrait` 或独立 Library。
- 将复杂的报表统计（如二次消费、消费走势）拆分为独立的 `ReportService`，避免 Model 臃肿。
3. **性能与数据库优化**：
- 针对 `BETWEEN ? AND ?` 及 `GROUP BY` 高频查询，建议 DBA 添加覆盖索引：`INDEX idx_merchant_shop_time (_merchant_id, _shop_id, _timestamp)`。
- 移除 `force index` 硬编码，改用 `EXPLAIN` 分析慢查询后针对性优化。
4. **现代化 PHP 规范**：
- 启用 `declare(strict_types=1);`。
- 为方法参数和返回值添加类型声明（如 `int`, `array`, `bool`）。
- 使用常量类或枚举（PHP 8.1+）替代散落的 `const` 定义，提升可读性。
> ⚠️ **局限性说明**：您提供的代码在末尾处被截断（`$this->ahead_yc_merchant_user_model->get_one(['_id' => $para`）。若截断部分包含权限校验、事务处理或核心业务逻辑，请补充完整代码后重新提交审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780898524
|
1780898524
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
590
|
22
|
72
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 订单退款，卡券退回
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `a40343fe7 ## 自动代码审查报告
**分支**: app-260616
**提交**: `a40343fe7c3db2748ec3113c18d9fedba16401a0`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-08 15:14:12
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖较全面，但实现方式较为粗糙。存在多处高危 SQL 注入隐患、明显的语法遗漏（缺失 `return`）、严重的 N+1 查询性能瓶颈以及大量拼写与命名不一致问题。部分核心方法（如 `refund`）严重违反单一职责原则，维护成本极高。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_pay_log_model.php` ~L248, L320 | **SQL 注入漏洞**：`$where` 与 `$up` 变量直接使用字符串拼接构造 SQL 条件，未使用框架查询构建器或参数绑定。若外部传入 `$relation_id` 或 `$pay_id` 未严格校验，将导致数据库注入。 | 全面改用框架 Query Builder 或参数绑定，禁止手动拼接 SQL 条件。 | `$this->db->where('_relation_id', $relation_id)->where_in('_status', [1,4]);`<br>`$this->db->set('_refund_amount', '_refund_amount + ' . floatval($refund_amount), FALSE)->update(...);` |
| 🔴 严重 | `Ahead_user_reward_model.php` ~L118 | **语法/逻辑缺陷**：`try-catch` 块中 `['success' => false, 'msg' => '优惠券添加失败'];` 缺少 `return` 关键字。异常捕获后实际返回 `null`，调用方无法正确判断失败状态。 | 补充 `return` 关键字，确保异常分支返回标准响应结构。 | `return ['success' => false, 'msg' => '优惠券添加失败'];` |
| 🔴 严重 | `Ahead_coupon_operation_log_model.php` ~L45 | **N+1 查询性能灾难**：在 `foreach` 循环中动态加载 Model 并执行 `get_one`。若 `$data` 有 100 条记录，将触发 400+ 次独立查询，极易导致数据库连接池耗尽与接口超时。 | 循环外统一加载 Model，收集所有 `user_id` 后使用 `where_in` 批量查询，再在内存中通过 `array_column` 映射数据。 | `$ids = array_column($data, 'user_id');`<br>`$users = $this->ahead_user_model->select(['where_in' => ['_id', $ids]], '_id,_nickname,_mobile');`<br>`$userMap = array_column($users, null, '_id');` |
| 🟠 警告 | `Ahead_pay_log_model.php` ~L400 | **事务处理不严谨**：`$this->db->trans_begin()` 后，多处分支直接调用 `throwError()` 但未显式 `trans_rollback()`。依赖框架隐式回滚在复杂嵌套调用中可能失效。 | 使用 `try-catch` 包裹事务块，或在抛出异常前显式回滚。 | `try { $this->db->trans_begin(); ... $this->db->trans_commit(); } catch (\Exception $e) { $this->db->trans_rollback(); throw $e; }` |
| 🟠 警告 | `Ahead_user_reward_model.php` ~L285 | **数组过滤未生效**：`array_filter($satisfy_shop_ids_arr);` 未接收返回值，导致空字符串/空值未被过滤，后续 `implode` 可能生成 `,,` 引发 SQL 语法错误。 | 修正赋值逻辑。 | `$satisfy_shop_ids_arr = array_filter($satisfy_shop_ids_arr);` |
| 🟠 警告 | 多个文件 | **拼写与键名不一致**：`recevie`→`receive`、`from_palce`→`from_place`、`fileds`→`fields`、`TYPR_DADA`→`TYPE_DATA`。且 `$data['_satisfy_shop_ids']` 与别名 `$data['satisfy_shop_ids']` 混用，易触发 `Undefined index`。 | 全局搜索替换修正拼写；统一使用 SQL 别名后的键名访问数组。 | `public function add_log_by_receive(...)`<br>`$data['satisfy_shop_ids']` |
| 🟡 建议 | 多个文件顶部 | **全局实例滥用**：文件顶部 `$CI = &get_instance();` 在类外部声明，违反 CI 生命周期规范，易引发上下文污染或内存泄漏。 | 移除顶部声明，在方法内部按需使用 `$this->load->model()` 或 `$this->db`。 | 删除顶部 `$CI = &get_instance();` 及 `$CI->load->model(...)` |
| 🟡 建议 | `Ahead_pay_log_model.php` ~L400 | **方法过长违反单一职责**：`refund()` 方法超 300 行，混合了事务控制、多模型更新、第三方退款、日志记录、Redis 操作等，可读性与可测试性极差。 | 拆分为独立 Service 类或提取私有方法（如 `processVipRefund()`, `callThirdPartyRefund()`, `updateRefundLog()`）。 | 建议引入 `RefundService` 处理核心退款流程，Model 仅负责数据持久化。 |
| 🟡 建议 | `Ahead_user_reward_model.php` ~L200 | **低效查询设计**：大量使用 `FIND_IN_SET` 查询逗号分隔的字符串字段，无法利用 B-Tree 索引，数据量增长后必现全表扫描。 | 建议将 `satisfy_shop_ids` 拆分为 `reward_shop_relation` 关联表，或改用 MySQL JSON 类型配合 `JSON_CONTAINS`。 | 架构优化建议，需结合业务排期重构。 |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即修复 SQL 注入风险**：所有手动拼接 `$where` 和 `$up` 字符串的地方必须替换为框架 Query Builder 或预处理参数。这是生产环境的高危红线。
2. **修复语法遗漏**：补充 `Ahead_user_reward_model::add_reg_reward()` 中缺失的 `return`，避免异常静默吞没导致业务状态不一致。
3. **消除 N+1 查询**：将 `Ahead_coupon_operation_log_model::get_log_list()` 中的循环内查询改为批量查询+内存映射，接口响应时间预计可下降 70% 以上。
### 🛠 后续重构与优化方向
1. **架构分层**：当前 Model 承载了过多业务逻辑（如退款流程、第三方 API 调用、Redis 操作、复杂报表计算）。建议引入 `Service` 层处理业务流程，`Model` 层仅保留数据访问与基础校验。
2. **统一规范与拼写**：使用 PHP-CS-Fixer 或 IDE 批量格式化代码，严格遵循 PSR-12。修复所有拼写错误，统一变量命名风格（推荐 `snake_case` 用于 DB 字段，`camelCase` 用于 PHP 变量）。
3. **事务与异常标准化**：建立全局事务处理模板，统一使用 `try-catch` 包裹事务块，确保 `trans_rollback()` 100% 执行。避免在 Model 中直接调用 `throwError()` 中断流程，建议改为抛出标准异常由 Controller 层统一捕获响应。
4. **数据库设计优化**：逐步废弃 `FIND_IN_SET` 查询模式，将逗号分隔的关联字段拆分为标准关系表，为后续引入分库分表或读写分离打好基础。
> 📝 **注**：本次审查基于代码呈现的 CodeIgniter 3 风格架构。若 `phpci` 为内部定制框架，部分生命周期调用（如 `$this->load->model()`）请以官方文档为准。由于 `Ahead_pay_log_model.php` 与 `Ahead_user_reward_model.php` 末尾代码被截断，部分边界逻辑（如 `mobile_refund` 返回值处理、`register_present_gift` 完整流程）未能完全覆盖，建议补充完整代码后进行二次审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780902852
|
1780902852
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
593
|
22
|
73
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 自助商家小程序兑换开房，续费的订单卡券要退回给用户
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `5999c22d1 ## 自动代码审查报告
**分支**: app-260616
**提交**: `5999c22d1ad94b48eae8ff43caaefa11d27932c6`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-08 15:21:04
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：该 Model 承担了支付流水记录、回调处理、账单查询、营业报表及核心退款等大量业务逻辑。整体功能完整，但存在明显的架构越界（Model 层混杂业务逻辑与第三方 API 调用）、SQL 拼接安全隐患、循环内 N+1 查询性能瓶颈以及事务管理不规范等问题。代码风格偏向传统 PHP 开发，缺乏现代 PHP 类型约束与 PSR-12 规范。
- **风险等级**：🔴 高（存在 SQL 注入隐患、事务回滚不一致、硬编码敏感信息）
> 📌 **框架说明**：代码中大量使用 `$CI = &get_instance()`、`$this->load->model()`、`$this->db->trans_begin()` 等特征，实际为 **CodeIgniter 3** 架构，而非 `phpci`（PHP 持续集成服务器）。以下审查基于 CI3 及现代 PHP 最佳实践进行。若为自研封装框架，请确保底层查询构建器支持参数化绑定。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `update_refund_amount` / `refund` 多处 | **SQL 注入风险**：直接使用字符串拼接构造 `$where` 和 `$up` 条件（如 `'_relation_id="' . $relation_id . '"...'`），未使用预处理或查询构建器，极易被恶意参数注入。 | 全面改用 CI3 查询构建器或参数化查询，禁止手动拼接 SQL 片段。 | `$this->db->where('_relation_id', $relation_id)->where('_type', $type)->where_in('_status', [1,4]);`<br>`$this->db->set('_status', 4)->set('_refund_amount', '_refund_amount+' . $refund_amount, FALSE)->update($this->table_name);` |
| 🔴 严重 | `refund` 方法 | **事务回滚逻辑缺陷**：在多处 `if (!$res)` 中重复调用 `$this->db->trans_rollback()`，且依赖自定义 `throwError()`。若 `throwError` 未正确抛出异常或终止执行，会导致事务状态不一致或数据库锁死。 | 使用 `try...catch` 统一包裹事务，仅在 `catch` 中执行一次回滚，并抛出标准 `Exception`。 | `try { $this->db->trans_begin(); /* 业务逻辑 */ $this->db->trans_commit(); } catch (\Exception $e) { $this->db->trans_rollback(); throw $e; }` |
| 🔴 严重 | `mobile_refund` 方法 | **硬编码敏感密钥**：`md5()` 中直接写死 `'1441600902'` 和 `'2017040606573534'`，疑似支付网关盐值或签名密钥，存在代码泄露导致资金安全风险。 | 将密钥移至 `config/payment.php` 或环境变量中，通过 `config_item()` 读取。 | `$salt = config_item('refund_sign_salt'); $data['refund_key'] = md5($order_info['_id'] . $order_info['_trade_no'] . $salt);` |
| 🟠 警告 | `get_bill_pay_log` / `get_refundable_pay_log` | **N+1 查询性能瓶颈**：在 `foreach` 循环内频繁调用 `get_one()`、加载模型及查询配置。当 `$log_data` 数据量较大时，将产生数十次甚至上百次 DB 请求，严重拖慢接口响应。 | 提前收集所有关联 ID，使用 `WHERE IN` 批量查询，在内存中构建映射表后循环赋值。 | `$ids = array_column($log_data, 'relation_id'); $exts = $this->ahead_yc_order_extension_model->select(['where_in' => ['_order_id', $ids]]); $extMap = turn_array_key($exts, '_order_id');` |
| 🟠 警告 | `add_order_pay_log` | **循环内修改传入参数**：`$params` 在循环中被直接覆盖（如 `$params['_pay_platform'] = $platform`），导致后续迭代使用被污染的数据，且 `$actual_pay` 累加逻辑可能失真。 | 使用局部变量或克隆参数数组，避免副作用。明确多支付金额计算边界。 | `foreach ($payPlatformArr as $platform) { $currentParams = $params; if ($payPlatform == 14) { $currentParams['_pay_platform'] = $platform; ... } }` |
| 🟠 警告 | `update_after_pay_call_back` | **类型比较不一致**：`$newPayLogData['_type'] == 1` 与后续 `== '1'` 混用，PHP 弱类型比较可能导致分支误判。常量键名为字符串 `'1'`，但传入值可能为整型。 | 统一使用严格比较 `===`，或在方法入口处进行类型标准化。 | `if ((string)$newPayLogData['_type'] === '1')` |
| 🟡 建议 | 全局 | **违反单一职责原则 (SRP)**：Model 层包含退款计算、第三方 API 调用、报表聚合、Redis 队列推送等大量业务逻辑，导致类臃肿且难以测试。 | 抽离业务逻辑至 `Service` 或 `Library` 层，Model 仅保留数据存取与基础查询。 | 创建 `RefundService::processRefund()` 处理退款流程，Model 仅提供 `updateStatus()`、`getById()` 等方法。 |
| 🟡 建议 | 全局 | **魔法数字泛滥**：大量硬编码的平台/状态码（如 `1, 2, 11, 15, 17-28`），可读性差且易遗漏维护。 | 充分利用已定义的 `const` 常量，或建立统一的 `PaymentPlatform` 枚举/配置类。 | `if ($platform == self::VIP_PRESENT_PAY)` 替代 `== 15` |
| 🟡 建议 | 全局 | **缺乏现代 PHP 规范**：无 `declare(strict_types=1);`，无参数/返回值类型声明，注释含口语化历史遗留说明（如 `//之前不知道为啥被注释掉...`）。 | 遵循 PSR-12，补充类型提示，清理无效注释，使用 PHPDoc 规范方法签名。 | `public function add_order_pay_log(int $merchantId, int $adminUid, string $type, int $relationId, array $params, array $multPayIndexArr = []): array` |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复 SQL 注入隐患**：立即替换所有字符串拼接的 `$where` 和 `$up` 逻辑，改用框架提供的查询构建器或参数化绑定。这是资金安全与数据完整性的底线。
2. **重构事务管理**：将 `refund()` 方法中的 `trans_begin/commit/rollback` 包裹在 `try...catch` 中，移除重复的 `trans_rollback()` 调用，确保异常发生时事务能安全回滚且不阻塞连接。
3. **移除硬编码密钥**：将 `mobile_refund` 中的签名盐值迁移至配置文件或环境变量，禁止在代码库中明文存储支付相关密钥。
### 🛠 后续重构与优化方向
1. **架构分层（MVC -> MVCS）**：
- 将 `refund()`、`update_after_pay_call_back()` 中的业务编排逻辑抽离至 `application/services/PaymentService.php`。
- Model 仅负责 `insert()`, `update()`, `select()`, `getOne()` 等数据操作。
- 第三方 API 调用（微信/支付宝退款）封装至 `application/libraries/PaymentGateway.php`。
2. **性能优化（消除 N+1）**：
- 对 `get_bill_pay_log` 和 `get_refundable_pay_log` 实施“批量预加载”策略。使用 `WHERE IN` 一次性拉取关联表数据，通过 `array_column` + `array_combine` 构建内存映射，将 DB 查询次数从 `O(N)` 降至 `O(1)`。
3. **规范化与可维护性**：
- 引入 PHP 7.4+ 类型声明与严格模式。
- 建立统一的 `PaymentPlatform` 与 `PayLogStatus` 常量/枚举类，彻底消除魔法数字。
- 清理历史注释，使用 PHPDoc 补充 `@param`、`@return`、`@throws` 说明。
> ⚠️ **局限性说明**：提供的代码在 `mobile_refund` 方法末尾被截断，未能完整审查支付宝退款回调处理及后续逻辑。建议补充完整代码以便进行全链路安全与异常流审查。若需针对特定 CI3 版本或自定义底层封装进行适配调整，请提供框架核心加载器或 DB 驱动实现细节。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780903264
|
1780903264
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
614
|
22
|
74
|
1
|
|
0
|
Merge pull request '合并0519' (#5) from app Merge pull request '合并0519' (#5) from app into app-260616...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1780972453
|
1780972468
|
1780972468
|
0
|
0
|
0
|
Edit
Delete
|
|
615
|
22
|
75
|
1
|
|
0
|
开房套餐转房增加房态判断。
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1780972488
|
1780972586
|
1780972493
|
0
|
0
|
0
|
Edit
Delete
|
|
616
|
22
|
76
|
1
|
|
0
|
🔍 代码审查报告：app - Merge pull request 'Merge pull 🔍 代码审查报告：app - Merge pull request 'Merge pull request '合并0519' (#...
|
## 自动代码审查报告
**分支**: app
**提交**: `518d00cf2a78cb28 ## 自动代码审查报告
**分支**: app
**提交**: `518d00cf2a78cb28e24dab21ba3d2f2827974f81`
**提交人**: zhangjunnan (121158035@qq.com)
**时间**: 2026-06-09 10:36:17
---
## 1. 审查摘要
- **代码质量评分**：4.5 / 10
- **总体评价**：代码已实现核心业务链路，但存在**严重的安全隐患**（硬编码凭证、非原子锁、SQL拼接）、**架构反模式**（Model内加载Model、巨型Switch、全局状态污染）及**性能瓶颈**（频繁建连Redis、循环内N+1查询）。整体工程化程度较低，不符合现代 PHP 及 CI3 最佳实践。
- **风险等级**：🔴 高
> 📌 **框架说明**：提交代码实际基于 **CodeIgniter 3** 架构（使用 `CI_Controller`、`get_instance()`、`$this->load->` 等）。若 `phpci` 为内部定制分支，部分加载机制可对照官方文档微调，但底层安全与性能原则通用。
---
## 2. 问题详情
| 严重程度 | 文件/位置 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `common_helper.php` / `alioss_*` 函数 | **硬编码云厂商 AccessKey/Secret**。直接暴露在公共 Helper 中，极易被泄露导致云资源被恶意操作。 | 移至 `application/config/oss.php` 或环境变量，通过 CI 配置加载。禁止在代码中明文存储密钥。 | `$config = $CI->config->item('aliyun_oss');`<br>`$param = $config['credentials'];` |
| 🔴 严重 | `common_helper.php` / `doRedisLockRelease` | **Redis 分布式锁非原子释放**。`GET` 校验后 `DEL` 存在竞态条件，可能误删其他请求持有的锁。 | 使用 Lua 脚本保证 `GET` 与 `DEL` 的原子性，或使用 `Redis::eval`。 | `lua = "if redis.call('get',KEYS[1]) == ARGV[1] then return redis.call('del',KEYS[1]) else return 0 end"`<br>`$redis->eval($lua, [$key, $value], 1);` |
| 🔴 严重 | `Ahead_book_order_model.php` / `refund_by_notify` | **字符串拼接构造 SQL 条件**：`$log_where = '_relation_id="' . $order_data['_id'] . '"...'` 存在 SQL 注入风险，且绕过 CI 查询构造器。 | 全面使用 CI Query Builder 或参数绑定，禁止手动拼接 WHERE 子句。 | `$this->db->where('_relation_id', $order_data['_id'])`<br>`->where('_status', 1)`<br>`->where_in('_type', [5,13]);` |
| 🟠 警告 | `Api.php` / `selfChangeRoom` | **全局状态污染**：直接修改 `$CI->merchant_id`、`$CI->admin_data`。在并发请求或 CLI 环境下会导致数据串扰。 | 通过方法参数传递上下文，或使用独立的 `Context` 对象/Session 存储，避免修改 CI 超全局实例。 | `// 移除 $CI->admin_data = ...`<br>`$orderService->handle($merchant_id, $admin_data, $params);` |
| 🟠 警告 | `MerchantAppServer.php` / `index()` | **上帝方法 (God Method)**：单方法超 500 行，包含巨型 `switch`，违反单一职责原则，极难维护与测试。 | 按 `function` 路由拆分至独立 Controller 或 Service 类。主方法仅负责鉴权与路由分发。 | `// 路由分发示例`<br>`$action = $request['function'];`<br>`$service = new MerchantService($this);`<br>`return $service->$action($request);` |
| 🟠 警告 | `common_helper.php` / `get_aliyun_redis_conn` | **频繁创建/关闭 Redis 连接**。每次调用都 `connect()` + `close()`，高并发下将耗尽文件描述符并引发严重延迟。 | 使用 CI 内置 Redis 驱动或单例模式复用连接。生产环境建议启用 `persistent` 连接。 | `$CI->load->driver('cache', ['adapter' => 'redis']);`<br>`$redis = $CI->cache->redis;` |
| 🟠 警告 | `Ahead_book_order_model.php` / `invalid_book` | **事务中断风险**：`throwError()` 直接 `exit/die`，导致 `$this->db->trans_start()` 未执行 `trans_complete()`，连接池可能泄漏或状态不一致。 | 改用抛出 `Exception` 或返回错误数组，在 `try-catch` 中统一处理事务回滚。 | `try { $this->db->trans_start(); ... $this->db->trans_complete(); } catch(\Exception $e) { $this->db->trans_rollback(); throw $e; }` |
| 🟡 建议 | 多个 Model 文件 | **Model 内部加载 Model**：`$CI = &get_instance(); $CI->load->model(...)` 破坏依赖注入，增加耦合。 | 模型应在 Controller 层加载，或通过 CI `autoload.php` 预加载。若需跨模型调用，应在 `__construct` 中注入。 | `// 移除内部 $CI->load->model()`<br>`// 在 Controller 中统一加载`<br>`$this->load->model('Ahead_yc_merchant_model');` |
| 🟡 建议 | `common_helper.php` / `decodeUnicode` | **使用已废弃函数**：`create_function` 在 PHP 7.2+ 已废弃，存在安全风险且性能差。 | 替换为匿名函数（闭包）。 | `return preg_replace_callback('/\\\\u([0-9a-f]{4})/i', fn($m) => mb_convert_encoding(pack('H*', $m[1]), 'UTF-8', 'UCS-2BE'), $str);` |
| 🟡 建议 | `Api.php` / `jsonEcho` | **冗余输出缓冲控制**：`ob_end_clean(); ob_start(); ... ob_end_flush();` 易引发 Header 已发送警告，且逻辑冗余。 | 直接设置 Header 后输出 JSON 并终止，CI 已处理基础缓冲。 | `header('Content-Type: application/json; charset=utf-8');`<br>`echo json_encode($result, JSON_UNESCAPED_UNICODE);`<br>`exit;` |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题（P0）
1. **立即移除硬编码凭证**：将 `common_helper.php` 中的 OSS Key/Secret 迁移至配置文件或环境变量，并轮换已泄露的密钥。
2. **修复分布式锁竞态条件**：将 `doRedisLockRelease` 改为 Lua 脚本原子操作，防止业务数据错乱。
3. **消除 SQL 注入隐患**：全面审查 `Ahead_book_order_model.php` 及其他 Model 中的字符串拼接 SQL，统一替换为 CI Query Builder 或 PDO 参数绑定。
4. **规范事务处理**：移除 `throwError()` 中的 `exit()`，改用异常机制确保 `$this->db->trans_rollback()` 必执行。
### 🛠 后续重构与优化方向
1. **架构解耦**：
- 拆分 `MerchantAppServer::index()` 巨型方法，采用 **路由分发器 + 独立 Service 类** 模式。
- 禁止在 Model 内部使用 `get_instance()->load->model()`，改为构造函数依赖注入或 CI 自动加载。
2. **性能提升**：
- 替换 `get_aliyun_redis_conn` 为 CI 官方 Redis 驱动，启用长连接。
- 解决 `get_list` / `get_detail` 中的 **N+1 查询问题**：使用 `JOIN` 或批量 `WHERE IN` 预加载关联数据。
3. **规范与安全**：
- 遵循 PSR-12 规范：统一类名/文件名大小写、移除重复函数定义（如 `rs_hmset`）、补充类型声明与 PHPDoc。
- 替换 `passport_encrypt` 弱加密算法为 `openssl_encrypt` 或 `sodium`。
- 限制 `Access-Control-Allow-Origin: *`，改为白名单校验或动态反射 Origin。
> ⚠️ **局限性说明**：部分文件（如 `MerchantAppServer.php`、`common_helper.php`、`Ahead_book_order_model.php`）末尾被截断，未完整覆盖所有分支逻辑与异常处理路径。建议补充完整代码后进行二次深度审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780972577
|
1780972577
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
617
|
22
|
77
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - Merge pull request '开房套餐 🔍 代码审查报告：app-260616 - Merge pull request '开房套餐转房增加房态判断。' (#75) from app...
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `48af25d1e ## 自动代码审查报告
**分支**: app-260616
**提交**: `48af25d1eb8d8e97ce1b9fc107d08baf2a9eb74e`
**提交人**: zhangjunnan (121158035@qq.com)
**时间**: 2026-06-09 10:36:26
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了较为完整的商家端业务逻辑，但整体架构偏向传统的“胖控制器+贫血模型”模式。存在**硬编码敏感密钥**、**SQL注入风险**、**N+1查询性能瓶颈**及**事务处理不规范**等关键问题。代码规范与 PSR-12 存在较大偏差，魔法数字泛滥，可维护性较低。
- **风险等级**：🔴 高
> 📌 *注：提示中提及 `phpci` 框架，但实际代码结构、加载方式（`$this->load->`、`get_instance()`、`BASEPATH`）均明确指向 **CodeIgniter 3.x**。以下审查将基于 CI3 最佳实践与通用 PHP 安全规范进行。*
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `MerchantAppServer.php`<br>(约 150-160 行) | **硬编码敏感 API 密钥**：科大讯飞 `APPID`、`APISecret`、`APIKey` 直接暴露在控制器中，极易导致密钥泄露与盗刷。 | 将敏感配置移至 `application/config/` 或环境变量中，通过 `$this->config->item()` 读取。生产环境严禁硬编码。 | `// config/xfyun.php<br>return ['appid'=>getenv('XFYUN_APPID'), ...];<br>// 控制器中<br>$config = $this->config->item('xfyun');` |
| 🔴 严重 | `Ahead_yc_merchant_user_model.php`<br>`get_user_shop_page()` 等方法 | **SQL 注入漏洞**：直接使用字符串拼接构造 `WHERE` 与 `LIKE` 条件（如 `"... like '%" . $params['name'] . "%'"`），未使用查询构建器转义。 | 全面改用 CI3 Query Builder 的 `like()`、`where()` 方法，或手动使用 `$this->db->escape()`。 | `$this->db->like('user._name', $params['name'], 'both');<br>// 替代原有的字符串拼接` |
| 🔴 严重 | `Ahead_yc_merchant_user_model.php`<br>`check_token()` | **弱加密验证 Token**：使用 `md5()` 拼接静态盐值验证 Token，存在碰撞风险且无过期机制，易被重放攻击。 | 升级为 `hash_hmac('sha256')` 或引入 JWT。Token 生成时应绑定时间戳并校验有效期。 | `hash_hmac('sha256', $payload, $secret_key);<br>// 结合 Redis 设置 TTL 实现会话管理` |
| 🟠 警告 | `Ahead_book_order_model.php`<br>`get_list()` | **N+1 查询性能瓶颈**：在 `foreach` 循环中频繁调用 `$this->ahead_yc_order_model->get_one()` 获取关联数据，导致数据库连接数激增。 | 使用 `WHERE IN` 批量查询或 `JOIN` 关联表一次性获取数据，或使用 CI 缓存机制。 | `$ids = array_column($order_info, 'relation_order_id');<br>$orders = $this->ahead_yc_order_model->get_by_ids($ids);<br>// 循环中通过数组映射获取` |
| 🟠 警告 | `Ahead_book_order_model.php`<br>`invalid_book()` | **事务回滚机制不健全**：在 `$this->db->trans_start()` 后直接调用自定义 `throwError()`，若该函数未抛出标准异常，可能导致事务挂起或数据不一致。 | 使用 `try-catch` 包裹事务逻辑，确保异常时强制 `trans_rollback()`，并依赖 CI 的 `trans_status()` 判断。 | `try { $this->db->trans_start(); ... } catch(\Exception $e) { $this->db->trans_rollback(); throw $e; }` |
| 🟠 警告 | `MerchantAppServer.php`<br>`index()` | **控制器职责过重 (Fat Controller)**：单方法超 300 行，混杂权限校验、菜单过滤、配置加载、数据组装，违反单一职责原则。 | 将业务逻辑下沉至 `Service` 或 `Repository` 层，控制器仅负责路由分发与请求/响应转换。 | `// 控制器<br>$data = $this->login_service->handleLogin($this->stream);<br>$this->success_response($data);` |
| 🟡 建议 | 全局多处 | **魔法数字与硬编码 ID 泛滥**：如 `permission_id = 534`、`$unsetMenuIds = [694, 849...]` 等散落在代码中，后期维护成本极高。 | 提取为常量类（如 `PermissionConst::RECHARGE_INVALID = 534`）或写入配置文件。 | `class MenuConst { const FILTER_IDS = [694, 849, 824, 1250]; }` |
| 🟡 建议 | 全局多处 | **命名规范与 PSR-12 不符**：类名使用下划线（`Ahead_book_order_model`），变量混用驼峰与下划线，注释含过期人员标记（`edit by nan 17.10.25`）。 | 遵循 PSR-12：类名 `PascalCase`，方法/属性 `camelCase`，清理历史注释，使用 PHPDoc 规范。 | `class AheadBookOrderModel extends Simple_model { ... }` |
| 🟡 建议 | `Ahead_yc_merchant_user_model.php` | **频繁使用 `get_instance()`**：在 Model 中多次调用 `$CI = &get_instance();` 获取 CI 实例，增加耦合且影响性能。 | CI3 的 Model 已继承 `CI_Model`，可直接使用 `$this->load->model()` 或 `$this->config->item()`。 | `// 移除 $CI = &get_instance();<br>$this->load->model('ahead_yc_merchant_model');` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即移除硬编码密钥**：将 `MerchantAppServer.php` 中的讯飞 API 密钥、加密盐值等迁移至配置文件或 `.env`，并检查版本控制历史是否已泄露。
2. **修复 SQL 注入漏洞**：全面审查 `Ahead_yc_merchant_user_model.php` 中所有动态拼接 SQL 的地方，替换为 CI Query Builder 或参数化查询。
3. **升级 Token 验证机制**：废弃 MD5 静态盐方案，引入带过期时间的 HMAC-SHA256 签名或标准 JWT，并在 Redis 中维护会话黑名单。
### 🛠 后续重构与优化方向
1. **架构分层改造**：
- 将 `MerchantAppServer::index()` 拆分为 `AuthService`、`MenuService`、`PermissionService`。
- 控制器仅保留：参数接收 -> 调用 Service -> 格式化响应。
2. **数据库查询优化**：
- 消除 `get_list()` 等方法的 N+1 查询，改用 `JOIN` 或批量 `WHERE IN`。
- 对高频读取的配置数据（如权限列表、菜单树）引入 Redis 缓存，设置合理 TTL。
3. **规范化与工程化**：
- 统一类名与文件命名规范，清理 `//edit by xxx` 等无效注释。
- 将散落的权限 ID、菜单 ID、状态码提取至 `config/constants.php` 或独立常量类。
- 引入静态分析工具（如 PHPStan / Psalm）与代码规范检查（PHP_CodeSniffer PSR-12），纳入 CI/CD 流程。
> ⚠️ **局限性说明**：`MerchantAppServer.php` 代码在 `case "1001"` 处被截断，未能完整审查该分支的结账取消逻辑。若该分支涉及资金扣减或订单状态变更，请补充完整代码以便进行事务与并发安全审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780972586
|
1780972586
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
639
|
22
|
78
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 需求 存取酒员工自存自取 16434
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `09c01c0bb ## 自动代码审查报告
**分支**: app-260616
**提交**: `09c01c0bb36b0f138829ca8465ba81ec4ba03103`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-06-09 16:03:54
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码实现了较为完整的寄存单业务闭环（创建、打印、列表查询、状态流转），具备一定的防 N+1 查询优化意识。但整体架构偏向“上帝类”模式，核心方法职责过重；存在明显的 SQL 注入隐患与全局变量滥用问题；未遵循现代 PHP 类型声明与 PSR-12 规范。代码末尾被截断，导致 `update_deposit` 方法后半部分无法评估。
- **风险等级**：🔴 高（主要源于 SQL 注入风险、事务与异常处理耦合不当、全局实例污染）
> 📌 **框架适配说明**：代码结构、`$CI = &get_instance()`、`$this->load->model()` 及事务调用方式高度符合 **CodeIgniter 3 (CI3)** 规范。若 `phpci` 为基于 CI3 的定制框架，请确认其底层 Query Builder 与事务管理器是否完全兼容。以下建议以 CI3 最佳实践为基准。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `search_deposit_list` 约第 280 行 | **SQL 注入漏洞**：直接拼接 `$uid` 到 WHERE 条件字符串中，若 `$uid` 未严格校验类型，可导致注入。 | 强制类型转换或使用 Query Builder 的 `where()` 方法构建安全条件。 | `$where['where'][] = '(deposit._admin_id = ' . (int)$uid . ' OR deposit._admin_id = 0)';` |
| 🔴 严重 | 文件顶部第 2 行 | **全局实例污染**：`$CI = &get_instance();` 在类外部声明，破坏封装性，且在 CLI/多请求环境下可能引发状态混乱。 | 移除文件级 `$CI` 声明。在模型内部统一使用 `$this->load` 或构造函数加载依赖。 | `// 删除文件顶部的 $CI = &get_instance();`<br>`// 在方法内使用 $this->load->model('xxx');` |
| 🟠 警告 | `add_deposit` 方法 | **违反单一职责原则 (SRP)**：单方法超 200 行，混合了参数校验、DB 事务、关联表写入、短信/微信推送、打印逻辑。任一环节失败都会导致调试困难。 | 拆分为独立服务类（如 `DepositCreateService`、`DepositNotifyService`、`DepositPrintService`），Model 仅负责数据持久化。 | `// 建议重构为：<br>$service = new DepositCreateService();<br>$result = $service->handle($merchantId, $adminId, $data);` |
| 🟠 警告 | `add_deposit` / `update_deposit` | **事务与异常处理不规范**：`trans_start()` 配合手动 `trans_rollback()` 与 `trans_complete()` 混用，CI3 中 `trans_complete()` 已内置自动回滚机制，手动回滚可能导致事务状态机异常。 | 统一使用 CI3 推荐的事务流，或改用 `trans_begin()` + `trans_commit()` + `trans_rollback()`。 | `try {<br> $this->db->trans_begin();<br> // ... 业务逻辑 ...<br> if ($this->db->trans_status() === FALSE) {<br> $this->db->trans_rollback();<br> return ['success'=>false, 'msg'=>'...'];<br> }<br> $this->db->trans_commit();<br>} catch (\Exception $e) { ... }` |
| 🟠 警告 | `search_deposit_list` 约第 260 行 | **动态 IN 查询未严格过滤**：`implode(',', array_unique($aheadUserIds))` 直接拼入 SQL，若数组元素含非数字字符将引发语法错误或注入。 | 使用 `array_map('intval', $ids)` 清洗数据，或交由框架 Query Builder 的 `where_in()` 处理。 | `$cleanIds = array_map('intval', array_unique($aheadUserIds));`<br>`$this->db->where_in('_ahead_user_id', $cleanIds);` |
| 🟡 建议 | 全局 | **命名规范与拼写错误**：`deposit_printer_opreating` 拼写错误；类名 `Ahead_deposit_model` 大小写混用；数组语法混用 `array()` 与 `[]`。 | 遵循 PSR-12：类名 `AheadDepositModel`，方法名 `depositPrinterOperating`，统一使用短数组语法 `[]`。 | `public function depositPrinterOperating(...)` |
| 🟡 建议 | 全局 | **缺失类型声明与返回值约束**：所有方法无参数类型、无返回值类型声明，降低 IDE 提示与静态分析能力。 | 逐步补充 PHP 7.4+ 类型声明（如 `public function add_deposit(int $merchantId, int $adminId, array $data): array`）。 | `public function add_deposit(int $merchantId, int $adminId, array $data): array` |
| 🟡 建议 | `add_deposit` / `search_deposit_list` | **重复加载模型**：多次调用 `$this->load->model()`，虽 CI3 有缓存机制，但影响可读性。 | 在 `__construct()` 中统一加载高频模型，或使用依赖注入容器。 | `public function __construct() {<br> parent::__construct();<br> $this->load->model(['ahead_yc_merchant_user_model', 'ahead_store_model', ...]);<br>}` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即修复 SQL 注入**：`search_deposit_list` 中的 `$uid` 拼接与 `IN` 查询数组必须经过 `(int)` 强转或使用 Query Builder 安全方法。
2. **清理全局 `$CI` 实例**：移除文件顶部的 `&get_instance()`，避免多进程/CLI 环境下的内存泄漏与状态污染。
3. **规范事务控制流**：统一采用 CI3 标准事务写法，避免 `trans_start()` 与手动 `trans_rollback()` 交叉使用导致的事务状态不一致。
### 🛠 后续重构与优化方向
- **架构分层**：将 `add_deposit` 拆分为 `Validator`（校验）→ `Repository`（数据操作）→ `Notifier`（消息推送）→ `Printer`（打印）。Model 层仅保留 `insert`/`update`/`query` 等纯数据操作。
- **引入 DTO/VO**：使用数组或轻量级对象封装 `$data` 与返回结果，替代魔法数组键名，提升类型安全与可维护性。
- **统一错误处理**：当前混用 `throwError()`（自定义全局函数）与 `try-catch`。建议统一抛出 `\RuntimeException` 或自定义业务异常，由全局异常处理器捕获并格式化返回。
- **补充单元测试**：针对 `search_deposit_list` 的复杂 WHERE 条件拼接、状态流转逻辑编写 PHPUnit 测试用例，覆盖边界条件（如空数组、非法类型、跨店权限）。
- **注意代码截断**：提供的 `update_deposit` 方法在 `$lastTa` 处中断，请补充完整代码以便审查状态机流转、库存扣减逻辑及事务回滚路径。
> 💡 **专家提示**：若 `phpci` 框架已支持 PHP 8+，强烈建议启用 `declare(strict_types=1);` 并全面引入类型声明。对于高频查询的 `search_deposit_list`，可考虑将 `listinfos` 替换为原生 Query Builder 链式调用，并针对 `_merchant_id`, `_shop_id`, `_status`, `_end_time` 建立复合索引以提升分页性能。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780992234
|
1780992234
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
641
|
22
|
79
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - Merge branch 'app-260616 🔍 代码审查报告：app-260616 - Merge branch 'app-260616' of https://gitea.g-hi.co...
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `c7aa286e2 ## 自动代码审查报告
**分支**: app-260616
**提交**: `c7aa286e291b5ee2230354e0e05df24e5d5a433d`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 16:14:18
---
## 1. 审查摘要
- **代码质量评分**：4.5 / 10 分
- **总体评价**：代码实现了较为复杂的商户端业务逻辑，但存在明显的架构反模式（胖控制器、模型内重复加载组件）、严重的安全隐患（硬编码敏感密钥、SQL拼接注入风险）及事务一致性缺陷。代码风格与 PSR-12 规范差距较大，可维护性与扩展性较低。
- **风险等级**：🔴 高
> 📌 **框架说明**：提交代码的结构、加载方式（`$this->load->`、`$this->db->`、`Simple_model`）高度符合 **CodeIgniter 3.x** 规范。若 `phpci` 为贵司内部定制框架，请结合其官方生命周期文档微调以下建议。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `MerchantAppServer.php`<br>~L295 | **硬编码第三方敏感密钥**：`xfyun_tts_config` 直接明文写入控制器，极易随代码库泄露，违反安全基线。 | 将密钥迁移至 `config/` 配置文件或 `.env` 环境变量中，通过 `config_item()` 或 `getenv()` 读取。 | ```php<br>// config/xfyun.php<br>return [<br> 'appid' => getenv('XFYUN_APPID'),<br> 'secret' => getenv('XFYUN_SECRET'),<br> 'key' => getenv('XFYUN_KEY')<br>];<br>// 控制器中<br>$this->config->load('xfyun');<br>$config = $this->config->item('xfyun');<br>``` |
| 🔴 严重 | `Ahead_deposit_model.php`<br>~L385 | **SQL 注入风险**：`$where['where'][] = '(deposit._admin_id = '.$uid.' or deposit._admin_id = 0)';` 未对 `$uid` 进行类型强转或参数绑定，若传入恶意字符串将导致注入。 | 强制类型转换或使用 CI 查询构建器安全拼接。 | ```php<br>// 推荐写法<br>$uid = (int)$uid;<br>$this->db->group_start();<br>$this->db->where('deposit._admin_id', $uid);<br>$this->db->or_where('deposit._admin_id', 0);<br>$this->db->group_end();<br>``` |
| 🔴 严重 | `Ahead_book_order_model.php`<br>~L115-180 | **事务与外部 API 调用耦合**：`invalid_book` 开启数据库事务后，在 `refund_by_notify` 中直接调用微信/银联退款 API。若外部退款成功但后续 DB 操作失败触发 `trans_rollback()`，将导致**资金已退但订单状态未更新**的资损问题。 | 将外部支付退款剥离出数据库事务。采用“先更新本地状态为退款中 -> 调用外部API -> 回调更新最终状态”的异步/补偿机制，或使用消息队列保证最终一致性。 | ```php<br>// 架构建议<br>$this->db->trans_start();<br>$this->update(['_status' => 3], ['_id' => $id]); // 3:退款处理中<br>$this->db->trans_complete();<br><br>// 事务外调用退款<br$refundRes = $this->refund_by_notify($order_data, ...);<br>if ($refundRes['status']) {<br> $this->update(['_status' => 4], ['_id' => $id]);<br>}<br>``` |
| 🟠 警告 | `MerchantAppServer.php`<br>~L100-450 | **控制器严重违反单一职责原则 (SRP)**：`index()` 方法通过 `switch` 处理登录、个人中心、打印机、支付配置等数十个接口，代码超 500 行，难以测试与维护。 | 按业务域拆分控制器（如 `LoginController`, `ProfileController`, `PrinterController`），或使用路由分发+Service 层处理业务逻辑。 | 拆分示例：<br>`class MerchantLoginController extends AplicationController { public function login() { ... } }` |
| 🟠 警告 | 所有 Model 文件<br>顶部 L2-L3 | **文件顶部全局实例化 CI 对象**：`$CI = &get_instance(); $CI->load->model('Simple_model');` 在文件加载时即执行，增加不必要的内存开销，且不符合框架懒加载机制。 | 移除文件顶部的实例化代码。模型继承 `Simple_model` 即可，CI 会自动处理基类加载。 | ```php<br>// 删除这两行<br>$CI = &get_instance();<br>$CI->load->model('Simple_model');<br>``` |
| 🟠 警告 | 所有文件<br>多处 | **模型内重复加载组件**：大量使用 `$this->load->model()` 在方法内部动态加载。CI 虽会缓存，但频繁调用仍影响性能且破坏依赖注入原则。 | 将高频使用的 Model 移至 `__construct()` 中加载，或配置 `autoload.php`。 | ```php<br>public function __construct() {<br> parent::__construct();<br> $this->load->model('ahead_yc_merchant_user_model');<br> $this->load->model('ahead_shop_config_model');<br>}<br>``` |
| 🟡 建议 | `MerchantAppServer.php`<br>~L10 | **拼写错误与命名不规范**：`AplicationController` 拼写错误；属性 `$createOutTradeNo` 使用驼峰，其余多用下划线，违反一致性。 | 修正拼写为 `ApplicationController`；统一属性命名风格（推荐 `snake_case` 或 `camelCase` 全量统一）。 | `class MerchantAppServer extends ApplicationController` |
| 🟡 建议 | `MerchantAppServer.php`<br>~L35 | **输入流处理不规范**：混用 `$_POST['post_content_data']` 与 `file_get_contents('php://input')`，未校验 Content-Type 与数据长度，易受畸形请求攻击。 | 统一使用 CI 提供的 `$this->input->raw_input_stream`，并增加 JSON 解析异常捕获与大小限制。 | ```php<br>$raw = $this->input->raw_input_stream;<br>if (strlen($raw) > 1024 * 100) exit('Payload too large');<br>$this->stream = json_decode($raw, true);<br>if (json_last_error() !== JSON_ERROR_NONE) {<br> $this->error_response('JSON格式错误');<br>}<br>``` |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **移除硬编码密钥**：立即将 `xfyun_tts_config` 等敏感信息迁移至环境变量或加密配置文件中，避免代码仓库泄露导致资损。
2. **修复 SQL 注入漏洞**：对 `Ahead_deposit_model.php` 中的 `$uid` 拼接进行强制类型转换 `(int)`，或全面改用 CI Query Builder 的 `where()` 链式调用。
3. **解耦事务与外部支付**：重构 `invalid_book` 与 `refund_by_notify` 的调用链路，确保数据库事务仅包裹本地数据操作，外部 API 调用移至事务外或采用异步补偿机制。
### 🛠 后续重构与优化方向
1. **架构分层**：当前控制器承担了路由分发、参数校验、业务逻辑、数据组装等多重职责。建议引入 **Service 层** 处理核心业务，控制器仅负责请求接收、参数校验与响应格式化。
2. **依赖管理优化**：清理文件顶部的 `$CI = &get_instance()` 滥用，统一在构造函数中声明依赖。对于高频调用的 Model，建议配置自动加载。
3. **规范与可维护性**：
- 遵循 PSR-12 规范统一命名风格（属性、方法、常量）。
- 为长方法（如 `index()`、`search_deposit_list()`）添加类型声明（PHP 7.4+ 支持 `array`, `int`, `string` 等）与 PHPDoc 注释。
- 引入静态分析工具（如 `PHPStan` 或 `Psalm`）与代码规范检查（`PHP_CodeSniffer`）纳入 CI/CD 流程。
> ⚠️ **局限性说明**：本次审查基于您提供的代码片段。由于 `MerchantAppServer.php` 末尾被截断，部分业务闭环逻辑（如 `case "1001"` 后续处理）未能完整评估。建议补充完整代码或提供对应单元测试用例，以便进行更精准的边界条件与异常流审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780992858
|
1780992858
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
642
|
22
|
80
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 预订列表
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `6c2219403 ## 自动代码审查报告
**分支**: app-260616
**提交**: `6c2219403bf4db53b66686f356e77bd5c4e06939`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 16:49:37
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务功能覆盖较完整，状态流转与权限校验逻辑较为清晰。但存在**高危 SQL 注入漏洞**、**事务处理机制不规范**、**严重的 N+1 查询性能瓶颈**，且代码风格偏向传统 PHP 写法，缺乏现代类型声明与 PSR-12 规范。部分方法逻辑冗长，职责划分不够清晰。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_book_model.php`<br>`add_book()` 方法 | **事务处理逻辑错误**。在 `try` 块中手动调用 `$this->db->trans_rollback()` 后直接 `return`，但方法末尾仍执行 `$this->db->trans_complete()`。在 CI 类框架中，`trans_complete()` 会自动根据状态提交或回滚，手动回滚会导致事务状态机混乱，可能引发“事务已提交/回滚”异常或数据不一致。 | 移除 `try-catch` 中的手动回滚，采用框架标准事务流：`trans_start()` → 业务逻辑 → `trans_complete()` → 通过 `trans_status()` 判断结果。异常捕获应仅用于记录日志或转换错误格式。 | ```php<br>$this->db->trans_start();<br>// ... 业务逻辑 ...<br>$this->db->trans_complete();<br>if ($this->db->trans_status() === FALSE) {<br> return ['success'=>false, 'msg'=>'预订失败'];<br>}<br>``` |
| 🔴 严重 | `Ahead_book_model.php`<br>`get_book_list()` ~L285 | **SQL 注入漏洞**。`$where_str[] = "(book._book_no LIKE '%" . $params['book_no'] . "%' ..."` 直接拼接用户输入，未做任何转义或参数绑定。攻击者可构造恶意输入破坏查询结构或拖库。 | 使用框架查询构建器的 `like()` 方法，或强制类型转换/使用 `escape()`。复杂条件建议拆分为多个 `or_group_start()`。 | ```php<br>$this->db->group_start();<br>$this->db->like('book._book_no', $params['book_no']);<br>$this->db->or_like('book._customer_contact', $params['book_no'], 'after');<br>// ... 其他条件 ...<br>$this->db->group_end();<br>``` |
| 🔴 严重 | `Ahead_book_model.php`<br>`update_book()` ~L195 | **SQL 注入漏洞**。`$where = "_merchant_id=" . $merchantId . " and _id in (" . implode(",", $ids) . ")";` 直接拼接字符串作为 WHERE 条件。 | 使用数组条件或 `where_in()`，避免手写 SQL 片段。 | ```php<br>$where = ['_merchant_id' => $merchantId];<br>$this->db->where_in('_id', $ids);<br>``` |
| 🟠 警告 | `Ahead_book_model.php`<br>`get_book_list()` ~L330-380 | **N+1 查询性能瓶颈**。在 `foreach ($list['rows'] as &$row)` 循环中频繁调用 `get_one()`、`get_vip_info_by_recharge()` 等模型方法。数据量稍大时将导致数据库连接耗尽、响应超时。 | 提取循环中所需的所有 ID，使用 `where_in` 批量查询，或在 SQL 层通过 `LEFT JOIN` 一次性获取。循环内仅做数组映射与格式化。 | ```php<br>$userIds = array_unique(array_filter(array_column($list['rows'], 'ahead_user_id')));<br>$users = $this->ahead_user_model->get_list(['_id' => $userIds]);<br>$userMap = array_column($users, '_mobile', '_id');<br>// 循环内直接 $row['customer_contact'] = $userMap[$row['ahead_user_id']] ?? '';<br>``` |
| 🟠 警告 | `Ahead_book_model.php`<br>`add_book()` ~L45 | **日期格式化逻辑错误且冗余**。`strtolower(date("Ymd H:30"))` 中 `H:30` 非标准格式（应为 `H:i`），且 `strtolower` 对日期字符串无意义。业务意图疑似“取整到半小时”。 | 使用数学取整或标准时间函数。若需取当前时间最近的半小时节点：`date('Y-m-d H:i', floor(time()/1800)*1800)`。 | `$thirtyTime = date('Y-m-d H:i', floor($nowTime / 1800) * 1800);` |
| 🟠 警告 | `Ahead_book_model.php`<br>全文多处 | **模型重复加载**。在多个方法内部反复调用 `$this->load->model()`，增加框架解析开销，且不利于依赖管理。 | 将高频使用的模型移至 `__construct()` 中统一加载，或采用服务容器/依赖注入。 | ```php<br>public function __construct() {<br> parent::__construct();<br> $this->load->model('ahead_shop_config_model');<br> $this->load->model('Ahead_yc_shop_model');<br>}<br>``` |
| 🟡 建议 | `Ahead_book_model.php`<br>`Ahead_book_order_operation_log_model.php` | **不符合 PSR-12 规范**。方法名使用下划线（如 `add_book`），缺少参数与返回类型声明，文件顶部 `$CI = &get_instance();` 暴露在全局作用域。 | 方法名改为驼峰式（`addBook`），补充类型提示（`int $merchantId, array $params`），移除全局 `$CI` 赋值，按需使用 `$this->ci` 或 `$CI =& get_instance()` 局部获取。 | `public function addBook(int $merchantId, int $uid, array $params): array` |
| 🟡 建议 | `Ahead_book_order_operation_log_model.php`<br>`get_log_list()` ~L45 | **循环内分支查询冗余**。根据 `source` 在循环中调用不同模型的 `get_one()`，可优化为分组批量查询。 | 按 `source` 分组收集 `user_id`，分别批量查询后合并映射，减少 DB 交互次数。 | *(逻辑同 N+1 优化，按 source 分组后批量 `where_in` 查询)* |
| 🟡 建议 | `Ahead_book_model.php`<br>`update_book()` 末尾 | **代码截断**。方法在 `if (isset($params['welcome_minutes']) && $params['welcome_minutes']) {` 处中断，无法评估完整逻辑与事务闭合情况。 | 请补充完整代码。审查当前仅基于已提供片段。 | N/A |
> 💡 **框架适配说明**：代码结构高度类似 CodeIgniter 3/4。若 `phpci` 为定制框架，其事务状态机（`trans_start/complete/status`）与查询构建器（`where_in/like`）的实现可能与原生 CI 存在差异。建议对照 `phpci` 官方文档确认事务自动回滚机制与参数绑定语法。
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即修复 SQL 注入**：替换所有字符串拼接的 WHERE/LIKE 条件，全面改用框架查询构建器或参数绑定。这是最高优先级的安全红线。
2. **重构事务控制流**：移除 `try-catch` 中的手动 `trans_rollback()`，统一使用 `trans_start() → trans_complete() → trans_status()` 标准模式，确保数据一致性。
3. **消除 N+1 查询**：将 `get_book_list()` 循环内的单条查询改为批量查询（`where_in`）或 SQL `JOIN`，预计可将列表接口响应时间降低 60%~80%。
### 🛠 后续重构与优化方向
- **分层架构优化**：当前 Model 承担了过多业务逻辑（如权限校验、短信发送、欢迎词编码、日志记录）。建议引入 `Service` 层处理业务流程，Model 仅负责数据存取，符合单一职责原则。
- **现代化 PHP 特性**：全面启用 PHP 7.4+/8.x 类型声明（`declare(strict_types=1);`、参数/返回类型、属性类型），提升代码健壮性与 IDE 提示体验。
- **配置与常量管理**：硬编码的魔法数字（如 `1, 2, -1, 7, 11, 34`）应提取为类常量或配置文件，提升可维护性。
- **异常处理规范化**：`throwError()` 疑似全局函数，建议替换为抛出标准 `InvalidArgumentException` 或自定义业务异常类，配合全局异常处理器统一返回 JSON 格式。
- **补充完整代码**：请提供 `update_book()` 方法的完整实现，以便审查其事务边界、数据校验与状态流转逻辑。
如需针对特定模块（如短信发送逻辑、权限校验链路）进行深度剖析或提供完整重构代码模板，可随时提供补充片段。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780994977
|
1780994977
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
643
|
22
|
81
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 预订列表
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `422c54fb5 ## 自动代码审查报告
**分支**: app-260616
**提交**: `422c54fb5bc9fc67619ac2b5c777d7be5b60df9e`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 16:52:26
---
## 1. 审查摘要
- **代码质量评分**：4.5 / 10 分
- **总体评价**：代码实现了较为复杂的商户端业务逻辑，但存在典型的“巨型控制器/模型”反模式。安全层面存在硬编码密钥与 SQL 注入风险；性能层面存在明显的 N+1 查询与重复资源加载；架构层面严重违反单一职责原则，且混用了 CodeIgniter 3 的底层特性（注：项目结构及语法特征高度吻合 CI3，非 `phpci` 框架，以下审查基于 CI3 最佳实践）。整体需进行系统性重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `MerchantAppServer.php` (约第350行) | **硬编码敏感凭证**：`xfyun_tts_config` 直接暴露了第三方 API 的 `APPID`、`APISecret` 和 `APIKey`，极易导致服务被盗刷或数据泄露。 | 将敏感配置移至 `application/config/` 下的独立配置文件或环境变量中，通过 `$this->config->item()` 读取。 | `// config/xfyun.php<br>return ['appid' => 'xxx', 'secret' => 'xxx'];<br><br>// Controller<br>$config = $this->config->item('xfyun');` |
| 🔴 严重 | `Ahead_book_model.php` (第5行) | **框架生命周期违规**：在类外部直接调用 `$CI = &get_instance();`。CI 框架在文件加载阶段尚未完成初始化，会导致致命错误或不可预期的行为。 | 移除文件顶部的全局调用。在类方法内部按需使用 `$this->load->` 或 `$CI = &get_instance()`。 | `// 删除文件顶部的 $CI = &get_instance();<br>// 在方法内部使用：<br>$CI =& get_instance();` |
| 🔴 严重 | `Ahead_book_model.php` (`get_book_list` 方法内) | **SQL 注入漏洞**：`$where_str[] = "(book._book_no LIKE '%" . $params['book_no'] . "%' ..."` 直接拼接用户输入，未进行转义或参数绑定。 | 使用 CI Query Builder 的 `like()` 方法或 `$this->db->escape_like_str()` 进行安全过滤。 | `$search = $this->db->escape_like_str($params['book_no']);<br>$this->db->group_start()<br> ->like('book._book_no', $search, 'both')<br> ->or_like('book._customer_contact', $search, 'after')<br>->group_end();` |
| 🔴 严重 | `MerchantAppServer.php` (`__construct`) | **初始化逻辑断裂**：通过 `if ($this->router->fetch_method() !== "uploadPic")` 跳过核心初始化。若其他未白名单方法被调用，将因 `$this->request`、`$this->stream` 等属性未定义而抛出 `Undefined property` 或空指针异常。 | 移除方法级白名单拦截。改为在 `__construct` 中安全初始化基础属性，或在具体方法开头按需校验。 | `// 移除 if 判断<br>$this->stream = json_decode(file_get_contents('php://input'), true) ?? [];` |
| 🟠 警告 | `Ahead_book_model.php` (`get_book_list` 循环内) | **N+1 查询性能瓶颈**：`foreach` 循环中逐行调用 `$this->ahead_vip_model->get_vip_info_by_recharge()`，数据量大时将导致数据库连接耗尽与响应延迟。 | 改为批量查询：先收集所有 `customer_contact`，使用 `WHERE IN` 一次性查出 VIP 信息，再在 PHP 中映射到对应行。 | `$contacts = array_column($list['rows'], 'customer_contact');<br>$vip_data = $this->ahead_vip_model->get_batch_by_mobiles($contacts);<br>// 循环中通过 $vip_data[$row['customer_contact']] 赋值` |
| 🟠 警告 | `MerchantAppServer.php` (`index` 方法) | **重复加载模型与配置**：多个 `case` 分支中重复执行 `$this->load->model()` 和 `$this->config->load()`，增加 I/O 开销与内存占用。 | 将公共依赖的模型/配置提取至 `__construct` 或类属性中按需懒加载。 | `// __construct 中统一加载<br>$this->load->model(['ahead_merchant_role_priv_model', 'ahead_yc_shop_model']);` |
| 🟠 警告 | `MerchantAppServer.php` (约第15行) | **CORS 策略过于宽松**：`header("Access-Control-Allow-Origin:*");` 允许任意域名跨域请求，易被恶意站点利用发起 CSRF 或数据窃取。 | 根据实际业务配置可信域名白名单，或限制为 App 端特定 `User-Agent`/`Origin`。 | `$allowed = ['https://app.yourdomain.com'];<br>if (in_array($_SERVER['HTTP_ORIGIN'] ?? '', $allowed)) {<br> header("Access-Control-Allow-Origin: {$_SERVER['HTTP_ORIGIN']}");<br>}` |
| 🟡 建议 | `MerchantAppServer.php` (`index` 方法) | **魔法数字与硬编码泛滥**：`case '0005'`、`$permission_id = 534`、`$unsetMenuIds = [694, 849...]` 等缺乏语义化定义，维护成本极高。 | 提取为类常量或独立配置文件（如 `config/permissions.php`），使用枚举或常量映射。 | `const FUNC_LOGIN = '0005';<br>const PERM_VIP_RECHARGE_INVALID = 534;` |
| 🟡 建议 | `MerchantAppServer.php` (约第200行) | **数组过滤逻辑低效**：使用 `foreach` + `unset` + `array_values()` 过滤菜单，可读性差且性能不佳。 | 改用 `array_filter()` 结合闭包函数，代码更简洁且符合函数式编程规范。 | `$menus = array_values(array_filter($menus, function($v) use ($unsetMenuIds) {<br> return !in_array($v['id'], $unsetMenuIds) && !in_array($v['parentid'], $unsetMenuIds);<br>}));` |
| 🟡 建议 | `MerchantAppServer.php` / `Ahead_book_model.php` | **命名规范与拼写错误**：`AplicationController` 拼写错误；变量命名混用驼峰与下划线；注释含大量历史标记（如 `add by nan 17.10.25`）。 | 遵循 PSR-12 规范：类名 PascalCase，方法/属性 camelCase，常量 UPPER_SNAKE_CASE。清理过期注释。 | `class MerchantAppServer extends ApplicationController` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即移除硬编码密钥**：将 `xfyun_tts_config` 等敏感配置迁移至环境变量或加密配置文件中，避免凭证泄露。
2. **修复 SQL 注入风险**：全面审查 `Ahead_book_model.php` 中的 `$where_str` 拼接逻辑，统一替换为 CI Query Builder 的安全绑定方法。
3. **纠正框架生命周期调用**：删除 `Ahead_book_model.php` 文件顶部的 `$CI = &get_instance();`，防止框架初始化冲突。
### 🛠 后续重构与优化方向
1. **架构拆分（Controller -> Service/Repository）**：
- 当前 `MerchantAppServer` 承担了路由分发、鉴权、权限校验、数据组装、业务逻辑等所有职责，违反单一职责原则。
- **建议**：按业务域拆分为独立控制器（如 `AuthController`, `OrderController`, `PrinterController`），并将复杂业务逻辑下沉至 `Service` 层，模型仅负责数据持久化。
2. **性能优化策略**：
- 解决 `get_book_list` 中的 N+1 查询问题，采用批量加载或 `JOIN` 预加载。
- 将重复的 `$this->load->model()` 移至构造函数或使用 CI 的自动加载配置。
- 对高频调用的权限校验（如 `check_priv`）引入 Redis 缓存，降低数据库压力。
3. **规范化与可维护性提升**：
- 建立全局常量/配置中心管理魔法数字（接口号、权限ID、菜单ID等）。
- 统一输入校验层：在 `__construct` 或中间件中统一解析 JSON、校验 Token 格式、过滤非法字符，避免在每个 `case` 中重复处理。
- 补充异常处理与日志记录：当前 `try-catch` 仅返回失败提示，建议接入统一日志组件（如 Monolog）记录堆栈，便于线上排查。
> 📌 **注**：项目代码结构、加载方式（`$this->load->`、`get_instance()`、`BASEPATH`）与 **CodeIgniter 3** 高度一致。若实际使用的是内部封装的 `phpci` 框架，请核对上述 CI3 特性是否被框架兼容或重写。建议查阅框架官方文档确认路由分发、模型加载及安全过滤的最佳实践。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780995146
|
1780995146
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
661
|
22
|
82
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 预订管理
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `e6fd8b8b7 ## 自动代码审查报告
**分支**: app-260616
**提交**: `e6fd8b8b7e1699eeac1d338194fc5d8217f67ea7`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 11:21:00
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖较全面，能处理预订、关联账单、权限校验及短信通知等复杂场景。但代码存在**严重的 SQL 注入风险**、**事务管理不规范**、**N+1 查询性能瓶颈**以及**MVC 职责严重越界**等问题。整体架构偏向老旧的 CodeIgniter 3 写法，缺乏现代 PHP 的类型约束、单一职责划分与防御性编程意识。
- **风险等级**：🔴 高
> 📌 **框架说明**：提交代码的语法特征（`&get_instance()`、`$this->load->model()`、`$this->db->trans_start()` 等）高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，以下建议基于 CI3 核心机制与现代 PHP 最佳实践，架构原则通用。
---
## 2. 问题详情
| 严重程度 | 文件/方法 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_book_list` / `update_book` | **SQL 注入风险**：直接将用户输入 `$params['book_no']` 拼接到 `LIKE` 语句，且使用 `implode(",", $ids)` 拼接 `IN` 条件，未做任何转义或参数绑定。 | 使用查询构造器（Query Builder）或 `$this->db->escape()` / `escape_like_str()` 进行安全处理。 | `$this->db->like('book._book_no', $params['book_no']);`<br>`$this->db->where_in('_id', $ids);` |
| 🔴 严重 | `add_book` | **事务管理混乱**：混用 `trans_start()`、手动 `trans_rollback()` 与 `try-catch`。若 `catch` 中直接 `return` 而未调用 `trans_complete()`，可能导致连接池事务未释放或死锁。 | 遵循框架标准事务流：移除手动 `trans_rollback()`，统一在 `trans_complete()` 后通过 `trans_status()` 判断结果。 | 见下方 `事务重构示例` |
| 🟠 警告 | `get_book_list` | **N+1 查询性能瓶颈**：在 `foreach ($list['rows'] as &$row)` 循环内多次调用 `get_one()`（查用户、查兑换记录、查VIP等）。数据量 >100 时将引发严重性能雪崩。 | 收集所有关联 ID，使用 `WHERE IN` 批量查询，在内存中通过 `array_column` 映射数据。 | 见下方 `批量查询优化示例` |
| 🟠 警告 | `add_book` / `update_book` | **MVC 职责越界**：Model 层直接处理短信发送、房态推送、操作日志记录、权限校验及业务规则判断。违反单一职责原则，难以测试与维护。 | 将非持久化逻辑抽离至 `Service` 层或控制器，Model 仅负责数据读写。复杂流程可改用事件/队列解耦。 | 模型仅保留 `$this->insert()`，业务逻辑移至 `BookService::create()` |
| 🟠 警告 | 文件顶部 | **全局作用域加载模型**：`$CI = &get_instance(); $CI->load->model('Simple_model');` 在类外部执行，易导致重复加载、内存泄漏或 CLI 环境报错。 | 移除全局代码，在 `__construct()` 中按需加载，或依赖框架自动加载机制。 | `public function __construct() { parent::__construct(); }` |
| 🟡 建议 | 全局 | **缺乏类型声明与防御性编程**：方法无参数类型/返回类型约束；`strtotime()` 未校验格式；`throwError()` 为全局函数，可能暴露堆栈信息。 | 添加 PHP 7+ 类型提示；对时间/手机号等关键参数做前置校验；使用框架标准异常类。 | `public function add_book(int $merchantId, int $uid, array $params): array` |
| 🟡 建议 | `add_book` | **魔法数字与硬编码**：大量使用 `1, 2, -1, 7, 11, 22, 34` 等状态码/模板ID，可读性差且易出错。 | 提取为类常量或配置项，如 `const SMS_TEMPLATE_BOOK_SUCCESS = 34;` | `const SMS_TEMPLATE_BOOK_ADMIN = 22;` |
### 🔧 核心代码重构示例
**1. 事务安全写法（替代原 `try-catch` + 手动回滚）**
```php
$this->db->trans_begin(); // 或 trans_start()
// 执行所有数据库操作...
$result = $this->insert($addData);
if ($result === false) {
$this->db->trans_rollback();
return ['success' => false, 'msg' => '预订失败'];
}
// 其他更新操作...
$this->db->trans_commit(); // 或 trans_complete()
if ($this->db->trans_status() === FALSE) {
$this->db->trans_rollback();
return ['success' => false, 'msg' => '数据库事务异常'];
}
// 事务成功后执行非DB操作（短信、通知等）
```
**2. N+1 查询优化（`get_book_list` 循环内）**
```php
// 1. 收集所有需要查询的ID
$userIds = array_unique(array_filter(array_column($list['rows'], 'ahead_user_id')));
$exchangeIds = array_unique(array_filter(array_column($list['rows'], 'group_platform_order_id')));
// 2. 批量查询
$userMap = $exchangeMap = [];
if ($userIds) {
$users = $this->ahead_user_model->get_list(['_id' => $userIds], '_id,_mobile');
$userMap = array_column($users, '_mobile', '_id');
}
if ($exchangeIds) {
$exchanges = $this->ahead_tuangou_exchange_log_model->get_list(['_id' => $exchangeIds], '_id,_platform_voucher_code');
$exchangeMap = array_column($exchanges, '_platform_voucher_code', '_id');
}
// 3. 循环内直接内存映射
foreach ($list['rows'] as &$row) {
if (empty($row['customer_contact']) && isset($userMap[$row['ahead_user_id']])) {
$row['customer_contact'] = $userMap[$row['ahead_user_id']];
}
// ... 其他逻辑同理
}
```
---
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **彻底修复 SQL 注入**：全局搜索 `$params` 直接拼接 SQL 的位置，统一替换为 `$this->db->where()`、`$this->db->like()` 或 `$this->db->query($sql, $bindings)`。
2. **规范事务生命周期**：移除 `try-catch` 中的手动 `trans_rollback()`，严格遵循 `trans_begin() -> 业务逻辑 -> trans_commit() -> trans_status()` 流程。
3. **消除循环查库**：对 `get_book_list` 中的关联数据查询实施批量加载（Batch Fetching），预计可提升列表接口响应速度 **60%~80%**。
### 🛠 后续重构方向
1. **架构分层（Service 模式）**：当前 Model 承担了 Controller 和 Service 的职责。建议创建 `BookService`，将权限校验、短信发送、房态同步、操作日志等逻辑移出 Model，Model 仅保留 `CRUD` 与基础数据校验。
2. **引入现代 PHP 特性**：
- 添加严格类型声明：`declare(strict_types=1);`
- 使用 PHP 8 属性/类型提示：`public function add_book(int $merchantId, int $uid, array $params): array`
- 替换全局函数 `throwError()` 为 `throw new \InvalidArgumentException('...')` 或框架标准异常。
3. **配置与常量管理**：将硬编码的短信模板 ID、状态映射、业务阈值抽离至 `config/book.php` 或类常量中，便于多环境部署与后期维护。
4. **代码格式化**：使用 `PHP-CS-Fixer` 或 `Prettier` 统一应用 PSR-12 规范，拆分超过 150 行的方法（如 `add_book` 可拆分为 `validate()`, `prepareData()`, `executeTransaction()`, `notify()`）。
> ⚠️ **局限性说明**：提供的代码在 `update_book` 方法末尾处截断（`if (isset($params['welcome_minutes']) && $params['welcome_minutes']) {`），未能审查完整逻辑。若该方法后续包含类似拼接或事务操作，请同步按上述标准修复。建议补充完整代码或进行二次审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781061660
|
1781061660
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
666
|
22
|
83
|
1
|
|
0
|
🔍 代码审查报告：app-260616 - 订单退款api
|
## 自动代码审查报告
**分支**: app-260616
**提交**: `fbf39c180 ## 自动代码审查报告
**分支**: app-260616
**提交**: `fbf39c1808c9d230b929c3e80aadf2d78042a9f2`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 16:23:08
---
## 1. 审查摘要
- **代码质量评分**：4.5 / 10 分
- **总体评价**：代码呈现典型的历史遗留系统特征，业务逻辑高度耦合，存在多处严重的安全隐患（硬编码密钥、明文密码比对、SQL 拼接风险）与架构反模式（上帝控制器、构造函数过重、直接操作超全局变量）。整体可维护性、扩展性与安全性均不达标，需进行系统性重构。
- **风险等级**：🔴 高
> 📌 **框架说明**：从 `CI_Controller`、`$this->load->`、`defined('BASEPATH')` 等特征判断，当前代码基于 **CodeIgniter 3** 架构。若 `phpci` 为贵司内部定制框架，以下安全与架构规范同样适用。具体组件调用请以 `phpci` 官方文档为准。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `MerchantAppServer.php`<br>`case '0005'` | **硬编码敏感凭证**：科大讯飞 TTS 的 `APISecret`、`APIKey` 直接暴露在业务逻辑中，极易通过版本库或反编译泄露。 | 移至独立配置文件或环境变量，通过框架配置类读取。禁止在代码中明文存储密钥。 | `// config/keys.php\n$config['xfyun'] = ['APPID' => '...', 'APISecret' => '...'];\n// 控制器中\n$xfyun = $this->config->item('xfyun');` |
| 🔴 严重 | `MerchantAppServer.php`<br>`case '00064'` | **明文密码存储与比对**：`_discount_pwd` 疑似明文存储，且直接与 `$_old_password` 比对，违反基础安全规范。 | 使用 `password_hash()` 加密存储，`password_verify()` 验证。 | `if (!password_verify($_old_password, $data['_discount_pwd'])) { ... }\n$hash = password_hash($_new_password1, PASSWORD_DEFAULT);` |
| 🔴 严重 | `Ahead_yc_order_model.php`<br>多处查询方法 | **SQL 注入风险**：`$addsql`、`$shop_ids`、`$_start_date` 等变量直接拼接入 SQL 字符串，未使用参数绑定。若上游传入恶意数据将导致注入。 | 全面改用 CI Query Builder 或严格使用 `?` 占位符。禁止拼接外部传入的 SQL 片段。 | `$this->db->where_in('_shop_id', explode(',', $shop_ids));\n$this->db->where('_timestamp >=', $start_date);` |
| 🔴 严重 | `Ahead_pay_log_model.php`<br>`update_refund_amount()` | **原始 WHERE 条件拼接**：`$where = '_relation_id="' . $relation_id . '" ...'` 存在注入风险且易引发语法错误。 | 使用 CI 数组条件或 Query Builder 方法构建查询。 | `$where = ['_relation_id' => $relation_id, '_type' => $type, '_status' => [1, 4]];` |
| 🟠 警告 | `Api.php`<br>`__construct()` & `jsonEcho()` | **输出缓冲滥用 & 未设响应头**：直接操作 `ob_*` 易触发 Warning，且未设置 `Content-Type: application/json`，可能导致客户端解析异常。 | 移除冗余缓冲操作，使用 CI 的 `output` 类统一响应。 | `$this->output->set_content_type('application/json')->set_output(json_encode($result, JSON_UNESCAPED_UNICODE));` |
| 🟠 警告 | `MerchantAppServer.php`<br>`__construct()` & `index()` | **构造函数过重 & 上帝方法**：构造函数执行鉴权、日志、配置加载；`index()` 包含数十个 `case`，严重违反单一职责原则，难以测试与维护。 | 将鉴权/日志移至基类控制器或中间件；按业务域拆分控制器，利用 CI 路由分发。 | 拆分为 `AuthController`、`OrderController`、`PrinterController` 等独立控制器。 |
| 🟠 警告 | `Api.php`<br>`selfChangeRoom()` | **直接修改 CI 超对象属性**：`$CI->merchant_id = ...` 破坏框架封装，易引发请求间状态污染与并发安全问题。 | 通过参数传递上下文数据至 Model，或使用 Session/Request 对象管理状态。 | `$this->load->model('order_model');\n$this->order_model->change_room($merchant_id, $admin_data, $params);` |
| 🟡 建议 | 全局多处 | **浮点数处理金额**：使用 `float` 进行金额加减（如 `bcsub` 未全面覆盖），在 PHP 中可能导致精度丢失（如 `0.1+0.2=0.30000000000000004`）。 | 金额统一以“分”为单位（整数）存储计算，或全面使用 `BCMath` 扩展。 | `$actual_pay = bcsub($pay, $refund, 2);\n$refund_amount = (int)round($refund * 100); // 转为分` |
| 🟡 建议 | 全局多处 | **模型重复加载**：在方法内部频繁 `$this->load->model()`，增加 I/O 开销且不符合 CI 最佳实践。 | 移至构造函数统一加载，或配置 `autoload.php` 自动加载高频模型。 | `public function __construct() { parent::__construct(); $this->load->model('vip_model'); }` |
## 3. 总结与行动建议
### 🚨 优先修复的关键问题（P0）
1. **移除硬编码密钥**：立即将 `MerchantAppServer.php` 中的 `xfyun_tts_config` 迁移至配置文件或密钥管理服务（如 Vault/Env），并轮换已泄露的密钥。
2. **修复密码安全漏洞**：对 `_discount_pwd` 字段执行一次性哈希迁移脚本，后续所有密码比对必须使用 `password_verify()`。
3. **封堵 SQL 注入入口**：全面审查 `$addsql`、`$shop_ids`、`$relation_id` 等变量的来源，替换所有字符串拼接 SQL 为 Query Builder 或参数绑定查询。
### 🛠 后续重构与优化方向
1. **架构解耦与路由规范化**：
- 废弃 `switch ($request['function'])` 的伪路由模式，改用 CI 原生路由配置（`config/routes.php`）映射到独立控制器。
- 将鉴权、日志记录、参数校验等横切关注点抽离至 `MY_Controller` 基类或中间件，保持业务控制器轻量。
2. **统一输入/输出处理**：
- 废弃直接读取 `$_POST`/`$_GET`，统一使用 `$this->input->post()`、`$this->input->get()` 或 `$this->input->raw_input_stream`，利用 CI 内置的 XSS/过滤机制。
- 封装统一的 `ApiResponse` 类，替代 `jsonEcho()`，自动处理 Header、状态码与 JSON 序列化。
3. **财务计算精度保障**：
- 建立全局金额处理规范，禁止使用 `float` 进行财务运算。引入 `BCMath` 或整数（分）计算，并在入库/出库时进行严格校验。
4. **代码规范与可维护性提升**：
- 遵循 PSR-12 规范，统一命名风格（如 `AplicationController` 拼写修正、模型类名大小写统一）。
- 消除魔法数字，将 `1, 2, 3, 15, 16` 等状态码/支付类型提取为类常量或枚举。
- 补充关键方法的 PHPDoc 注释与类型声明（PHP 7.4+ 推荐），提升 IDE 提示与静态分析能力。
> 💡 **提示**：由于提供的代码片段存在截断，部分全局函数（如 `throwError`、`request_frequency`）及基类 `Simple_model` 的实现未完全展示。建议在完整代码库中结合静态分析工具（如 `PHPStan`、`SonarQube`）进行全量扫描，以覆盖潜在的类型不匹配与未捕获异常。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781079788
|
1781079788
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
669
|
22
|
84
|
1
|
|
0
|
0616合并
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1781141689
|
1781141696
|
1781141696
|
0
|
0
|
0
|
Edit
Delete
|
|
678
|
22
|
85
|
1
|
|
0
|
退款用户加手机号显示
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1782697385
|
1782697392
|
1782697392
|
0
|
0
|
0
|
Edit
Delete
|
|
679
|
22
|
86
|
1
|
|
0
|
退款用户加手机号显示
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1782697413
|
1782697422
|
1782697422
|
0
|
0
|
0
|
Edit
Delete
|
|
686
|
22
|
87
|
1
|
|
0
|
需求 门店名称限制 16489
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1782720336
|
1782720353
|
1782720353
|
0
|
0
|
0
|
Edit
Delete
|
|
698
|
22
|
88
|
1
|
|
0
|
订台汇总test
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1784696635
|
1784696641
|
1784696641
|
0
|
0
|
0
|
Edit
Delete
|
|
704
|
22
|
89
|
1
|
|
0
|
一些bug
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1784801850
|
1784801855
|
1784801855
|
0
|
0
|
0
|
Edit
Delete
|
|
705
|
22
|
90
|
1
|
|
0
|
一些bug
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1784801871
|
1784871268
|
1784801879
|
0
|
0
|
0
|
Edit
Delete
|
|
707
|
22
|
91
|
1
|
|
0
|
Merge pull request '一些bug' (#90) from app Merge pull request '一些bug' (#90) from app into app-260728...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1784871262
|
1784871267
|
1784871267
|
0
|
0
|
0
|
Edit
Delete
|
|
718
|
22
|
92
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1785224312
|
1785224318
|
1785224318
|
0
|
0
|
0
|
Edit
Delete
|
|
719
|
22
|
93
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1785224329
|
1785224335
|
1785224335
|
0
|
0
|
0
|
Edit
Delete
|
|
727
|
22
|
94
|
1
|
|
0
|
需求 新版智控线路手动控制
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1786341304
|
1786341310
|
1786341310
|
0
|
0
|
0
|
Edit
Delete
|
|
728
|
22
|
95
|
1
|
|
0
|
需求 新版智控线路手动控制
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1786341320
|
1786341325
|
1786341325
|
0
|
0
|
0
|
Edit
Delete
|
|
734
|
22
|
96
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1786672452
|
1786672460
|
1786672460
|
0
|
0
|
0
|
Edit
Delete
|
|
735
|
22
|
97
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1786672471
|
1786672480
|
1786672480
|
0
|
0
|
0
|
Edit
Delete
|
|
736
|
22
|
98
|
1
|
|
0
|
需求 包厢暂停营业播报 16703
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1786672508
|
1786672513
|
1786672513
|
0
|
0
|
0
|
Edit
Delete
|
|
750
|
22
|
99
|
1
|
|
0
|
需求 分账
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1787640046
|
1787640054
|
1787640054
|
0
|
0
|
0
|
Edit
Delete
|
|
751
|
22
|
100
|
1
|
|
0
|
需求 分账
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1787640079
|
1787640085
|
1787640085
|
0
|
0
|
0
|
Edit
Delete
|