|
175
|
22
|
8
|
1
|
|
0
|
🔍 代码审查报告：app-260519 - 1
|
## 自动代码审查报告
**分支**: app-260519
**提交**: `2aabf960a ## 自动代码审查报告
**分支**: app-260519
**提交**: `2aabf960a677f132ad3491a05777a43b01aeec7b`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-19 13:15:08
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码实现了复杂的预订、退款、开房及订单流转业务，功能覆盖较全。但存在明显的事务管理混乱、N+1 查询性能瓶颈、SQL 拼接安全隐患及大量硬编码魔法数字。部分方法职责过重（如 `refund_by_notify` 超过 200 行），违反单一职责原则。
- **风险等级**：🔴 高（存在事务不一致、潜在 SQL 注入及外部支付调用与数据库事务耦合风险）
> 📌 **说明**：代码结构高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请对照其官方文档调整事务控制与模型加载机制。末尾 `update_by_change_room` 方法代码被截断，本次审查仅基于已提供内容。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_book_order_model.php` ~L130 | **SQL 注入风险**：`$log_where` 使用字符串拼接构建查询条件，若 `_id` 未严格过滤，将导致注入漏洞。 | 废弃字符串拼接，统一使用框架查询构造器或参数绑定。 | `$this->db->where('_relation_id', $order_data['_id'])->where_in('_type', [5,13])->where('_status', 1)->update(...)` |
| 🔴 严重 | `Ahead_book_order_model.php` ~L60, ~L330 | **事务与外部 API 耦合**：`invalid_book` 开启事务后调用 `refund_by_notify`，后者内部执行微信/银联退款请求。若外部 API 超时或失败，数据库已回滚但资金可能已退还，造成数据与资金不一致。 | 将外部支付调用移出数据库事务。采用“先更新本地状态为退款中 → 调用外部 API → 根据回调更新最终状态”的异步/补偿事务模式。 | 见下方架构建议 |
| 🔴 严重 | `Ahead_book_order_model.php` ~L60, ~L330 | **事务控制混乱**：混用 `trans_start()` 与手动 `trans_rollback()`/`trans_complete()`。若 `throwError` 为 `exit/die` 而非抛出异常，将导致事务挂起或死锁。 | 统一使用 CI 标准事务流：`trans_start()` → 业务逻辑 → `trans_complete()` → 检查 `trans_status()`。确保 `throwError` 抛出 `\Exception`。 | `try { $this->db->trans_start(); ... $this->db->trans_complete(); if ($this->db->trans_status() === FALSE) throw new \Exception('事务失败'); } catch (\Exception $e) { $this->db->trans_rollback(); throw $e; }` |
| 🟠 警告 | `Ahead_book_order_model.php` ~L230 | **N+1 查询性能瓶颈**：`get_list` 循环内调用 `get_one` 查询关联订单，列表数据量大时会导致数十次额外 DB 请求。 | 提取所有 `relation_order_id`，使用 `WHERE IN` 批量查询或 `LEFT JOIN` 一次性获取。 | `$ids = array_filter(array_column($order_info, 'relation_order_id')); if($ids) { $orders = $this->ahead_yc_order_model->select(['where_in'=>['_id',$ids]], '_id,_machine_name'); $orderMap = array_column($orders, '_machine_name', '_id'); }` |
| 🟠 警告 | `Ahead_book_order_model.php` ~L1, `Ahead_yc_order_process_model.php` ~L1 | **全局实例获取违规**：在类外部直接 `$CI =& get_instance();` 违反框架生命周期，在 CLI 或单元测试中易引发未初始化错误。 | 移至构造函数中初始化，或按需通过 `$this->CI =& get_instance()` 获取。 | `public function __construct() { parent::__construct(); $this->CI =& get_instance(); }` |
| 🟠 警告 | `Ahead_book_order_model.php` ~L110 | **敏感数据日志泄露**：`doLog(var_export($res, true), ...)` 直接记录支付网关完整响应，可能包含商户密钥、用户 OpenID、交易流水等敏感信息。 | 脱敏后记录，仅保留状态码、订单号、错误信息。 | `doLog(json_encode(['order_id'=>$order_data['_id'], 'status'=>$res_data['response']['result_status']??'', 'msg'=>$res_data['response']['error_msg']??'']), 'BookOrderWxRefund');` |
| 🟡 建议 | `Ahead_book_order_model.php` 多处 | **魔法数字泛滥**：大量使用 `1, 2, 3, 4, 5, 13, 17...` 表示状态/支付渠道，可读性差且维护成本高。 | 提取为类常量或独立配置类，如 `const STATUS_PAID = 1; const PAY_WX = 1;`。 | `if ($book_order['_status'] === self::STATUS_PAID) { ... }` |
| 🟡 建议 | `Ahead_book_order_model.php` ~L105 | **弱类型比较隐患**：`$is_refund == '1'` 使用松散比较，PHP 隐式转换可能导致非预期分支执行。 | 统一使用严格比较 `===` 或在入口处进行类型强转。 | `$is_refund = (int)$is_refund; if ($is_refund === 1) { ... }` |
| 🟡 建议 | `Ahead_yc_order_process_model.php` ~L100 | **重复代码**：`send_to_screen` 与 `order_refund_send_to_screen` 构建 WebSocket 请求数组逻辑高度重复。 | 抽取私有方法 `build_screen_request(array $params)` 复用，提升可维护性。 | `private function build_screen_request($data) { return ['header'=>['data_type'=>'proxy',...], 'request'=>$data]; }` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **重构事务边界**：立即将 `refund_by_notify` 中的微信/银联退款调用移出数据库事务。采用 **本地状态预更新 → 异步/同步调用支付网关 → 依赖回调更新最终状态** 的模式，避免资金与数据不一致。
2. **修复 SQL 拼接漏洞**：全局替换 `$this->model->up($raw_string)` 类用法，全面改用框架 Query Builder 或预处理参数绑定。
3. **消除 N+1 查询**：对 `get_list`、`get_detail` 等列表/详情接口实施批量查询优化，预计可降低 60% 以上的数据库 I/O 耗时。
4. **规范异常处理**：确认 `throwError` 实现机制。若仅为 `die()`，请替换为 `throw new \RuntimeException()` 以确保 `try-catch` 与事务回滚能正确捕获。
### 🛠 后续重构与优化方向
- **单一职责拆分**：`refund_by_notify` 方法过长（>200行），建议拆分为：
- `prepareRefundData()`：数据组装与校验
- `executePaymentRefund()`：支付网关调用
- `updateRefundLogs()`：日志与流水更新
- `handleCommunityRefund()`：社区业务逻辑
- **引入常量/枚举管理**：建立 `OrderStatus`, `PayPlatform`, `ProcessStep` 等配置类，彻底消除魔法数字。
- **模型懒加载优化**：频繁在方法内 `$this->load->model()` 会增加开销。建议在 `__construct` 中集中加载高频模型，或启用框架的自动加载机制。
- **日志脱敏规范**：制定全局日志规范，支付、用户隐私字段必须经过 `mask_sensitive_data()` 过滤后再落盘。
- **补充单元测试**：针对 `invalid_book`、`openRoomByCommunityOrder` 等核心链路编写 Mock 测试，覆盖事务回滚、外部 API 超时、并发开房等边界场景。
> 💡 **框架适配提示**：若 `phpci` 对事务或模型加载有特殊封装（如自动事务代理、依赖注入容器），请优先遵循其官方文档。上述建议基于标准 CI3/PHP 最佳实践，可直接平滑迁移。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779167708
|
1779167708
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
242
|
23
|
8
|
1
|
|
0
|
🔍 代码审查报告：admin-260616 - 需求 聚旺设备标识
|
## 自动代码审查报告
**分支**: admin-260616
**提交**: `3920c00 ## 自动代码审查报告
**分支**: admin-260616
**提交**: `3920c0037aa16f59e5d7ea7f0ed8963bce23b75d`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-05-20 15:36:39
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：该 Model 承载了过多的业务逻辑（查询构建、数据格式化、外部服务通信、权限校验等），属于典型的“上帝类”。代码中存在严重的安全隐患（SQL 拼接未转义）、数据一致性风险（多表操作无事务）以及大量硬编码魔法值。整体可维护性较低，需进行职责拆分与安全加固。
- **风险等级**：🔴 高
> 📌 **框架说明**：代码结构、API 调用（如 `$this->load->model()`、`$this->db->query()`、`get_instance()`）高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请确认底层 Query Builder、事务 API 及模型加载机制是否与以下建议兼容。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `update_version` 方法 | 手动拼接 `WHERE` 与 `SET` 子句，且未对 `$v` 进行转义或参数化。若参数含单引号或恶意字符，将直接导致 **SQL 注入**。 | 废弃手动拼接，全面改用框架 Query Builder 或预处理语句。 | `$this->db->where($where)->update($this->table_name, $up);` |
| 🔴 严重 | `untying` 方法 | 涉及多表 `update`/`del` 及日志写入，但未包裹数据库事务。若中途抛出异常或网络超时，将导致 **数据不一致**（如包厢已删但门店计数未减）。 | 使用 `$this->db->trans_start()` 包裹核心数据操作，失败自动回滚。 | `见下方事务示例` |
| 🔴 严重 | 文件顶部 (L3-L4) | 在类外部直接调用 `get_instance()` 并加载模型。该代码会在每次 `include/require` 时执行，违反框架生命周期，易引发 **单例污染或内存泄漏**。 | 移除文件级代码，依赖加载应移至 `__construct()` 或按需调用。 | `public function __construct() { parent::__construct(); }` |
| 🟠 警告 | `get_list` 方法 | 方法体超 300 行，混合了条件组装、多表 JOIN、数据格式化、二维码生成、限速判断等逻辑，严重违反 **单一职责原则 (SRP)**。 | 拆分为 `buildQueryConditions()`, `formatRoomData()`, `resolveQrCodes()` 等私有方法，主方法仅负责流程编排。 | (结构拆分建议) |
| 🟠 警告 | `copy_data` 方法 | 在 `foreach` 循环中逐条调用 `$this->insert()`。当模板包厢数较多时，会产生大量数据库往返，极易触发 **连接超时或性能瓶颈**。 | 收集数据后使用 `insert_batch()` 批量插入，大幅降低 I/O 开销。 | `$this->db->insert_batch($this->table_name, $addData);` |
| 🟠 警告 | `wb_execute_command` | 直接透传 `$command` 至 WebSocket 服务端，未做白名单、长度或危险字符过滤。若下游直接执行系统命令，存在 **命令注入/越权风险**。 | 增加参数校验，或建立命令白名单映射机制。 | `if (!in_array($command, $allowed_cmds)) throwError('非法命令');` |
| 🟡 建议 | 全局多处 | 大量使用魔法数字/字符串（如 `'9066'`, `'P960'`, `'-1'`, `'audio.g-hi.com'`），降低可读性且难以统一维护。 | 提取为类常量或独立配置文件。 | `const FUNC_FORMAT = 9066; const WS_HOST = 'audio.g-hi.com';` |
| 🟡 建议 | `check_room_data` | 方法体完全为空，属于无效死代码，易误导后续开发者。 | 补充业务逻辑或直接删除该方法。 | `// 待实现或移除` |
| 🟡 建议 | 频繁 `$this->load->model()` | 在多个方法内部重复加载同一模型（如 `Ahead_yc_merchant_model`），增加不必要的 I/O 与解析开销。 | 移至构造函数统一加载，或配置 CI 自动加载 (`autoload.php`)。 | `__construct() { $this->load->model('Ahead_yc_merchant_model'); }` |
**🔧 关键代码修复示例：**
```php
// 1. update_version 安全重构
public function update_version($where, $up, $join_table = [])
{
// 使用 CI Query Builder 自动转义，杜绝 SQL 注入
$builder = $this->db->table($this->table_name);
if (!empty($join_table)) {
foreach ($join_table as $j) {
$builder->join($j[0], $j[1], $j[2] ?? 'left');
}
}
$builder->where($where)->update($up);
$affected = $this->db->affected_rows();
if ($affected > 0) {
$this->update_after($up, $where);
}
return $affected;
}
// 2. untying 事务包裹示例
public function untying($room_id)
{
$this->db->trans_start();
try {
// ... 原有数据查询与校验逻辑 ...
$this->del(['_id' => $room_data['_id']]);
$this->Ahead_open_room_log_model->update(...);
$this->ahead_agent_authenticate_model->update(...);
$this->Ahead_yc_shop_model->up(...);
// ... 其他表操作 ...
$this->db->trans_complete();
if ($this->db->trans_status() === FALSE) {
$this->db->trans_rollback();
throwError('解绑失败，数据已回滚');
}
return true;
} catch (\Exception $e) {
$this->db->trans_rollback();
throwError('解绑异常: ' . $e->getMessage());
}
}
```
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **修复 SQL 注入漏洞**：立即替换 `update_version` 中的手动 SQL 拼接逻辑，全面启用框架 Query Builder 或 PDO 预处理。
2. **引入数据库事务**：为 `untying`、`copy_data` 等涉及多表写操作的方法添加事务控制，确保数据原子性。
3. **清理文件级代码**：删除顶部的 `$CI = &get_instance();`，将模型依赖移至构造函数或按需加载。
### 🛠 后续重构方向
1. **职责拆分（SRP）**：将 `get_list` 拆分为 `查询条件构建器`、`数据格式化器`、`外部服务调用器`。建议将 WebSocket 通信逻辑抽离至独立的 `Service/WebSocketService.php`，Model 仅负责数据持久化。
2. **常量与配置管理**：建立 `config/room_constants.php` 或在类顶部定义 `const`，集中管理状态码、功能码、产品型号、WebSocket 地址等硬编码值。
3. **性能优化**：
- `copy_data` 改用 `insert_batch()`。
- 减少方法内重复的 `$this->load->model()`，利用 CI 的自动加载或构造函数预加载。
- 对 `get_list` 中的 `formatBytes`、日期转换等重复计算可考虑缓存或数据库层 `DATE_FORMAT` 处理。
4. **规范与可维护性**：
- 补充 `@param`、`@return` 类型声明，统一 DocBlock 格式。
- 删除 `check_room_data` 等空方法。
- 若团队已启用 PHP 7.4+，建议为属性添加类型声明（如 `public array $mural_status = [];`）。
> ⚠️ **局限性说明**：您提供的代码在 `upt_room_online_time` 方法末尾（`$data = send_web_socket`）被截断，未能审查该方法的完整逻辑及后续代码。建议补充完整文件后再次进行深度审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779262599
|
1779262599
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
773
|
85
|
8
|
4
|
|
0
|
按省市统计设备月度新增上线数量报表
|
## 功能概述
在 PC 管理后台「数据报表」模块，按省市统计各 KTV 设备每月新增上线数量，支 ## 功能概述
在 PC 管理后台「数据报表」模块，按省市统计各 KTV 设备每月新增上线数量，支持全国省份分布视图 + 城市下钻。
## 完整产物链路
| 阶段 | 产物 | 链接 | 状态 |
|------|------|------|------|
| 需求意图 | Intent 文档 | PR #1 | ✅ 已合并 |
| 设计原型 | Design 文档 | PR #3 | ✅ 已合并 |
| 技术方案（后台） | Plan 文档 | PR #6 | ✅ 已合并 |
| 开发实现（后台） | admin 仓库代码 | admin PR #1 | ✅ 已合并 |
| 开发实现（Web前端） | admin 仓库代码 | admin PR #2 | ✅ 已合并 |
| 测试验收 | — | 本 Issue | 🧪 待测试 |
## 验收标准
- 后台 API 返回省份/城市维度的月度新增上线设备数，分页正确
- PC 管理后台菜单「数据报表 > 设备上线分布」可正常点击进入
- 省份视图正常渲染（横向柱状图），点击下钻到城市视图
- 城市视图面包屑可返回全国
- 月份切换后数据刷新正确
- 无数据月份显示空状态，不报错...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1789379887
|
1789380277
|
0
|
0
|
2
|
0
|
Edit
Delete
|
|
814
|
124
|
8
|
5
|
|
0
|
fix(AUD-EXIT): keep fulfilling original obligation fix(AUD-EXIT): keep fulfilling original obligations after cooperation exit...
|
Exit used to freeze an exiting cooperation: delive Exit used to freeze an exiting cooperation: deliveries were rejected, so
open promises had no continued-handling path (ALI01, AC30).
- deliver accepts active or exiting cooperations, still capped by each
original obligation; after-exit entries are tagged phase:'after-exit'
with the acting principal.
- exit records which obligations were open at that moment; there is still
no action that adds new obligations, and approve stays draft-only.
- new completeExit moves exiting -> exited only when every original
obligation is delivered (OBLIGATIONS_OPEN otherwise); exited rejects
further deliveries. Finance reconcile is unaffected and funds remain
unverified.
- store workbench mirrors the rules and adds a 完成退出 action.
- ALI01/AC30 named test mapping for the new rule test.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791559039
|
1791559044
|
1791559044
|
0
|
0
|
0
|
Edit
Delete
|
|
851
|
83
|
8
|
5
|
|
0
|
feat(integration): Web/开发基座只绑回环、D-12 packages/* 受控 feat(integration): Web/开发基座只绑回环、D-12 packages/* 受控通道与 H20 出站事实注册表...
|
## 改了什么
本 PR 有三部分；已合入 main 的 PR #9（D-10 API 回环）与 ## 改了什么
本 PR 有三部分；已合入 main 的 PR #9（D-10 API 回环）与 PR #10（H23 根与 apps/* 依赖）均已并入本分支。
### 1. Web / 开发基座只绑回环（蓝图 §4.9）
- Web `next dev` / `next start` 与 UI 验收的 `next start` 加 `-H 127.0.0.1`；开发基座 `apps/api-nestjs/docker-compose.yml` 的 PostgreSQL / Redis 端口只发布到回环（此前发布到全部网卡，PG 口令 `app`、Redis 无口令）。已在运行的基座容器须 `docker compose up -d` 重建才生效。
- 在 #9 的 `scripts/lib/listen-host-guard.mjs` 内新增 `exposureViolations`（同一守卫），接入 `check:dual-backend`；单测 9/9。
- 文档：owner-matrix 去掉 D3 前「双后端 / Fastify」措辞；Scope 来源接入文档改正 IdentityLink 归属（enterprise-idp 拥有）。
### 2. D-12：`packages/*` 受控升级通道（关 H24，蓝图 §4.10）
- D3 保护此前逐字节冻结全部 `packages/*`（215 个受保护文件），台账只放行产品目录。现在同一台账 `docs/post-d3-product-updates.json` 增加 `packageUpdates`：编号 `PKG-NNN` 递增，须写来源与理由，仅限 D3 冻结清单内的 `packages/*` 文件，逐文件从冻结 hash 链式放行；任一条目不合规即整体不放行。
- `check:d3` 30/30：新增登记后放行、未登记 / 台账缺失判红，以及断链、前后相同、非 hash、缺理由、缺来源、编号格式错、编号不递增、空文件、包外路径、非冻结文件、字段类型错均判红。变异探针：删掉「理由与来源」或「链式校验」规则，测试各判红 1 项，已还原。
### 3. H20：出站事实注册表（蓝图 §4.10）
- contracts 新增 `ENTERPRISE_FACT_ENVELOPE_SCHEMAS`（`factType@schemaVersion` → 信封 Schema，当前只有 `EmploymentTransferred@1`）与 `EnterpriseFactTypeUnregisteredError`；`buildEnterpriseFactEnvelope` 按注册表分派。EmploymentTransferred v1 的 Schema 不变。
- relay 去掉全部类型强转；未登记类型或版本记 `FACT_TYPE_UNREGISTERED`，立即进死信、不调用 Publisher（部署补登记后可经既有重放恢复）。
- contracts 两个文件经台账 **PKG-001** 放行（登记前 `check:d3` 对这两个文件判红）。
- `check:app-manifest` M6 新增注册表与 `facts.owns` 双向一致检查（3 个负例）。
- 测试：contracts 新增 4 例；新文件 `apps/api-nestjs/test/enterprise-fact-relay-registry.test.ts` 真实 DB 3 例。原 `enterprise-fact-relay.test.ts` 受 D3 冻结未改，12 例仍全绿。
## 验证（本地，2026-10-27，干净提交 `effed90`）
- `check:runtime` 462/462、`check:ui` 46/46（一次性 PostgreSQL / Redis，跑完删除）
- typecheck、lint、`check:d3`、`check:dual-backend`、`check:app-manifest`、`check:docs-truth`、`check:governance-docs`、`check:migrations`、`check:database-rls` 等静态门禁通过
- `check:governance` 剩 2 项，与 main 相同：O22（D-8，等 OS 写入授权）、`osProductReleaseTargetRemotes` 2→1（D-9）
## 未覆盖
- 远端 CI 未查验（本机无 Gitea API 令牌）
- 新事实（`OrganizationUnitMoved` 等）仍待 Catalog 登记；H21（Scope 消费端写死来源应用）仍开放
- `packages/*` 内的 vitest 2.1.9 与 `client-fact` rc.3 现可经台账升级，本 PR 未做
- 00 框架同一监听默认值（他层仓）；README 第 3 步提到的 `.env.example` 不存在（既有不一致）
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793111425
|
1793124194
|
0
|
0
|
3
|
0
|
Edit
Delete
|
|
885
|
116
|
8
|
5
|
|
0
|
feat(notification): bind signing credentials to co feat(notification): bind signing credentials to controlled M2 access...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793610788
|
1793610798
|
1793610798
|
0
|
0
|
0
|
Edit
Delete
|
|
9
|
2
|
9
|
1
|
|
0
|
🔍 代码审查报告：main 分支 - 修改测试111
|
## 自动代码审查报告
**分支**: main
**提交**: `252852cf322c50f ## 自动代码审查报告
**分支**: main
**提交**: `252852cf322c50ffd2a193223a08e9e4b567bf78`
**时间**: 2026-03-17 11:14:20
**审查模型**: qwen3.5-plus
---
[2026-03-17 11:13:48] 正在调用 Qwen API 进行代码审查...
[2026-03-17 11:14:20] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260317-111348.md
## 📋 审查摘要
- **变更文件数**: 0
- **严重问题**: 0
- **高危问题**: 0
- **中危问题**: 0
- **建议优化**: 1
## 🐛 发现的问题
### [输入缺失] 未检测到可审查的代码内容
- **严重程度**: 严重 (阻塞审查)
- **文件**: 无
- **行号**: N/A
- **问题描述**: 在提供的输入中，“变更文件内容”部分为空。没有具体的代码片段、文件路径或项目结构信息，无法执行语法检查、逻辑分析或安全审计。
- **修复建议**: 请补充需要审查的具体代码内容。建议提供以下信息：
1. 文件的相对路径或名称。
2. 完整的代码片段或 Diff 内容。
3. 如果有跨文件调用，请提供相关依赖文件的上下文。
## ✅ 代码亮点
暂无代码可供评估。
## 📝 总体建议
当前输入不包含任何代码内容，因此无法进行实质性的审查。作为代码审查专家，我已准备好从以下维度为您服务：
1. **语法与规范**：检查语言特性使用是否正确，是否符合 PSR/PEP/Google 等规范。
2. **逻辑与健壮性**：识别潜在的空指针、边界条件错误及资源泄露。
3. **安全性**：扫描 SQL 注入、XSS、敏感信息泄露等风险。
4. **架构与依赖**：分析方法调用链及模块耦合度。
5. **可维护性**：评估代码复杂度、重复率及注释完整性。
**请粘贴您需要审查的代码，我将立即开始工作。**
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773717260
|
1773717260
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
36
|
18
|
9
|
1
|
|
0
|
🔍 代码审查报告：pc-260331 分支 - 测试
|
## 自动代码审查报告
**分支**: pc-260331
**提交**: `b1ef432b25 ## 自动代码审查报告
**分支**: pc-260331
**提交**: `b1ef432b25dddc514fa2569dcac5be00d4a0b2f4`
**时间**: 2026-03-28 11:18:27
**审查模型**: qwen3.5-plus
---
[2026-03-28 11:17:39] 正在调用 Qwen API 进行代码审查...
[2026-03-28 11:18:27] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260328-111739.md
基于您提供的项目结构和代码变更内容，我对 `application/controllers/AdMachine.php` 进行了代码审查。以下是分析结果、发现的问题以及修正建议。
### 🚨 关键问题：语法错误
在 `getScreensaver` 方法定义中存在明显的拼写错误，会导致 PHP 解析失败（Parse Error）。
**错误代码：**
```php
public function getScreensaver()dddd
```
**问题说明：**
方法名后面多出了 `dddd` 字符，这不符合 PHP 函数定义语法。
---
### ✅ 修正后的代码
以下是修复了语法错误并优化了部分结构的代码建议：
```php
<?php
defined('BASEPATH') OR exit('No direct script access allowed');
// 建议：如果 PcServer 是基础控制器，最好通过 CI 的核心机制加载，而不是手动 include
// 但为了保持现有逻辑，保留 include，建议改为 require_once 确保只加载一次
require_once FCPATH . 'application' . DIRECTORY_SEPARATOR . 'controllers' . DIRECTORY_SEPARATOR . 'PcServer.php';
// 广告机
class AdMachine extends PcServer
{
/**
* 获取屏保内容
* @return void
*/
public function getScreensaver() // 修复了这里的 dddd 错误
{
$this->load->model('Ahead_shop_ad_machine_set_model');
// 建议：增加参数验证或错误处理
$res = $this->Ahead_shop_ad_machine_set_model->get_data($this->merchant_id, $this->param);
$this->stream['request']['result'] = $res;
$this->success_response();
}
/**
* 设置屏保内容
* @return void
*/
public function setScreensaver()
{
$this->load->model('Ahead_shop_ad_machine_set_model');
// 建议：增加参数验证，确保 $this->param 安全
$this->Ahead_shop_ad_machine_set_model->set_data($this->merchant_id, $this->param);
$this->success_response();
}
}
```
---
### 📊 依赖与架构分析
1. **框架识别**:
* 根据 `defined('BASEPATH')`、`$this->load->model`、`FCPATH` 等特征，该项目基于 **CodeIgniter (CI)** 框架（可能是 CI2 或 CI3 版本）。
2. **父类依赖 (`PcServer`)**:
* **文件位置**: `application/controllers/PcServer.php` (存在于您提供的文件列表中)。
* **加载方式**: 当前代码使用 `include` 手动加载。在 CodeIgniter 中，通常建议将基础控制器放在 `application/core/` 目录下（例如 `MY_Controller.php`），以便框架自动加载，避免在每个子类中手动 `include`。
* **继承属性**: 代码中使用了 `$this->merchant_id`, `$this->param`, `$this->stream`, `$this->success_response()`，这些属性和方法应该定义在 `PcServer` 父类中。
3. **模型依赖 (`Ahead_shop_ad_machine_set_model`)**:
* **文件位置**: 应在 `application/models/Ahead_shop_ad_machine_set_model.php`。
* **注意**: 您提供的文件列表中没有包含 `models` 目录的文件列表，请确保该模型文件存在且包含 `get_data` 和 `set_data` 方法。
4. **代码规范建议**:
* **模型加载**: 如果 `Ahead_shop_ad_machine_set_model` 在该控制器的多个方法中���用，建议在 `__construct` 构造函数中加载，而不是在每个方法中重复加载。
* **安全性**: `$this->param` 直接传入模型，建议在控制器层或模型层进行严格的数据验证和过滤，防止 SQL 注入或逻辑漏洞。
* **包含语句**: 建议使用 `require_once` 代替 `include`，防止文件缺失时继续执行导致致命错误，同时也避免重复包含。
### 📋 下一步建议
1. **修复语法错误**：立即移除 `getScreensaver` 方法名后的 `dddd`。
2. **验证模型文件**：确认 `application/models/Ahead_shop_ad_machine_set_model.php` 是否存在。
3. **检查父类逻辑**：确认 `PcServer.php` 中是否正确初始化了 `$this->merchant_id` 和 `$this->param`，否则这两个变量在当前类中可能未定义。
4. **优化加载机制**：考虑将 `PcServer` 迁移至 `application/core/` 目录以符合 CI 框架规范。
如果您需要我协助检查 `PcServer.php` 的内容或生成模型文件模板，请提供相关文件内容。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774667907
|
1774667907
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
176
|
21
|
9
|
1
|
|
0
|
🔍 代码审查报告：pay-260519 - 1
|
## 自动代码审查报告
**分支**: pay-260519
**提交**: `aa880e506 ## 自动代码审查报告
**分支**: pay-260519
**提交**: `aa880e50672277db9650328ede3b2b261906d202`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-19 13:20:10
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了较为复杂的门店业务逻辑，但存在多处严重的安全隐患（SQL注入）、状态管理缺陷（类属性污染）、MVC 职责越界以及性能瓶颈（N+1查询、PHP层距离计算）。部分代码未遵循 CI 框架生命周期规范，且命名与硬编码缺乏一致性。需进行系统性重构。
- **风险等级**：🔴 高
> 📌 **框架说明**：根据目录结构（`system/`, `application/`）及语法特征（`get_instance()`, `$this->load->model()`, `$this->db->`），判定该代码基于 **CodeIgniter 3.x** 架构。以下审查与建议均基于 CI3 最佳实践。若 `phpci` 为内部定制框架，请对照其官方文档调整组件调用方式。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_community_shop_list` 约第 280 行 | **变量名拼写错误**：使用未定义的 `$param['bookFrom']`，实际参数名为 `$params`。将导致 PHP Notice 且业务逻辑失效。 | 修正为 `$params['bookFrom']`，并增加类型校验。 | `if (!empty($params['bookFrom'])) { ... }` |
| 🔴 严重 | `get_date_shop_no_business_time` 约第 560 行 | **类属性状态污染**：使用 `$this->shop_no_business_time` 累加数据。同一请求多次调用该方法会导致数据无限叠加，引发内存泄漏与逻辑错乱。 | 改为局部变量，方法结束后自动销毁。 | `$shop_no_business_time = [];`<br>`// 后续逻辑中替换 $this->shop_no_business_time` |
| 🔴 严重 | 多处 (`get_community_shop_list`, `get_shop_list_order_by_distance` 等) | **SQL 注入漏洞**：`FIND_IN_SET({$operational_scene}, ...)` 与 `LIKE '%{$shop_name}%'` 直接拼接用户输入，未做转义或参数化。 | 使用 CI 查询构建器绑定参数，或严格过滤输入类型。 | `$this->db->where("FIND_IN_SET(?, shop._operational_scene)", (int)$operational_scene);`<br>`$shop_name = $this->db->escape_like_str($shop_name);` |
| 🟠 警告 | 文件顶部 第 8-9 行 | **违反框架生命周期**：在文件作用域直接调用 `get_instance()` 加载模型。CI 尚未完全初始化时会引发 Fatal Error。 | 移除文件顶部代码，统一在 `__construct()` 中加载依赖。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🟠 警告 | `getAliAuthInfo` 约第 180 行 | **变量拼写错误**：`$this->db->select($filed)` 中 `$filed` 未定义，应为 `$fields`。 | 修正变量名，避免查询字段为空。 | `$this->db->select($fields)` |
| 🟠 警告 | `get_shop_list_order_by_distance` 约第 350 行 | **原始 SQL 拼接风险**：`_id in (...)` 与 `FIELD()` 循环拼接未校验数据类型，若传入非数字将破坏 SQL 语法或引发注入。 | 使用 `array_map('intval', $arr)` 过滤，并改用 CI Query Builder 或严格转义。 | `$safe_ids = implode(',', array_map('intval', $shop_id_arr_order_distance));`<br>`$sql .= " AND _id IN ({$safe_ids})";` |
| 🟠 警告 | 多个列表方法 (`get_info`, `get_community_shop_list` 等) | **N+1 查询性能瓶颈**：在 `foreach` 循环中逐条调用 `get_miniprogram_consumption_methods` 等模型方法，数据量大时数据库压力剧增。 | 提取 ID 列表批量查询，或引入 Redis/内存缓存。 | `$shop_ids = array_column($shop_data, 'shop_id');`<br>`$configs = $this->ahead_shop_config_second_model->get_batch_by_shop_ids($shop_ids);` |
| 🟡 建议 | 全局 | **MVC 职责越界**：Model 中直接调用 `throwError()` / `showErrorView()`。Model 应仅负责数据存取，视图与异常响应应由 Controller 处理。 | 改为抛出 `Exception` 或返回错误码，由 Controller 统一拦截处理。 | `if (empty($data)) { throw new \Exception('门店错误', 404); }` |
| 🟡 建议 | 全局 | **命名规范与魔法数字**：方法名混用驼峰(`getAliAuthInfo`)与下划线；大量使用 `86400`, `1`, `-1` 等硬编码。 | 统一使用下划线命名法（CI 规范）；定义类常量提升可读性。 | `const STATUS_ACTIVE = 1;`<br>`const TEST_SHOP_FLAG = -1;`<br>`public function get_ali_auth_info($condition)` |
| 🟡 建议 | `get_cache_shop_data` 约第 100 行 | **缓存序列化隐患**：`json_encode($shop_data)` 未处理失败情况，且 `$redis->close()` 可能破坏连接池复用。 | 增加 JSON 编码校验；使用 CI 内置 Cache 驱动管理连接。 | `if ($json === false) { log_message('error', 'Redis cache encode failed'); return $shop_data; }` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复 SQL 注入**：立即替换所有字符串拼接的 SQL 条件，改用 CI Query Builder 的 `where()`, `where_in()`, `like()` 或参数化绑定。特别是 `FIND_IN_SET` 和 `LIKE` 场景。
2. **消除状态污染**：将 `get_date_shop_no_business_time` 中的 `$this->shop_no_business_time` 改为局部变量，避免并发或多次调用时的数据交叉污染。
3. **修正致命拼写错误**：修复 `$param` vs `$params` 及 `$filed` vs `$fields`，防止运行时崩溃。
4. **规范框架初始化**：移除文件顶部的 `get_instance()` 调用，将依赖模型加载移至 `__construct()`。
### 🛠 后续重构与优化方向
- **架构分层**：严格遵循 MVC。Model 仅返回数据或抛出 `Exception`，将 `throwError`/`showErrorView` 移至 Controller 或全局异常处理器。
- **性能优化**：
- **批量查询**：将循环内的单条查询改为 `WHERE IN` 批量获取，或使用 Redis Hash 缓存门店配置。
- **距离计算下沉**：PHP 层循环计算经纬度距离（Haversine）消耗 CPU。建议在 MySQL 中使用空间函数或 `ST_Distance_Sphere`，或在查询时直接返回排序结果。
- **代码规范**：
- 统一方法命名为 `snake_case`。
- 将 `public` 属性改为 `protected`，避免外部随意篡改。
- 提取魔法数字为类常量（如 `const CACHE_TTL = 86400;`）。
- **缓存策略升级**：建议弃用自定义 `get_aliyun_redis_conn()`，改用 CI 官方 `Cache` 驱动（`$this->load->driver('cache', ['adapter' => 'redis'])`），以获得更好的连接池管理与序列化支持。
> 💡 **提示**：若需对 `get_shop_list_order_by_distance` 中的 `FIELD()` 排序进行深度优化，可考虑在数据库层面使用临时表或 `CASE WHEN` 语句，避免在 PHP 中动态拼接超长 SQL。如有特定业务约束无法修改，请确保所有传入参数经过 `intval()` 或白名单过滤。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779168010
|
1779168010
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
181
|
22
|
9
|
1
|
|
0
|
🔍 代码审查报告：app-260519 - 1
|
## 自动代码审查报告
**分支**: app-260519
**提交**: `fc28a1961 ## 自动代码审查报告
**分支**: app-260519
**提交**: `fc28a1961ee774bff6f5ca865687e04f6002fe8f`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-19 13:34:18
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：该 Model 承载了核心支付流水、退款、报表统计等关键业务逻辑，功能完整但架构耦合度较高。代码中存在明显的 SQL 注入风险、N+1 查询性能瓶颈、事务与外部 API 调用顺序导致的财务一致性隐患，且部分方法严重违反单一职责原则。整体可维护性与安全性亟待提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `update_refund_amount` / `refund` | **SQL 注入风险**：使用字符串拼接构造 `WHERE` 与 `UPDATE` 语句（如 `'_relation_id="' . $relation_id . '"...'`），未进行参数绑定或转义，若参数可控将导致严重数据泄露或篡改。 | 全面改用框架查询构造器（Query Builder）或数组条件传参，杜绝原生字符串拼接。 | `$this->db->where('_relation_id', $relation_id)->where('_type', $type)->where_in('_status', [1,4])->get()->row();`<br>`$this->db->set('_refund_amount', '_refund_amount+' . floatval($refund_amount), FALSE)->update($this->table_name, ['_id' => $pay_id]);` |
| 🔴 严重 | `refund` 方法末尾 | **事务与外部 API 顺序导致财务不一致**：`$this->db->trans_commit()` 执行后才调用 `mobile_refund()`。若支付网关退款失败，数据库已标记退款成功，造成资金账实不符。 | 引入异步消息队列处理网关退款；或改为同步调用并包裹 `try-catch`，失败时触发补偿逻辑/人工工单。 | `// 建议改为异步任务<br>$this->load->library('queue');<br>$this->queue->push('refund_gateway', ['pay_id' => $pay_id, 'amount' => $refund_amount]);` |
| 🟠 警告 | `get_bill_pay_log` / `get_refundable_pay_log` | **N+1 查询性能瓶颈**：在 `foreach` 循环内多次调用 `get_one()` 及 `get_custom_pay_platform()`，数据量超 50 条时将引发数百次 DB 请求，严重拖慢响应。 | 循环前批量查询，使用 `where_in` 获取数据并构建映射数组，循环内直接读取。 | `$ids = array_column($log_data, 'relation_id');<br>$extensions = $this->ahead_yc_order_extension_model->get_list(['where_in' => ['_order_id', $ids]]);<br>$extMap = array_column($extensions, null, '_order_id');<br>// 循环内直接 $extMap[$v['relation_id']] ?? null` |
| 🟠 警告 | `mobile_refund` | **硬编码敏感配置**：MD5 签名盐值 `'1441600902'`、`'2017040606573534'` 直接写死在代码中，违反安全规范且不利于多环境部署。 | 移至配置文件（如 `config/payment.php`），通过 `$this->config->item()` 读取。 | `$salt = $this->config->item('wx_refund_salt');<br>$data['refund_key'] = md5($order_info['_id'] . $order_info['_trade_no'] . $salt);` |
| 🟠 警告 | 文件顶部 | **全局实例化框架对象**：`$CI = &get_instance();` 在类外部全局调用。若文件在框架未完全初始化时被 `include`，将触发致命错误。 | 移除全局调用。依赖 CI 的自动加载机制，或在 `__construct()` 中按需加载。 | `// 删除顶部两行<br>public function __construct() {<br> parent::__construct();<br> $this->load->model('Simple_model');<br>}` |
| 🟠 警告 | `update_after_pay_call_back` | **回调逻辑缺乏事务保护**：连续更新多个关联模型（营收、用户消费、地图数据、销售日志），任一失败将导致数据不一致。 | 包裹在 `$this->db->trans_begin()` 中，或改为发布事件/消息队列异步解耦。 | `$this->db->trans_begin();<br>try { /* 更新逻辑 */ $this->db->trans_commit(); } catch(\Exception $e) { $this->db->trans_rollback(); throw $e; }` |
| 🟡 建议 | 全局方法命名 | **违反 PSR-12 命名规范**：方法名使用蛇形命名（`snake_case`），不符合现代 PHP 标准，降低 IDE 提示与团队协作效率。 | 统一改为驼峰命名（`camelCase`），如 `addOrderPayLog`、`updateAfterPayCallback`。 | `public function addOrderPayLog($merchantId, $adminUid, $type, $relationId, $params, $multPayIndexArr = [])` |
| 🟡 建议 | `refund` 方法 | **严重违反单一职责原则 (SRP)**：方法长达 300+ 行，混合了事务控制、VIP 余额计算、日志落库、网关调用、打印通知、跨日处理等逻辑。 | 拆分为独立私有方法或提取为 `RefundService` 服务类，主方法仅负责流程编排。 | `private function processVipRefund($payData, $amount) { ... }`<br>`private function callPaymentGateway($payData, $amount) { ... }` |
| 🟡 建议 | `add_by_vip_recharge_order` | **代码截断导致审查受限**：文件末尾 `$this-` 处被截断，无法评估完整逻辑。 | 请补充完整代码以便进行闭环审查。 | *(待补充)* |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即修复 SQL 注入漏洞**：`update_refund_amount` 与 `refund` 中的字符串拼接是最高危项。请全面替换为框架提供的参数化查询或数组条件构造方式。
2. **解决退款事务与网关调用不一致**：财务类操作必须保证最终一致性。建议将 `mobile_refund` 移至异步队列执行，或在同步调用失败时记录补偿任务并告警。
3. **消除 N+1 查询**：报表与流水列表接口是高频调用点，务必将循环内的单条查询改为批量 `WHERE IN` 查询，预计可提升 60%~80% 的接口响应速度。
### 🛠 后续重构与优化方向
- **架构解耦**：当前 Model 承担了过多业务逻辑。建议引入 **Service 层** 处理退款、报表聚合等复杂流程，Model 仅负责数据持久化。符合 `phpci`/CI 框架的 `MVC` 最佳实践。
- **配置集中化**：将支付平台映射、MD5 盐值、业务类型常量等硬编码内容统一收敛至 `config/` 目录，支持多商户/多环境动态加载。
- **规范与可维护性**：
- 逐步将方法名重构为 `camelCase`，属性名统一为 `protected/private` 并添加类型声明（PHP 7.4+ 支持）。
- 清理历史注释（如 `// 之前不知道为啥被注释掉...`），补充关键业务分支的 PHPDoc 说明。
- 对 `throwError()` 等全局异常函数进行统一封装，建议替换为 `throw new \RuntimeException()` 配合全局异常处理器，便于日志追踪与 API 标准化返回。
> 💡 **框架适配说明**：本代码高度符合 CodeIgniter 3 的语法特征。若 `phpci` 为基于 CI 深度定制的框架，上述 Query Builder 用法、事务控制及配置加载方式通常完全兼容。如遇特定生命周期钩子差异，建议查阅 `phpci` 官方文档中关于 `Database` 与 `Transaction` 的章节进行微调。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779168858
|
1779168858
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
247
|
23
|
9
|
1
|
|
0
|
🔍 代码审查报告：admin-260519 - 1
|
## 自动代码审查报告
**分支**: admin-260519
**提交**: `54fa716 ## 自动代码审查报告
**分支**: admin-260519
**提交**: `54fa716f5c2bf0244ff4d953b8bda6d527b1f008`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-20 17:40:48
---
基于您提供的代码结构与内容，该文件明显遵循 **CodeIgniter 3.x** 的架构规范（注：`phpci` 应为 `CodeIgniter` 的笔误）。以下是对 `Ahead_songs_sales_pay_log_model.php` 的深度审查报告。
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码完整实现了支付流水查询与复杂的退款业务链路，功能覆盖全面。但存在明显的 **SQL 注入隐患**、**事务控制不规范**、**循环内查库（N+1）** 以及 **方法职责过重** 等问题。部分逻辑直接拼接原生 SQL，绕过了框架的安全转义机制，且硬编码密钥降低了系统可维护性。
- **风险等级**：🔴 高（主要源于安全漏洞、事务状态不一致风险及核心业务逻辑耦合度过高）
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_list` ~L48 | **SQL 注入风险**：`$where['where']` 直接拼接 `$title` 变量。若 `Simple_model` 未做底层转义，恶意输入可破坏 SQL 结构或窃取数据。 | 使用参数绑定或框架查询构造器。若必须使用原生条件，需强制转义。 | `$safe_title = $this->db->escape_like_str($title);`<br>`$where['where'] = ['(log._title LIKE "%'.$safe_title.'%" OR book._group_package_name LIKE "%'.$safe_title.'%" OR book._group_platform_name LIKE "%'.$safe_title.'%")'];` |
| 🔴 严重 | `refund` ~L115, L138, L152, L171, L185, L201, L221, L248, L268 | **事务回滚机制缺陷**：多次调用 `$this->db->trans_rollback()` 后直接 `throwError()` 终止执行，导致末尾的 `$this->db->trans_complete()` 永远不会被调用。在 CI3 中，这可能导致数据库连接锁未释放或事务状态残留。 | 移除手动 `trans_rollback()`，统一通过 `$this->db->trans_status(false)` 标记失败，或使用 `try-catch-finally` 确保 `trans_complete()` 必执行。 | `try { $this->db->trans_start(); /* 业务逻辑 */ if ($fail) $this->db->trans_status(false); } catch(\Throwable $e) { $this->db->trans_status(false); throw $e; } finally { $this->db->trans_complete(); }` |
| 🔴 严重 | `refund` ~L236 | **硬编码敏感密钥**：`md5(... . '1441600902')` 将业务密钥写死在代码中，违反安全规范，且多环境部署时极易引发事故。 | 将密钥迁移至 `application/config/config.php` 或环境变量，通过配置读取。 | `$secret = $this->config->item('wx_refund_sign_key');`<br>`$refund_data['refund_key'] = md5($log_info['_id'] . $log_info['_trade_no'] . $secret);` |
| 🟠 警告 | `get_list` ~L85-L95 | **N+1 查询性能瓶颈**：在 `foreach` 循环中调用 `$this->ahead_user_reward_model->get_reward_name()`，数据量增大时将产生大量冗余数据库请求。 | 批量预加载奖励名称，或在主查询中 `LEFT JOIN` 奖励表，循环内仅做数组映射。 | `$reward_ids = array_filter(array_column($data, 'reward_id'));`<br>`$reward_map = $this->ahead_user_reward_model->get_names_map($reward_ids);`<br>`$v['reward_name'] = $reward_map[$v['reward_id']] ?? '';` |
| 🟠 警告 | `refund` ~L118, L135, L149, L168, L182, L198, L218, L245, L265 | **原始 SQL 拼接更新**：多处使用字符串拼接构造 `UPDATE` 的 `SET` 子句（如 `'_order_refund_num=_order_refund_num+1...'`），绕过框架自动转义，易引发语法错误或注入。 | 改用 CI 查询构造器数组传参，或确保自定义 `up()` 方法内部使用 `set()` 安全处理。 | `$this->ahead_open_room_log_model->up([`<br>` '_order_refund_num' => '_order_refund_num+1',`<br>` '_order_refund_amount' => '_order_refund_amount+' . floatval($pay_log['_actual_pay'])`<br>`], $open_where);` |
| 🟠 警告 | `get_list` ~L28-L29 | **边界条件处理缺失**：`strtotime('')` 返回 `false`，直接传入 SQL 会导致 `WHERE col >= false`（即 `>= 0`），可能引发全表扫描或逻辑错误。 | 增加时间格式校验，空值时跳过该条件或赋予合理默认值。 | `if (!empty($params['start_time'])) { $start_time = strtotime($params['start_time']); if ($start_time === false) throwError('开始时间格式无效'); }` |
| 🟡 建议 | 全局 | **方法过长违反单一职责**：`refund` 方法超 300 行，混合了参数校验、多表状态更新、第三方 API 调用、日志记录等，可读性与可测试性极差。 | 拆分为独立私有方法：`validateRefund()`, `processOrderRefund()`, `processVipRefund()`, `callWxRefund()` 等，主方法仅负责流程编排。 | `private function processOrderRefund($pay_log, $remark) { ... }`<br>`private function callWxRefund($log_info, $remark) { ... }` |
| 🟡 建议 | `get_list` ~L10 & `refund` ~L285 | **冗余的 `$CI` 实例获取**：模型顶部 `$CI = &get_instance();` 非必需，且末尾再次赋值。CI 模型中可直接使用 `$this->config`、`$this->load` 等。 | 删除全局 `$CI` 赋值，直接使用框架内置属性。若需访问控制器上下文，应通过参数传递。 | 直接移除 `$CI = &get_instance();`，使用 `$this->config->item('order_pay_platform')` 替代。 |
| 🟡 建议 | 全局 | **现代 PHP 规范缺失**：类名使用蛇形命名，方法无类型声明，不符合 PSR-12 及 PHP 7.4+ 最佳实践。 | 若项目允许，添加类型提示并遵循驼峰命名规范，提升 IDE 提示与静态分析能力。 | `public function get_list(array $params, bool $export = false): array { ... }` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复 SQL 注入漏洞**：立即替换 `get_list` 中 `$title` 的拼接逻辑，并全面审查 `refund` 中所有字符串拼接的 `UPDATE/SET` 语句，确保全部走参数绑定或框架构造器。
2. **重构事务控制逻辑**：废弃 `trans_rollback() + throwError()` 的粗暴中断模式。采用 `try-catch-finally` 或 CI3 标准的 `$this->db->trans_status(false)` 机制，确保事务状态闭环，防止数据库连接池泄漏。
3. **消除硬编码密钥**：将 `1441600902` 等敏感标识移至配置文件，并考虑使用 `hash_hmac` 替代裸 `md5` 提升签名安全性。
### 🛠 后续重构与优化方向
- **架构解耦**：将 `refund` 拆分为 **策略模式** 或 **服务层（Service Layer）**。支付流水模型仅负责数据持久化，退款业务逻辑（如微信退款、团购撤销、会员积分回滚）应下沉至独立的 `RefundService` 类中，便于单元测试与后续扩展。
- **性能优化**：针对 `get_list` 的 N+1 查询，建议采用 `IN` 查询批量预加载关联数据，或在数据库层面建立合理的联合索引（如 `(_shop_id, _create_time, _status)`）。
- **框架适配升级**：若项目计划向 CI4 或现代 PHP 迁移，建议逐步替换 `Simple_model` 中的原生 SQL 封装，全面转向 Query Builder 或 ORM，并引入 PHPStan/Psalm 进行静态代码分析。
> 💡 **注**：由于未提供 `Simple_model` 及 `throwError()` 的具体实现，部分关于底层转义与异常处理的建议基于 CodeIgniter 3 标准实践。若 `Simple_model` 已内置安全过滤，请忽略对应警告，但仍建议统一使用框架原生构造器以提升可维护性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779270048
|
1779270048
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
291
|
6
|
9
|
5
|
|
0
|
治理上线 ②④: 恢复 CI 硬门禁(豁免感知) + 文档对账机制 + Phase2 单测全绿
|
## 模块治理摘要
- 模块：
- 战区：
- lane：
- 阶段：
- 分类：`模 ## 模块治理摘要
- 模块：
- 战区：
- lane：
- 阶段：
- 分类：`模块专项通过 / 工作区漂移 / 阻断`
- 结论：
- 战役卡：
- 验收卡：
- runner evidence bundle：
## 验证命令
| 命令 | 结果 | 备注 |
| --- | --- | --- |
| | | |
| | | |
| | | |
## 审计命中
- `audit:tenant`：
- `audit:events`：
- `audit:route-contract`：
- `audit:docs-single-source`：
- 如未跑全量，请说明原因：
## 前端 / 页面验收
- canonical 入口：
- smoke / auth-scope：
- alias / compat 回归：
## 残余风险
- 风险 1：
- 风险 2：
## 文档同步
- [ ] `CLAUDE.md`
- [ ] `AGENTS.md`
- [ ] `README.md`
- [ ] `GEMINI.md`
- [ ] 治理经验库 / 专项记录
## 口径确认
- [ ] 本 PR 只宣称模块级通过，不把模块结果外推为工作区全绿
- [ ] 若存在工作区漂移，已明确标记且未误记到当前模块
- [ ] 若存在热修 lane，已与全域补证据 / build-out lane 隔离
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1779504052
|
1779504064
|
1779504064
|
0
|
0
|
0
|
Edit
Delete
|
|
815
|
124
|
9
|
5
|
|
0
|
test: move remaining hardcoded booking fixtures on test: move remaining hardcoded booking fixtures onto future dates...
|
The CRM04/BRD04 20:30 failures were already fixed The CRM04/BRD04 20:30 failures were already fixed on main (3e41516,
4941806). Running the self-contained suites under a shifted clock found
two more booking fixtures that would start failing once their date passed:
- composition-fixtures.mjs booked 2026-10-20, so all seven PER07 tests
fail from 20:30 Asia/Shanghai that day.
- prototype-enhancement.persistence.test.mjs defaulted to 2026-10-12 and
used 10-13/14/15 for its other bookings.
Both now go through futureBookingDate(), which keeps weekdays and distinct
dates while staying ahead of the server day.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791559271
|
1791559275
|
1791559275
|
0
|
0
|
0
|
Edit
Delete
|
|
852
|
83
|
9
|
5
|
|
0
|
fix(api): D-10 默认只监听 127.0.0.1，对外暴露须显式设 HOST
|
> 状态说明：本 PR 已于 2026-10-27 合入 `main`（合并提交 `7835e > 状态说明：本 PR 已于 2026-10-27 合入 `main`（合并提交 `7835e3b`），合并时 CI 检查尚未运行；远端 CI 结果以本 PR 的检查为准。
## 概要
执行蓝图 D-10（关闭 H22）：HR API 默认只监听 `127.0.0.1`，需要容器或局域网访问时必须显式设 `HOST`（如 `0.0.0.0`，双栈用 `::`）。改动前后都实测了 04 runtime 容器的可达性。记录见 `docs/domain/enterprise-control-plane-integration-blueprint-2026-10-27.md` §4.7。
## 改动
- **监听地址单源** `apps/api-nestjs/src/listen-host.ts`：未设或空 `HOST` → `127.0.0.1`，否则用显式值。`main.ts` 只经 `resolveListenHost()` 取值，启动日志写明是否只在回环上监听。
- **Web 默认目标**：`next.config.ts` 代理与 `useRealtime` 的 WS 默认值从 `localhost:3001` 改为 `127.0.0.1:3001`，避免 `localhost` 先解析到 `::1` 时连不上只听 IPv4 回环的 API；`.claude/launch.json` 同步。
- **防回潮**：`check:dual-backend` 新增 `scripts/lib/listen-host-guard.mjs`（默认值必须是回环，`main.ts` 不得绕过解析函数），node 单测 9 个负例；把默认值改回 `0.0.0.0` 的变异探针判红 2 处。行为单测 `apps/api-nestjs/test/listen-host.test.ts` 3 例。
- **文档同步**：CLAUDE.md（常用命令、运行时拓扑、G23 中 H22 移为已关闭）、README、后端规范、Scope 来源接入、蓝图 §4.7 / H22 / D-10、给 04 的答复草稿、框架同步评估 F-7。
## 实测证据
证据文件：`docs/evidence/d10-listen-host-2026-10-27.json`（本机 Docker Desktop；对 04 容器只做只读 GET，未改 04 的容器或配置）。
**改动前**，用宿主上的最小 HTTP 探针：
| 绑定 | 宿主回环 | 本机局域网 IP | 04 容器经 `host.docker.internal` |
| --- | --- | --- | --- |
| `0.0.0.0` | 可达 | 可达 | 可达 |
| `127.0.0.1` | 可达 | 不可达 | 可达（宿主看到来源 `127.0.0.1`） |
结论：改为回环默认后，04 的 Scope 来源探针（`SCOPE_HR_SOURCE_URL` 指向 `host.docker.internal:65281`）仍可达。
**改动后**，用真实构建的 HR API 配本仓一次性 PostgreSQL/Redis（跑完已删除）：
| `HOST` | lsof | 局域网 `/api/health` | 04 容器 `/api/health` |
| --- | --- | --- | --- |
| 未设 | `127.0.0.1` | 不可达 | 200，database/redis up |
| 空串 | `127.0.0.1` | 不可达 | 200，database/redis up |
| `0.0.0.0` | `*` | 200 | 200，database/redis up |
另外，Web 不设 `API_PROXY_TARGET` 时，经 Next 代理访问 `/api/health` 返回 200。
**全量验收**（本地，clean `9c53371`，一次性库，跑完已删，报告无口令命中）：
- `check:runtime` 455/455：比 452 多出的是新增的 3 例；冻结行为、多实例 Outbox、依赖故障三项均通过。
- `check:ui` 46/46。
- `check:governance` 仅剩已知两项回退：O22（OS main 不含 0.19.6）、`osProductReleaseTargetRemotes` 2→1；来源指纹违规 0。
- 远端 CI 以本 PR 的运行为准。
**未覆盖**：
- 未用 Scope 来源真实凭据调用签名查询，只证明网络可达。
- Linux 原生 Docker 经 `host-gateway` 连不到宿主回环，这类部署必须显式设 `HOST`。
- 仓外 65281 实例当前无进程；它的源码 `326ae3e` 不读 `HOST`，恢复前须换到含本改动的提交。
- `apps/web` 的 `next dev` / `next start` 仍按 Next 默认监听全部网卡，不在本 PR 范围。
## 待裁：00 框架 F-7
00 框架 `apps/api-nestjs/src/main.ts:8` 仍是 `process.env.HOST ?? "0.0.0.0"`；`backend-standard.md:7`、`COMPATIBILITY.md:45`、ADR-0010 与迁移手册都把 `0.0.0.0` 写成默认，理由是容器内可达。HR 因此有意与框架不同。本 PR 未改框架仓，请框架 Owner 二选一：
- **(a)** 框架也改为回环默认，容器部署在 Dockerfile / compose 里显式设 `HOST=0.0.0.0`。
- **(b)** 维持 `0.0.0.0`，在 `COMPATIBILITY.md` 登记「消费者可偏离为回环」。
无论选哪个，下一次框架同步或叠加仓 `replace: true`（F-5）都不得把 HR 的回环默认冲回去。详见 `docs/framework-sync-assessment-2026-10-27.md` F-7。
## 另外发现（不在本 PR 范围）
- `apps/api-nestjs/docker-compose.yml` 的开发 PostgreSQL（口令为 `app`）与 Redis 映射到 `0.0.0.0`，属同类暴露。
- `CLAUDE.md` 基线表 runtime 行仍是 10-25 的读数 452：报告指纹包含文档，验收后改文档会使报告失效，留待下次改文档时一并更新。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793111729
|
1793114019
|
1793111736
|
0
|
1
|
0
|
Edit
Delete
|
|
886
|
116
|
9
|
5
|
|
0
|
docs(credential): 补齐M2真实请求键与双侧授权验收前置
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793610828
|
1793610838
|
1793610838
|
0
|
0
|
0
|
Edit
Delete
|
|
10
|
2
|
10
|
1
|
|
0
|
🔍 代码审查报告：main 分支 - 修改测试111
|
## 自动代码审查报告
**分支**: main
**提交**: `252852cf322c50f ## 自动代码审查报告
**分支**: main
**提交**: `252852cf322c50ffd2a193223a08e9e4b567bf78`
**时间**: 2026-03-17 11:15:38
**审查模型**: qwen3.5-plus
---
[2026-03-17 11:13:55] 正在调用 Qwen API 进行代码审查...
[2026-03-17 11:15:38] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260317-111355.md
## 📋 审查摘要
- **变更文件数**: 1
- **严重问题**: 1
- **高危问题**: 3
- **中危问题**: 4
- **建议优化**: 3
## 🐛 发现的问题
### [安全隐患] 越权访问风险 (IDOR)
- **严重程度**: 严重
- **文件**: pc/application/controllers/Bill.php
- **行号**: 135 (getDetail 方法)
- **问题描述**: 在 `getDetail` 方法中，查询账单详情时仅使用了 `bill_no` 和 `unique_key`，未校验当前登录用户的 `merchant_id` 是否拥有该账单的权限。攻击者若获取到其他商户的账单号和唯一键，可越权查看敏感账单信息。
- **修复建议**: 在调用模型查询时，必须传入 `merchant_id` 并在模型层或控制器层进行权限校验。
```php
// 修复示例
$bill_detail = $this->ahead_bill_model->get_detail($bill_no, $unique_key, $merchant_id);
// 或者查询后校验
if (empty($bill_detail) || $bill_detail['_merchant_id'] != $merchant_id) {
$this->error_response('账单不存在或无权访问');
}
```
### [安全隐患] SQL 注入风险
- **严重程度**: 高危
- **文件**: pc/application/controllers/Bill.php
- **行号**: 256, 266 (export 方法)
- **问题描述**: 在 `export` 方法中，`$bill_no` 等参数直接来自 `$_GET`，虽然使用了 `trim`，但在构建查询条件 `$bill_where['a._bill_no like'] = '%' . $bill_no;` 时，直接拼接字符串。如果底层模型 `ahead_bill_model` 未对数组值进行严格的转义处理，存在 SQL 注入风险。
- **修复建议**: 使用框架提供的参数绑定或转义函数，不要直接拼接 SQL 片段。
```php
// 修复建议：确保模型层使用预处理，或在此处转义
$bill_no = $this->db->escape_like_str($bill_no);
$bill_where['a._bill_no like'] = '%' . $bill_no . '%';
```
### [逻辑 BUG] 金额计算错误
- **严重程度**: 高危
- **文件**: pc/application/controllers/Bill.php
- **行号**: 167 (getDetail 方法)
- **问题描述**: `$bill_detail['manager_discounts']` 计算逻辑疑似错误，将 `goods_info` 中的折扣与 `bill_detail` 自身的折扣相加，但变量名重复，可能是想加其他字段，或者是重复计算了自身。
```php
// 当前代码
$bill_detail['manager_discounts'] = number_format($bill_detail['goods_info']['manager_discounts'] + $bill_detail['manager_discounts'], 2, '.', '');
```
- **修复建议**: 确认业务逻辑，如果是累加，确保变量来源正确；如果是赋值，移除加法。
```php
// 修复示例（假设是累加）
$bill_detail['manager_discounts'] = number_format(($bill_detail['goods_info']['manager_discounts'] ?? 0) + ($bill_detail['manager_discounts'] ?? 0), 2, '.', '');
```
### [代码质量] 方法名拼写错误
- **严重程度**: 中危
- **文件**: pc/application/controllers/Bill.php
- **行号**: 76 (getList 方法)
- **问题描述**: 调用模型方法 `get_clean_datedd` 多了一个 'd'，而在 `export` 方法（行 242）中调用的是 `get_clean_date`。这会导致 `getList` 方法报错或调用不存在的方法。
- **修复建议**: 统一修正为正确的方法名。
```php
// 修复
$start_date = $this->ahead_setting_clean_data_model->get_clean_date($merchant_id, 2, $start_date);
```
### [代码质量] 响应格式不一致
- **严重程度**: 中危
- **文件**: pc/application/controllers/Bill.php
- **行号**: 232, 236, 240 (export 方法)
- **问题描述**: `export` 方法在错误处理时直接使用 `exit()` 输出字符串，而 `getList` 等其他方法使用 `$this->error_response()` 返回标准 JSON 格式。这会导致前端处理逻辑不一致，且 `exit` 可能绕过框架的日志记录或后续清理逻辑。
- **修复建议**: 统一使用框架的响应方法。
```php
// 修复
if (empty($merchant_id)) {
$this->error_response('账号异常，请重新登录');
return;
}
```
### [安全隐患] 输入来源不一致
- **严重程度**: 中危
- **文件**: pc/application/controllers/Bill.php
- **行号**: 222 (export 方法)
- **问题描述**: `export` 方法直接使用 `$_GET` 获取参数，而 `getList` 使用 `$this->param`。`$this->param` 通常在父类 `PcServer` 中经过统一 sanitization 或合并处理，直接使用 `$_GET` 可能绕过安全过滤。
- **修复建议**: 统一使用 `$this->param` 获取输入参数。
```php
// 修复
$param = $this->param;
```
### [语法/规范] 多余的分号
- **严重程度**: 低危
- **文件**: pc/application/controllers/Bill.php
- **行号**: 191 (printInvoice 方法)
- **问题描述**: 行尾出现双分号 `;;`。
- **修复建议**: 删除多余分号。
```php
// 修复
$invoice_amount = is_numeric($param['invoice_amount']) ? floatval($param['invoice_amount']) : $this->error_response('请输入正确的发票金额');
```
### [代码质量] 代码重复
- **严重程度**: 低危
- **文件**: pc/application/controllers/Bill.php
- **行号**: 45-115 与 222-290
- **问题描述**: `getList` 和 `export` 方法中构建查询条件 (`$bill_where`, `$bill_count_where`) 的逻辑高度重复。
- **修复建议**: 提取公共逻辑到私有方法 `_build_bill_query_conditions()`，减少维护成本。
## ✅ 代码亮点
1. **入口安全**: 文件开头使用了 `defined('BASEPATH') or exit(...)` 防止直接访问，符合 CodeIgniter 安全规范。
2. **辅助函数加载**: 在需要使用 `isDateDayTime` 前正确加载了 `check` helper。
3. **权限意识**: 大部分查询都考虑了 `merchant_id` 和 `priv_shop_ids` 的权限隔离（除了 `getDetail` 的疏漏）。
## 📝 总体建议
1. **安全优先**: 立即修复 `getDetail` 的越权漏洞和 `export` 的 SQL 注入风险。这是最严重的问题。
2. **统一规范**: 统一控制器中的参数获取方式（建议全部使用 `$this->param`）和错误响应方式（全部使用 `$this->error_response`）。
3. **重构优化**: `Bill` 控制器过于臃肿，建议将查询构建逻辑、数据格式化逻辑抽取到 Model 或 Service 层，控制器只负责流程调度。
4. **事务管理**: 在 `printInvoice` 方法中，更新账单表和插入发票记录表应该放在数据库事务中，保证数据一致性。
5. **测试覆盖**: 修复 `get_clean_datedd` 拼写错误后，需回归测试 `getList` 接口，确保日期过滤功能正常。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773717339
|
1773717339
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
37
|
18
|
10
|
1
|
|
0
|
🔍 代码审查报告：pc-260331 - 测试1
|
## 自动代码审查报告
**分支**: pc-260331
**提交**: `2474bdff2b ## 自动代码审查报告
**分支**: pc-260331
**提交**: `2474bdff2b5538fc38dbbdf768ff9283785ce682`
**时间**: 2026-03-28 12:39:06
---
## 📋 审查摘要
- **变更文件数**: 1
- **严重问题**: 0
- **高危问题**: 5
- **中危问题**: 2
- **建议优化**: 1
## 🐛 发现的问题
### <font color="red">[跨文件调用] 父类 PcServer 未在项目中定义</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: application/controllers/AdMachine.php
- **行号**: 7
- **问题描述**: 类 `AdMachine` 继承自 `PcServer`，且使用 `include` 手动加载 `PcServer.php`。在提供的「项目结构」文件中，不存在 `application/controllers/PcServer.php`。如果该文件不存在，将导致致命错误（Class 'PcServer' not found）。此外，在 CodeIgniter 中手动 `include` 控制器文件不符合框架规范，通常应通过 `core/MY_Controller.php` 扩展基类。
- **修复建议**:
1. 确认 `PcServer.php` 是否存在于 `application/controllers/` 目录下。
2. 建议将公共逻辑移至 `application/core/MY_Controller.php`，并让 `AdMachine` 继承 `MY_Controller`。
3. 移除手动的 `include` 语句，依赖框架自动加载。
### <font color="red">[跨文件调用] 模型 Ahead_shop_ad_machine_set_model 未找到</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: application/controllers/AdMachine.php
- **行号**: 16, 29
- **问题描述**: 代码中加载了模型 `Ahead_shop_ad_machine_set_model` (`$this->load->model('Ahead_shop_ad_machine_set_model')`)。在提供的「项目结构」中，未包含 `application/models/` 目录及该模型文件。如果模型文件不存在或命名不匹配（如大小写、下划线），将导致运行时错误。
- **修复建议**:
1. 确认 `application/models/Ahead_shop_ad_machine_set_model.php` 文件存在。
2. 确保模型类名与文件名一致（CodeIgniter 3 通常要求类名为 `Ahead_shop_ad_machine_set_model`，文件名为 `Ahead_shop_ad_machine_set_model.php`）。
### <font color="red">[跨文件调用] 调用了未定义的方法和属性</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: application/controllers/AdMachine.php
- **行号**: 17, 18, 30, 31
- **问题描述**: 代码中使用了 `$this->merchant_id`, `$this->param`, `$this->stream`, `$this->success_response()`。这些属性和方法未在 `AdMachine` 类中定义，依赖于父类 `PcServer`。由于 `PcServer` 类未被验证存在（见问题 1），这些成员变量和方法视为未定义。如果父类中不存在这些定义，将导致 "Undefined property" 或 "Call to undefined method" 错误。
- **修复建议**:
1. 检查父类 `PcServer` 是否确实定义了这些属性和方法。
2. 建议在代码中添加 `isset` 检查或在使用前初始化这些属性，例如：`if (!isset($this->param)) { $this->param = $this->input->post(); }`。
### [安全隐患] 潜在的 SQL 注入风险
- **严重程度**: 高危
- **文件**: application/controllers/AdMachine.php
- **行号**: 17, 30
- **问题描述**: 控制器将 `$this->param` 直接传递给模型的 `get_data` 和 `set_data` 方法。如果模型内部直接使用这些参数拼接 SQL 语句而没有使用查询绑定（Query Binding）或转义，将导致 SQL 注入漏洞。`$this->param` 的来源不明（可能是用户输入）。
- **修复建议**:
1. 审查 `Ahead_shop_ad_machine_set_model` 的实现，确保使用 `$this->db->query($sql, $bindings)` 或 Active Record 模式。
2. 在控制器层对 `$this->param` 进行验证和过滤，不要信任来自客户端的数据。
### [安全隐患] 潜在的越权访问 (IDOR) 风险
- **严重程度**: 高危
- **文件**: application/controllers/AdMachine.php
- **行号**: 17, 30
- **问题描述**: 操作依赖于 `$this->merchant_id`。如果该 ID 来自用户输入（如 URL 参数或 POST 数据）且未与当前登录会话进行校验，攻击者可以篡改 `merchant_id` 来访问或修改其他商户的数据。
- **修复建议**:
1. 确保 `$this->merchant_id` 从安全的会话（Session）中获取，而不是用户输入。
2. 在执行数据库操作前，验证当前用户是否有权操作该 `merchant_id` 对应的数据。
### [代码质量] 模型加载位置不当
- **严重程度**: 中危
- **文件**: application/controllers/AdMachine.php
- **行号**: 16, 29
- **问题描述**: 在每个方法（`getScreensaver`, `setScreensaver`）内部都调用了 `$this->load->model()`。在 CodeIgniter 中，模型应在构造函数 `__construct()` 中加载一次，或在 `config/autoload.php` 中自动加载。重复加载虽然框架有保护机制，但影响代码整洁性和微性能。
- **修复建议**:
将模型加载移至构造函数：
```php
public function __construct()
{
parent::__construct();
$this->load->model('Ahead_shop_ad_machine_set_model');
}
```
### [代码质量] 控制器继承架构不规范
- **严重程度**: 中危
- **文件**: application/controllers/AdMachine.php
- **行号**: 5, 7
- **问题描述**: `AdMachine` 继承自另一个控制器 `PcServer`。在 CodeIgniter 最佳实践中，控制器应继承自 `CI_Controller` 或自定义的核心控制器 `MY_Controller`。控制器之间相互继承会导致耦合度高，生命周期管理混乱（如构造函数执行顺序）。
- **修复建议**:
重构 `PcServer` 中的公共逻辑，将其移至 `application/core/MY_Controller.php` 或一个独立的 Library/Helper 中，让 `AdMachine` 直接继承 `MY_Controller`。
## ✅ 代码亮点
- 使用了 `DIRECTORY_SEPARATOR` 来构建文件路径，增强了跨平台兼容性。
- 包含了 `defined('BASEPATH') OR exit('No direct script access allowed');` 安全检查，防止文件被直接访问。
- 方法命名清晰（`getScreensaver`, `setScreensaver`），符合 RESTful 风格的操作意图。
## 📝 总体建议
该代码片段存在严重的**跨文件依赖风险**，因为所依赖的父类 `PcServer` 和模型 `Ahead_shop_ad_machine_set_model` 在提供的项目结构中均未找到。这可能导致代码在生产环境中直接崩溃。
**优先修复项：**
1. **确认依赖文件存在**：必须确保 `PcServer.php` 和 `Ahead_shop_ad_machine_set_model.php` 存在于正确的位置。
2. **重构继承关系**：避免控制器继承控制器，改用 `MY_Controller` 模式。
3. **安全加固**：重点审查 `$this->param` 和 `$this->merchant_id` 的数据来源和处理逻辑，防止 SQL 注入和越权访问。
4. **代码规范**：将模型加载移至构造函数，遵循 CodeIgniter 标准开发规范。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774672746
|
1774672746
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
178
|
21
|
10
|
1
|
|
0
|
🔍 代码审查报告：pay-260519 - 1
|
## 自动代码审查报告
**分支**: pay-260519
**提交**: `4de14a2cf ## 自动代码审查报告
**分支**: pay-260519
**提交**: `4de14a2cff5bc6855ae3992c46f2b1b3e911c7bc`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-19 13:21:45
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码整体实现了门店查询、缓存、距离排序及多端适配等核心业务，但存在明显的**安全漏洞**（SQL注入）、**线上阻断风险**（遗留调试代码）及**性能瓶颈**（N+1查询、PHP层全量排序）。代码结构高度耦合，重复逻辑较多，未充分利用框架的查询构造器与缓存机制，可维护性与扩展性有待提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_shop_list_order_by_distance` (~L560) | **SQL 注入漏洞**：`$shop_name` 未经过滤直接拼接到 SQL 字符串中，攻击者可构造恶意输入破坏查询或窃取数据。 | 使用框架查询构造器或参数绑定，禁止字符串拼接。 | `$this->db->like('_name', $shop_name, 'both');` |
| 🔴 严重 | `get_shop_list_order_by_distance` (~L585) | **线上阻断风险**：遗留 `echo $this->db->last_query(); exit;`，一旦部署将直接中断所有请求。 | 立即删除该行，调试信息应通过日志记录而非直接输出并终止。 | 删除 `echo...exit;` 代码块 |
| 🔴 严重 | 文件顶部 (~L4) | **全局状态污染**：在类外部执行 `$CI = &get_instance();` 并加载模型，违反框架生命周期，易导致依赖冲突或内存泄漏。 | 移除文件顶部代码，依赖加载统一移至 `__construct()` 中。 | 删除顶部 `$CI = &get_instance();` 相关代码 |
| 🟠 警告 | `get_community_shop_list` 等多处 | **N+1 查询性能瓶颈**：在 `foreach` 循环中逐行调用 `get_miniprogram_consumption_methods()`，门店数量多时将产生海量数据库请求。 | 收集所有 `shop_id` 后使用 `where_in` 批量查询，或在主 SQL 中 `JOIN` 关联配置表一次性获取。 | `$ids = array_column($shop_data, 'shop_id'); $configs = $this->config_model->get_batch($ids);` |
| 🟠 警告 | `get_id_by_distance` (~L230) | **内存与计算瓶颈**：全量拉取门店数据后在 PHP 中循环计算距离并排序，数据量稍大即触发 OOM 或超时。 | 将距离计算下推至数据库层（MySQL 空间函数或 `ORDER BY` 近似公式），或使用 Redis GEO。 | `ORDER BY (latitude - ?)^2 + (longitude - ?)^2 ASC` |
| 🟠 警告 | `get_community_shop_list` (~L430) | **迭代安全隐患**：在 `foreach` 中直接使用 `unset($shop_data[$k])` 过滤数据，可能导致数组键断裂或后续逻辑索引错乱。 | 使用 `array_filter` 进行安全过滤，或先收集需剔除的 ID 再统一处理。 | `$shop_data = array_values(array_filter($shop_data, fn($v) => !empty($v['book_operational_scene'])));` |
| 🟡 建议 | 全文多处 | **违反 DRY 原则**：坐标转换、地址拼接、运营场景解析、营业时间格式化等逻辑在 5+ 个方法中重复编写。 | 抽取为私有辅助方法（如 `private function formatShopDisplayData(array $shop, string $from)`）。 | 统一封装处理逻辑，主方法仅负责查询与调用格式化。 |
| 🟡 建议 | 全文多处 | **硬编码魔法值**：`$special_city_id = [2, 3, 4, 5, 34, 35]` 在多处重复出现，后期维护极易遗漏。 | 提取为类常量或独立配置文件。 | `const SPECIAL_CITY_IDS = [2, 3, 4, 5, 34, 35];` |
| 🟡 建议 | `get_cache_shop_data` (~L60) | **缓存序列化风险**：`json_encode($shop_data)` 未处理非 UTF-8 字符或资源类型，且 `set` + `expireAt` 为两次网络请求。 | 使用 `JSON_UNESCAPED_UNICODE`，或改用框架缓存驱动/Redis `SETEX` 原子操作。 | `$redis->setex($redis_key, 86400, json_encode($shop_data, JSON_UNESCAPED_UNICODE));` |
| 🟡 建议 | `get_shop_list_order_by_distance` (~L555) | **类型安全缺失**：`$satisfy_shop_ids` 直接 `implode` 拼接，若包含非数字字符串将导致 SQL 语法错误。 | 强制类型转换或过滤，确保仅传入合法 ID。 | `$safe_ids = array_map('intval', array_filter($satisfy_shop_ids, 'is_numeric'));` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题（P0/P1）
1. **立即移除调试代码**：删除 `get_shop_list_order_by_distance` 中的 `echo ... exit;`，避免线上服务雪崩。
2. **修复 SQL 注入**：将 `get_shop_list_order_by_distance` 中的原生 SQL 拼接全面替换为 CI 查询构造器（Query Builder）或预处理语句，特别是 `LIKE` 和 `IN` 条件。
3. **解决 N+1 查询**：对 `get_community_shop_list`、`get_shop_list_order_by_distance` 等列表接口，将循环内的单条查询改为批量查询（`where_in`）或 SQL `JOIN`，预计可降低 70%+ 数据库负载。
### 🛠 后续重构与优化方向
1. **逻辑下沉与 DRY 重构**：
- 创建 `private function processShopForDisplay(array $shopData, string $source = 'web')`，统一处理：坐标转换、距离计算、地址拼接、运营场景映射、营业时间格式化。
- 将重复的 `$this->load->model()` 移至类顶部或构造函数，避免运行时重复加载。
2. **距离排序架构升级**：
- 若门店数据量 > 1000，强烈建议废弃 PHP 层 `get_distance()` 循环。改用 MySQL `ST_Distance_Sphere` 或 Redis `GEODIST` 实现数据库/缓存层排序，性能可提升 10 倍以上。
3. **框架规范适配**：
- 注：当前代码结构高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请确认其是否兼容 CI3 的 `DB_driver` 与 `load` 机制。建议统一使用框架提供的缓存组件（如 `$this->cache->redis->get()`）替代原生 `get_aliyun_redis_conn()`，以便统一连接池管理与异常捕获。
- 为所有公开方法添加 PHPDoc `@return` 类型声明，并逐步引入 PHP 7.4+ 类型提示（如 `public function get_info(array $param): array`）。
4. **安全与健壮性加固**：
- 对 `$param` 输入增加统一过滤层（如 `filter_var` 或框架验证器），避免依赖零散的 `intval()`/`trim()`。
- 敏感操作（如 `add_shop_after`）建议加入数据库事务包裹，确保多表插入的原子性。
> 💡 **局限性说明**：由于未提供 `Simple_model` 基类、全局辅助函数（`locationTurnTxMap`, `get_distance`, `throwError` 等）及数据库表结构，部分业务逻辑的边界条件验证依赖于假设。建议在完整上下文中补充单元测试覆盖核心查询与缓存逻辑。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779168105
|
1779168105
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
191
|
22
|
10
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1779171090
|
1779171196
|
1779171094
|
0
|
0
|
0
|
Edit
Delete
|
|
248
|
23
|
10
|
1
|
|
0
|
🔍 代码审查报告：admin-260519 - 1
|
## 自动代码审查报告
**分支**: admin-260519
**提交**: `11690e6 ## 自动代码审查报告
**分支**: admin-260519
**提交**: `11690e685c519ec01d0f4b7b355c0b441a4afd8a`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-20 17:44:38
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖较全面，退款流程考虑了多场景（预订单、普通单、会员、微信、团购等）。但存在**严重的 SQL 注入风险**、**N+1 查询性能瓶颈**、**框架生命周期误用**及**事务管理不规范**等问题。代码中大量使用魔法数字与硬编码字符串拼接，可维护性与安全性亟待提升。
- **风险等级**：🔴 高
> 📌 **框架说明**：从目录结构、`get_instance()`、`$this->load->model()`、`$this->db->trans_start()` 等特征判断，实际运行框架为 **CodeIgniter 3 (CI3)**，而非 `phpci`（phpci 通常为持续集成服务器）。以下审查基于 CI3 核心机制与 PHP 现代最佳实践进行。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_list` ~L48 | **SQL 注入漏洞**：`$where['where']` 中直接拼接 `$title` 变量到 SQL 字符串，未做任何转义或参数绑定。 | 使用查询构造器或参数化查询。若 `Simple_model` 支持数组条件，应使用占位符或框架安全方法。 | `$this->db->where("(log._title LIKE ? OR book._group_package_name LIKE ? OR book._group_platform_name LIKE ?)", ["%$title%", "%$title%", "%$title%"]);` |
| 🔴 严重 | `refund` ~L145, L168, L185 | **SQL 注入与精度丢失**：`$log_where`、`$open_update`、`log_update` 等直接使用字符串拼接构建 SQL 条件/更新语句。若变量含特殊字符或浮点数精度异常，将导致数据错乱或注入。 | 统一使用 CI3 查询构造器或 `set()` + `where()`，强制类型转换，避免手写 SQL 片段。 | `$this->db->set('_order_refund_num', '_order_refund_num+1', FALSE)<br>->set('_order_refund_amount', '_order_refund_amount+' . (float)$pay_log['_actual_pay'], FALSE)<br>->where($open_where)<br>->update('ahead_open_room_log');` |
| 🔴 严重 | 文件顶部 L2-L3 | **框架生命周期违规**：`$CI = &get_instance();` 在类外部执行。文件被 `include/require` 时即运行，此时 CI 核心可能尚未初始化，极易引发 `Call to undefined function get_instance()` 致命错误。 | 移至 `__construct()` 中，或直接在方法内使用 `$this->load->model()`。CI3 模型加载器已自动处理实例化。 | `public function __construct() {<br> parent::__construct();<br> $this->load->model('Simple_model');<br>}` |
| 🟠 警告 | `refund` ~L95-L100 | **事务管理冗余/冲突**：CI3 开启 `trans_start()` 后，若 `trans_strict` 为 `TRUE`，框架会自动在 `trans_complete()` 时根据状态回滚。手动多次调用 `trans_rollback()` 可能干扰事务状态机，导致后续操作报错。 | 移除手动 `trans_rollback()`，统一在方法末尾通过 `$this->db->trans_status()` 判断，或使用 `try...catch` 包裹。 | `// 移除所有 $this->db->trans_rollback();<br>// 在 trans_complete() 后统一判断：<br>$this->db->trans_complete();<br>if ($this->db->trans_status() === FALSE) { throwError('退款事务失败'); }` |
| 🟠 警告 | `get_list` ~L85 | **N+1 查询性能瓶颈**：`foreach` 循环内调用 `$this->ahead_user_reward_model->get_reward_name()`，数据量大时将产生大量数据库请求。 | 提取所有 `reward_id`，批量查询后构建映射数组，在循环内直接取值。 | `$reward_ids = array_filter(array_column($data, 'reward_id'));<br>$reward_map = $this->ahead_user_reward_model->get_names_by_ids($reward_ids);<br>foreach ($data as &$v) { $v['reward_name'] = $reward_map[$v['reward_id']] ?? ''; }` |
| 🟠 警告 | `get_list` ~L38 | **分页逻辑缺陷**：仅当 `$page == 1` 时计算 `$count`。用户翻页时 `$count` 未定义，导致分页组件总数丢失或报错。 | 分页总数应始终计算，或使用框架分页组件自动处理。 | 移除 `if ($page == 1 || $export)` 条件包裹，始终执行 `$count = $this->count($where);` |
| 🟡 建议 | 全局多处 | **魔法数字泛滥**：大量硬编码状态值（如 `1,2,3,5,7`、`4`、`3`、`8`、`9`、`13`），业务含义不透明，后期维护成本极高。 | 定义类常量或配置文件枚举，提升代码可读性与可维护性。 | `const STATUS_PAID = 1; const STATUS_REFUNDED = 4; const TYPE_BOOKING = 3; const PAY_WECHAT = 1;` |
| 🟡 建议 | `refund` ~L265 | **动态属性污染**：`$CI->add_book_order_operation_log = true;` 直接修改 CI 实例属性，属于全局状态污染，易引发并发冲突或难以追踪的副作用。 | 通过返回值传递状态，或使用独立的日志/事件服务记录操作结果。 | `return ['status' => true, 'log_added' => true, 'book_order' => $book_order];` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **彻底消除 SQL 拼接**：将 `refund` 与 `get_list` 中所有手写 SQL 字符串替换为 CI3 查询构造器（`$this->db->where()`, `$this->db->set()`, `$this->db->update()`）或 `Simple_model` 提供的安全绑定方法。
2. **修复事务控制逻辑**：移除分散的 `trans_rollback()`，依赖 CI3 的自动事务回滚机制，或在关键节点使用 `try { ... } catch (\Exception $e) { $this->db->trans_rollback(); throwError($e->getMessage()); }` 确保异常安全。
3. **解决 N+1 查询**：将 `get_reward_name` 改为批量查询，避免循环内访问数据库。
4. **修正模型加载时机**：将 `$CI = &get_instance();` 及模型加载逻辑移入 `__construct()`，确保符合 CI3 生命周期。
### 🛠 后续重构与优化方向
- **引入常量/枚举管理**：建立 `config/payment_status.php` 或在类顶部定义 `const`，替换所有魔法数字。例如：`if ($pay_log['_status'] == self::STATUS_PAID)`。
- **统一错误处理机制**：当前混用全局 `throwError()` 与 `return`。建议统一抛出 `\RuntimeException` 或返回标准结构 `['code' => 0, 'msg' => '', 'data' => null]`，便于上层控制器统一拦截与 API 响应格式化。
- **数据库索引优化**：针对高频查询字段 `_merchant_id`, `_shop_id`, `_order_id`, `_status`, `_create_time` 建立联合索引，可显著提升 `get_list` 与退款校验的查询效率。
- **并发安全考量**：退款涉及余额扣减与状态变更，建议在核心更新语句中加入乐观锁（如 `WHERE _version = ?`）或使用 `SELECT ... FOR UPDATE` 防止并发退款导致的数据不一致。
- **框架规范对齐**：若项目计划向现代 PHP 演进，建议逐步将模型类名改为 `PascalCase`（如 `AheadSongsSalesPayLogModel`），并采用 PSR-4 自动加载替代 `$this->load->model()`。
> 💡 **提示**：若 `Simple_model` 为项目自研基类，请确保其内部已实现完整的参数绑定与 SQL 预处理机制。对于不确定的框架特定用法，建议查阅 **CodeIgniter 3 官方文档 - Database Reference** 及 **Query Builder** 章节。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779270278
|
1779270278
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
816
|
124
|
10
|
5
|
|
0
|
test: dedicated evidence for 14 P1/P2 requirements test: dedicated evidence for 14 P1/P2 requirements and strict licence dates...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791559454
|
1791559459
|
1791559459
|
0
|
0
|
0
|
Edit
Delete
|
|
853
|
83
|
10
|
5
|
|
0
|
chore(deps): H23 根与 apps/* 对齐框架 0.27.0 安全基线
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793114286
|
1793115451
|
1793114295
|
0
|
0
|
0
|
Edit
Delete
|
|
888
|
116
|
10
|
5
|
|
0
|
docs(credential): 补齐恢复与生产签收待签材料
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793617972
|
1793617980
|
1793617980
|
0
|
0
|
0
|
Edit
Delete
|
|
11
|
2
|
11
|
1
|
|
0
|
🔍 代码审查报告：main 分支 - 修改测试调用不存在的model
|
## 自动代码审查报告
**分支**: main
**提交**: `82563bcddd8d325 ## 自动代码审查报告
**分支**: main
**提交**: `82563bcddd8d32587bc08757c8fdba4bb728fdbe`
**时间**: 2026-03-17 13:08:18
**审查模型**: qwen3.5-plus
---
[2026-03-17 13:06:57] 正在调用 Qwen API 进行代码审查...
[2026-03-17 13:08:18] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260317-130657.md
基于提供的文件内容和跨文件引用分析，我对 `pc/application/controllers/Bill.php` 进行了代码审查。以下是发现的严重错误、潜在隐患及改进建议：
### 1. 严重错误 (Critical Errors)
**问题描述：** 在 `getDetails` 方法中存在调用不存在模型和方法的代码，这将导致服务器内部错误 (500 Error)。
**位置：** `Bill.php` 第 135 行
**代码：**
```php
$this->non_existent_model->some_method();
```
**影响：** 该模型未在构造函数中加载，且项目中显然不存在该模型文件。执行到此行时程序会崩溃。
**建议：** 立即删除该行代码，除非它是未完成的开发遗留代码且需要替换为正确的逻辑。
### 2. 拼写错误与冗余代码 (Typos & Redundancy)
**问题 A：模型名称拼写错误**
**位置：** `getDetails` 方法第 136-137 行
**代码：**
```php
$this->load->model('ahead_room_changesss_model'); // 多写了 'ss'
$this->load->model('ahead_room_changesss_mode2l'); // 多写了 'ss' 且 'model' 写成了 'mode2l'
```
**分析：** 紧接着第 139 行加载了正确的模型 `$this->load->model('ahead_room_change_model');`。前两行看起来是复制粘贴错误或调试遗留代码。
**建议：** 删除第 136-137 行。
**问题 B：方法名拼写不一致**
**位置：** `getList` 方法第 105 行 vs `export` 方法第 275 行
**代码：**
```php
// getList 中
$start_date = $this->ahead_setting_clean_data_model->get_clean_datedd(...); // 多写了 'dd'
// export 中
$start_date = $this->ahead_setting_clean_data_model->get_clean_date(...); // 正常
```
**分析：** `get_clean_datedd` 极有可能是 `get_clean_date` 的拼写错误。如果模型中不存在 `get_clean_datedd` 方法，`getList` 接口也会报错。
**建议：** 统一修改为 `get_clean_date`，并确认模型文件中该方法的实际名称。
**问题 C：未使用的模型加载**
**位置：** `getList` 方法第 103 行
**代码：**
```php
$this->load->model('ahead_setting_clean_data_models'); // 复数 'models'
```
**分析：** 下一行立即加载了单数版本 `ahead_setting_clean_data_model` 并使用它。第 103 行是冗余的。
**建议：** 删除第 103 行。
### 3. 安全风险 (Security Risks)
**问题 A：SQL 注入风险**
**位置：** `printInvoice` 方法第 192 行
**代码：**
```php
$this->ahead_bill_model->up($up, "_unique_key='" . $unique_key . "'");
```
**分析：** 这里直接拼接 SQL 字符串。虽然 `$unique_key` 来自数据库查询结果 `$bill_info`，相对安全，但最佳实践是使用参数化查询或框架提供的 escape 方法。
**建议：** 使用 CodeIgniter 的 `$this->db->escape()` 或模型层提供的安全更新方法。
**位置：** `getList` 方法第 114-123 行
**代码：**
```php
$bill_where['where'] = ['a._book_admin_type=1 and a._book_admin_id=' . intval($admin_id)];
```
**分析：** 虽然使用了 `intval` 进行强制类型转换，降低了风险，但在 `where` 数组中直接拼接 SQL 片段不符合现代框架的安全规范。
**建议：** 尽量使用框架提供的数组条件格式，例如 `['a._book_admin_type' => 1, 'a._book_admin_id' => $admin_id]`。
### 4. 逻辑一致性 (Logic Consistency)
**问题：** 日期处理逻辑不一致
**位置：** `getList` vs `export`
**分析：**
* `getList` 中默认结束时间是 `time()`。
* `export` 中默认开始时间是 30 天前，结束时间是 `time()`。
* `export` 中增加了 `32 天` 的限制检查，`getList` 中没有。
**建议：** 确保两个接口的日期默认值和限制逻辑保持一致，避免数据导出与页面展示不一致。
### 5. 修正后的代码片段建议
**针对 `getDetails` 方法的修正：**
```php
public function getDetails()
{
$param = $this->param;
// ... (前略)
$unique_key = !empty($param['unique_key']) ? trim($param['unique_key']) : '';
// [删除] $this->non_existent_model->some_method();
// [删除] $this->load->model('ahead_room_changesss_model');
// [删除] $this->load->model('ahead_room_changesss_mode2l');
$bill_detail = $this->ahead_bill_model->get_detail($bill_no, $unique_key);
if (empty($bill_detail)) {
$this->error_response('账单不存在');
}
// 转房信息
$this->load->model('ahead_room_change_model');
$bill_detail['change_room_msg'] = $this->ahead_room_change_model->get_change_room_info($bill_detail['unique_key']);
// ... (后略)
}
```
**针对 `getList` 方法的修正：**
```php
// ...
$this->load->model('ahead_setting_clean_data_model'); // [修正] 删除复数 models 行，统一使用单数
// 设置了截至日期
$start_date = $this->ahead_setting_clean_data_model->get_clean_date($merchant_id, 2, $start_date); // [修正] 去掉多余的 dd
// ...
```
### 总结
该文件存在**阻断性错误**（调用不存在模型），必须优先修复。同时存在多处明显的拼写错误和冗余代码，建议在进行功能测试前进行清理。安全方面建议逐步重构 raw SQL 拼接部分。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773724098
|
1773724098
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
38
|
18
|
11
|
1
|
|
0
|
🔍 代码审查报告：pc-260331 - 测试122
|
## 自动代码审查报告
**分支**: pc-260331
**提交**: `1923fd10d2 ## 自动代码审查报告
**分支**: pc-260331
**提交**: `1923fd10d20a13d25d4753a96667f7373c599738`
**时间**: 2026-03-28 12:50:00
---
## 1. 审查摘要
- **代码质量评分**：2/10
- **总体评价**：代码存在严重的语法错误，导致无法运行。架构模式高度类似 CodeIgniter，但存在不规范的文件包含方式和潜在的拼写错误。缺乏基本的错误处理与安全验证逻辑。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | AppModule.php:14 | **语法错误**：方法体开始处存在非法字符 `s` (`{s`)，会导致 PHP 解析错误 (Parse Error)，服务直接崩溃。 | 删除多余的字符 `s`，确保方法体语法正确。 | `public function getPcModule()`<br>`{`<br>` // 逻辑代码`<br>`}` |
| 🔴 严重 | AppModule.php:15 | **语法错误/拼写错误**：`$this->load->smodel` 疑似拼写错误（标准为 `model`），且行尾有多余字符 `s`。 | 确认框架是否自定义了 `smodel` 方法。若为笔误，应修正为 `model` 并删除行尾 `s`。 | `$this->load->model('ahead_common_config_model');` |
| 🟠 警告 | AppModule.php:4 | **不规范的包含方式**：在控制器内部直接 `include` 父控制器类文件。这可能导致类重复声明错误，且违背框架自动加载机制。 | 建议将 `PcServer` 配置为框架的基类控制器，通过自动加载或入口文件引入，而非在每个子类中手动 include。 | `// 移除 include 语句`<br>`// 确保框架配置中指定 BaseController 为 PcServer` |
| 🟠 警告 | AppModule.php:16 | **缺乏错误处理**：直接调用模型方法并使用返回数据，未检查 `$data` 是否为 null 或 false。 | 增加对模型返回值的判断，若获取失败应返回错误响应，避免后续逻辑处理空数据。 | `if (empty($data)) {`<br>` $this->error_response('Config not found');`<br>` return;`<br>`}` |
| 🟠 警告 | AppModule.php:17 | **潜在安全风险 (XSS/信息泄露)**：直接将数据库配置数据赋值给响应流 `$this->stream`，未进行敏感数据过滤。 | 确保 `success_response()` 内部有数据转义逻辑，或在赋值前过滤敏感字段（如密钥、密码等）。 | `// 确保敏感字段被移除`<br>`$safe_data = $this->filterSensitiveData($data);`<br>`$this->stream['request']['result'] = $safe_data;` |
| 🟡 建议 | AppModule.php:9 | **注释与代码不符**：类注释为 `/** 账单 */`，但类名为 `AppModule` 且方法获取的是 `pc_module_config`。 | 更新类注释以准确反映该控制器的业务功能，避免误导维护者。 | `/**`<br>` * PC 模块配置管理`<br>` */` |
| 🟡 建议 | AppModule.php:13 | **命名规范**：方法名 `getPcModule` 采用驼峰式。若框架路由配置为默认模式，URL 可能需要访问 `/appmodule/get_pc_module`。 | 确认框架路由规则。若遵循 PSR-12 及现代 PHP 习惯，驼峰式可接受，但需确保路由配置一致。 | `// 保持现状，但需确认路由配置`<br>`// 或改为 public function get_pc_module()` |
| 🟡 建议 | AppModule.php:1 | **硬编码路径**：虽然使用了 `FCPATH` 和 `DIRECTORY_SEPARATOR`，但手动拼接路径仍显冗余。 | 若框架支持，建议使用框架提供的自动加载机制或命名空间来管理类依赖。 | `// 依赖框架自动加载机制` |
## 3. 总结与行动建议
### 优先修复的关键问题
1. **立即修复语法错误**：第 14 行和 15 行的多余字符 `s` 必须删除，否则代码无法部署。
2. **确认模型加载方法**：核实 `phpci` 框架是否确实存在 `smodel` 方法。如果是基于 CodeIgniter 的二次开发，通常应为 `$this->load->model()`。
3. **重构基类加载**：移除 `include PcServer.php`。基类控制器应在框架入口文件（如 `index.php`）或核心配置中加载，避免在每个控制器中重复引入。
### 后续重构或优化方向
1. **统一响应处理**：检查 `PcServer` 基类中的 `success_response()` 和 `$this->stream` 结构，确保所有输出数据经过统一的安全过滤（如 XSS 过滤）。
2. **增强健壮性**：在控制器中增加 `try-catch` 块捕获模型层可能抛出的异常，防止数据库错误直接暴露给用户。
3. **权限验证**：确认 `PcServer` 构造函数中是否包含了登录态校验或权限验证逻辑。若未包含，需在此方法或基类中补充。
4. **框架规范对齐**：该代码结构高度类似 CodeIgniter 2/3。建议查阅 `phpci` 官方文档（若为内部框架）或参照 CodeIgniter 最佳实践，确保目录结构、类命名和加载机制符合框架标准。
### 修正后的代码参考
```php
<?php
defined('BASEPATH') or exit('No direct script access allowed');
// 建议：移除手动 include，确保 PcServer 通过框架自动加载
// require_once FCPATH . 'application/controllers/PcServer.php';
/**
* PC 模块配置管理
*/
class AppModule extends PcServer
{
/**
* 获取 PC 模块配置
* @return void
*/
public function getPcModule()
{
// 修正加载方法名
$this->load->model('ahead_common_config_model');
// 获取数据
$data = $this->ahead_common_config_model->get_data('pc_module_config');
// 增加数据有效性检查
if ($data === null) {
$this->error_response('Configuration not found');
return;
}
// 赋值给响应流 (确保基类中已处理安全输出)
$this->stream['request']['result'] = $data;
// 返回成功响应
$this->success_response();
}
}
```
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774673400
|
1774673400
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
179
|
21
|
11
|
1
|
|
0
|
🔍 代码审查报告：pay-260519 - 1
|
## 自动代码审查报告
**分支**: pay-260519
**提交**: `e0309e97c ## 自动代码审查报告
**分支**: pay-260519
**提交**: `e0309e97c19e9542fbbf526e45cc4d0aff8b24e8`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-19 13:22:54
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：该 Model 承载了大量门店业务逻辑，功能覆盖较全，但存在明显的架构与编码规范问题。核心隐患集中在 **SQL 注入风险、N+1 查询性能瓶颈、框架生命周期误用** 以及 **变量拼写错误**。代码整体偏向“脚本式”堆砌，缺乏面向对象设计原则（如单一职责、依赖注入）的约束，后期维护与扩展成本较高。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 全局/第3-4行 | 文件顶部直接调用 `get_instance()` 并加载 `Simple_model`。此代码在文件被 `include/require` 时即执行，而非实例化时，严重违反 CI/PHPCI 框架生命周期，易导致类未定义或单例污染。 | 移除全局调用。模型依赖应通过 CI 自动加载配置或移至 `__construct()` 中。 | `// 删除顶部这两行<br>$CI = &get_instance();<br>$CI->load->model('Simple_model');` |
| 🔴 严重 | `get_shop_list_order_by_distance` (~L750) | 原始 SQL 拼接未对 `$shop_name`、`$satisfy_shop_ids`、`$city_id` 进行转义或参数绑定，存在高危 **SQL 注入** 漏洞。 | 使用 CI 查询构建器（Query Builder）或 `$this->db->escape()` / `escape_like_str()` 进行安全过滤。 | `$safe_name = $this->db->escape_like_str($shop_name);<br>$sql .= " AND _name LIKE '%{$safe_name}%'";` |
| 🔴 严重 | `get_community_shop_list` (~L630) | 变量名拼写错误：`if (!empty($param['bookFrom']))`，实际入参为 `$params`。将导致 `Undefined variable` 警告，且后续 JOIN 逻辑永远不执行。 | 修正为 `$params['bookFrom']`，并建议开启 `error_reporting(E_ALL)` 在开发环境拦截此类错误。 | `if (!empty($params['bookFrom'])) { ... }` |
| 🟠 警告 | `get_community_shop_list` & `get_shop_list_order_by_distance` | **N+1 查询性能瓶颈**：在 `foreach` 循环中逐条调用 `$this->ahead_shop_config_second_model->get_miniprogram_consumption_methods()`。若返回 100 家门店，将产生 100+ 次额外 DB 请求。 | 提取所有 `shop_id` 数组，编写批量查询方法 `get_batch_methods($merchant_id, $shop_ids)`，在循环外一次性获取，再在内存中映射。 | `$ids = array_column($shop_data, 'shop_id');<br>$batch_data = $this->ahead_shop_config_second_model->get_batch_methods($merchant_id, $ids);<br>foreach ($shop_data as &$v) { $config = $batch_data[$v['shop_id']] ?? []; ... }` |
| 🟠 警告 | `get_date_shop_no_business_time` (~L840) | 循环步长依赖外部模型属性 `$this->ahead_shop_book_time_info_model->min_minute_unit_time`。若该值为 `0` 或未初始化，将导致 **死循环** 耗尽 CPU/内存。 | 增加步长安全校验，或提取为类常量。 | `$step = max(1, $this->ahead_shop_book_time_info_model->min_minute_unit_time);<br>for ($i = 0; $i < $business_from; $i += $step) { ... }` |
| 🟠 警告 | 全局属性 `$shop_no_business_time` | 使用 `public $shop_no_business_time = [];` 累积状态。在并发请求或多次调用时会产生 **数据污染**，且破坏方法纯函数特性。 | 改为方法局部变量返回，或封装为独立 DTO/Value Object。 | `// 移除 public 属性<br>public function get_date_shop_no_business_time(...) {<br> $no_business_times = []; // 局部变量<br> // ... 逻辑<br> return ['result' => $result, 'no_business_times' => $no_business_times];<br>}` |
| 🟡 建议 | 全文 | 方法命名风格不统一（如 `getAliAuthInfo` 驼峰 vs `get_list_for_search` 下划线），不符合 PSR-12 规范。 | 统一使用 `camelCase` 命名方法，属性使用 `snake_case`。 | `public function getAliAuthInfo()` → `public function getAliAuthInfo()` (保持) 或 `get_ali_auth_info()` (统一) |
| 🟡 建议 | `get_cache_shop_data` | Redis 过期时间使用 `expireAt` 配合 `time()+86400` 非原子操作。若 `set` 成功但 `expireAt` 失败，将产生永久缓存。 | 使用原子命令 `setex` 或 `set($key, $val, ['ex' => 86400])`。 | `$redis->setex($redis_key, 86400, json_encode($shop_data));` |
| 🟡 建议 | `get_list_by_id` | `explode(",", $shop_id)` 在传入空字符串时会返回 `['']`，导致后续 `WHERE IN ('')` 语法错误或全表扫描。 | 增加空值过滤与类型转换。 | `$shop_id = array_filter(array_map('intval', explode(',', $shop_id)));<br>if (empty($shop_id)) return [];` |
## 3. 总结与行动建议
### 🚨 优先修复的关键问题
1. **立即修复 SQL 注入**：`get_shop_list_order_by_distance` 中的原始 SQL 拼接必须替换为 CI Query Builder 或严格转义。这是最高优先级的安全红线。
2. **修正致命拼写错误**：`get_community_shop_list` 中的 `$param` 必须改为 `$params`，否则业务逻辑将静默失效。
3. **清理全局实例化代码**：删除文件顶部的 `$CI = &get_instance();`，将其移至 `__construct()` 或依赖 CI 的 `autoload.php` 配置。
### 🛠 后续重构与优化方向
1. **性能架构升级**：
- **批量查询替代循环查询**：针对 `get_community_shop_list` 等列表接口，将配置、评论、距离计算等依赖改为 `IN` 批量查询或 Redis Pipeline 获取。
- **距离计算下沉**：PHP 循环计算 `get_distance()` 在数据量大时极慢。建议改用 MySQL 空间函数（如 `ST_Distance_Sphere`）或 Redis `GEO` 模块在数据库/缓存层完成排序。
2. **代码规范与可维护性**：
- **统一命名与类型声明**：全面采用 PSR-12 规范，为所有方法添加 PHP 7.4+ 类型提示（如 `public function get_info(array $param): array`），提升 IDE 提示与静态分析能力。
- **消除魔法数字**：将 `86400`、`1`、`-1`、`5` 等硬编码提取为类常量（如 `const CACHE_TTL = 86400; const STATUS_ACTIVE = 1;`）。
- **异常处理标准化**：当前混用 `throwError()` 和 `showErrorView()`。建议统一使用 PHP 原生 `throw new \RuntimeException()` 或框架标准异常，便于全局异常拦截与日志记录。
3. **框架适配建议**：
- 基于代码结构，该项目高度类似 **CodeIgniter 3**。若确为 `phpci` 定制框架，请确认其是否支持 CI4 的依赖注入容器。若支持，建议逐步将 `$this->load->model()` 替换为构造函数注入，以提升单元测试覆盖率。
- `Simple_model` 作为基类，建议封装统一的 `batchSelect()`、`cache()` 方法，避免各子类重复实现缓存与批量逻辑。
> 💡 **注**：由于未提供 `Simple_model` 基类及全局辅助函数（如 `locationTurnTxMap`, `get_distance`）的实现，部分性能与逻辑评估基于常规实践推断。建议在修复上述高危问题后，结合 Xdebug 或 Blackfire 进行实际压测，验证 N+1 与距离计算优化效果。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779168174
|
1779168174
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
192
|
22
|
11
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1779171106
|
1779171217
|
1779171109
|
0
|
0
|
0
|
Edit
Delete
|
|
256
|
23
|
11
|
1
|
|
0
|
🔍 代码审查报告：admin-260519 - 1
|
## 自动代码审查报告
**分支**: admin-260519
**提交**: `2d4d2b5 ## 自动代码审查报告
**分支**: admin-260519
**提交**: `2d4d2b55bfb87cabc2bc226b8c7fcbf22183ce22`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-20 18:39:49
---
## 1. 审查摘要
- **代码质量评分**：6.0 / 10 分
- **总体评价**：代码完整实现了预购订单查询、导出、退款及支付流水管理等核心业务逻辑，整体结构清晰。但存在**严重的 SQL 注入风险**、**核心退款流程缺失事务控制**、以及多处违反 MVC 分层原则与 CI3/现代 PHP 规范的反模式代码。若直接上线，极易引发数据不一致与安全漏洞。
- **风险等级**：🔴 高
> 📌 **框架说明**：代码结构、语法及调用方式（如 `$CI = &get_instance()`、`$this->load->model()`、`$this->db->trans_start()`）高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请结合其官方文档对底层 ORM/DB 驱动的差异进行适配。以下审查基于 CI3 最佳实践。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_songs_sales_pay_log_model.php`<br>~第 64 行 | **SQL 注入漏洞**：`$title` 参数未经过滤直接拼接到 `LIKE` 语句中，攻击者可构造恶意输入破坏查询或拖库。 | 使用框架查询构建器或参数化查询替代字符串拼接。 | `$this->db->group_start();`<br>`$this->db->like('log._title', $title);`<br>`$this->db->or_like('book._group_package_name', $title);`<br>`$this->db->or_like('book._group_platform_name', $title);`<br>`$this->db->group_end();` |
| 🔴 严重 | `Ahead_preorder_order_model.php`<br>`refund()` 方法<br>(约 133-210 行) | **缺失数据库事务**：退款涉及主订单状态更新、会员余额回滚、多表明细插入、支付流水更新等 10+ 次 DB 操作。若中途失败，将导致资金账目不平、状态不一致。 | 使用 `$this->db->trans_start()` 包裹所有写操作，失败时 `trans_rollback()`。 | `$this->db->trans_start();`<br>`// ... 所有 update/insert 操作 ...`<br>`if ($this->db->trans_status() === false) {`<br>` $this->db->trans_rollback();`<br>` return ['status'=>false, 'msg'=>'退款失败'];`<br>`}`<br>`$this->db->trans_complete();` |
| 🟠 警告 | `Ahead_songs_sales_pay_log_model.php`<br>~第 238 行 | **动态 SQL 拼接风险**：`$log_where` 使用字符串拼接构造 `WHERE` 条件，未做类型校验或转义，存在注入隐患且难以维护。 | 改用数组条件或查询构建器链式调用。 | `$log_where = [`<br>` '_merchant_id' => $pay_log['_merchant_id'],`<br>` '_shop_id' => $pay_log['_shop_id'],`<br>` '_relation_id' => $pay_log['_order_id'],`<br>` '_status' => 1,`<br>` '_type IN' => [1, 5, 13]`<br>`];` |
| 🟠 警告 | `Ahead_preorder_order_model.php`<br>~第 149-152 行 | **JSON 解析未校验**：`json_decode($order['_mult_pay'], true)` 若字段为空或格式错误将返回 `null`，后续 `foreach` 会触发 `Warning: Invalid argument supplied for foreach()`。 | 增加类型判断与默认值处理。 | `$mult_pay = json_decode($order['_mult_pay'], true);`<br>`if (is_array($mult_pay)) {`<br>` foreach ($mult_pay as $pay) { ... }`<br>`}` |
| 🟠 警告 | `Ahead_preorder_order_model.php`<br>~第 100 行 | **模型层直接终止脚本**：`exit('成功');` 破坏了 MVC 架构，导致控制器无法统一封装响应格式（如 JSON/HTTP 状态码），且不利于单元测试。 | 移除 `exit`，由控制器统一处理输出与响应。 | `// 模型中直接 return 结果`<br>`return ['status' => true, 'msg' => '导出成功'];`<br>`// 控制器中处理`<br>`$this->output->set_content_type('application/json')->set_output(json_encode($res));` |
| 🟡 建议 | 两个文件顶部<br>(第 3-4 行) | **类外获取 CI 实例**：`$CI = &get_instance();` 放在类定义外部会在文件被 `include` 时立即执行，不符合 OOP 规范，且可能引发加载顺序问题。 | 移至构造函数或按需调用。 | `public function __construct() {`<br>` parent::__construct();`<br>` $this->load->model('Simple_model');`<br>`}` |
| 🟡 建议 | `Ahead_preorder_order_model.php`<br>~第 63 行 | **数组键未定义检查**：`$this->pay_platform_arr[$v['pay_platform']]` 若数据库值不在映射表中会触发 `Undefined index` 警告。 | 使用空合并运算符或 `isset()` 防御。 | `$v['pay_platform_name'] = $this->pay_platform_arr[$v['pay_platform']] ?? '未知支付';` |
| 🟡 建议 | 多处 `up()` 调用<br>(如 165, 268, 298 行) | **原始 SQL 字符串更新**：`'_account=_account+' . $vip_pay_amount` 依赖自定义 `up()` 方法解析，易引发注入且丧失框架查询构建器的类型安全与日志追踪能力。 | 优先使用 `$this->db->set()` 或严格类型转换 `(float)`。 | `$this->db->set('_account', '_account + ' . (float)$vip_pay_amount, FALSE);`<br>`$this->db->where(['_merchant_id' => $merchant_id, '_vip_card' => $order['_vip_card']]);`<br>`$this->db->update('vip_table');` |
---
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **修复 SQL 注入**：立即替换 `Ahead_songs_sales_pay_log_model.php` 中所有字符串拼接的 `WHERE` 与 `LIKE` 条件，改用 CI3 查询构建器或参数绑定。
2. **补充事务控制**：为 `Ahead_preorder_order_model::refund()` 添加完整的事务包裹。当前 `Ahead_songs_sales_pay_log_model::refund()` 虽开启了事务，但调用的子模型退款方法无事务，一旦子方法内部报错或返回失败，外层事务可能无法正确回滚关联数据。
3. **移除模型层 `exit`**：将 `export()` 方法中的 `exit('成功')` 改为返回状态码，由 Controller 统一输出，保持架构纯净。
### 🛠 后续重构与优化方向
1. **统一输入校验**：所有来自 `$param` 的时间、金额、ID 等字段，应在入口处进行类型强转与合法性校验（如 `strtotime()` 返回 `false` 时应拦截或赋默认值）。
2. **封装全局函数**：代码中大量使用 `throwError()`、`check_export_datetime()`、`change_number_format()`、`wx_refund()` 等全局函数。建议将其迁移至 `application/helpers/` 或 `application/libraries/`，并通过 `$this->load->helper()` 或 `$this->load->library()` 按需加载，提升可测试性与命名空间隔离。
3. **优化模型加载策略**：频繁在方法内调用 `$this->load->model()` 会增加运行时开销。建议在 `__construct()` 中预加载高频模型，或启用 CI3 的 `autoload.php` 自动加载机制。
4. **金额计算精度**：涉及财务退款与余额回滚的逻辑（如 `_room_account` 与 `_account` 拆分退款），建议统一使用 `bcmath` 扩展或严格保留 2 位小数的浮点运算，避免 PHP 浮点数精度丢失导致的账目偏差。
> 💡 **局限性说明**：本次审查基于提供的两个 Model 文件。由于未提供 `Simple_model` 底层实现、控制器调用链路及全局函数定义，部分自定义方法（如 `up()`、`select()`、`get_one()`）的底层安全性与性能表现需结合实际框架源码进一步评估。建议结合 CI Profiler 或 Xdebug 进行全链路压测与 SQL 审计。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779273589
|
1779273589
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
817
|
124
|
11
|
5
|
|
0
|
test: dedicated evidence for 14 P1/P2 requirements test: dedicated evidence for 14 P1/P2 requirements and strict licence dates...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1791559480
|
1791559480
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
854
|
83
|
11
|
5
|
|
0
|
docs(os-product): 回灌 OS 侧 0.19.6 导入结果与 relay 整表授权放 docs(os-product): 回灌 OS 侧 0.19.6 导入结果与 relay 整表授权放宽原因（H11）...
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793143165
|
1793143171
|
1793143171
|
0
|
0
|
0
|
Edit
Delete
|
|
890
|
116
|
11
|
5
|
|
0
|
feat(payment): add candidate provider port and det feat(payment): add candidate provider port and deterministic simulator; verify pure Node safety cases...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793674092
|
1793674128
|
1793674128
|
0
|
0
|
0
|
Edit
Delete
|
|
12
|
2
|
12
|
1
|
|
0
|
🔍 代码审查报告：main 分支 - 修改测试调用不存在的model
|
## 自动代码审查报告
**分支**: main
**提交**: `82563bcddd8d325 ## 自动代码审查报告
**分支**: main
**提交**: `82563bcddd8d32587bc08757c8fdba4bb728fdbe`
**时间**: 2026-03-17 13:08:32
**审查模型**: qwen3.5-plus
---
[2026-03-17 13:06:57] 正在调用 Qwen API 进行代码审查...
[2026-03-17 13:08:32] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260317-130657.md
基于提供的文件内容和跨文件引用分析，以下是对 `pc/application/controllers/Bill.php` 的代码审查与风险分析报告。
### 1. 严重运行时错误 (Critical Runtime Error)
在 `getDetails` 方法中存在一处会导致 fatal error 的代码，必须立即修复。
* **位置**: `getDetails` 方法，第 133 行
* **代码**: `$this->non_existent_model->some_method();`
* **问题**: 该类并未通过 `$this->load->model()` 加载，且模型名称 `non_existent_model` 暗示这是一个占位符或调试遗留代码。
* **后果**: 运行时将抛出 `Undefined property: Bill::$non_existent_model` 错误，随后调用 `some_method()` 会导致脚本终止。
* **建议**: 删除该行代码，或替换为正确的模型调用。
### 2. 冗余与拼写错误的模型加载 (Redundant & Typo Model Loads)
在 `getDetails` 和 `getList` 方法中，存在加载了未使用的模型或模型名称拼写错误的情况，虽然不一定会导致报错（如果文件存在），但属于代码污染。
* **位置**: `getDetails` 方法，第 134-135 行
* `$this->load->model('ahead_room_changesss_model');` (多写了 'ss')
* `$this->load->model('ahead_room_changesss_mode2l');` (多写了 'ss' 且 'model' 拼写为 'mode2l')
* **现状**: 后续代码使用的是正确的 `$this->ahead_room_change_model`。
* **建议**: 删除第 134-135 行的无效加载。
* **位置**: `getList` 方法，第 74 行
* `$this->load->model('ahead_setting_clean_data_models');` (多写了 's')
* **现状**: 第 76 行又加载了正确的 `$this->ahead_setting_clean_data_model` 并使用之。
* **建议**: 删除第 74 行的无效加载。
### 3. 逻辑不一致 (Logic Inconsistencies)
`getList` (列表查询) 与 `export` (导出) 方法中的查询逻辑存在差异，可能导致导出的数据与前端列表展示的数据不一致。
* **账单号模糊查询**:
* `getList` (第 120 行): `'%'. $bill_no . '%'` (前后匹配)
* `export` (第 263 行): `'%' . $bill_no` (仅前匹配)
* **建议**: 统一为前后匹配 `'%'. $bill_no . '%'`。
* **参数获取方式**:
* `getList`: 使用 `$this->param` (通常经过框架处理)。
* `export`: 直接使用 `$_GET` (第 229 行)。
* **建议**: 统一使用 `$this->param` 以确保参数 sanitization 和一致性。
* **错误处理方式**:
* `getList`: 使用 `$this->error_response()` (返回 JSON 格式错误)。
* `export`: 使用 `exit()` (直接输出字符串并终止)。
* **建议**: 在 `export` 方法中也应尽量统一错误处理格式，或者确保前端能处理 `exit` 输出的纯文本错误。
### 4. 依赖引用分析修正 (Dependency Analysis Correction)
提供的引用分析列表中包含了 `$this->non_existent_model`，这确实是一个引用，但它是一个**无效引用**。其他模型引用基本正确，但需注意以下模型在代码中多次加载（虽不影响运行，但可优化）：
* `ahead_shop_model`: 在 `getList` 和 `export` 中均有加载。
* `ahead_yc_merchant_user_model`: 在 `getList` 和 `export` 中均有加载。
* `ahead_pay_log_model`: 在 `getDetails` 中加载了两次（第 159 行和第 176 行）。
### 5. 修复建议代码片段
针对 `getDetails` 方法中的严重错误和冗余代码，建议修改如下：
```php
public function getDetails()
{
$param = $this->param;
$merchant_id = $this->merchant_id ? $this->merchant_id : $this->error_response('账号异常，请重新登录');
$bill_no = !empty($param['bill_no']) ? trim($param['bill_no']) : '';
$unique_key = !empty($param['unique_key']) ? trim($param['unique_key']) : '';
// [修复] 删除无效的模型调用
// $this->non_existent_model->some_method();
$bill_detail = $this->ahead_bill_model->get_detail($bill_no, $unique_key);
if (empty($bill_detail)) {
$this->error_response('账单不存在');
}
// [修复] 删除冗余的模型加载
// $this->load->model('ahead_room_changesss_model');
// $this->load->model('ahead_room_changesss_mode2l');
// 转房信息
$this->load->model('ahead_room_change_model');
$bill_detail['change_room_msg'] = $this->ahead_room_change_model->get_change_room_info($bill_detail['unique_key']);
// ... 后续代码保持不变
}
```
### 6. 总结
该文件主要存在**代码清理不彻底**的问题。`non_existent_model` 的调用是最高优先级的修复项，否则该接口无法正常使用。其次建议统一 `getList` 和 `export` 的查询逻辑与错误处理机制，以保证系统行为的一致性。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773724112
|
1773724112
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
39
|
18
|
12
|
1
|
|
0
|
🔍 代码审查报告：pc-260331 - 测试12233
|
## 自动代码审查报告
**分支**: pc-260331
**提交**: `96f8a7732d ## 自动代码审查报告
**分支**: pc-260331
**提交**: `96f8a7732da6f05607f7752e00b0728e0c68d8f9`
**时间**: 2026-03-28 12:59:46
---
## 1. 审查摘要
- **代码质量评分**：1/10
- **总体评价**：代码存在严重的语法错误，无法通过 PHP 解析器编译，属于不可运行状态。代码结构看似基于 CodeIgniter 架构（尽管称为 phpci），但存在违反框架生命周期管理的用法（如在控制器中手动 include 其他控制器）。此外，命名规范、模型加载方式及潜在的安全隐患均需修正。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | AppModule.php:9 | **语法错误**：函数定义后存在非法字符 `ssss`，导致解析失败。 | 移除非法字符，确保函数签名符合 PHP 规范。 | `public function getPcModule()` |
| 🔴 严重 | AppModule.php:10 | **语法错误**：左大括号后存在非法字符 `s`，导致解析失败。 | 移除非法字符，使用标准的大括号包裹代码块。 | `{` |
| 🔴 严重 | AppModule.php:11 | **拼写错误**：`$this->load->smodel` 应为 `$this->load->model`，会导致方法不存在错误。 | 修正为框架标准的模型加载方法。 | `$this->load->model('ahead_common_config_model');` |
| 🟠 警告 | AppModule.php:4 | **框架违规**：在控制器中手动 `include` 另一个控制器文件 (`PcServer.php`)。在 CI/phpci 架构中，控制器应由框架路由自动加载，手动 include 可能导致生命周期混乱或重复加载。 | 移除 `include` 语句。确保 `PcServer` 类位于框架可自动加载的路径（如 `core` 目录）或作为基类正确配置。 | `// 移除 include 语句` |
| 🟠 警告 | AppModule.php:12 | **模型命名惯例**：加载模型后直接访问 `$this->ahead_common_config_model`。需确认框架是否自动去除 `_model` 后缀。 | 检查 phpci 模型加载规范。通常建议加载时指定别名或确认默认行为。 | `$this->load->model('ahead_common_config_model', 'config_model');`<br>`$this->config_model->get_data(...)` |
| 🟠 警告 | AppModule.php:13 | **属性未定义**：`$this->stream` 属性未在类中声明，依赖父类 `PcServer`。若父类未初始化，会导致 Notice 警告或错误。 | 在类属性中声明 `$public $stream` 或在构造函数中初始化，确保父类已正确构造。 | `public $stream = [];` |
| 🟡 建议 | AppModule.php:9 | **命名规范**：函数名 `getPcModule` 采用驼峰式。若 phpci 基于 CodeIgniter，通常建议使用蛇形命名法 (`get_pc_module`) 以保持一致性。 | 遵循框架统一的命名规范（PSR-1 或框架特定规范）。 | `public function get_pc_module()` |
| 🟡 建议 | AppModule.php:1 | **安全头**：虽然已有 `defined('BASEPATH')` 检查，但建议确保 `BASEPATH` 常量在全局严格定义，防止绕过。 | 保持现有检查，确保入口文件严格定义常量。 | (无需修改，保持现状) |
| 🟡 建议 | AppModule.php:12 | **性能优化**：`pc_module_config` 看似为配置数据，若频繁请求，建议增加缓存机制，避免每次请求都查询数据库。 | 使用框架缓存库（Cache）包裹数据库查询。 | `$data = $this->cache->get('pc_config'); if(!$data){ ... }` |
## 3. 总结与行动建议
### 优先修复的关键问题
1. **修复语法错误**：立即移除第 9 行和第 10 行的非法字符 (`ssss`, `{s`)，修正 `$this->load->smodel` 为 `$this->load->model`。这是代码无法运行的根本原因。
2. **修正继承与加载机制**：删除 `include FCPATH...PcServer.php` 语句。在 phpci/CodeIgniter 架构中，若 `AppModule` 需继承 `PcServer`，`PcServer` 应放置在 `application/core/` 目录下（命名为 `MY_Controller` 或类似），以便框架自动加载，而非在控制器中手动 include。
3. **验证父类依赖**：确认 `PcServer` 父类中是否已正确初始化 `$this->stream` 数组及 `success_response()` 方法，避免运行时错误。
### 后续重构或优化方向
1. **规范代码风格**：遵循 PSR-12 及框架特定的命名规范（建议统一使用蛇形命名法 `snake_case` 处理方法和变量，除非框架强制要求驼峰）。
2. **增强安全性**：
* 虽然当前方法看似只读取配置，但需确认 `pc_module_config` 中是否包含敏感信息。若有，需增加权限验证（如检查登录状态）。
* 确保 `success_response()` 方法内部正确处理了输出编码，防止 XSS 风险（即使这里是 JSON 数据，也建议设置正确的 `Content-Type`）。
3. **性能提升**：配置类数据通常变动频率低，建议在模型层或控制器层引入缓存机制（如 Redis 或 File Cache），减少数据库压力。
4. **错误处理**：增加 `try-catch` 块捕获模型查询可能产生的数据库异常，并返回友好的错误信息，而不是让框架抛出原始错误。
### 修正后的代码参考
```php
<?php
defined('BASEPATH') or exit('No direct script access allowed');
// 假设 PcServer 是核心控制器，应放在 application/core/ 并由框架自动加载，无需 include
/**
* 账单模块控制器
*/
class AppModule extends PcServer
{
// 显式声明属性以提高可读性
public $stream = [];
/**
* 获取 PC 模块配置
* @return void
*/
public function get_pc_module()
{
// 加载模型，建议确认是否需要别名
$this->load->model('ahead_common_config_model');
// 获取数据
$data = $this->ahead_common_config_model->get_data('pc_module_config');
// 设置响应数据
$this->stream['request']['result'] = $data;
// 发送成功响应
$this->success_response();
}
}
```
> **注意**：由于提供的代码片段过短且存在严重语法错误，以上审查主要基于框架通用规范及可见代码结构。若 `phpci` 框架有特殊的生命周期或加载机制，请务必查阅官方文档进行适配。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774673986
|
1774673986
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
180
|
21
|
12
|
1
|
|
0
|
🔍 代码审查报告：pay-260519 - 1
|
## 自动代码审查报告
**分支**: pay-260519
**提交**: `6a5abf32f ## 自动代码审查报告
**分支**: pay-260519
**提交**: `6a5abf32f15e89bfd3709efd2e39ebefdfb97ebf`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-19 13:33:35
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了支付流水记录的核心业务逻辑，功能结构完整。但存在**高危 SQL 注入漏洞**、**N+1 查询性能瓶颈**、**严重违反 DRY 原则**以及**框架生命周期误用**等问题。代码中大量使用魔法数字、松散比较和冗余的模型加载，可维护性与安全性亟待提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `get_vip_pay_log` (~380行)<br>`get_vip_pay_log_min` (~490行) | **SQL 注入漏洞**：直接使用字符串拼接构造 `WHERE` 条件（如 `'_merchant_id=' . $merchant_id`），未进行任何转义或参数绑定，恶意输入可导致数据泄露或篡改。 | 废弃字符串拼接，全面改用框架查询构造器（Query Builder）或参数化查询。 | `$this->db->where('_merchant_id', $merchant_id)->where('_vip_card', $vip_card)->get(...)` |
| 🔴 严重 | 文件顶部 (1-4行) | **框架生命周期误用**：在类外部执行 `$CI = &get_instance();` 并加载模型。该代码会在文件被 `include/require` 时立即执行，此时框架可能尚未完成初始化，易引发致命错误或内存泄漏。 | 移除顶部全局代码，直接继承父类。模型加载应移至构造函数或按需调用。 | `class Ahead_pay_log_model extends Simple_model { public function __construct() { parent::__construct(); } }` |
| 🔴 严重 | `add_by_footbath_order` (~345行) | **逻辑错误/数据不一致**：`ManageMap` 数据准备中误将 `$order['_registration_fee']` 赋值给 `actual_pay`，应为 `$order['_actual_pay']`。且核心调用已被注释，保留冗余代码易引发后续维护混淆。 | 修正字段名；若业务明确暂不启用，请直接删除整段数据准备代码。 | `'actual_pay' => $order['_actual_pay'],` |
| 🟠 警告 | `get_bill_pay_log`<br>`get_order_pay_log`<br>`get_vip_pay_log_min` | **N+1 查询性能瓶颈**：在 `foreach` 循环中频繁调用 `get_one()` 或 `get_custom_pay_platform()`。当返回 100 条记录时，将触发 100+ 次独立数据库查询，严重拖慢响应速度。 | 收集所有关联 ID，使用 `WHERE IN` 批量查询，或在内存中构建映射数组（Map）进行关联。 | `$ids = array_column($log_data, 'admin_id'); $users = $this->model->get_by_ids($ids); $map = array_column($users, '_name', '_id');` |
| 🟠 警告 | 多个 `add_by_*` 方法 | **违反 DRY 原则**：`ManageMap->add_income` 与 `ahead_songs_sales_pay_log_model->add_data` 的调用逻辑在 6 个方法中高度重复，且参数组装方式不一致。 | 抽取为私有统一处理方法 `_sync_related_logs($data, $map_type, $sales_type)`，集中管理第三方/关联表同步逻辑。 | `private function _sync_related_logs($data, $map_type, $sales_type) { if($data['_actual_pay']>0) { ... } $this->load->model('...'); ... }` |
| 🟡 建议 | 全文多处 | **魔法数字泛滥**：支付平台 ID (`1,2,3,11,14,16...`)、状态值硬编码散落各处，与 `PAY_LOG_TYPE_MAP` 未形成联动，业务变更时代价极高。 | 定义支付平台常量类（如 `PayPlatform`），或在配置文件中集中管理，代码中统一引用常量。 | `if ($platform === PayPlatform::WECHAT) { ... }` |
| 🟡 建议 | `get_bill_pay_log` (~108行) | **三元运算符滥用**：`$pay_platform == 11 ? ($v['actual_pay'] = $v['present_amount']) : '';` 利用三元运算符执行赋值操作，违反 PSR-12 规范且降低可读性。 | 改用标准 `if` 条件语句。 | `if ($pay_platform == 11) { $v['actual_pay'] = $v['present_amount']; }` |
| 🟡 建议 | 全文多处 | **松散比较 (`==`) 与类型不一致**：如 `$order['_pay_platform'] == 14` 与 `== '14'` 混用，PHP 隐式类型转换可能在边界条件下引发逻辑误判。 | 统一使用严格比较 `===`，并在方法入口对关键参数进行类型校验或强制转换。 | `if ((int)$order['_pay_platform'] === 14) { ... }` |
## 3. 总结与行动建议
### 🚨 优先修复的关键问题
1. **立即修复 SQL 注入**：`get_vip_pay_log` 与 `get_vip_pay_log_min` 中的字符串拼接 `WHERE` 条件必须替换为查询构造器或参数绑定。这是当前最高危的安全漏洞。
2. **清理顶部全局代码**：删除文件开头的 `$CI = &get_instance();` 及模型加载逻辑，确保模型生命周期符合 MVC 框架规范。
3. **修正 `add_by_footbath_order` 数据错误**：核对 `actual_pay` 字段赋值来源，并清理已注释的无效代码块。
### 🛠 后续重构与优化方向
1. **消除 N+1 查询**：对涉及循环内查库的方法进行批量查询重构。可引入 `array_column` + `WHERE IN` 模式，将数据库交互次数从 `O(N)` 降至 `O(1)`。
2. **抽象重复同步逻辑**：将 `ManageMap` 营收同步与 `ahead_songs_sales_pay_log_model` 销售日志同步封装为私有方法。建议采用**策略模式**或**观察者模式**解耦支付流水与下游业务系统的依赖。
3. **建立常量/枚举体系**：将散落的支付平台 ID、订单类型、状态码提取为独立的常量类或配置文件。结合 `PAY_LOG_TYPE_MAP` 实现前后端/数据库的统一映射。
4. **框架适配说明**：当前代码结构高度契合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请重点确认：
- `Simple_model::select()` 是否自动处理 SQL 转义？
- `$this->load->model()` 是否支持自动缓存？（CI3 默认支持，但显式重复加载仍影响可读性）
- 建议查阅 `phpci` 官方文档中关于 `Query Builder` 与 `Model 基类` 的最佳实践，确保底层驱动与当前写法兼容。
> 💡 **提示**：若需对 `Simple_model` 的底层实现或 `ManageMap` 同步逻辑进行更精准的架构级审查，请提供相关基类代码。当前审查基于标准 MVC/CI 范式给出。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779168815
|
1779168815
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
193
|
22
|
12
|
1
|
|
0
|
🔍 代码审查报告：app-260519 - Merge remote-tracking branch 🔍 代码审查报告：app-260519 - Merge remote-tracking branch 'app/app-260519' into...
|
## 自动代码审查报告
**分支**: app-260519
**提交**: `3e251a363 ## 自动代码审查报告
**分支**: app-260519
**提交**: `3e251a363b22be804cc9de7ae816688b5476f852`
**提交人**: zhangjunnan (121158035@qq.com)
**时间**: 2026-05-19 14:13:05
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑覆盖较全，支付、退款、预订等核心流程具备基础实现。但存在多处 **SQL 注入风险**、**事务管理不规范** 及 **逻辑缺陷**，且代码风格偏向老旧写法，未充分利用现代 PHP 特性与框架查询构造器。整体可维护性与安全性有待大幅提升。
- **风险等级**：🔴 高
> ⚠️ **局限性说明**：`Ahead_pay_log_model.php` 文件末尾代码被截断（`$this-`），无法评估完整逻辑。以下审查基于已提供片段，建议补全后二次复核。
> 📌 **框架说明**：代码结构特征（`$CI = &get_instance()`、`Simple_model`、`$this->load->model()`、`$this->db->trans_*`）高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制或衍生框架，请结合其官方文档调整生命周期与组件调用方式。
---
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_pay_log_model.php`<br>`update_refund_amount` / `refund` | **SQL 注入漏洞**：WHERE 条件与 UPDATE 语句直接使用字符串拼接（如 `'_relation_id="' . $relation_id . '"`），未使用参数绑定或查询构造器，恶意输入可导致数据篡改或泄露。 | 全面改用 CI3 查询构造器（Query Builder）或 `$this->db->query()` 配合占位符。 | `$this->db->where('_relation_id', $relation_id)<br>->where('_type', $type)<br>->where_in('_status', [1, 4]);` |
| 🔴 严重 | `Ahead_preorder_order_model.php`<br>`open_room` 方法内循环 | **逻辑缺陷/数据错乱**：`$in_process[] = $data;` 错误地将外部查询结果 `$data` 插入流程数组，应为当前循环变量 `$in_process_v`。将导致流程记录错乱或插入空数据。 | 修正变量引用，确保插入正确的流程状态对象。 | `// 错误：$in_process[] = $data;<br>// 正确：<br>$in_process[] = $in_process_v;` |
| 🔴 严重 | `Ahead_book_order_model.php`<br>`refund_by_notify` | **SQL 注入漏洞**：`$log_where = '_relation_id="' . $order_data['_id'] . '" and _status=1...'` 直接拼接。同属高危注入点。 | 使用模型封装的查询构造器方法，避免手写 SQL 字符串。 | `$this->ahead_pay_log_model->db->where('_relation_id', $order_data['_id'])<br>->where('_status', 1)<br>->where_in('_type', [5, 13]);` |
| 🟠 警告 | 多个 Model 文件 | **事务管理不规范**：手动调用 `trans_begin()` / `trans_rollback()` / `trans_commit()`，但业务中大量使用 `throwError()`。若该函数抛出异常或 `exit`，将跳过 `rollback`，导致脏数据或资金不一致。 | 使用 `try...catch` 包裹事务块，或改用 CI3 自动事务模式 `trans_start()` / `trans_complete()`。 | `try {<br> $this->db->trans_start();<br> // 业务逻辑<br> $this->db->trans_complete();<br>} catch (\Exception $e) {<br> $this->db->trans_rollback();<br> throw $e;<br>}` |
| 🟠 警告 | 多个 Model 文件 | **性能瓶颈 (N+1查询)**：在业务方法中频繁调用 `$this->load->model()`，部分甚至在循环或高频回调中加载。每次加载都会触发文件包含与实例化，拖慢响应。 | 将依赖模型统一移至 `__construct()` 初始化，或使用服务容器/依赖注入。 | `public function __construct() {<br> parent::__construct();<br> $this->load->model(['model_a', 'model_b', 'model_c']);<br>}` |
| 🟠 警告 | `Ahead_pay_log_model.php`<br>`add_order_pay_log` | **类型比较隐患**：`in_array($type, array_keys(self::PAY_LOG_TITLE_MAP))` 未开启严格模式。常量键为字符串 `'1'`，若传入整型 `1` 会因松散比较返回 `true`，可能引发类型混淆。 | 开启严格比较或统一类型转换。 | `in_array((string)$type, array_keys(self::PAY_LOG_TITLE_MAP), true)` |
| 🟡 建议 | 全局代码 | **代码规范与可维护性**：大量使用 `array()` 而非 `[]`；存在硬编码（如 `md5` 盐值、模板索引 `10`）；注释包含个人调试记录（如 `2020-09-30，hlj`）；未使用类型声明。 | 遵循 PSR-12；提取配置/常量；清理调试注释；启用 `declare(strict_types=1)` 与参数类型提示。 | `const WX_REFUND_SALT = '1441600902';<br>public function refund(int $pay_id, float $amount): bool` |
| 🟡 建议 | `Ahead_yc_order_process_model.php`<br>`add_order_process` | **空值判断不严谨**：`is_null($userInfo)` 用于判断数组返回值。若查询失败返回 `false` 或空数组，`is_null` 将失效。 | 使用 `empty()` 或 `isset()` 进行安全判断。 | `$userName = !empty($userInfo['_name']) ? $userInfo['_name'] : '';` |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **立即修复 SQL 注入点**：全局搜索 `'_xxx="' . $var . '"` 或 `'_xxx=' . $var` 模式，全部替换为 CI3 Query Builder 的 `where()`、`where_in()`、`set()` 方法。支付与退款模块涉及资金安全，必须作为 P0 级任务处理。
2. **修正 `open_room` 变量赋值 Bug**：`$in_process[] = $data;` 会导致流程日志写入错误数据，直接影响订单状态追踪与售后排查。
3. **统一事务异常处理机制**：将 `trans_begin/rollback/commit` 包裹在 `try...catch` 中，或全面迁移至 `trans_start()/trans_complete()` 自动模式，确保任何异常（包括 `throwError`）都能安全回滚。
### 🛠 后续重构与优化方向
| 优化维度 | 具体建议 |
| :--- | :--- |
| **架构解耦** | 当前 Model 承担了过多业务逻辑（支付、退款、库存、消息推送、报表统计）。建议抽取 `PaymentService`、`RefundService`、`OrderProcessService` 等独立服务类，Model 仅负责数据持久化。 |
| **性能提升** | 1. 模型依赖集中初始化，避免运行时重复加载。<br>2. 报表类查询（如 `get_business`）增加数据库索引覆盖（`_create_time`, `_pay_platform`, `_status`）。<br>3. 高频读取的配置（如支付平台映射）建议接入 Redis 缓存。 |
| **规范升级** | 1. 全面启用 `declare(strict_types=1)` 与 PHP 7.4+ 类型声明。<br>2. 替换 `array()` 为 `[]`，统一使用 PSR-12 缩进与命名规范。<br>3. 移除硬编码，将支付盐值、模板 ID、业务状态映射移至配置文件或数据库字典表。 |
| **框架适配** | 若项目确为 `phpci` 定制版，请确认其是否重写了 `Simple_model` 的生命周期或查询构造器。建议查阅官方文档确认 `load->model()` 的缓存机制与事务钩子，避免与原生 CI3 行为冲突。 |
> 💡 **审查专家提示**：支付与退款链路是系统的“资金命脉”。建议在修复上述问题后，补充 **单元测试**（覆盖正常支付、部分退款、并发退款、异常回滚等场景）与 **集成测试**，并引入静态代码分析工具（如 `PHPStan` 或 `Psalm`）进行常态化质量门禁。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779171185
|
1779171185
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
263
|
23
|
12
|
1
|
|
0
|
🔍 代码审查报告：admin-260616 - 包厢管理列表新增-聚旺
|
## 自动代码审查报告
**分支**: admin-260616
**提交**: `14959e0 ## 自动代码审查报告
**分支**: admin-260616
**提交**: `14959e0aff3557062973d1f1ea8cadb5e9095985`
**提交人**: linyangrui (yangruilin888@gmail.com)
**时间**: 2026-05-21 10:55:17
---
> **审查假设**：基于代码语法（`slot-scope`、`Vue.axios`、`this.$refs`、`layer` 弹窗、`$.ajax`）推断本项目为 **Vue 2 + Element UI + jQuery + Layer** 的混合技术栈。审查将基于 Vue 2 最佳实践及现代前端工程标准进行。
### 1. 总体评价
> **综合评分：5 / 10**
>
> **优点**：业务功能覆盖全面，搜索、列表、多类弹窗配置及权限控制逻辑完整；使用了 `v-model`、`v-if/v-show` 等 Vue 基础指令，具备一定组件化意识。
>
> **缺点**：该组件属于典型的“上帝组件”，单文件承载了过多独立业务模块（搜索表单、数据表格、10+ 个独立弹窗/面板、权限树解析、远程终端交互等），严重违反单一职责原则。架构上混用 Vue 响应式、jQuery DOM 操作、Select2 插件与 Layer 弹窗，破坏了 Vue 的数据驱动理念，导致状态管理混乱、内存泄漏风险高。存在明显的语法错误（赋值代替比较）与安全隐患（URL 拼接未编码）。整体可维护性与扩展性较差，亟需重构与技术栈统一。
### 2. 问题详情清单
| 严重等级 | 位置/行号 | 问题分类 | 问题描述 | 建议修改方案 |
| :---: | :---: | :---: | :---: | :---: |
| 🔴 严重 | `mounted` 函数内 | 逻辑缺陷 | `if (_this.$route.params.bxl_name = 'shop')` 误用赋值运算符 `=` 代替比较运算符 `===`，导致路由参数被意外篡改且条件恒为真。 | 改为严格相等判断：`if (this.$route.params.bxl_name === 'shop')`。 |
| 🔴 严重 | `getListTable` / `bxlExport` 等方法 | 规范/架构 | 大量使用 `$.ajax` 与 jQuery 读取 DOM（如 `$(_this.$refs.xxx).val()`），与 `Vue.axios` 混用导致全局拦截器失效，且违背 Vue 响应式原则。 | 彻底移除 jQuery 依赖，所有表单字段通过 `v-model` 绑定至 `data`，统一使用 `axios` 发起请求。 |
| 🟡 警告 | `bxlExport` 方法 | 安全性 | `window.location.href` 直接拼接查询参数，未使用 `encodeURIComponent` 编码。若用户输入含 `&`、`#`、空格等字符，将导致请求截断或潜在 XSS。 | 使用 `URLSearchParams` 或 `encodeURIComponent()` 处理参数，或改用 `axios` 请求 Blob 流触发下载。 |
| 🟡 警告 | `getOperAuth` 方法 | 可维护性/性能 | 权限校验使用 4 层 `forEach` 嵌套硬编码 ID（如 `'373'`、`'374'`），时间复杂度高，新增按钮需修改核心逻辑，极易出错。 | 将菜单树扁平化，提取 `权限ID -> 状态变量` 的映射配置表，使用 `find` 或递归工具函数替代硬编码遍历。 |
| 🟡 警告 | 模板多处 `v-for` (如 PAC/ROOM 配置) | 规范/性能 | 使用 `:key="index"` 作为列表键值。在动态增删项时会导致 Vue 复用错误、输入框状态错乱及不必要的重渲染。 | 为每项生成唯一业务 ID（如 `item.id` 或 `uuid()`），严禁在可变列表中使用 `index` 作为 `key`。 |
| 🟢 建议 | 全局 `data` 与模板 | 规范 | 变量名大量使用 `bxl_` 前缀（推测为“包厢”拼音缩写），语义模糊且不符合 `camelCase` 规范（如 `bxl_youc`、`bxl_mac`）。 | 采用语义化英文命名，如 `boxVersion`、`shopName`、`macAddress`、`roomList` 等，提升代码自解释性。 |
| 🟢 建议 | 模板结构 | 可维护性 | 单文件超千行，包含大量独立弹窗 DOM，违反单一职责原则，团队协作与代码审查成本极高。 | 拆分为独立子组件：`<BoxSearchForm />`、`<BoxDataTable />`、`<RemoteControlModal />`、`<ConfigPanel />` 等。 |
| 🟢 建议 | `layer.confirm` / `layer.open` 调用 | 架构 | 使用 `layer` 直接挂载 DOM 节点（`content: $(_this.$refs.xxx)`）会脱离 Vue 生命周期管理，组件销毁时易引发内存泄漏与事件未清理。 | 替换为 Element UI 的 `<el-dialog>` 或 Vue 原生条件渲染，保持数据驱动与生命周期同步。 |
### 3. 优化代码示例
```javascript
// 选取关键问题片段，提供重构后的最佳实践代码
// 1. 修复路由赋值 Bug 并统一 Axios 请求模式
mounted() {
// ✅ 修复：使用严格相等，避免意外修改路由参数
if (this.$route.params.bxl_name === 'shop') {
this.bxl_shop = this.$route.params.bxl_val;
}
this.initPage();
},
methods: {
async initPage() {
// ✅ 优化：并行请求初始化数据，提升加载性能
await Promise.all([this.fetchModels(), this.fetchSkins(), this.getListTable(1)]);
this.getOperAuth();
},
// ✅ 优化：移除 jQuery，数据完全由 Vue 响应式管理
async getListTable(page = 1) {
this.loading = true;
try {
// 假设模板中已改为 v-model="searchForm.roomVersion" 等
const params = {
page,
page_size: this.GLOBAL.PAGESIZE,
room_version: this.searchForm.roomVersion,
merchant_name: this.searchForm.merchantName,
shop_name: this.searchForm.shopName,
// ... 其他字段直接读取 data，无需 $.trim($().val())
};
const res = await this.$axios.post('room/getList', params);
if (res.data.code === 0) {
this.list_total = Number(res.data.result.total);
this.list_table = res.data.result.rows;
} else {
this.$message.error(res.data.msg || '获取列表失败');
}
} catch (error) {
console.error('列表请求异常:', error);
this.$message.error('网络异常，请稍后重试');
} finally {
this.loading = false;
}
},
// ✅ 优化：安全导出，避免 URL 拼接风险
bxlExport() {
const params = new URLSearchParams({
room_version: this.searchForm.roomVersion,
merchant_name: this.searchForm.merchantName,
// ... 其他参数
}).toString();
// 使用安全拼接或 axios 下载流
window.location.href = `${Vue.ctUrl}room/exportList?${params}`;
},
// ✅ 优化：权限校验配置化，消除深层嵌套
getOperAuth() {
const authMap = {
'373': 'show_config_btn',
'374': 'show_box_update_btn',
'375': 'show_music_update_btn',
'376': 'show_unbind_btn',
'377': 'show_remote_btn',
'378': 'show_controller_btn',
'379': 'show_IMAX_btn',
'380': 'show_mural_btn'
};
// 假设已提供扁平化菜单工具函数 flattenMenu(menuTree)
const allBtns = flattenMenu(this.menunew);
allBtns.forEach(btn => {
if (authMap[btn.id]) {
this[authMap[btn.id]] = true;
}
});
}
}
```
### 4. 总结与行动建议
1. **立即修复逻辑与安全缺陷**：修正 `mounted` 中的赋值错误；为所有 URL 拼接添加 `encodeURIComponent` 或改用 `URLSearchParams`；统一错误提示文案，避免暴露内部状态。
2. **剥离 jQuery 与第三方 DOM 库**：逐步移除 `$.ajax`、`select2` 和 `layer` 的 DOM 挂载逻辑。表单交互全面转向 `v-model`，弹窗改用 `<el-dialog>`，确保 Vue 虚拟 DOM 与生命周期完整接管。
3. **实施组件拆分与状态收敛**：将当前文件按职责拆分为 `SearchForm`、`DataTable`、`RemoteControl`、`ConfigModal` 等子组件。将分散的 `data` 状态收敛至 Vuex/Pinia 或组合式函数中，降低组件耦合度。
**推荐 Lint 规则配置**：
```json
{
"rules": {
"vue/no-mutating-props": "error",
"vue/no-v-html": "warn",
"vue/require-v-for-key": "error",
"vue/valid-v-for": "error",
"no-eval": "error",
"no-implied-eval": "error",
"eqeqeq": ["error", "always"],
"no-param-reassign": "warn",
"vue/component-name-in-template-casing": ["error", "PascalCase"]
}
}
```
建议配合 `eslint-plugin-vue` 与 `prettier` 进行自动化格式化，并在 CI/CD 流程中强制拦截 `🔴` 级别问题。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779332117
|
1779332117
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
818
|
124
|
12
|
5
|
|
0
|
feat(GOV01): renew development test registration w feat(GOV01): renew development test registration without rotating identities...
|
The registered development scope had no renewal pa The registered development scope had no renewal path: rerunning the script
only re-validated the file, and bindings expire 30 days after generation.
- renewDevelopmentRegistration() keeps tenant, store, registration, binding
and person UUIDs; only active bindings get expiresAt = now + days
(1-30, default 30). Revoked/suspended people are never revived. Each
renewal records time, authorization and previous expiry in `renewals`.
- ops/register-development-test-scope.mjs --renew "<authorization>".
- Renewed the fixture once: 2026-11-07T02:15Z -> 2026-11-08T15:33Z.
- Running registered gateways read their release copy and need a redeploy.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791560333
|
1791560339
|
1791560339
|
0
|
0
|
0
|
Edit
Delete
|
|
855
|
83
|
12
|
5
|
|
0
|
fix(os-product): D-9 发布审计沿快照本机远端解析到真实 OS 仓，只读 ls-r fix(os-product): D-9 发布审计沿快照本机远端解析到真实 OS 仓，只读 ls-remote 其发布远端...
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793145336
|
1793145343
|
1793145343
|
0
|
0
|
0
|
Edit
Delete
|
|
891
|
116
|
12
|
5
|
|
0
|
docs: archive IdentityWeb launcher preparation and docs: archive IdentityWeb launcher preparation and independent v2 evidence...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793674146
|
1793674154
|
1793674154
|
0
|
0
|
0
|
Edit
Delete
|
|
13
|
2
|
13
|
1
|
|
0
|
🔍 代码审查报告：pc-260401 分支 - 11111
|
## 自动代码审查报告
**分支**: pc-260401
**提交**: `81010fc873 ## 自动代码审查报告
**分支**: pc-260401
**提交**: `81010fc873c5b5fe0d871fdccb321c9f9015e9ae`
**时间**: 2026-03-17 15:19:00
**审查模型**: qwen3.5-plus
---
[2026-03-17 15:18:04] 正在调用 Qwen API 进行代码审查...
[2026-03-17 15:19:00] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260317-151804.md
基于您提供的项目结构、变更内容及文件列表，以下是完成后的代码分析报告。由于大部分文件仅提供了路径而未提供具体代码内容，且变更文件 `pc/alilog.php` 的内容为非代码文本，因此类提取和引用分析受到限制。
## 项目分析报告
### 1. 项目框架识别
根据文件结构（`application/config/`, `application/controllers/`, `autoload.php`, `database.php` 等），该项目基于 **CodeIgniter (CI)** 框架开发。
### 2. 变更文件分析
#### 文件：pc/alilog.php
- **变更内容**: `修改提交 dddd`
- **分析**: 该文件当前内容并非有效的 PHP 代码，看起来像是提交记录、占位符或日志标记。
- **影响**: 由于不包含可执行代码，此次变更不会影响业务逻辑，但需确认该文件是否应为 PHP 脚本。如果是日志文件，建议确认其扩展名或存储位置是否规范。
### 3. 项目中定义的类和方法 (基于提供内容)
#### PHP 类:
```text
未检测到。
原因：提供的文件列表中仅包含文件路径，未提供控制器（controllers）及配置文件的具體代码内容，无法提取类定义。
推测：根据 CodeIgniter 规范，pc/application/controllers/ 下的每个文件通常包含一个与文件名相同的控制器类（例如 AdMachine.php 对应 AdMachine 类）。
```
#### PHP 模型文件:
```text
未检测到。
原因：文件列表中未显示 models 目录，且未提供相关模型文件内容。
推测：模型可能位于未列出的 pc/application/models/ 目录中，或该项目采用事务脚本模式直接在控制器中处理数据库逻辑。
```
### 4. 跨文件引用分析
#### pc/alilog.php 中的引用:
```text
无。
原因：文件内容 "修改提交 dddd" 不包含 PHP 语法，因此不存在 require, include, use, 或类实例化等引用关系。
```
### 5. 潜在风险与建议
1. **文件内容异常**: `pc/alilog.php` 的内容看起来不像正常的 PHP 代码。如果这是一个日志文件，建议移至专门的日志目录（如 `logs/`）；如果这是代码文件，请恢复其实际 PHP 逻辑。
2. **代码完整性**: 当前分析仅基于文件路径列表。若要進行完整的依赖分析、漏洞扫描或重构建议，需要提供 `controllers` 和 `config` 目录下关键文件的具体代码内容。
3. **敏感配置**: 文件列表中包含 `database.php`, `redis.php`, `merchant.php` 等配置文件，请确保这些文件在版本控制中已被忽略（.gitignore），防止敏感信息泄露。
---
**总结**: 本次提交仅修改了 `pc/alilog.php`，内容为非代码文本，未检测到实际的逻辑变更或类结构变化。建议核实该文件的用途。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773731940
|
1773731940
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
40
|
18
|
13
|
1
|
|
0
|
🔍 代码审查报告：pc-260331 - 测试12233rrr
|
## 自动代码审查报告
**分支**: pc-260331
**提交**: `8bf63cfaf4 ## 自动代码审查报告
**分支**: pc-260331
**提交**: `8bf63cfaf44e00bebb635fe640a15bea40a38ed0`
**时间**: 2026-03-28 13:09:25
---
## 1. 审查摘要
- **代码质量评分**：2/10
- **总体评价**：提交代码存在**严重语法错误**（包含无效字符），无法通过编译/解析。架构设计存在隐患，控制器继承方式不符合常规框架规范，前端组件通信耦合度过高。代码中混入大量无意义字符，疑似编辑失误，需立即修正。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `AppModule.php`:9 | **语法错误**：函数体内存在无效字符 `sssss` 和 `ddd`，导致 PHP 解析失败。 | 删除所有无意义字符，确保语法合法。 | `public function getPcModule() {` |
| 🔴 严重 | `App.vue`:1-4 | **文件格式错误**：Vue 单文件组件不应以 PHP 注释风格 `/** */` 开头，且代码块被标记为 `php`。 | 移除顶部 PHP 风格注释，确保文件为标准 Vue SFC 格式。 | `<template>` 应作为文件起始内容之一。 |
| 🔴 严重 | `App.vue`:6 | **模板语法错误**：模板中存在无效文本 `wd` 和 `se`，会导致渲染异常。 | 删除模板中的无意义字符。 | `<v-header ref="vheader"></v-header>` |
| 🟠 警告 | `AppModule.php`:5 | **架构隐患**：直接在 Controller 中使用 `include` 引入父类 `PcServer.php`。若框架基于 CodeIgniter，应通过核心文件加载或自动加载机制。 | 建议将 `PcServer` 移至 `application/core/MY_Controller.php` 或框架指定的基类目录，利用自动加载。 | `class AppModule extends PcServer` (需确保基类已加载) |
| 🟠 警告 | `AppModule.php`:12 | **安全隐患**：获取配置接口未见权限验证逻辑。若 `pc_module_config` 含敏感信息，存在泄露风险。 | 在 `PcServer` 基类构造函数或本方法中添加登录态/权限校验。 | `if (!$this->auth->is_logged_in()) { show_error('Unauthorized'); }` |
| 🟠 警告 | `App.vue`:35 | **脆弱代码**：通过 `$refs.vheader.$children[1]` 直接访问子组件实例，耦合度过高，重构易崩溃。 | 使用 Vuex 状态管理或事件总线（EventBus）进行通信，避免直接操作子组件内部结构。 | `this.$emit('firstMenu', this.menunew[i]);` (配合父组件监听) |
| 🟠 警告 | `App.vue`:45 | **硬编码**：菜单 ID `'544'` 硬编码在逻辑中，业务变更需修改代码。 | 将特殊菜单配置提取到配置文件或后端动态下发。 | `if (selectedMenu.id == CONFIG.SPECIAL_MENU_ID)` |
| 🟡 建议 | `AppModule.php`:8 | **注释规范**：`@return void` 不准确，该方法实际输出响应流。 | 修改注释为 `@return void` (输出流) 或描述具体行为。 | `@description 获取 PC 模块配置并返回 JSON 响应` |
| 🟡 建议 | `App.vue`:22 | **内存泄漏风险**：使用 `$root.$on` 监听事件，未在组件销毁时移除监听。 | 在 `beforeDestroy` 生命周期钩子中调用 `$off`。 | `beforeDestroy() { this.$root.$off('firstMenuClick'); }` |
| 🟡 建议 | `AppModule.php`:1 | **框架一致性**：项目结构高度类似 CodeIgniter，但提示为 phpci。需确认 `success_response` 是否为框架标准方法。 | 查阅 phpci 文档，确认响应封装方法是否符合框架规范。 | N/A |
## 3. 总结与行动建议
### 优先修复的关键问题
1. **清除无效字符**：立即删除 `AppModule.php` 中的 `sssss`, `ddd` 以及 `App.vue` 中的 `wd`, `se` 和顶部错误注释。这是导致代码无法运行的阻断性问题。
2. **修正文件类型标记**：`App.vue` 不应被识别为 PHP 文件，确保构建工具能正确解析 Vue 组件。
3. **权限验证加固**：确认 `AppModule` 继承的 `PcServer` 基类中是否包含统一的权限校验逻辑。若无，必须在接口入口显式添加，防止配置信息泄露。
### 后续重构或优化方向
1. **控制器基类规范化**：
* 当前 `include .../PcServer.php` 的方式不符合现代 PHP 框架（包括 CodeIgniter）的最佳实践。
* **建议**：如果 `phpci` 基于 CI 架构，请将 `PcServer` 重命名为 `MY_Controller` 并放置于 `application/core/` 目录下，框架会自动加载，无需手动 `include`。
2. **前端组件解耦**：
* `App.vue` 中通过 `$children[1]` 调用子组件方法是非常危险的写法。
* **建议**：改用 **Props/Events** 模式或 **Vuex**。例如，`App.vue` 监听路由变化，更新 Vuex 中的 `activeMenu` 状态，`Header` 组件订阅该状态自动更新，而非由父组件强行调用子组件方法。
3. **配置动态化**：
* 移除前端硬编码的菜单 ID `'544'`。建议后端在返回菜单配置时，增加字段标识特殊行为（如 `is_special: true`），前端根据标识执行逻辑，而非判断具体 ID。
4. **代码审查流程优化**：
* 本次提交包含大量明显的编辑错误（乱码字符），表明提交前未经过基本的本地语法检查或 Diff 审查。建议引入 **Git Pre-commit Hook** 或 **CI 静态检查**（如 PHP lint, ESLint），阻止此类低级错误进入仓库。
### 修正后的代码片段参考
**application/controllers/AppModule.php**
```php
<?php
defined('BASEPATH') or exit('No direct script access allowed');
// 建议：确认框架是否支持自动加载核心控制器，否则保留 require_once 但需确保路径正确
// 如果 PcServer 是基类，建议移至 application/core/MY_Controller.php
require_once APPPATH . 'controllers/PcServer.php';
class AppModule extends PcServer
{
/**
* 获取 PC 模块配置
* @return void (输出 JSON 响应)
*/
public function getPcModule()
{
// 建议：在此处或基类构造函数中确认权限
// if (!$this->session->userdata('logged_in')) { ... }
$this->load->model('ahead_common_config_model');
$data = $this->ahead_common_config_model->get_data('pc_module_config');
// 确保 stream 数组初始化，避免 Notice 警告
if (!isset($this->stream)) {
$this->stream = [];
}
$this->stream['request']['result'] = $data;
$this->success_response();
}
}
```
**web/youc_business_operate_pc/src/App.vue**
```vue
<template>
<div id="app">
<v-header ref="vheader"></v-header>
<router-view ref="index_content"></router-view>
</div>
</template>
<script>
import header from '@/components/header.vue';
export default {
name: 'App',
// ... 其他配置
beforeDestroy() {
// 清理事件监听，防止内存泄漏
this.$root.$off('firstMenuClick');
},
methods: {
// 建议：避免直接访问 $children，通过 $refs 组件实例暴露的方法更安全
secondMenu(selectedMenu) {
const indexContent = this.$refs.index_content;
if (indexContent && typeof indexContent.secondMenu === "function") {
// 移除硬编码 ID，建议由配置驱动
if (selectedMenu.id === '544') {
// ...
}
}
}
}
}
</script>
<!-- 样式部分保持不变 -->
```
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774674565
|
1774674565
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
182
|
21
|
13
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1779169219
|
1779169354
|
1779169234
|
0
|
0
|
0
|
Edit
Delete
|
|
194
|
22
|
13
|
1
|
|
0
|
🔍 代码审查报告：app - Merge pull request '1' (#10 🔍 代码审查报告：app - Merge pull request '1' (#10) from app-260519 into...
|
## 自动代码审查报告
**分支**: app
**提交**: `582c55ea02676486 ## 自动代码审查报告
**分支**: app
**提交**: `582c55ea0267648620394ada0a2763c7094e68e1`
**提交人**: zhangjunnan (121158035@qq.com)
**时间**: 2026-05-19 14:13:16
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：业务逻辑覆盖较全，但代码结构呈现典型的“上帝方法”特征，职责边界模糊。存在明显的 SQL 注入风险、N+1 查询性能瓶颈、事务管理不规范及硬编码泛滥等问题。整体可维护性与安全性亟待提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_preorder_order_model.php`<br>`open_room()` 方法内 | **逻辑错误/数据污染**：在 `foreach ($in_arr as $order)` 循环中，错误地将未定义的 `$data` 推入 `$in_process` 数组，导致流程日志插入脏数据或引发 PHP Warning。 | 修正为当前循环变量 `$in_process_v`。 | `$in_process[] = $in_process_v;` |
| 🔴 严重 | `Ahead_book_order_model.php`<br>`refund_by_notify()` 等多处 | **SQL 注入风险**：多处使用字符串拼接构造 `WHERE` 或 `UPDATE` 条件，如 `'_relation_id="' . $order_data['_id'] . '" and _status=1...'`，未做参数绑定或转义。 | 全面替换为框架查询构造器（Query Builder）或参数化查询。 | `$this->db->where('_relation_id', $order_data['_id'])->where('_status', 1)->update(...)` |
| 🟠 警告 | `Ahead_book_order_model.php`<br>`get_list()` 方法 | **N+1 查询性能瓶颈**：在 `foreach ($order_info as &$v)` 循环中逐条调用 `$this->ahead_yc_order_model->get_one()`，订单量稍大时将导致数据库连接耗尽与响应超时。 | 提取所有 `relation_order_id`，使用 `WHERE IN` 批量查询后映射到数组。 | `$ids = array_column($order_info, 'relation_order_id'); $orders = $this->ahead_yc_order_model->select(['where_in' => ['_id', $ids]]);` |
| 🟠 警告 | `Ahead_book_order_model.php`<br>`Ahead_pay_log_model.php` 等 | **事务管理隐患**：混合使用 `$this->db->trans_start()` 与手动 `trans_rollback()`，且 `throwError()` 若直接 `exit/die` 会导致事务未提交/回滚，引发数据库锁表或连接泄漏。 | 统一采用 `trans_begin()` + `try/catch` + `trans_status()` 标准模式。 | 见下方重构示例 |
| 🟠 警告 | 全局 Model 文件顶部 | **框架生命周期违规**：在类外部直接执行 `$CI =& get_instance(); $CI->load->model('Simple_model');`，违反 CI3 加载规范，且每次请求都会无条件执行。 | 移除文件顶部代码，将模型加载移至 `__construct()` 或具体业务方法内。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🟡 建议 | 全局业务逻辑 | **魔法数字泛滥**：支付类型(`1,2,14`)、订单状态(`1,4`)、时间阈值(`30`)等硬编码散落各处，后期维护极易出错。 | 提取为类常量或独立配置文件，增强语义化。 | `const STATUS_PAID = 1; const PAY_PLATFORM_WX = 1; const BOOK_TIME_LIMIT_MIN = 30;` |
| 🟡 建议 | `Ahead_book_order_model.php`<br>`openRoomByCommunityOrder()` | **方法职责过重（God Method）**：单方法超 300 行，混合了订单创建、库存扣减、支付回调、硬件通知、日志记录等逻辑，违背单一职责原则。 | 拆分为 `OrderService`、`PaymentService`、`DeviceNotifyService` 等独立组件。 | 采用事件驱动或 Service 层模式解耦 |
| 🟡 建议 | 全局命名规范 | **命名不一致**：`setTablename` 与 `set_table_name` 混用；类名大小写不规范；注释代码未清理。 | 严格遵循 PSR-12 及项目统一命名约定，清理废弃代码。 | 统一使用 `set_table_name()`；删除 `//` 注释块 |
**🔧 事务安全重构示例（推荐替换现有 `trans_start` 写法）：**
```php
$this->db->trans_begin();
try {
// 1. 执行数据库操作
$this->db->insert('orders', $data);
$this->db->update('logs', $logData);
// 2. 检查事务状态
if ($this->db->trans_status() === FALSE) {
throw new Exception('数据库操作失败');
}
$this->db->trans_commit();
return true;
} catch (Exception $e) {
$this->db->trans_rollback();
// 记录日志或抛出业务异常，避免直接 exit 导致连接泄漏
log_message('error', 'Transaction failed: ' . $e->getMessage());
throw $e;
}
```
## 3. 总结与行动建议
### 🚨 优先修复的关键问题
1. **修复 SQL 注入漏洞**：立即排查所有使用字符串拼接构造 SQL 条件的地方，替换为 CI3 的 `$this->db->where()`、`$this->db->set()` 或原生预处理语句。
2. **修正循环赋值 Bug**：修复 `Ahead_preorder_order_model::open_room()` 中的 `$in_process[] = $data;` 错误，避免生产环境流程日志错乱。
3. **规范事务控制**：将 `trans_start()` 与 `throwError()` 的混用改为标准的 `try-catch` + `trans_begin()` 模式，防止异常中断导致数据库死锁。
4. **消除 N+1 查询**：重构 `get_list()` 及报表类方法，将循环内单条查询改为批量 `WHERE IN` 查询或 `JOIN`。
### 🛠 后续重构与优化方向
1. **架构分层**：当前 Model 承载了过多业务逻辑（支付、退款、硬件通信、报表计算）。建议引入 **Service 层** 处理复杂业务流程，Model 仅负责数据存取（CRUD），Controller 负责请求路由与参数校验。
2. **常量与配置集中化**：建立 `config/payment.php`、`config/order_status.php` 等配置文件，将硬编码的状态码、支付渠道、业务阈值统一管理。
3. **框架适配说明**：*注：您提供的代码结构（`$CI =& get_instance()`、`$this->load->model()`、`$this->db->trans_start()`）属于典型的 **CodeIgniter 3** 规范，而非 `phpci`（phpci 通常为持续集成工具）。若项目确为 CI3，请严格遵循其生命周期；若为自研框架，请确保基类 `Simple_model` 已正确封装事务与查询构造器。建议查阅对应框架官方文档确认组件加载机制。*
4. **自动化测试覆盖**：针对退款、开房等核心资金流转逻辑，补充单元测试与集成测试，确保金额计算、状态流转、第三方回调的幂等性与一致性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779171196
|
1779171196
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
264
|
23
|
13
|
1
|
|
0
|
🔍 代码审查报告：admin-260616 - 编译
|
## 自动代码审查报告
**分支**: admin-260616
**提交**: `c5c70d3 ## 自动代码审查报告
**分支**: admin-260616
**提交**: `c5c70d3ca38b6414ee6244a1c9e5834733f0ee99`
**提交人**: linyangrui (yangruilin888@gmail.com)
**时间**: 2026-05-21 10:58:22
---
## 📋 审查摘要
- **变更文件数**: 3
- **严重问题**: 0
- **高危问题**: 2
- **中危问题**: 2
- **建议优化**: 3
> ⚠️ **重要说明**：本次提供的变更文件均为前端 Webpack 打包压缩后的 JavaScript 文件（Vue.js + Element UI + Axios + Vuex），**未包含任何 PHP 文件**。因此，针对 `CodeIgniter 框架` 的模型/控制器命名规范、`$this->load->model()` 等后端检查项无法执行。以下审查将严格基于提供的前端代码进行。
## 🐛 发现的问题
### <font color="red">[跨文件调用] 依赖全局未声明变量可能导致运行时崩溃</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `web/backstage_pc/dist/js/index.04c3de3b.js` / `index.7ad47ea2.js`
- **行号**: 压缩代码中多处（如 `layer.open`, `$(".repeat-send")`, `de["default"].axios`）
- **问题描述**: 代码中直接使用了 `layer` (Layui)、`$` (jQuery)、`de["default"]` (Axios 实例) 等全局变量。在 Webpack 模块化环境中，若未在 `main.js` 或 `vue.config.js` 中通过 `ProvidePlugin` 注入或显式 `import`，浏览器运行时会抛出 `ReferenceError: layer is not defined` 等致命错误。
- **修复建议**:
1. 若使用 Webpack，请在 `vue.config.js` 中配置：
```javascript
const webpack = require('webpack');
module.exports = {
configureWebpack: {
plugins: [
new webpack.ProvidePlugin({
$: 'jquery',
jQuery: 'jquery',
layer: 'layui-layer', // 需确保已安装对应包
})
]
}
}
```
2. 或在组件顶部显式引入：`import axios from 'axios'; import layer from 'layui-layer';`
### [安全隐患] 敏感接口调用缺乏 CSRF 防护与明文传输风险
- **严重程度**: 高危
- **文件**: `web/backstage_pc/dist/js/index.04c3de3b.js`
- **行号**: 约 `login()`, `showMainEdit()`, `resetCode()`, `verifyforget()` 方法内
- **问题描述**:
1. 所有 POST 请求（如 `User/public_login`, `User/api_changePasswordByOldPwd`, `Sms/public_send`）均未在请求头或参数中携带 CSRF Token。若后端未做同源策略限制，极易遭受 CSRF 攻击。
2. 密码字段 `old_password`, `new_password_1` 等直接以明文形式通过 HTTP/HTTPS 传输。虽然现代 Web 依赖 HTTPS 加密，但前端未做任何防重放或基础混淆处理。
- **修复建议**:
1. 配置 Axios 拦截器自动附加 CSRF Token：
```javascript
axios.interceptors.request.use(config => {
config.headers['X-CSRF-TOKEN'] = document.querySelector('meta[name="csrf-token"]')?.content;
return config;
});
```
2. 确保全站强制 HTTPS，并在后端启用 `SameSite` Cookie 策略。
### [代码质量] Vue 组件中混用 jQuery 直接操作 DOM
- **严重程度**: 中危
- **文件**: `web/backstage_pc/dist/js/index.04c3de3b.js`
- **行号**: `resetCode()` 方法内
- **问题描述**: 在 `setInterval` 回调中使用了 `$(".repeat-send").attr("disabled","true").addClass("disabled")`。Vue 采用虚拟 DOM 管理视图，直接使用 jQuery 操作真实 DOM 会导致状态不同步、内存泄漏或组件销毁后定时器未清理的 Bug。
- **修复建议**: 改用 Vue 响应式数据控制：
```javascript
data() { return { isSendingCode: false, countdown: 0 } },
methods: {
resetCode() {
this.isSendingCode = true;
this.countdown = 60;
const timer = setInterval(() => {
this.countdown--;
if (this.countdown <= 0) {
clearInterval(timer);
this.isSendingCode = false;
}
}, 1000);
}
}
// 模板中: <button :disabled="isSendingCode" :class="{ disabled: isSendingCode }">
```
### [代码质量] 变量命名拼写错误
- **严重程度**: 中危
- **文件**: `web/backstage_pc/dist/js/index.04c3de3b.js`
- **行号**: 包厢管理表单区域 (`bxl-manage-wrap`)
- **问题描述**: 数据绑定变量 `inclue_real_mac` 存在明显拼写错误（`inclue` 应为 `include`）。虽然不影响当前运行，但会降低代码可读性，且若后续后端接口字段修正为 `include_real_mac`，将导致前后端字段不一致。
- **修复建议**: 全局搜索替换为 `include_real_mac`，并同步检查后端接口文档是否匹配。
## ✅ 代码亮点
1. **路由与状态管理清晰**：使用 Vuex 集中管理用户登录状态、菜单权限 (`menunew`) 和标签页 (`selectTabs`)，符合 Vue 大型项目最佳实践。
2. **表单验证基础完善**：手机号正则 `/^1[0-9]{10}$/` 校验准确，密码修改包含二次确认逻辑，降低了用户误操作风险。
3. **组件化程度高**：将 `v-header`、`v-menu`、登录页、后台布局拆分为独立组件，并通过 `provide/inject` 传递根级状态，架构合理。
## 📝 总体建议
1. **前端工程化规范**：当前代码为高度压缩的产物，建议保留并审查未压缩的 `.vue` 源码。压缩代码中混用 jQuery 与 Vue 是典型的技术债，建议逐步用 Vue 原生指令（`v-model`, `:disabled`, `@click`）替代 jQuery DOM 操作。
2. **安全加固**：务必在 Axios 全局拦截器中统一处理 CSRF Token、请求超时重试及错误码映射（当前 `catch` 仅提示 `"出错啦"`，不利于排查）。
3. **后端对接提醒**：由于未提供 PHP 代码，请确保后端 CodeIgniter 控制器中：
- 对 `User/public_login` 等接口启用 `csrf_protection`。
- 密码存储必须使用 `password_hash()` / `password_verify()`，严禁明文或 MD5 存储。
- 模型文件命名严格遵循 `Xxx_model.php`，控制器遵循 `Xxx.php`，避免 CI 自动加载失败。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779332302
|
1779332302
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
819
|
124
|
13
|
5
|
|
0
|
feat(GOV01): allow 90-day development registration feat(GOV01): allow 90-day development registration renewals and renew again...
|
At the user's request the renewal cap and defa At the user's request the renewal cap and default go from 30 to 90 days
(DEVELOPMENT_RENEWAL_MAX_DAYS); first-time generation still issues 30 days.
Renewed the fixture again with the same identities: bindings now expire
2027-01-07T15:38:57Z. Both renewals are recorded in `renewals`.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791560579
|
1791560584
|
1791560584
|
0
|
0
|
0
|
Edit
Delete
|
|
856
|
83
|
13
|
5
|
|
0
|
docs(integration): 蓝图补服务级对接矩阵，按 04 CHG-061 订正镜像与 H docs(integration): 蓝图补服务级对接矩阵，按 04 CHG-061 订正镜像与 H1/H5/H21...
|
## 内容
- 对接蓝图新增 §2.5 服务级对接矩阵：HR 与 02/03/04 共 20 个对 ## 内容
- 对接蓝图新增 §2.5 服务级对接矩阵：HR 与 02/03/04 共 20 个对接点，HR 侧写到 file:line，并标明 dev 实际跑通过的只有 3 条：事实写入、Scope 来源、OS 装载产品包。
- 按 04 `e50aad416`（CHG-061）订正：
- 应用清单镜像已合到 0.19.6，工作区镜像剩 5 处差异（H1）；
- 4 条 HR 事实已在 Catalog 登记为 draft，H5 的 Catalog 半边完成；
- Scope 改按 Catalog producer 接受 `juhai.hr` 组织事实，H21 部分关闭，剩载荷约束与投影器现场摄取。
- 新增缺口：
- H25：OS 清单契约仍强制双后端，HR 清单里的 Fastify 声明是占位，不能单方删除；
- H26：门店归属事实没有生产者；
- H27：IdP 令牌 `aud` 是生成的 UUID，不是 `juhai.hr`；
- H28：数字智能层自带一份 0.19.5 副本。
- §2.5 补本机现场读数（2026-10-28 00:35 UTC 前后）：
- HR API 未运行；
- 04 runtime 开了事实受理但缺 `FACT_APPLICATION_BINDINGS_JSON`，IdP 签发的机器令牌会被拒；
- Scope 来源 65281 不可达；
- OS 0.19.6 分支只在本机、未推 Gitea。
- 已确认三项：
- client-fact rc.3 与 rc.8 摘要实现逐行相同（H7 只剩发布）；
- EES 运行时以 `READ_ONLY_PROJECTION` 拒绝已撤回人域命令；
- Scope 证据 `owner: juhai.hr` 由 04 按 HR 签名组装。
- H22：本机开发基座容器已于 00:40 UTC 重建为只绑回环，数据卷沿用。
- **CI 修复**（`a174e46`）：本 PR 首次运行 26581 停在「Build locked OS compatibility sources」，原因是兼容锁指向 D3 后的 OS `1b8ae0d7`，该步却仍写死 `--filter=api-fastify`。
- 现改为经 `scripts/lib/os-host-backends.mjs` 读锁定 OS 的 `backend-profile.json` 决定宿主范围；
- `check:docs-truth` 新增 `ci-os-host-backends` 断言防回潮，对旧工作流负向判红。
- 同步落地状态：CLAUDE.md G23 / G14、两份 CHG 草稿、人域对照表。
- 已合入 `main` `46da429`：CLAUDE.md 冲突以 main 为准，只叠加本分支的 H21 订正。
## 验证（本地，PR 头 `25aecd7`，报告绑定 `a174e46`、clean）
- 通过：`check:runtime` 462/462（地板 452）、`check:ui` 46/46。用的是一次性 PostgreSQL 16 / Redis 7，只绑回环，跑完已删。
- 静态链如实停在 O22（OS main 尚无 0.19.6）。其后 governance-docs、docs-truth、lint、typecheck 通过。`check:governance` 只剩 O22，`reportProvenanceViolations` 0。
- CI 构建步骤：本地覆盖 5 种情形（nestjs-production、D3 前双宿主、缺 profile、非法 profile、profile 与 Fastify 冲突），以及锁定 OS 快照上的 turbo dry-run。
- 远端 CI 见本页检查结果。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793145750
|
1793194416
|
0
|
0
|
1
|
0
|
Edit
Delete
|
|
892
|
116
|
13
|
5
|
|
0
|
chore(reports): adopt Runtime static originals at chore(reports): adopt Runtime static originals at dc13b515 and clarify evidence snapshot...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793674166
|
1793710134
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
14
|
2
|
14
|
1
|
|
0
|
🔍 代码审查报告：pc-260401 分支 - 11111
|
## 自动代码审查报告
**分支**: pc-260401
**提交**: `81010fc873 ## 自动代码审查报告
**分支**: pc-260401
**提交**: `81010fc873c5b5fe0d871fdccb321c9f9015e9ae`
**时间**: 2026-03-17 15:19:03
**审查模型**: qwen3.5-plus
---
[2026-03-17 15:18:06] 正在调用 Qwen API 进行代码审查...
[2026-03-17 15:19:03] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260317-151806.md
### pc/alilog.php 中的引用:
由于提供的文件内容 `修改提交 dddd` 并非有效的 PHP 代码，无法进行静态语法分析来确定具体的 `include`、`require` 或类调用关系。但基于项目结构（CodeIgniter 框架风格）和文件命名规范，可以进行以下架构层面的推断分析：
1. **潜在被引用位置**:
* **配置文件**: `pc/application/config/autoload.php`
* **推断**: 此类工具文件通常会在自动加载配置中注册，以便全局使用。如果修改了日志逻辑，所有依赖自动加载的模块都会受影响。
* **控制器层**: `pc/application/controllers/` 下的所有控制器文件 (共 70+ 个)
* **推断**: 如 `Index.php`, `Order.php`, `Manage.php` 等业务控制器极有可能调用日志记录功能。
* **核心入口**: `pc/index.php` (未列出但通常存在)
* **推断**: 可能在框架初始化阶段被引入。
2. **潜在引用外部依赖**:
* **阿里云 SDK**: 文件名 `alilog` 暗示可能依赖 Alibaba Cloud Log Service SDK。
* **系统扩展**: 可能依赖 `curl`, `openssl` 等 PHP 扩展。
* **配置文件**: 可能读取 `pc/application/config/config.php` 或 `pc/application/config/log.php` (未列出) 中的配置项。
3. **变更影响评估**:
* **风险等级**: **高**
* **理由**: 日志组件属于横切关注点（Cross-Cutting Concern）。即使代码看似简单，日志功能的变更可能导致：
* 关键业务日志丢失，影响故障排查。
* 日志写入性能变化，影响接口响应速度。
* 敏感信息泄露（如果日志格式变更）。
* **建议**: 需要审查实际代码差异，确认是否改变了日志接口签名、写入逻辑或配置依赖。建议在测试环境验证日志写入是否正常。
---
## 总结与建议
1. **代码内容缺失**: 当前提供的变更内容 `修改提交 dddd` 为占位符，无法进行实质性的代码审查。请提供实际的 PHP 代码 diff。
2. **类与模型未识别**: 报告中 "PHP 类" 和 "PHP 模型文件" section 为空，建议补充具体类定义以便分析继承和依赖关系。
3. **框架识别**: 项目结构高度符合 **CodeIgniter 2/3** 框架特征。修改 `pc/alilog.php` 时需注意是否符合框架的 Library 或 Helper 规范。
4. **后续行动**:
* 获取 `pc/alilog.php` 的实际代码变更。
* 检查 `pc/application/config/autoload.php` 确认该文件是否被自动加载。
* 在测试环境验证日志服务连通性。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773731943
|
1773731943
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
41
|
18
|
14
|
1
|
|
0
|
🔍 代码审查报告：pc-260331 - 测试12233rrr333
|
## 自动代码审查报告
**分支**: pc-260331
**提交**: `ad827e7e9b ## 自动代码审查报告
**分支**: pc-260331
**提交**: `ad827e7e9b8908010adeebb35ed0062bf94b8c82`
**时间**: 2026-03-28 13:11:15
---
### 1. 总体评价
> **综合评分：4/10**
>
> **概述**：
> 代码存在严重的**卫生问题**（模板中存在无意义字符）和**架构反模式**（直接操作子组件实例、事件监听未清理）。虽然基本功能逻辑可运行，但维护成本极高，容易引发内存泄漏和运行时错误。
>
> **主要优点**：
> - 文件结构清晰，遵循了 Vue 单文件组件的基本结构。
> - 使用了 Vuex 进行状态管理（尽管用法有待优化）。
>
> **主要缺点**：
> - **代码污染**：模板中存在明显的无用字符。
> - **耦合度过高**：通过 `$children` 索引访问子组件，极度脆弱。
> - **内存泄漏风险**：全局事件总线监听未在组件销毁时移除。
> - **类型安全缺失**：大量隐式类型转换和 `any` 隐式存在。
### 2. 问题详情清单
| 严重等级 | 位置/行号 | 问题分类 | 问题描述 | 建议修改方案 |
| :---: | :---: | :---: | :--- | :--- |
| 🔴 严重 | Template L3-L4 | 代码规范 | 模板中存在无意义字符 `wd`, `se`, `ddww`，疑似调试残留或误触。 | **立即删除**所有无意义文本字符。 |
| 🔴 严重 | Script L47 | 逻辑/维护性 | 通过 `$refs.vheader.$children[1]` 直接访问子组件实例，耦合度极高，子组件结构变化会导致崩溃。 | 使用 `$refs.vheader` 暴露的方法或通过 Vuex/EventBus 通信，避免访问 `$children`。 |
| 🔴 严重 | Script L39-L41 | 性能/内存 | `$root.$on` 注册全局事件未在组件销毁时移除 (`$off`)，会导致内存泄漏。 | 在 `beforeDestroy` 生命周期中调用 `this.$root.$off('firstMenuClick')`。 |
| 🟡 警告 | Script L26 | 性能/逻辑 | 计算属性中执行 `JSON.parse`，若 store 状态频繁更新会导致重复解析，且 store 应存储对象而非字符串。 | 在 Vuex mutation 中解析 JSON，确保 state 存储的是对象结构。 |
| 🟡 警告 | Script L54 | 逻辑/安全 | 硬编码魔法字符串 `'544'`，业务逻辑与特定 ID 耦合，难以维护。 | 提取为常量配置，或通过后端配置菜单行为类型。 |
| 🟡 警告 | Script L55, L57 | 逻辑/健壮性 | 直接访问 `selectedMenu.submenu[0]` 未检查数组长度，易引发 `undefined` 错误。 | 增加可选链操作 `?.` 或长度检查 `if (submenu && submenu.length > 0)`。 |
| 🟡 警告 | Script L33 | 代码规范 | 使用 loose equality `!=` 而非 `!==`，可能导致类型转换意外。 | 统一使用严格相等运算符 `!==`。 |
| 🟢 建议 | Script L19 | 代码规范 | 组件名 `app` 应为 PascalCase (`App`) 以符合 Vue 风格指南。 | 修改 `name: 'App'`。 |
| 🟢 建议 | Script L20, L45 | 代码规范 | `data` 返回空对象，`mounted` 为空函数，存在冗余代码。 | 移除空的 `data` 返回值（若无需响应式数据）及空的生命周期钩子。 |
| 🟢 建议 | Script L16 | 代码规范 | 导入组件名 `header` 为小写，建议与文件名保持一致或使用语义化命名。 | 建议 `import Header from '@/components/headerw.vue'`。 |
### 3. 优化代码示例
```vue
<template>
<div id="app">
<!-- 移除无用字符 wd, se -->
<v-header ref="vheader"></v-header>
<router-view ref="index_content"></router-view>
</div>
</template>
<script>
// 命名建议大写，符合组件规范
import Header from '@/components/headerw.vue';
// 提取魔法字符串为常量，便于维护
const SPECIAL_MENU_ID = '544';
export default {
name: 'App',
components: {
'v-header': Header
},
computed: {
// 假设 Vuex store 中存储的已经是对象，避免在计算属性中 parse
menuList() {
return this.$store.state.menunew || [];
}
},
created() {
// 无需 $nextTick，路由信息在 created 中通常已可用
this.selectMenu();
// 绑定事件处理上下文
this._handleFirstMenuClick = this.secondMenu.bind(this);
this.$root.$on('firstMenuClick', this._handleFirstMenuClick);
},
beforeDestroy() {
// 🔴 关键修复：组件销毁前移除事件监听，防止内存泄漏
this.$root.$off('firstMenuClick', this._handleFirstMenuClick);
},
methods: {
selectMenu() {
const path = this.$router.currentRoute.path;
// 使用严格相等
if (path !== '/') {
const aPath = path.split('/');
const targetPath = aPath[1];
// 使用 find 替代 for 循环，代码更语义化
const activeMenu = this.menuList.find(item => item.url === targetPath);
if (activeMenu && this.$refs.vheader) {
// 🔴 关键修复：避免使用 $children[1]，建议 header 组件暴露特定方法或使用 $emit 向上通信
// 此处假设 header 组件有 handleFirstMenu 方法
if (typeof this.$refs.vheader.handleFirstMenu === 'function') {
this.$refs.vheader.handleFirstMenu(activeMenu);
}
}
}
},
secondMenu(selectedMenu) {
const indexContent = this.$refs.index_content;
// 健壮性检查
if (!indexContent || !selectedMenu) return;
if (typeof indexContent.secondMenu === 'function') {
// 安全访问 submenu
const firstSub = selectedMenu.submenu?.[0];
if (!firstSub) return;
if (selectedMenu.id === SPECIAL_MENU_ID) {
const secondSub = firstSub.submenu?.[0];
// 确保三级菜单存在再调用
if (secondSub && typeof indexContent.thirdMenu === 'function') {
indexContent.thirdMenu(selectedMenu, firstSub, secondSub);
}
} else {
indexContent.secondMenu(selectedMenu, firstSub);
}
}
}
}
}
</script>
<style lang="less">
@import "./assets/css/common";
html, body {
overflow: hidden;
height: 100%;
}
body {
min-width: 970px;
background: #f5f5f5;
}
#app {
height: 100%;
padding: 72px 0 0;
font-size: 14px;
font-family: 'Microsoft YaHei', Avenir, Helvetica, Arial, sans-serif;
-webkit-font-smoothing: antialiased;
-moz-osx-font-smoothing: grayscale;
}
</style>
```
### 4. 总结与行动建议
1. **立即清理代码污染**：
务必删除模板中的 `wd`, `se`, `ddww` 等无意义字符，这严重影响代码严肃性且可能导致渲染异常。
2. **修复内存泄漏与耦合**：
- 必须在 `beforeDestroy` 中清理 `$root.$on` 监听。
- 重构 `App.vue` 与 `Header` 组件的通信方式，禁止使用 `$children` 索引访问，改为通过 `$refs` 调用公开方法或使用 Vuex 状态驱动。
3. **强化类型与健壮性检查**：
- 所有数组访问（如 `submenu[0]`）前必须检查长度或使用可选链。
- 统一使用 `===` 和 `!==` 进行比较。
- 建议引入 **TypeScript** 或至少配置 **ESLint** 强规则。
**推荐 Lint 配置项：**
- `vue/no-unused-components`: 检查未使用组件。
- `vue/require-component-is`: 确保组件命名规范。
- `no-unused-vars`: 防止未使用变量。
- `eqeqeq`: 强制使用严格相等 (`eslint rule: "error"`).
- `vue/no-direct-mutate-state`: 防止直接修改 Vuex 状态。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774674676
|
1774674676
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
183
|
21
|
14
|
1
|
|
0
|
🔍 代码审查报告：pay-260519 - Merge remote-tracking branch 🔍 代码审查报告：pay-260519 - Merge remote-tracking branch 'pay/pay-260519' into...
|
## 自动代码审查报告
**分支**: pay-260519
**提交**: `b44bc1364 ## 自动代码审查报告
**分支**: pay-260519
**提交**: `b44bc1364c13ad1c28c37124418000afeb7d209c`
**提交人**: zhangjunnan (121158035@qq.com)
**时间**: 2026-05-19 13:40:30
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了较为完整的支付流水、套餐查询与门店管理业务，但整体呈现“过程式”堆砌特征。存在明显的 **SQL 注入风险**、**N+1 查询性能瓶颈**、**财务浮点数精度隐患**，且方法过长、职责混杂。代码实际基于 **CodeIgniter 3** 架构（非 `phpci`），建议后续统一遵循 CI3 规范与现代 PHP 工程实践。
- **风险等级**：🔴 高（涉及资金流水与核心业务数据，安全与事务一致性需优先处理）
---
## 2. 问题详情
| 严重程度 | 文件/方法 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_pay_log_model.php`<br>`get_vip_pay_log()` | **SQL 注入风险**：使用字符串拼接构造 `WHERE` 条件，`$vip_card` 未做任何过滤或参数绑定。 | 使用 CI 查询构造器或参数化查询，彻底杜绝拼接。 | `$this->db->where('_merchant_id', $merchant_id)->where('_vip_card', $vip_card)->get($this->table_name)->result_array();` |
| 🔴 严重 | `Ahead_room_package_infos_model.php`<br>`get_book_package_list()` / `get_hot_sale_top5()` | **SQL 注入风险**：原生 SQL 中直接拼接 `$shop_name` 等外部参数，未使用 `escape` 或预处理。 | 改用 CI Query Builder 的 `like()` 方法，或 `$this->db->escape()`。 | `$this->db->like('shop._name', $shop_name);` |
| 🔴 严重 | `Ahead_pay_log_model.php`<br>`get_bill_pay_log()` / `add_by_order()` | **财务精度丢失**：使用 `float` 直接进行金额加减（`$total_actual_pay += $v['actual_pay']`），PHP 浮点运算会导致分位误差。 | 财务计算统一使用 `bcmath` 扩展，或转为“分”为单位的整数运算。 | `$total_actual_pay = bcadd($total_actual_pay, $v['actual_pay'], 2);` |
| 🟠 警告 | `Ahead_pay_log_model.php`<br>`get_bill_pay_log()` | **N+1 查询与重复加载**：`foreach` 循环内反复执行 `$this->load->model()` 和 `get_one()`，数据库 IO 压力极大。 | 提前收集所有 `admin_id`，使用 `where_in` 批量查询；模型应在构造函数或方法入口加载一次。 | `$ids = array_unique(array_column($log_data, 'admin_id')); $users = $this->ahead_yc_merchant_user_model->get_list(['where_in' => ['_id', $ids]]);` |
| 🟠 警告 | `Ahead_pay_log_model.php`<br>`add_by_order()` / `add_by_book_order()` | **缺乏数据库事务**：支付流水插入后，紧接着调用多个关联模型更新营收、积分、地图数据。若中途失败，将导致数据不一致。 | 使用 `$this->db->trans_start()` 包裹核心写入逻辑，失败时 `trans_rollback()`。 | `$this->db->trans_start(); /* 插入与关联更新 */ $this->db->trans_complete();` |
| 🟠 警告 | `Ahead_shop_model.php`<br>`get_cache_shop_data()` | **缓存异常处理缺失**：未捕获 Redis 连接失败或 `json_decode` 解析异常，可能引发致命错误。 | 增加 `try-catch` 或条件判断，降级回源数据库查询。 | `if ($redis->ping()) { $data = $redis->get($key); }` |
| 🟡 建议 | 全局多个文件 | **魔法数字泛滥**：大量硬编码状态值（如 `1, 2, 3, 11, 14, 16, 17`）散落在逻辑判断中，可读性与可维护性差。 | 提取为类常量或独立配置类，如 `const PAY_TYPE_ORDER = 1;`。 | `if ($order['_type'] == self::PAY_TYPE_ORDER) { ... }` |
| 🟡 建议 | `Ahead_room_package_infos_model.php`<br>`get_package_price_list()` | **方法过长 & 违反单一职责**：单个方法超 200 行，混合了数据查询、时间计算、折扣逻辑、数组重组与视图格式化。 | 拆分为 `fetchPackages()`、`calculateDiscount()`、`formatForView()`，或使用 Service/DTO 模式。 | (架构级重构建议) |
| 🟡 建议 | 全局 | **框架适配说明**：代码实际使用 **CodeIgniter 3** 规范。CI3 推荐在 `__construct()` 中预加载高频模型，避免运行时重复 `load->model()`。 | 将 `$this->load->model('xxx')` 移至构造函数，或配置 `autoload.php`。 | `public function __construct() { parent::__construct(); $this->load->model('ahead_shop_config_model'); }` |
---
## 3. 总结与行动建议
### 🔑 优先修复的关键问题（P0/P1）
1. **立即修复 SQL 注入漏洞**：替换所有字符串拼接的 `WHERE` 条件与原生 `LIKE` 查询，全面改用 CI Query Builder 或参数绑定。
2. **引入财务计算安全机制**：所有涉及金额累加、抵扣、分账的逻辑，强制使用 `bcmath` 函数族（`bcadd`, `bcsub`, `bcmul`, `bcdiv`）或转为整数（分）计算。
3. **补充数据库事务控制**：支付流水写入属于强一致性场景，必须使用 `$this->db->trans_start()` / `trans_complete()` 包裹，防止部分成功导致账目不平。
### 🛠 后续重构与优化方向
1. **性能优化**：
- 消除循环内查询（N+1），改为批量 `IN` 查询或 `JOIN`。
- 高频只读接口（如套餐列表、门店信息）引入 Redis 缓存层，并设置合理的缓存失效策略（避免缓存穿透/雪崩）。
2. **架构与规范**：
- **拆分上帝类**：当前 Model 承载了过多业务逻辑（折扣计算、时间判断、视图格式化）。建议将业务逻辑抽离至 `Service` 层，Model 仅负责数据存取。
- **统一常量管理**：建立 `config/constants.php` 或独立 `Enum` 类管理支付类型、订单状态、运营场景等枚举值。
- **遵循 PSR-12**：统一命名规范（方法名驼峰、属性名小写）、补充类型声明（PHP 7.4+ 支持属性类型、方法返回类型）、规范注释块。
3. **框架适配提示**：
> 注：代码特征高度匹配 **CodeIgniter 3**。若项目确为内部定制框架 `phpci`，请核对 `load->model()`、`$this->db` 等核心组件的生命周期是否与 CI3 一致。CI3 已停止官方维护，若为长期项目，建议规划向 CI4、Laravel 或 Symfony 迁移，以获得更好的类型安全、依赖注入与现代化生态支持。
### ⚠️ 审查局限性说明
- 提供的 `Ahead_room_package_infos_model.php` 与 `Neworderservice.php` 文件末尾被截断，部分业务逻辑（如 `getOrderTypeInfo` 后半段、事务回滚逻辑、异常抛出机制）未能完整评估。建议补充完整代码后进行二次深度审查。
- 未提供数据库表结构，部分字段类型（如 `_actual_pay` 是否为 `DECIMAL`）及索引设计无法评估，建议结合 `EXPLAIN` 分析慢查询。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779169230
|
1779169230
|
0
|
0
|
0
|
0
|
Edit
Delete
|