|
164
|
18
|
114
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - Merge pull request '合并051 🔍 代码审查报告：pc-260616 - Merge pull request '合并0519最新' (#112) from pc into...
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `a90eac9c51 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `a90eac9c517a111152e50694d0cad9a322db0dcd`
**提交人**: zhangjunnan (121158035@qq.com)
**时间**: 2026-05-19 09:42:49
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码整体业务逻辑清晰，能够完成预订单列表、详情查询与退款等核心功能。但存在明显的 **N+1 查询性能瓶颈**、**事务缺失导致的数据一致性风险** 以及 **变量未初始化引发的运行时警告**。部分写法偏离现代 PHP 规范与框架最佳实践，需进行结构性优化。
- **风险等级**：🟠 中（存在性能隐患与数据一致性风险，无直接高危安全漏洞，但需尽快修复）
> 💡 **框架说明**：从目录结构、`get_instance()`、`$this->load->model()` 等特征判断，该项目高度疑似基于 **CodeIgniter 3** 或深度定制的 `phpci` 框架。以下审查基于 CI 架构规范、PSR-12 标准及现代 PHP 最佳实践。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_book_order_model.php`<br>`get_list` 方法内 | **N+1 查询与循环内写库**：在 `foreach` 中频繁调用 `ahead_user_model->get_one()` 与 `update_book_mobile()`。数据量稍大时将导致数据库连接耗尽、响应超时，且破坏事务原子性。 | 改为批量处理：收集缺失手机号的 `ahead_user_id`，使用 `WHERE IN` 一次性查询，再统一更新；或直接在初始 SQL 中 `LEFT JOIN` 用户表获取手机号。 | `// 收集ID后批量查询<br>$ids = array_column($missing_mobiles, 'ahead_user_id');<br>$users = $this->ahead_user_model->get_batch(['_id' => $ids], '_id,_mobile');<br>// 映射回原数组并批量更新` |
| 🔴 严重 | `Ahead_book_order_model.php`<br>`update_book_mobile` / `refund` | **资金/状态操作未使用事务**：涉及订单金额、状态变更及多表更新，若中途发生异常或网络中断，将导致主表与关联表数据不一致。 | 使用框架事务机制包裹关键写操作，失败时自动回滚。 | `$this->db->trans_start();<br>$this->update(...);<br>$this->ahead_book_model->update(...);<br>if ($this->db->trans_status() === FALSE) {<br> $this->db->trans_rollback();<br> throw new Exception('更新失败');<br>}<br>$this->db->trans_commit();` |
| 🟠 警告 | `Ahead_book_order_model.php`<br>`get_detail` 方法内 | **未初始化变量直接访问**：`$refund_admin` 仅在 `if (!empty($order_refund['_admin_id']))` 分支内定义，后续在 `foreach` 中访问 `$refund_admin['_name']` 时，若条件未满足将触发 `PHP Warning: Undefined variable`。 | 在方法开头初始化 `$refund_admin = [];`，或统一使用空合并运算符安全访问。 | `$refund_admin = []; // 初始化<br>...<br>$v['operator'] = $refund_admin['_name'] ?? '';` |
| 🟠 警告 | `Ahead_book_order_model.php`<br>全局第 3-4 行 | **类外部加载模型违反框架生命周期**：`$CI = &get_instance(); $CI->load->model('Simple_model');` 在类定义外执行，可能导致未初始化调用、内存泄漏或自动加载冲突。 | 移除全局加载，依赖框架自动加载或在构造函数中初始化父类依赖。 | `public function __construct() {<br> parent::__construct();<br> $this->load->model('Simple_model');<br>}` |
| 🟠 警告 | `Ahead_book_order_model.php`<br>多处方法内 | **方法内重复加载模型**：`$this->load->model()` 在 `get_list`、`get_detail` 中多次调用，增加 I/O 开销且违背单一职责。 | 将高频依赖模型统一移至 `__construct()` 中加载，或通过属性声明。 | `public function __construct() {<br> parent::__construct();<br> $this->load->model(['ahead_user_model', 'ahead_book_model', 'ahead_yc_order_model']);<br>}` |
| 🟡 建议 | `Ahead_book_order_model.php`<br>全局/方法签名 | **缺乏类型声明与规范不一致**：混合使用 `array()` 与 `[]`，无参数/返回值类型提示，不符合 PSR-12 与现代 PHP (7.4+) 规范。 | 统一短数组语法，补充类型声明，提升可读性与静态分析能力。 | `public function get_list(array $where, int $page = 0, int $page_size = 0): array` |
| 🟡 建议 | `Ahead_book_order_model.php`<br>`refund` 方法 | **依赖全局函数破坏封装**：`throwError()` 与 `bookOrderRefund()` 为全局函数，难以进行单元测试、异常追踪与依赖替换。 | 改用 `throw new \RuntimeException()`，将退款逻辑抽离为独立 Service 类。 | `if (empty($result['status'])) {<br> throw new \RuntimeException($result['msg'] ?? '退款失败');<br>}` |
| 🟡 建议 | `main.js` (前端文件) | **全局挂载非标准 Vue 实践**：将 `Vue.ctUrl`、`Vue.accMul` 等直接挂载到 Vue 构造函数，易造成全局污染且不符合 Vue 2/3 官方推荐。 | 使用 `Vue.prototype.$xxx` (Vue2) 或封装为独立工具模块按需引入。 | `Vue.prototype.$accMul = function(arg1, arg2) { ... };` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **消除 N+1 查询**：`get_list` 中的循环查库与写库是性能杀手。务必改为 `JOIN` 查询或批量 `WHERE IN` 处理，预计可将列表接口响应时间降低 60%~80%。
2. **补充数据库事务**：所有涉及金额计算、状态流转、多表更新的逻辑（`update_book_mobile`、`refund`）必须包裹在 `$this->db->trans_start()` 与 `$this->db->trans_commit()` 中，防止资金/订单数据不一致。
3. **修复未定义变量警告**：`get_detail` 中的 `$refund_admin` 需提前初始化，避免生产环境日志污染及潜在逻辑中断。
### 🛠 后续重构与优化方向
- **架构分层优化**：将 `bookOrderRefund` 等全局业务逻辑迁移至独立的 `Service` 层（如 `application/services/RefundService.php`），Model 仅负责数据存取，Controller 负责流程编排，提升可测试性与可维护性。
- **规范与现代化**：全面启用 `declare(strict_types=1);`，为所有方法补充参数与返回值类型声明。统一使用 `[]` 数组语法，遵循 PSR-12 缩进与命名规范。
- **前端工程化**：`main.js` 中的全局工具函数建议抽离至 `src/utils/` 目录，通过 `import` 按需引入或使用 Vue 插件机制注册，避免全局命名空间污染。
- **安全加固**：虽然当前未发现直接注入点，但建议对 `$where` 传入参数进行严格白名单校验；敏感字段（如手机号、支付金额）在日志输出时应脱敏处理。
> 若需针对 `phpci` 框架的特定组件（如自定义 ORM、缓存驱动或路由机制）进行深度适配审查，请提供框架核心加载文件或官方文档链接，我将进一步补充针对性建议。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779154969
|
1779154969
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
627
|
18
|
194
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 用户申请退款后设置
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `ad53eaf6e2 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `ad53eaf6e225d82432bb6e89ba0de587a54cf0c6`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-09 11:20:14
---
## 1. 审查摘要
- **代码质量评分**：4/10 分
- **总体评价**：当前代码主要为静态业务配置数组，但存在严重的框架生命周期误用、内存性能隐患及代码截断问题。配置数据硬编码在模型实例属性中，不符合现代 PHP 架构与 PSR-12 规范，可维护性与扩展性较差。
- **风险等级**：🟠 中（主要源于架构设计缺陷与潜在内存开销，无直接安全漏洞）
> 📌 **框架说明**：根据目录结构（`system/`, `application/`）及 `get_instance()`、`$this->load->model()` 等特征，该代码高度符合 **CodeIgniter 3 (CI3)** 架构。若 `phpci` 为基于 CI3 的二次开发框架，以下建议同样适用；若为独立框架，请以 `phpci` 官方文档的模型加载规范为准。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 1-3 | 模型文件顶部直接使用 `$CI = &get_instance();` 与 `$CI->load->model('Simple_model');`。该代码会在文件被 `include/require` 时立即执行，破坏框架生命周期，极易导致 `CI_Controller` 未初始化报错或重复加载。 | 移除顶部过程式代码。模型继承关系由 PHP 原生机制处理，若需加载依赖或初始化父类，应在 `__construct()` 中调用 `parent::__construct()`。 | `class Ahead_community_shop_model extends Simple_model {<br> public function __construct() {<br> parent::__construct();<br> }<br>}` |
| 🟠 警告 | 全文 | 超大嵌套数组直接定义为**实例属性** `$operational_scene_config`。每次 `new` 该模型都会重复分配数百 KB 内存，高并发下易引发内存泄漏与 GC 压力。 | 改为 `private static` 属性，或抽离至 `application/config/` 配置文件，或使用 CI 缓存组件。模型仅负责读取/转换。 | `private static $operational_scene_config = [...];<br>public static function getSceneConfig(int $sceneId): array {<br> return self::$operational_scene_config[$sceneId] ?? [];<br>}` |
| 🟠 警告 | 末尾 | 代码在 `'config_params'` 处被截断，缺失数组闭合 `];`、类闭合 `}` 及后续场景配置。无法验证语法完整性与逻辑闭环。 | 补充完整代码。确保所有嵌套数组正确闭合，并检查 PHP 语法错误（如 `php -l` 静态检查）。 | *(需补充完整代码后重新审查)* |
| 🟡 建议 | 1 | 类名 `Ahead_community_shop_model` 使用蛇形命名，不符合 PSR-12 的 `PascalCase` 规范，也不利于现代 IDE 自动补全与静态分析。 | 重命名为 `AheadCommunityShopModel`，并全局同步更新引用路径。若框架强制要求蛇形，请保留但补充 `@property` 注释。 | `class AheadCommunityShopModel extends Simple_model` |
| 🟡 建议 | 全文 | 配置结构极度嵌套且硬编码在业务模型中，违反“配置与逻辑分离”原则。后续新增场景或修改字段需改动模型文件，增加发布风险。 | 将配置抽离为独立配置文件（如 `config/scene_config.php`）或 JSON/YAML，模型通过 `$this->config->load()` 或 `json_decode()` 读取。 | `// application/config/scene_config.php<br>return [<br> '1' => [...],<br> '2' => [...]<br>];` |
| 🟡 建议 | 全文 | 缺乏类型声明与结构化注释。PHP 7.4+ 支持属性类型与数组形状定义，缺失会导致 IDE 提示弱、重构困难。 | 添加 `declare(strict_types=1);`，使用 PHPDoc 定义数组结构，提升可读性与静态检查能力。 | `/** @var array<string, array<string, mixed>> */<br>public array $operational_scene_config = [];` |
## 3. 总结与行动建议
### 🔑 优先修复项
1. **移除顶部 `get_instance()` 滥用**：立即删除文件第 1-3 行的过程式代码，改用标准 CI 模型构造函数初始化。此问题可能导致线上环境白屏或 Fatal Error。
2. **补全截断代码并验证语法**：使用 `php -l application/models/Ahead_community_shop_model.php` 进行语法检查，确保数组与类结构完整闭合。
3. **内存优化**：将 `$operational_scene_config` 改为 `static` 属性或移至配置文件，避免每次实例化重复分配内存。
### 🛠 后续重构方向
- **配置与模型解耦**：当前配置属于“视图/表单渲染元数据”，不应与数据访问层（Model）耦合。建议迁移至 `application/config/` 或独立 JSON 文件，通过配置加载器按需读取。
- **引入配置校验机制**：硬编码数组缺乏运行时校验。建议在读取配置时增加类型/必填字段校验（如使用 `symfony/options-resolver` 或自定义验证器），防止前端传参异常导致渲染崩溃。
- **遵循 PSR-12 与现代化 PHP 规范**：统一类名命名规范，补充严格类型声明与 PHPDoc。若项目已升级至 PHP 8.0+，可考虑使用 `readonly` 属性或 `enum` 替代魔法数字（如 `'1'`, `'2'` 场景标识）。
- **安全输出提醒**：若该配置后续用于生成 HTML 表单，请确保在视图层使用 `htmlspecialchars()` 或 CI 的 `form_*` 辅助函数进行 XSS 过滤，避免配置中的 `tips`、`remark` 字段被恶意注入。
> 💡 **局限性说明**：由于提交的代码在末尾被截断，本次审查仅基于可见部分进行静态分析与架构评估。完整代码提交后，可进一步审查业务方法逻辑、数据库交互及异常处理机制。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780975214
|
1780975214
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
586
|
18
|
188
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 其他
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `b41c93c7e5 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `b41c93c7e520401cc90f6f3def5c2625f45983ce`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-06-08 13:55:15
---
## 1. 审查摘要
- **代码质量评分**：4 / 10 分
- **总体评价**：代码存在明显的逻辑断层、SQL 语法错误及潜在的安全注入风险。整体架构风格高度类似 CodeIgniter 3，但缺乏现代 PHP 的类型约束、异常处理机制及规范的模型加载方式。若直接投入生产环境，极易引发查询失败、数据越权或 SQL 注入问题。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 顶部全局作用域 | 在类外部使用 `$CI = &get_instance(); $CI->load->model()` 加载模型。这违反了框架生命周期，可能导致未定义变量、重复加载或内存泄漏。 | 移除顶部加载代码。依赖框架自动加载器，或在控制器/构造函数中按需加载。若 `Simple_model` 为基类，应确保其已被正确注册。 | `// 删除顶部两行代码`<br>`class AheadSkinShopSetModel extends Simple_model { ... }` |
| 🔴 严重 | `get_list` / `$fields` 定义 | SQL 字段字符串存在双逗号 `,,`，将直接导致底层数据库驱动抛出 SQL 语法错误，查询中断。 | 清理冗余符号，保持字段列表规范。 | `$fields = "_skin_type as skin_type, _skin_scene as skin_scene, _skin_key as skin_key";` |
| 🔴 严重 | `get_list` / `$where` 逻辑 | 传入的 `$shop_id` 和 `$merchant_id` 完全未被使用，查询条件硬编码为默认值 `0`。业务逻辑与接口预期严重不符，可能导致返回错误数据。 | 将动态参数注入 `$where` 数组，或明确区分“全局默认配置”与“门店专属配置”的查询分支。 | `$where['_merchant_id'] = $merchant_id;`<br>`$where['_shop_id'] = $shop_id;` |
| 🟠 警告 | `get_list` / `$order` 参数 | `$order` 参数未经校验直接传入底层查询方法。若攻击者传入恶意字符串（如 `1; DROP TABLE users--` 或复杂子查询），将引发 **ORDER BY SQL 注入**。 | 实施白名单校验，仅允许预定义的排序字段与方向。 | `if (!preg_match('/^(skin_type\|skin_scene)\s+(ASC\|DESC)$/i', $order)) { $order = 'skin_type ASC'; }` |
| 🟠 警告 | `get_list` / `foreach` | 未校验 `$data['rows']` 是否为数组。若底层 `listinfos` 返回空值或结构变更，将触发 `Warning: Invalid argument supplied for foreach()`。 | 增加类型与存在性检查，提升代码健壮性。 | `if (!empty($data['rows']) && is_array($data['rows'])) { foreach ($data['rows'] as &$v) { ... } }` |
| 🟠 警告 | `get_shop_skin` / `$fields` | 字段列表重复定义 `_skin_scene as skin_scene` 两次，属于冗余代码，部分严格模式的数据库驱动会抛出警告。 | 移除重复字段，保持查询精简。 | `$fields = "_skin_scene, _skin_key, _skin_id";` |
| 🟡 建议 | 全局类定义 | 类名 `Ahead_skin_shop_set_model` 使用蛇形命名，不符合 PSR-12 规范（类名应为 PascalCase）。方法参数无类型声明，降低可读性与 IDE 提示能力。 | 重命名类为 `AheadSkinShopSetModel`，补充 PHP 7+ 类型提示与返回值声明。 | `class AheadSkinShopSetModel extends Simple_model {`<br>`public function get_list(int $merchant_id, array $params, int $page, int $page_size, string $order): array {` |
| 🟡 建议 | `get_list` / 错误处理 | `throwError()` 为全局函数调用，未使用现代 PHP 异常机制。若框架未全局捕获，将直接暴露堆栈信息或导致脚本终止。 | 改用标准异常类或框架内置异常，便于上层统一拦截与日志记录。 | `throw new \InvalidArgumentException('请选择门店');` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复 SQL 语法错误**：立即修正 `get_list` 中 `$fields` 的双逗号问题，否则该接口将 100% 报错。
2. **补全业务逻辑**：将 `$merchant_id` 与 `$shop_id` 正确映射到 `$where` 条件中，或明确当前方法是否仅用于查询“全局默认皮肤配置”。若需查询门店配置，需调整过滤逻辑。
3. **防御 ORDER BY 注入**：对 `$order` 参数实施严格的白名单过滤，禁止直接拼接用户输入到 SQL 语句中。
### 🛠 后续重构与优化方向
- **统一模型加载机制**：若 `phpci` 框架基于 CodeIgniter 架构，请遵循其模型自动加载规范。避免在文件顶层使用 `get_instance()`，推荐在控制器中 `$this->load->model('ahead_skin_shop_set_model')` 或通过依赖注入容器管理。
- **引入类型约束与异常处理**：全面补充参数类型声明（`int`, `array`, `string`）及返回值类型。将全局错误函数替换为 `try-catch` 或框架标准异常，提升可维护性与调试效率。
- **查询逻辑优化**：`get_shop_skin` 中的“先查门店，后查默认”逻辑可考虑合并为单次查询（例如：`ORDER BY _merchant_id DESC, _shop_id DESC LIMIT 1`），减少数据库往返开销。
- **框架适配确认**：代码中使用的 `listinfos`、`get_one` 及 `_field>` 语法属于 `Simple_model` 的自定义实现。建议查阅 `phpci` 官方文档或 `Simple_model` 源码，确认其是否已内置参数绑定（Prepared Statements）与防注入机制。若未内置，需手动使用查询构建器或 PDO 预处理。
> 💡 **注**：当前代码结构高度契合 CodeIgniter 3.x 规范。若 `phpci` 为内部定制框架，请确保其生命周期与 CI3 保持一致。建议在修复上述问题后，补充单元测试覆盖边界条件（如空参数、非法排序、无匹配数据等）。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780898115
|
1780898115
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
348
|
18
|
153
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 预留清洁时间设置
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `b63a006a4b ## 自动代码审查报告
**分支**: pc-260616
**提交**: `b63a006a4b4bd55a499e916ea3dfdfdcdf7991a2`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-26 17:51:27
---
## 1. 审查摘要
- **代码质量评分**：4/10 分
- **总体评价**：当前提交的代码片段仅包含一个庞大的配置数组定义，缺乏实际业务方法。代码存在严重的语法截断、框架生命周期违规调用以及架构设计反模式。配置数据硬编码在 Model 中导致内存冗余与维护困难，需进行结构性重构。
- **风险等级**：🔴 高
> 📌 **框架说明**：根据提供的目录结构（`system/`, `application/`, `DB_driver.php` 等），该代码高度符合 **CodeIgniter 3** 架构。若 `phpci` 为内部定制框架，请确保其核心加载机制与 CI3 一致。以下审查基于标准 PHP 与 CI3 最佳实践。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 文件顶部 (1-3行) | 在类定义外直接执行 `$CI = &get_instance();` 与 `$CI->load->model()`。PHP 在解析/引入该文件时，CI 核心实例尚未初始化，将触发 `Fatal Error` 或破坏框架生命周期。 | 移除顶部过程式代码。父模型应通过自动加载机制（如 `application/core/MY_Model.php`）或 `require` 引入，无需在模型文件中手动加载。 | `// 直接删除以下两行：<br>$CI = &get_instance();<br>$CI->load->model('Simple_model');` |
| 🔴 严重 | 文件末尾 | 代码在 `'config_params` 处被截断，数组未闭合。直接运行将导致 `Parse error: syntax error, unexpected end of file`，系统无法启动。 | 补全缺失的数组闭合符号 `], ], ], ];`，并确保所有嵌套层级正确匹配。 | `// 补全结构示例：<br>...<br> ]<br> ]<br> ]<br> ];<br>}<br>` |
| 🟠 警告 | 类属性定义 | `$operational_scene_config` 作为公共实例属性，每次 `new` 该模型时都会在内存中完整实例化该巨型数组，造成不必要的内存开销与 GC 压力。 | 改为 `public static $operational_scene_config`，或彻底抽离至独立配置文件。 | `public static $operational_scene_config = [...];` |
| 🟠 警告 | 架构设计 | 将庞大的 UI/表单 Schema 配置硬编码在 Model 中，严重违反单一职责原则（SRP）。Model 应专注数据持久化，配置管理应独立。 | 将配置移至 `application/config/community_shop_config.php`，通过 `$this->config->load()` 或 `config_item()` 按需读取。 | `// application/config/community_shop_config.php<br>$config['scene_config'] = [...];<br>// 模型中调用：<br>$this->config->load('community_shop_config', TRUE);<br>$config = $this->config->item('scene_config');` |
| 🟡 建议 | 类定义 | 缺少标准构造函数，未调用 `parent::__construct()`。在 CI3 中，若父类有初始化逻辑（如加载 DB、Helper），跳过构造函数会导致父类功能失效。 | 补充标准构造函数，显式调用父类初始化。 | `public function __construct() {<br> parent::__construct();<br>}` |
| 🟡 建议 | 数组结构 | 场景 ID 使用魔法数字 `'1'`, `'2'`，可读性差且易出错。配置项高度重复，后期扩展维护成本极高。 | 使用类常量或枚举定义场景标识；考虑将配置转为 JSON 存储或迁移至数据库动态管理，实现后台可视化配置。 | `const SCENE_KTV = 1;<br>const SCENE_BILLIARDS = 2;<br>public $operational_scene_config = [<br> self::SCENE_KTV => [...],<br> self::SCENE_BILLIARDS => [...]<br>];` |
## 3. 总结与行动建议
### 🚨 优先修复项（必须处理）
1. **修复语法截断**：立即补全数组闭合符号，确保文件可被 PHP 解析器正常加载。
2. **移除顶部过程式代码**：删除 `$CI = &get_instance();` 及 `$CI->load->model()`，避免框架启动阶段崩溃。
3. **补充构造函数**：添加 `__construct()` 并调用 `parent::__construct()`，确保继承链完整。
### 🛠 后续重构与优化方向
1. **配置与逻辑分离**：当前配置数据量极大且结构固定，强烈建议将其迁移至 `application/config/` 目录。若配置需支持后台动态修改，应设计独立的 `shop_config` 数据表，通过 Model 提供 `getConfig($scene_id)` 方法读取，而非硬编码。
2. **内存与性能优化**：若暂不迁移配置，务必将 `$operational_scene_config` 声明为 `static`，避免多实例化时的内存重复分配。
3. **类型安全与可维护性**：引入 PHP 7+ 类型声明（如 `array`、`string`），为配置结构定义 DTO 类或使用 `json_decode` + `json_validate` 进行运行时校验，防止前端传入非法结构导致渲染异常。
4. **框架适配确认**：若 `phpci` 为自研框架，请确认其模型加载机制是否支持 `application/core/` 下的基类自动加载。若不支持，需在 `composer.json` 或框架入口文件中配置 PSR-4 自动加载规则。
> ⚠️ **局限性说明**：本次审查仅基于提供的配置数组片段。由于未包含任何业务方法（如 CRUD、校验逻辑、事务处理等），无法对数据库查询效率、SQL 注入防护、权限校验等核心安全与逻辑维度进行深度评估。建议补充完整业务逻辑代码后再次提交审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779789087
|
1779789087
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
477
|
18
|
184
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 1
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `bd3330752c ## 自动代码审查报告
**分支**: pc-260616
**提交**: `bd3330752ce9ecce765904957b4de347d25b21f0`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 18:13:07
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10
- **总体评价**：代码实现了核心报表查询与数据组装逻辑，但存在明显的架构设计缺陷（全局实例获取位置错误）、严重的 N+1 查询性能瓶颈、潜在的 SQL 注入风险以及多处逻辑边界处理不严谨。整体可维护性与安全性需大幅提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 顶部 1-2 行 | `$CI = &get_instance();` 置于类定义外部。PHP 会在文件被 `include` 时立即执行，此时框架可能未完成初始化，极易引发 `Fatal Error` 或 `$CI` 为空。且类内部未复用该变量。 | 移除顶部代码。在 `__construct()` 中统一加载依赖模型，或使用 `$this->load->model()` 按需加载。 | `public function __construct() { parent::__construct(); $this->load->model(['Report_model', 'ahead_yc_shop_model', ...]); }` |
| 🔴 严重 | `get_community_revenues_list` 约 115-125 行 | `$pay_platform_where` 数组直接拼接用户输入参数 `$pay_platform` 与 `$pay_platform_arr[1]`。若底层 `select()` 未做严格转义，将导致 **SQL 注入**。 | 禁止手动拼接 SQL 片段。改用框架查询构建器的 `or_where`/`where_in`，或对参数进行严格类型转换与转义。 | `$this->db->group_start(); foreach(...) { $this->db->or_where('a._pay_platform', (int)$p); ... } $this->db->group_end();` |
| 🔴 严重 | `get_community_revenues_list` 约 155-175 行 | `foreach ($data as &$v)` 循环内，当 `order_type == '1'` 时逐行调用 `get_one()` 查询预订信息。数据量 >100 时将引发严重的 **N+1 查询**，拖垮数据库。 | 提取所有需查询的 `order_id`，使用 `where_in` 批量查询，在内存中通过 `order_id` 映射关联数据。 | `$ids = array_filter(array_column($data, 'order_id')); $books = $this->ahead_book_order_model->get_data_by_ids($ids, ...); $bookMap = array_column($books, null, '_id');` |
| 🟠 警告 | `get_community_revenues_list` 约 135-140 行 | `$count` 与 `$sum_data` 仅在 `$params['page'] == '1'` 时赋值。请求第 2 页及以上时，变量未定义，虽被 `??` 掩盖，但违背业务语义（分页应始终返回总数与合计）。 | 移除 `if ($params['page'] == '1')` 限制，或初始化默认值。分页统计应在第一页或独立接口计算。 | `$count = 0; $sum_data = ['total_amount' => 0]; if ((int)$params['page'] === 1) { ... }` |
| 🟠 警告 | `get_community_revenues_list` 约 85-88 行 | `strtotime($params['start_time'])` 未校验时间格式。非法字符串返回 `false`，导致数据库查询条件变为 `a._create_time >= 0`，可能查出全量历史数据。 | 增加时间格式校验，失败时抛出异常或返回空结果。 | `if (!preg_match('/^\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}$/', $params['start_time'])) { return ['count'=>0, 'data'=>[]]; }` |
| 🟠 警告 | `get_search_params` 约 65 行 | `json_decode($params['order_type_arr'])` 未传第二个参数 `true`，默认返回 `stdClass` 对象。后续 `is_array()` 判断恒为 `false`，导致参数被强制置空。 | 补充 `true` 参数，或使用 `json_decode($str, true) ?? []`。 | `$params['order_type_arr'] = json_decode($params['order_type_arr'], true) ?? [];` |
| 🟡 建议 | 全局 | 类名 `Jh_community_shop_revenues_detail_model` 不符合 PSR-12 大驼峰命名规范。CI3 传统为下划线，若项目强制 PSR-12 需统一。 | 改为 `JhCommunityShopRevenuesDetailModel`，并确保 Composer 自动加载或框架路由兼容。 | `class JhCommunityShopRevenuesDetailModel extends Report_model` |
| 🟡 建议 | `get_community_revenues_list` 约 105-110 行 | `join` 条件中 `if/else` 分支未统一指定连接类型（默认 `INNER` vs 显式 `LEFT`），可能导致运营场景过滤时数据丢失或冗余。 | 显式声明连接类型，保持逻辑一致性。 | `$where['join'][] = ['ktv_online.ahead_merchant_room_type b', 'b._id=a._room_type', 'left'];` |
| 🟡 建议 | 全局 | 频繁调用 `$this->load->model()`。CI 框架虽有缓存，但重复调用仍增加 I/O 开销。 | 将模型加载收敛至构造函数，或使用 `$this->load->model('xxx', '', TRUE)` 避免重复实例化。 | 见 🔴 严重第 1 条示例 |
> ⚠️ **框架适配说明**：代码结构高度类似 **CodeIgniter 3**。若 `phpci` 为自研或定制框架，请重点核对底层 `select($where)`、`enforce_con_db()` 及 `where_in` 的实现机制是否与 CI 一致。若底层未对 `$where` 数组做自动转义，🔴 SQL 注入风险将直接生效。
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **移除全局 `$CI` 实例获取**：改为构造函数依赖注入或按需加载，避免文件加载期致命错误。
2. **修复 SQL 注入隐患**：彻底废弃手动拼接 `WHERE` 字符串，全面改用框架查询构建器或参数化绑定。
3. **解决 N+1 查询**：将循环内的单条 `get_one()` 替换为批量 `where_in` 查询 + 内存映射，预计可降低 80%+ 数据库交互次数。
4. **修正 JSON 解析与分页统计逻辑**：`json_decode` 补充 `true` 参数；确保 `$count` 与 `$sum_data` 在任意分页下均有合理默认值。
### 🛠 后续重构与优化方向
- **参数校验层前置**：建议在 Controller 或独立 Validator 中统一校验 `$params`（时间格式、数组类型、分页范围），Model 层仅负责纯净的数据查询。
- **消除魔法值**：将 `'1'`, `'2'`, `'8'`, `'9'` 等硬编码提取为类常量（如 `const PAY_PLATFORM_DOUYIN = '9_1';`），提升可读性与后期维护效率。
- **类型声明与 PSR-12 规范**：若运行环境为 PHP 7.4+，建议为属性与方法参数添加类型提示（如 `public function get_search_params(int $merchant_id, int $shop_id): array`），并统一使用大驼峰命名。
- **依赖解耦**：`filter_emoji`、`change_number_format` 等全局函数建议封装为 `Helper` 或 `Service` 类，通过依赖注入调用，避免隐式全局依赖导致的测试困难。
- **数据库连接切换安全**：`enforce_con_db()` 与 `enforce_con_db(2)` 疑似读写分离或分库切换。建议增加 `try-finally` 块确保连接状态恢复，防止异常中断导致后续请求路由错乱。
> 📝 **局限性说明**：本次审查基于提供的单文件代码。由于未包含 `Report_model` 基类实现、全局 Helper 函数定义及框架底层查询构建器源码，部分安全与性能评估基于 CI3 标准实践推断。建议结合完整项目上下文进行集成测试验证。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780395187
|
1780395187
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
332
|
18
|
146
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 需求 赠时报表 16382
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `bd90284947 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `bd90284947362fa60b96e2b14246fd25bcf4ae79`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-05-26 15:38:27
---
## 1. 审查摘要
- **代码质量评分**：N/A（待补充代码）
- **总体评价**：当前提交仅包含项目目录结构，**未提供具体的变更代码内容**。基于您提供的 `system/` 目录结构，该项目高度疑似基于 CodeIgniter 3/4 架构（或内部定制版 `phpci`）。在缺乏实际代码的情况下，无法进行逻辑、安全及性能维度的深度审查。
- **风险等级**：未知（需代码补充后评估）
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 全局/未提供 | **缺失变更代码**：未提交实际修改的 PHP 文件内容或 Diff 信息，导致审查无法开展。 | 请提供 `git diff` 输出、具体文件路径及变更代码片段。建议包含上下文至少 10-20 行。 | `git diff HEAD~1` 或粘贴具体文件内容 |
| 🟡 建议 | 架构层面 | 目录结构显示为传统 MVC 框架布局。若为 CI3，需注意 PHP 8.x 兼容性；若为自研 `phpci`，请确认是否已实现 PSR-4 自动加载。 | 建议明确框架版本及 PHP 运行环境版本，以便针对性审查语法兼容性与依赖注入规范。 | - |
## 3. 总结与行动建议
- **优先修复的关键问题**：
1. 请补充本次提交的实际变更代码（建议以 Diff 格式或完整文件内容提供）。
2. 明确 `phpci` 框架的具体版本及底层依赖（如是否基于 CI3/CI4 二次开发，或为完全自研框架）。
- **后续重构或优化的方向性指导**（待代码补充后将重点审查）：
- **安全性**：重点检查 `DB_driver.php` 相关查询是否严格使用参数绑定（防 SQL 注入）；`security_helper.php` 与 `form_helper.php` 是否正确使用 `xss_clean()` 或 `htmlspecialchars()`；`cookie_helper.php` 与 `Session` 驱动是否启用 `httponly`、`secure` 及 `SameSite` 策略。
- **性能优化**：审查 `Cache/` 与 `Session/` 驱动配置，避免在循环中执行数据库查询或重复实例化类；检查 `Upload.php` 与 `Image_lib.php` 是否合理处理大文件内存占用（如使用流式读取/分块处理）。
- **代码规范**：确保新增代码遵循 PSR-12，类名/方法名符合驼峰命名，合理使用类型声明（PHP 7.4+/8.0+），并补充完整的 PHPDoc 注释。移除冗余的 `require/include`，优先使用框架自动加载机制。
- **框架适配**：若使用 `phpci` 生命周期钩子或事件系统，请确认未破坏核心路由或请求分发机制；数据库操作建议统一使用 Query Builder 或 ORM，避免原生 SQL 拼接；若涉及自定义 Helper/Library，请确认是否遵循框架的加载规范（如 `$this->load->helper()` 或自动加载配置）。
> 💡 **提示**：请回复补充具体的变更代码片段（或提供 `git diff` 输出）。收到后我将立即按照上述维度输出完整、可落地的深度审查报告，并提供可直接替换的优化代码。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779781107
|
1779781107
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
169
|
18
|
117
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 测试
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `bfaec9f980 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `bfaec9f9802097c0f7a2014143a64f3a74fc6d20`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-05-19 10:59:07
---
## 1. 审查摘要
- **代码质量评分**：6/10
- **总体评价**：代码实现了计时开房详情的核心 CRUD 逻辑与基础业务校验，但存在明显的变量拼写错误、框架使用不规范、重复加载依赖及时间边界处理隐患。整体逻辑可跑通，但健壮性、可维护性与安全性需重点优化。
- **风险等级**：🟠 中（存在运行时致命错误隐患、潜在 SQL 注入风险及性能冗余）
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `add_room_timing_detail` (约第58行) | 循环内误用未定义变量 `$param` 而非 `$params`，将触发 PHP `Notice/Error`，导致 VIP 价格数据写入失败或覆盖为默认值。 | 修正变量名，确保与参数声明一致。 | `$addData['_vip_level' . $i . '_price'] = $params['vip_level' . $i . '_price'] ?? 0;` |
| 🔴 严重 | 文件顶部 (第5-6行) | 在类外部使用 `$CI = &get_instance();` 并加载模型。违反 OOP 封装原则，在 CLI 环境或并发请求下易引发未定义行为或内存泄漏。 | 移除全局 `$CI` 调用。依赖加载应移至构造函数或具体业务方法内。 | 删除顶部两行，改用 `$this->load->model()` 按需加载。 |
| 🟠 警告 | `_validate_params` (约第118行) | `strtotime()` 未校验返回值。若传入非法时间格式（如 `2023-13-01`），将返回 `false`，后续 `-1` 及大小比较将产生类型错误与逻辑漏洞。 | 增加时间解析校验，失败时直接拦截。 | `if ($startTime === false || $endTime === false) throwError('时间格式不合法');` |
| 🟠 警告 | `get_timing_holiday_list` (约第88行) | `$where['_name LIKE '] = '%' . $params['name'] . '%';` 依赖底层 `Simple_model` 是否自动转义。若未使用预处理，存在 SQL 注入风险。 | 使用框架标准查询构造器或显式转义，避免手动拼接 LIKE 条件。 | `$this->db->like('_name', $params['name']);` (若底层支持) |
| 🟠 警告 | 多处方法内 | 重复调用 `$this->load->model('ahead_vip_level_model')` 及 `count(...)`。频繁加载模型与访问静态常量增加 I/O 与 CPU 开销。 | 将模型加载移至构造函数，将 VIP 等级上限提取为类属性缓存。 | `private $vipMaxLevel; public function __construct() { parent::__construct(); $this->load->model('ahead_vip_level_model'); $this->vipMaxLevel = count(ahead_vip_level_model::VIP_LEVEL_DEFAULT_NAME); }` |
| 🟡 建议 | `_validate_params` (价格校验循环) | 使用 `strpos` 匹配价格字段不够严谨，且 `$value < 0` 未做类型转换，字符串比较在 PHP 8+ 中可能抛出 `TypeError`。 | 明确价格字段白名单，或使用 `is_numeric()` + 强制浮点转换。 | `if (in_array($key, $priceKeys, true) && (float)$value < 0) throwError('...');` |
| 🟡 建议 | 全局函数 `throwError()` | 使用全局函数中断流程不符合现代 PHP 异常处理规范，不利于单元测试、错误堆栈追踪与统一响应拦截。 | 建议替换为抛出标准异常（如 `InvalidArgumentException`），由控制器或中间件统一捕获。 | `throw new \InvalidArgumentException('节假日名称不能为空');` |
| 🟡 建议 | 类/方法命名 | 类名 `Ahead_room_timing_detail_model` 与方法名未遵循 PSR-12 驼峰命名规范，降低代码可读性与 IDE 自动补全体验。 | 建议重构为 `AheadRoomTimingDetailModel` 及 `deleteRoomTimingDetail()` 等。 | 符合 PSR-12 命名约定。 |
| 🟡 建议 | `get_timing_holiday_list` 字段拼接 | 循环内使用 `$fields .= ...` 拼接 SQL 字段字符串，效率低且不易维护。 | 使用数组收集字段名，循环结束后通过 `implode()` 合并。 | `$fieldsArr = []; foreach(...) $fieldsArr[] = "..."; $fields = implode(',', $fieldsArr);` |
> 💡 **框架适配说明**：当前代码结构、`$this->load->model()`、`$this->insert()` 等调用方式高度契合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请确认其底层 `Simple_model` 是否已内置 PDO 预处理与自动转义机制。若未内置，上述 SQL 注入风险需优先处理。
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复致命拼写错误**：立即将 `add_room_timing_detail` 中的 `$param` 更正为 `$params`，否则新增 VIP 价格功能将直接失效。
2. **移除全局 `$CI` 实例**：将 `$CI = &get_instance();` 及外部 `load->model()` 删除，改为在类内部按需加载，避免上下文污染。
3. **强化时间校验**：为 `strtotime()` 增加返回值校验，防止非法时间字符串导致业务逻辑崩溃或产生错误的时间重叠判断。
### 🛠 后续重构与优化方向
- **统一异常处理机制**：逐步废弃全局 `throwError()`，引入 `try-catch` 与自定义异常类（如 `BusinessException`），配合全局错误处理器返回标准化 JSON 响应。
- **提取 VIP 配置逻辑**：将 VIP 等级字段生成、价格校验逻辑抽离为独立方法（如 `buildVipFields()`、`validateVipPrices()`），消除 `add`/`update`/`list` 中的重复代码，提升可维护性。
- **规范数据库交互**：若底层 `Simple_model` 不支持自动转义，请全面改用框架提供的 Query Builder 链式调用（如 `$this->db->where()`, `$this->db->like()`），彻底杜绝 SQL 注入隐患。
- **遵循 PSR-12 规范**：在后续迭代中逐步重命名类与方法，补充 PHPDoc 类型声明（如 `@param int $relationId`），并启用 `PHP_CodeSniffer` 进行静态检查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779159547
|
1779159547
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
475
|
18
|
182
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 其他
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `c11b7058b3 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `c11b7058b3d315da90b1a82cf4fb60358419403d`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-06-02 17:57:55
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码实现了核心业务逻辑，具备基本的分页、导出与数据聚合能力。但存在**严重的 SQL 注入风险**与**JSON 解析逻辑缺陷**，且类外部调用 `get_instance()` 违反面向对象规范。多处模型重复加载、缺乏输入校验及硬编码魔法值，影响了可维护性与执行效率。
- **风险等级**：🔴 高（存在直接可利用的安全漏洞与逻辑阻断点）
> 📌 **框架说明**：代码结构高度类似 `CodeIgniter 3`。若 `phpci` 为内部定制框架，请结合其官方文档对查询构建器、模型加载机制进行适配。以下建议基于通用 PHP 最佳实践与 CI 架构规范。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 约 145-155 行 | **SQL 注入漏洞**：`$pay_platform_where` 数组拼接时直接使用了未过滤的用户输入 `$pay_platform` 与 `$pay_platform_arr[1]`。攻击者可通过构造恶意参数闭合引号执行任意 SQL。 | 严禁字符串拼接。若框架支持参数绑定请使用占位符；否则必须强制类型转换或使用 `$this->db->escape()`。 | `$pay_platform_where[] = '(a._pay_platform=' . (int)$pay_platform . ' AND a._second_pay_platform=' . (int)$pay_platform_arr[1] . ')';` |
| 🔴 严重 | 约 118, 138 行 | **JSON 解析逻辑缺陷**：`json_decode()` 未传入 `true` 参数，默认返回 `stdClass` 对象。后续 `is_array()` 判断恒为 `false`，导致参数被错误重置为空数组，筛选功能失效。 | 添加 `true` 参数强制返回关联数组，并增加 JSON 格式校验。 | `$decoded = json_decode($params['order_type_arr'], true);`<br>`$params['order_type_arr'] = is_array($decoded) ? $decoded : [];` |
| 🟠 警告 | 第 3 行 | **反模式：全局 `get_instance()`**：在类外部直接调用 `get_instance()` 违反 OOP 封装原则，且在非标准加载流程或单元测试中极易引发 `Fatal Error`。 | 移除全局调用，将依赖模型统一移至 `__construct()` 中加载。 | `public function __construct() { parent::__construct(); $this->load->model('Report_model'); }` |
| 🟠 警告 | 约 105-106 行 | **时间参数未校验**：`strtotime()` 直接处理 `$params['start_time']` 和 `$params['end_time']`。若传入非法格式将返回 `false`，导致查询条件异常或数据库报错。 | 增加时间格式校验，或使用框架验证器。失败时提前返回明确错误。 | `if (!strtotime($params['start_time'] ?? '') || !strtotime($params['end_time'] ?? '')) { return ['error' => '时间格式无效']; }` |
| 🟠 警告 | 约 168 行 | **类型比较隐患**：`$params['page'] == '1'` 使用弱类型比较，若传入整数 `1` 或字符串 `'1'` 虽能匹配，但缺乏严谨性，且未处理非数字/空值情况。 | 使用严格类型判断，并设置默认值。 | `if ((int)($params['page'] ?? 1) === 1) { ... }` |
| 🟡 建议 | 第 6 行 | **命名规范不符 PSR-12**：类名 `Jh_community_shop_revenues_detail_model` 使用下划线命名，不符合 PSR-12 的 `PascalCase` 规范。 | 重命名为 `JhCommunityShopRevenuesDetailModel`，并同步更新项目中的引用路径。 | `class JhCommunityShopRevenuesDetailModel extends Report_model` |
| 🟡 建议 | 全文多处 | **重复加载模型**：在多个方法中重复调用 `$this->load->model()`，增加 I/O 开销且不符合框架最佳实践。 | 将高频使用的模型统一在 `__construct()` 中加载，或使用依赖注入（若框架支持）。 | `__construct() { parent::__construct(); $this->load->model(['ahead_yc_shop_model', 'ahead_user_model', 'ahead_book_order_model']); }` |
| 🟡 建议 | 约 218 行 | **循环内调用复杂函数**：`minToStr()` 在 `foreach` 循环中频繁调用。若内部涉及复杂计算或 DB 查询，将显著拖慢列表/导出性能。 | 评估 `minToStr` 实现，考虑批量计算、结果缓存或移至 SQL 层（如 `TIMESTAMPDIFF`）处理。 | *(视具体实现优化，建议提取至独立服务层或添加静态缓存)* |
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **修复 SQL 注入**：立即对 `$pay_platform_where` 拼接逻辑进行类型强转 `(int)` 或改用框架查询构建器。这是最高优先级的安全红线。
2. **修正 JSON 解析**：全局替换 `json_decode($str)` 为 `json_decode($str, true)`，并增加 `json_last_error()` 校验，防止脏数据导致业务中断。
3. **移除全局 `get_instance()`**：将模型加载迁移至构造函数，确保类实例化符合面向对象生命周期。
### 🛠 后续重构方向
- **输入校验层**：建议在 Controller 层或独立 Validator 中对 `$params` 进行统一校验（时间格式、数组结构、枚举值范围），避免脏数据渗透至 Model 层。
- **查询构建器标准化**：当前 `$where` 数组结构为自定义实现，建议逐步迁移至框架原生 Query Builder（如 `$this->db->where()`, `$this->db->join()`），以提升可读性并自动防御注入。
- **常量与配置抽离**：将 `ORDER_TYPE`、`revenues_pay_platform_arr` 等硬编码数组移至独立配置类或数据库字典表，便于后续动态扩展与多语言适配。
- **性能优化**：若导出量级较大，建议将 `get_community_revenues_list` 拆分为 `getList()` 与 `getExportData()`，导出时采用游标查询（`unbuffered query`）或分块写入，避免内存溢出。
> 💡 **局限性说明**：本次审查基于提供的单文件代码。由于强依赖父类 `Report_model` 的自定义 `$where` 解析逻辑、`enforce_con_db()` 连接切换机制及全局辅助函数（如 `filter_emoji`, `minToStr`），部分性能与框架适配细节需结合完整项目上下文进一步验证。建议补充父类核心方法实现以便进行深度链路分析。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780394275
|
1780394275
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
632
|
18
|
198
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 需求 盘点单增加时间选择
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `c9caaf5c47 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `c9caaf5c47e86a80287dbba7b932ce23cc35c06a`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-06-09 14:19:45
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑链路完整，具备基础的事务控制与 N+1 查询优化意识。但存在多处严重架构违规（模型层直接 `exit`、顶层实例化 CI 对象）、类型混淆导致的逻辑缺陷，以及不符合现代 PHP 规范的命名与类型声明问题。整体可维护性与健壮性需重点提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 文件顶部 | 顶层直接执行 `$CI = & get_instance();`。在框架未完全初始化或模型被提前加载时会导致 Fatal Error，且违反 MVC 分层原则。 | 移除顶层代码。在方法内部按需使用 `$this->load->model()` 或框架提供的实例获取方式。 | `// 删除文件顶部的 $CI = & get_instance(); 及后续加载逻辑` |
| 🔴 严重 | `stocktaking_report_export` 方法内 | 模型层使用 `exit()` 强制中断脚本执行。破坏框架生命周期，导致后续中间件、日志记录、事务回滚无法执行，且无法被上层捕获。 | 改为抛出标准异常或返回统一错误结构，由控制器层统一处理响应。 | `if (empty($exportFields)) { throw new \InvalidArgumentException('导出字段不能为空'); }` |
| 🔴 严重 | `add_merchant_stocktaking` 约第 38 行 | 字符串与整数混合运算及比较：`$now_business_date['end_business_day_str'] > $lastTimeBusiness['end_business_day_str'] + 86400*15`。PHP 会将字符串转为 `0` 或触发 Warning，导致时间校验完全失效。 | 统一转换为时间戳后再进行数学运算与比较。 | `$nowTs = strtotime($now_business_date['end_business_day_str']);<br>$lastTs = strtotime($lastTimeBusiness['end_business_day_str']);<br>if ($nowTs > $lastTs + 15 * 86400) { throwError("..."); }` |
| 🟠 警告 | `add_merchant_stocktaking` 事务块 | 混用 CI 自动事务 (`trans_start`) 与手动提交/回滚 (`trans_commit`/`trans_rollback`)。若 `insert` 失败未抛异常，事务状态可能不一致。 | 使用显式事务控制 `trans_begin()` + `try-catch` + `trans_commit/rollback`，或统一使用 `trans_start()` + `trans_complete()`。 | `$this->db->trans_begin();<br>try { /* 业务逻辑 */ $this->db->trans_commit(); } catch (\Exception $e) { $this->db->trans_rollback(); throw $e; }` |
| 🟠 警告 | `get_stocktaking_detail` 约第 115 行 | 动态修改共享模型表名 `set_table_name()`。在并发请求或同一请求多次调用时，会污染模型全局状态，导致后续查询错乱。 | 使用查询构建器别名或创建临时查询实例，避免修改模型属性。 | `$this->db->select("SUM($amountField * stocktaking._profit_loss) AS amount_total")<br> ->from($orginalTableName . ' stocktaking')<br> ->join('ahead_merchant_goods goods', 'stocktaking._merchant_goods_id = goods._id', 'LEFT')<br> ->where('stocktaking._stocktaking_id', $takingDetail['id'])<br> ->get()->row_array();` |
| 🟠 警告 | `search_stocktaking_list` / `get_stocktaking_report` | `strtotime($params['start_time'])` 未校验返回值。若传入非法时间字符串将返回 `false`，传入 Query Builder 可能引发 SQL 语法错误或注入风险。 | 增加时间格式校验，失败时跳过条件或赋予安全默认值。 | `if (!empty($params['start_time'])) { $ts = strtotime($params['start_time']); if ($ts !== false) $where['stocktaking._create_time >= '] = $ts; }` |
| 🟡 建议 | 全局类/方法定义 | 类名 `Ahead_merchant_goods_stocktaking_model` 与方法名不符合 PSR-12 规范。缺乏 PHP 7+ 类型声明，降低 IDE 提示与静态分析能力。 | 类名改为 `PascalCase`，方法名改为 `camelCase`，补充参数类型与返回值声明。 | `class AheadMerchantGoodsStocktakingModel extends Simple_model {<br> public function addMerchantStocktaking(int $merchantId, int $adminId, string $adminName, array $data): array { ... }` |
| 🟡 建议 | `stocktaking_report_export` | 全量查询 `get_stocktaking_report(..., 0, 0)` 后直接加载至内存。数据量大时极易触发 `Allowed memory size exhausted`。 | 采用游标查询、分块处理 (`chunk`) 或流式导出，避免一次性加载全部结果集。 | `// 建议 Export_model 内部实现分批查询，或在此处使用 while 循环配合 limit/offset 流式写入文件` |
| 🟡 建议 | 多处 SQL 条件拼接 | `where_in` 与 `LIKE` 直接拼接数组/字符串。若 `Simple_model` 底层未使用预处理语句，存在 SQL 注入隐患。 | 确认 `Simple_model` 是否使用 PDO 预处理。若为原生拼接，请改用框架提供的 `like()`、`where_in()` 方法或手动绑定参数。 | `$this->db->like('info._goods_name', $params['goods_name'], 'both');` |
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **移除顶层 `get_instance()`**：将其移至具体方法内部，或直接依赖 `$this->load->model()` 的自动加载机制。
2. **替换模型层 `exit()`**：所有业务中断逻辑必须改为 `throw new \Exception()` 或返回 `['success' => false, 'msg' => '...']`，确保框架能正常接管错误处理与事务回滚。
3. **修复时间比较逻辑**：将 `end_business_day_str` 统一通过 `strtotime()` 转为时间戳后再进行 `+ 86400*15` 运算与大小比较，杜绝隐式类型转换导致的逻辑漏洞。
### 🛠 后续重构与优化方向
1. **事务规范化**：统一采用 `$this->db->trans_begin()` 配合 `try-catch` 显式控制事务，避免与框架自动事务机制冲突。建议在 `Simple_model` 基类中封装标准事务模板方法。
2. **类型安全与规范**：全面引入 PHP 7+ 类型声明（`int`, `string`, `array`, `bool`），类名与方法名严格遵循 PSR-12。可借助 `PHP_CodeSniffer` 或 `PHPStan` 进行静态扫描。
3. **大数据导出优化**：报表导出方法应改为流式处理。若 `Export_model` 不支持，可在此模型中实现 `yield` 生成器，按批次（如每次 500 条）查询并写入临时文件，彻底解决内存瓶颈。
4. **框架适配确认**：*注：代码特征高度符合 CodeIgniter 3 架构。若 `phpci` 为内部定制框架，请重点核对 `Simple_model` 的 `where_in`、`LIKE` 底层实现是否默认启用参数绑定，并查阅官方文档确认 `trans_*` 系列方法的生命周期行为。*
> 💡 **审查局限性说明**：本次审查仅基于提供的单文件代码。`Simple_model`、`throwError`、`get_business_date` 等外部依赖的具体实现未提供，部分安全性与性能评估基于常规 CI 架构假设。建议结合完整项目上下文进行集成测试验证。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780985985
|
1780985985
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
402
|
18
|
174
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 需求-包厢停用报表
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `d61d705db3 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `d61d705db33011e2e6852e1d744afd459e894c7b`
**提交人**: caihongyuchy (1091045324@qq.com)
**时间**: 2026-05-28 15:24:28
---
## 📋 审查摘要
- **变更文件数**: 2
- **严重问题**: 0
- **高危问题**: 1
- **中危问题**: 2
- **建议优化**: 3
## 🐛 发现的问题
### <font color="red">[跨文件调用] 路由组件引用与导出名称不匹配</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: web/youc_business_operate_pc/src/router/index.js
- **行号**: 约 348 行
- **问题描述**: 在路由配置中使用了 `component: pages.bpsm`，但在 `pages.js` 中对应的导入和导出名称为 `or_bpsm_report`。`pages` 对象中不存在 `bpsm` 属性，这将导致 Vue Router 在匹配该路由时抛出 `TypeError: Cannot read properties of undefined (reading 'bpsm')` 或组件加载失败，访问该路径时页面白屏。
- **修复建议**: 将 `index.js` 中的 `component: pages.bpsm` 修改为 `component: pages.or_bpsm_report`，或在 `pages.js` 的 `export default` 对象中增加别名映射 `bpsm: or_bpsm_report`。
### [安全隐患] 前端路由守卫依赖直接状态访问，存在越权绕过风险
- **严重程度**: 中危
- **文件**: web/youc_business_operate_pc/src/router/index.js
- **行号**: 约 540 行
- **问题描述**: `router.beforeEach` 中直接使用 `!store.state.usermobile` 判断登录状态。客户端状态（Vuex/LocalStorage）极易被用户通过浏览器控制台篡改或注入，导致未授权用户绕过登录页直接访问受保护路由。此外，若 Vuex 初始化未完成或状态被清空，`store.state.usermobile` 可能为 `undefined`，导致正常用户被误拦截或陷入重定向循环。
- **修复建议**: 1. 结合后端 Token 验证（如 Axios 拦截器统一处理 401 状态码并跳转）。2. 使用 Vuex Getter 或封装统一的鉴权函数 `isAuthenticated()`。3. 增加路由元信息 `meta: { requiresAuth: true }` 进行统一拦截，避免硬编码判断特定字段。
### [代码质量] 部分组件导入缺少 `.vue` 扩展名
- **严重程度**: 中危
- **文件**: web/youc_business_operate_pc/src/router/pages.js
- **行号**: 约 268, 270, 272 行
- **问题描述**: `custom_music_manage`、`screen_ad_set`、`door_plate_set` 的 `import` 语句中省略了 `.vue` 后缀。虽然现代构建工具（Webpack/Vue CLI）通常能通过 `resolve.extensions` 自动补全，但在某些严格配置、迁移构建工具（如 Vite）或 CI/CD 环境中会导致模块解析失败或警告。
- **修复建议**: 统一补全扩展名，例如：`import custom_music_manage from '../views/system_set/custom_music_manage.vue'`。保持项目导入规范的一致性，降低环境差异带来的构建风险。
## ✅ 代码亮点
- **导航故障处理规范**：正确重写了 `VueRouter.prototype.push` 并捕获错误（`.catch(err => err)`），有效避免了 Vue Router 3.x 中常见的 `NavigationDuplicated` 控制台报错，提升了用户体验。
- **模块化路由配置**：通过 `pages.js` 集中管理所有视图组件的导入与导出，实现了路由配置与组件引用的解耦，结构清晰，便于后续按需加载改造。
- **注释清晰**：在 `index.js` 中保留了清晰的代码分割注释（如 `// route level code-splitting`），有利于后续团队理解路由加载策略。
## 📝 总体建议
1. **路由配置巨型文件拆分**：当前 `index.js` 包含超过 150 个路由配置，单文件过长导致维护困难、冲突率高。强烈建议按业务模块（如 `sale_manage.js`, `finance_report.js`, `system_set.js`）拆分为多个子路由文件，再通过 `import` 合并到主路由数组中。
2. **统一属性访问语法**：`index.js` 中混用了 `pages.xxx` 和 `pages["xxx"]`（如 `pages["wx_order_home"]`、`pages["ticket_manage"]`）。建议统一使用点语法 `pages.xxx`，除非键名包含特殊字符或需动态计算，以提升代码可读性与一致性。
3. **引入动态路由/权限路由机制**：当前所有路由均为静态硬编码。对于企业级后台系统，建议改为根据用户角色/权限从后端动态下发路由表，前端通过 `router.addRoutes()` 动态挂载。这不仅能从根本上解决前端越权访问问题，还能大幅减少首屏 JS 体积。
4. **规范代码风格**：文件中单引号 `'` 与双引号 `"` 混用，部分对象末尾存在多余逗号。建议配置 `ESLint` + `Prettier` 进行自动化格式化与静态检查，统一团队代码规范。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779953068
|
1779953068
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
665
|
18
|
203
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 1
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `d7e5996ab6 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `d7e5996ab65bb821f278a19b971a31f3a3f1c4b9`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-10 16:18:15
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10
- **总体评价**：代码实现了退款数据的查询、关联与格式化逻辑，基础功能完整。但存在明显的架构反模式（如全局作用域加载、方法内重复加载依赖）、硬编码魔法数字、JSON 解析缺乏容错、以及数据获取与视图格式化严重耦合。整体可维护性、健壮性与性能有较大优化空间。
- **风险等级**：🟠 中（主要隐患在于 JSON 解析异常导致崩溃、硬编码维护成本高、潜在的性能损耗及框架生命周期管理不规范）
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 文件顶部 (全局作用域) | 在类外部使用 `$CI =& get_instance();` 并加载 `Simple_model`。此写法破坏面向对象封装，易引发依赖冲突、内存泄漏，且不符合现代 PHP 框架规范。 | 移除全局 `$CI` 调用。基类 `Simple_model` 应由框架自动加载或继承时自动解析，无需手动 `load`。 | `// 删除顶部代码\n// $CI =& get_instance();\n// $CI->load->model('Simple_model');` |
| 🔴 严重 | `get_refund_data` / `get_refund_log` 循环内 | `json_decode()` 未做严格类型校验。若数据库字段值为 `"null"`、`"false"` 或损坏 JSON，`json_decode` 将返回 `null`，后续 `foreach` 会触发 `Warning: Invalid argument supplied for foreach()`。 | 增加返回值类型强校验，或使用 `is_array()` 兜底。 | `$data = json_decode($json, true);\n$v['refund_amount_info'] = is_array($data) ? $data : [];` |
| 🟠 警告 | `get_refund_data` / `get_refund_log` 方法内 | 在业务方法中频繁调用 `$this->load->model()` 与 `$this->config->load()`。每次调用均触发框架 Loader 检查，造成冗余 I/O 与性能损耗。 | 将依赖加载统一移至 `__construct()` 中，或使用框架的自动加载/依赖注入机制。 | `public function __construct() {\n parent::__construct();\n $this->load->model('ahead_yc_order_refund_infos_model');\n $this->load->model('ahead_shop_config_model');\n $this->load->model('ahead_yc_merchant_user_model');\n $this->config->load('merchant', TRUE);\n}` |
| 🟠 警告 | `get_refund_data` & `get_refund_log` | 硬编码支付平台 ID 数组 `[17, 18, 19, 20, 23, 24, 25, 26, 27, 28]` 重复出现。业务变更时需多处修改，极易遗漏且可读性差。 | 提取为类常量或独立配置文件，使用 `in_array($id, self::CUSTOM_PAY_IDS, true)` 提升安全性。 | `const CUSTOM_PAY_PLATFORMS = [17, 18, 19, 20, 23, 24, 25, 26, 27, 28];\n// 使用时\nif (in_array($vv['pay_platform'], self::CUSTOM_PAY_PLATFORMS, true)) { ... }` |
| 🟠 警告 | `get_refund_data` 方法内 | `$this->setTableName()` 修改表名后，若中间逻辑抛出异常，将导致后续所有查询使用错误的表名（状态未回滚）。 | 使用 `try...finally` 确保表名必定恢复，或封装为独立查询方法避免污染全局状态。 | `try {\n $this->setTableName($this->table_name.' a');\n // ... 查询逻辑\n} finally {\n $this->setTableName($table_name);\n}` |
| 🟠 警告 | `get_refund_log` 循环内 | `$total_refund_amount += $v['refund_amount'];` 未进行类型安全转换。若数据库返回字符串类型金额，可能触发 PHP 警告或浮点精度丢失。 | 累加前强制转换为浮点数，并处理空值。 | `$total_refund_amount += (float) ($v['refund_amount'] ?? 0);` |
| 🟡 建议 | 类定义行 | 类名 `Ahead_yc_order_refund_model` 使用蛇形命名，不符合 PSR-12 规范（类名应使用大驼峰 PascalCase）。 | 重命名为 `AheadYcOrderRefundModel`，并全局同步更新引用。 | `class AheadYcOrderRefundModel extends Simple_model` |
| 🟡 建议 | 方法返回值 | `return $refund_info ?$refund_info : array();` 存在语法空格不规范，且三元表达式冗余。 | 使用空合并运算符简化，提升可读性与执行效率。 | `return $refund_info ?? [];` |
| 🟡 建议 | 架构设计 | 模型层承担了过多视图格式化职责（如拼接 `【套餐配送】`、日期格式化、金额字符串拼接）。违反单一职责原则 (SRP)。 | 将数据格式化逻辑剥离至 `Service` 层或 `ViewModel`，模型仅负责纯净的数据查询与返回。 | `// 模型返回原始数组\n// Service/ViewModel 层负责格式化\n$formatter = new RefundDataFormatter();\nreturn $formatter->format($rawData);` |
| 🟡 建议 | PHPDoc 注释 | 方法注释缺少 `@return` 类型声明及参数类型提示，不利于 IDE 静态分析与团队协作。 | 补充完整 PHPDoc，明确参数与返回值类型。 | `/**\n * @param int $order_id\n * @param string|int $order_type\n * @param int $shop_id\n * @return array\n */` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **移除全局 `$CI` 调用**：彻底清理文件顶部的 `$CI =& get_instance();`，避免破坏框架依赖树。
2. **JSON 解析容错**：对所有 `json_decode` 结果进行 `is_array()` 校验，防止脏数据导致循环崩溃。
3. **依赖加载前置**：将 `load->model()` 与 `config->load()` 统一收敛至构造函数，消除运行时重复加载开销。
4. **硬编码提取**：将支付平台 ID 列表提取为类常量或配置文件，降低后续维护成本。
### 🛠 后续重构与优化方向
- **架构分层**：当前模型混合了 `数据查询` 与 `展示层格式化`。建议引入 `Service` 层处理业务编排，或使用 `DTO/ViewModel` 处理前端展示所需的字符串拼接与格式化，保持 Model 的纯粹性。
- **状态安全管理**：`setTableName()` 属于框架级状态修改，务必配合 `try...finally` 或封装为闭包查询，防止异常中断导致全局状态污染。
- **类型声明升级**：若运行环境为 PHP 7.4+，建议为方法参数与返回值添加类型声明（如 `public function get_refund_data(int $order_id, string $order_type = '', int $shop_id = 0): array`），提升代码健壮性。
- **框架适配说明**：代码结构高度契合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请确认其 Loader 机制、配置缓存策略与 CI3 是否完全一致。对于不确定的生命周期行为，建议查阅 `phpci` 官方文档中关于 `Model 初始化` 与 `Config 缓存` 的最佳实践。
> 💡 **局限性提示**：本次审查基于提供的单文件代码。由于未提供 `Simple_model` 基类实现、`$this->select()` 底层 SQL 构建逻辑及完整框架配置，部分安全性（如底层是否自动参数绑定防注入）与性能评估基于通用框架经验推断。建议结合完整项目上下文进行集成测试验证。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1781079495
|
1781079495
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
357
|
18
|
160
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 1
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `dc9bd12ca6 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `dc9bd12ca6caa2d9642d99e6e6d30d23474d89ba`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-27 13:11:40
---
## 1. 审查摘要
- **代码质量评分**：4/10 分
- **总体评价**：当前提交代码主要为超大型静态配置数组，未包含任何业务方法或数据交互逻辑。存在严重的架构设计缺陷（在类外部直接调用框架实例、配置硬编码于 Model 层），且代码在末尾被截断，无法评估完整生命周期与数据流。整体可维护性、性能与框架适配度均不达标，需进行结构性重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 第2-3行 | 在类定义外部直接执行 `$CI = &get_instance();` 并加载模型。PHP 文件被 `include/require` 时会立即执行该代码，若框架尚未完全初始化将引发 Fatal Error；若已初始化，则每次请求都会无条件重复加载，极易导致循环依赖、内存泄漏或状态污染。 | 移除文件顶部的全局执行代码。若需加载依赖，应移至类的构造函数 `__construct()` 中，并确保调用父类构造方法。 | ```php\npublic function __construct()\n{\n parent::__construct();\n // 按需加载其他组件\n}\n``` |
| 🟠 警告 | 全文/类属性 | 将数百行、深层嵌套的业务配置硬编码在 Model 类中。违反单一职责原则（Model 应专注数据访问），增加单次请求内存开销，且无法实现多环境（开发/测试/生产）配置隔离。 | 将配置迁移至 `application/config/` 目录下的独立配置文件（如 `scene_config.php`），通过框架配置加载器读取，或持久化至数据库/Redis 并配合缓存。 | ```php\n// config/scene_config.php\nreturn [\n '1' => [ /* KTV配置 */ ],\n '2' => [ /* 台球配置 */ ]\n];\n// 模型中调用\n$this->config->load('scene_config');\n$scene = $this->config->item('scene_config');\n``` |
| 🟠 警告 | 末尾截断处 | 代码在 `'config_params'` 处被意外截断，未提供类闭合括号、业务方法及父类 `Simple_model` 定义。无法审查数据验证、SQL 交互、异常处理及框架生命周期适配情况。 | 请补充完整文件内容。审查需基于完整上下文，否则无法保证逻辑正确性与安全性。 | （需提交完整代码） |
| 🟡 建议 | 第5行起 | 数组嵌套层级过深（5~6层），键名使用字符串数字 `'1'`, `'2'`。在 PHP 弱类型比较中易引发 `'1' == 1` 的隐式转换隐患，且降低 IDE 静态分析与代码可读性。 | 使用类常量定义场景标识，将部分配置扁平化或拆分为独立配置块。若配置需动态下发，建议采用 DTO（数据传输对象）模式封装。 | ```php\nclass Ahead_community_shop_model extends Simple_model\n{\n public const SCENE_KTV = 1;\n public const SCENE_BILLIARDS = 2;\n \n public $operational_scene_config = [\n self::SCENE_KTV => [...],\n self::SCENE_BILLIARDS => [...]\n ];\n}\n``` |
| 🟡 建议 | 全文 | 数组缩进与对齐风格不统一，部分 `option` 数组末尾逗号缺失或多余。未遵循 PSR-12 规范中的控制结构与数组声明格式。 | 使用 PHP-CS-Fixer 或 IDE 自动格式化工具统一代码风格。补充类级 `@package`、`@author` 及 `@property` 注释。 | （通过工具自动修复） |
## 3. 总结与行动建议
### 🔑 优先修复项
1. **立即移除顶部全局代码**：删除 `$CI = &get_instance();` 及 `$CI->load->model(...)`，避免在文件包含阶段执行框架调用。
2. **配置与模型解耦**：将 `$operational_scene_config` 迁移至独立配置文件或数据库。若为静态只读配置，使用 `config/` 目录；若需后台动态修改，建议落库并增加缓存层（如 Redis/File Cache）。
3. **补全代码上下文**：提交完整的类定义、构造函数、数据读写方法及父类继承链，以便进行逻辑与安全审查。
### 🛠 后续重构与优化方向
- **架构分层**：遵循 MVC 规范，Model 仅负责数据持久化与查询。配置数据应通过 `Config` 组件或 `Service` 层注入，避免在 Model 中定义业务规则字典。
- **类型安全与校验**：若该配置用于前端渲染或 API 返回，建议在读取时增加类型校验（如 `filter_var`、`ctype_digit`）或引入 `Symfony/Validator` 等组件，防止非法值导致下游逻辑崩溃。
- **框架适配说明**：从目录结构（`system/helpers/`, `system/libraries/`, `application/models/`）判断，该项目高度疑似基于 **CodeIgniter 3** 架构。若确为 `phpci` 框架，请查阅其官方文档中关于 `Config Loader` 与 `Model Lifecycle` 的规范，确保组件加载时机与 OPcache 缓存策略兼容。
- **性能优化**：超大型数组在每次请求解析时会消耗 CPU 与内存。建议启用 OPcache 预编译，或对配置启用序列化缓存（`serialize()` / `json_encode()` + 文件缓存），将解析开销降至最低。
> 💡 **提示**：当前代码片段仅包含配置声明，未涉及数据库操作或用户输入处理，故暂未发现 SQL 注入/XSS 等直接安全漏洞。但若该配置后续用于动态拼接 SQL 或输出至前端，请务必使用参数化查询与 `htmlspecialchars()` 转义。请补充完整业务逻辑代码以便进行深度安全审计。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779858700
|
1779858700
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
347
|
18
|
152
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 预留清洁时间设置
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `e16a99d52b ## 自动代码审查报告
**分支**: pc-260616
**提交**: `e16a99d52b5a155505262c6b45952ba22deefa5f`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-26 17:49:18
---
## 1. 审查摘要
- **代码质量评分**：4 / 10 分
- **总体评价**：该文件主要定义了一个庞大的运营场景配置数组，但存在严重的架构设计缺陷。代码在类外部直接调用框架核心函数与加载模型，违反面向对象原则与框架生命周期；超大配置硬编码在 Model 中导致维护成本极高且影响性能。此外，代码片段在末尾截断，无法评估完整业务逻辑。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 全局作用域 (第1-3行) | 在类定义外部调用 `get_instance()` 与 `load->model()`。此时框架可能尚未完成初始化，极易引发 `Fatal Error` 或全局状态污染，且破坏单例模式。 | 移除全局作用域代码，将依赖加载移至类的 `__construct()` 中，或依赖框架自动加载机制。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🔴 严重 | 类属性定义行 | 将数百行的 UI 表单配置 Schema 硬编码在 Model 属性中。Model 应专注数据访问与业务逻辑，配置数据应独立管理，否则会导致内存常驻过高、解析缓慢且极难维护。 | 将 `$operational_scene_config` 抽离至 `application/config/` 独立配置文件，或持久化至数据库/缓存中按需加载。 | `// application/config/community_shop_schema.php<br>return [ '1' => [...], '2' => [...] ];` |
| 🟠 警告 | 类名定义行 | 类名 `Ahead_community_shop_model` 不符合 PSR-12 规范（应使用大驼峰命名法 `StudlyCaps`），且可能导致自动加载器无法正确映射文件。 | 重命名为 `AheadCommunityShopModel`，并确保物理文件名同步修改为 `AheadCommunityShopModel.php`。 | `class AheadCommunityShopModel extends Simple_model` |
| 🟠 警告 | 数组内部结构 | 配置数组存在大量重复的嵌套结构（如 `radio`、`select`、`checkbox` 的 `option` 定义），属于典型的“样板代码”，易引发复制粘贴错误。 | 使用工厂方法或配置构建器动态生成，或提取为公共配置模板进行合并。 | `protected function buildRadioOption($field, $label, $options) { return [...]; }` |
| 🟠 警告 | 安全/下游逻辑 | 当前仅为静态配置，但若后续直接用于渲染前端表单或处理用户提交数据，未做严格类型校验与白名单过滤，可能引发批量赋值（Mass Assignment）或 XSS 隐患。 | 在接收前端配置提交时，必须使用框架验证器（如 `Form_validation`）进行字段白名单校验，并对输出进行 `htmlspecialchars` 转义。 | `$this->form_validation->set_rules('config_key', '配置项', 'required|in_list[1,-1]');` |
| 🟡 建议 | 魔法数字使用 | 数组中大量使用 `'1'`, `'-1'`, `'2'` 等字符串数字作为状态标识，可读性差且易出错。 | 定义类常量或 PHP 8.1+ 枚举（Enum）替代魔法值，提升类型安全与 IDE 提示体验。 | `const SCENE_KTV = 1; const STATUS_ON = 1; const STATUS_OFF = -1;` |
| 🟡 建议 | 文件末尾 | 代码在 `'config_params` 处截断，缺失闭合括号、类方法及业务逻辑，无法评估异常处理、边界条件及完整数据流。 | 补充完整代码片段，以便进行深度逻辑审查与安全审计。 | - |
## 3. 总结与行动建议
### 🔑 优先修复项
1. **移除全局作用域代码**：立即删除文件头部的 `$CI = &get_instance();` 与 `$CI->load->model()`，将其迁移至 `__construct()` 方法中。这是导致潜在崩溃的最高危问题。
2. **配置数据解耦**：将 `$operational_scene_config` 移出 Model，存放于 `application/config/` 目录或数据库配置表中。Model 仅通过 `config_item()` 或缓存读取，遵循单一职责原则。
3. **规范命名与自动加载**：将类名改为 `AheadCommunityShopModel`，确保符合 PSR-12 与框架自动加载规范。
### 🛠 后续重构方向
- **架构适配说明**：注：提供的代码结构（`$CI = &get_instance();`、`system/`、`application/` 目录树）高度符合 **CodeIgniter 3** 框架特征。若 `phpci` 为定制分支或笔误，请参照 CI3 生命周期规范；若为独立框架，请查阅其官方文档确认模型加载与配置管理的最佳实践。
- **配置动态化**：当前配置实为“前端表单渲染 Schema”。建议引入配置构建器模式，将重复的 `config_params` 结构抽象为基类或辅助函数，通过传入元数据动态生成完整数组，减少 70% 以上的冗余代码。
- **类型安全与验证**：若项目已升级至 PHP 8.0+，强烈建议为配置数组添加类型声明，并在 Controller 层接收配置更新时，使用严格的白名单验证机制，防止越权修改或非法数据入库。
- **性能优化**：若配置最终保留在代码中，请使用 `opcache` 缓存，或将其序列化为 JSON 文件按需 `json_decode`，避免每次请求重复解析超大 PHP 数组。
> ⚠️ **局限性提示**：当前提交的代码仅包含属性定义且末尾截断，无法审查实际业务方法（如增删改查、配置校验、缓存策略等）。请补充完整类文件后，可进一步进行逻辑链路与安全深度审计。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779788958
|
1779788958
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
325
|
18
|
145
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 需求 赠时报表 16382
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `e59f808813 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `e59f80881314e0b44bbb1e846529fd54e49b9105`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-05-26 14:27:56
---
## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：控制器承载了过多业务逻辑与导出模板配置，功能实现完整但架构臃肿。存在明显的参数过滤缺失、内存管理不当及大量重复代码。若数据量增长，极易引发 OOM 或响应超时。需进行服务层抽离与安全加固。
- **风险等级**：🔴 高
> 📌 **框架说明**：代码中出现的 `BASEPATH`、`$this->load->model()`、`$this->param`、`$this->success_response()` 等特征高度符合 **CodeIgniter 3** 规范。若项目实际使用 `phpci`（可能为 CI 二次开发或定制版），请结合其官方文档核对生命周期与输入过滤机制。以下审查基于 CI3 最佳实践。
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 全局导出方法 | **直接使用 `$_GET` 传入模型，未做安全过滤**。若底层模型未使用 Query Builder 或预处理语句，将导致 SQL 注入。 | 统一使用框架输入类获取参数，并在控制器层进行类型/范围校验。模型层必须使用参数绑定。 | `$params = $this->input->get(null, true); // 启用 XSS 过滤`<br>`$params['start_date'] = $this->input->get('start_date', true);` |
| 🔴 严重 | `managerDiscountExport()` 等 | **滥用 `exit()` 终止脚本**。绕过框架输出生命周期，导致响应头丢失、日志未记录、数据库连接未释放，且不利于后续中间件/钩子执行。 | 移除 `exit()`，改用框架标准响应输出或让模型直接输出文件流后由控制器返回。 | `// 移除 exit();<br>// 若模型已输出文件流，可返回空响应：<br>$this->output->set_status_header(200);` |
| 🟠 警告 | `getWaresCountExport()` 等 | **硬编码 `ini_set("memory_limit", "500M")` 应对大数据导出**。治标不治本，易引发服务器资源争抢，且未恢复原值，影响后续请求。 | 采用分块查询（`LIMIT/OFFSET` 或游标）、生成器（`yield`）或数据库层聚合。若必须调高内存，应在方法结束前恢复。 | `$oldLimit = ini_get('memory_limit');<br>ini_set('memory_limit', '500M');<br>// ... 业务逻辑 ...<br>ini_set('memory_limit', $oldLimit);` |
| 🟠 警告 | 全局导出方法 | **导出逻辑高度重复（DRY 原则违反）**。列定义、字段过滤、表头配置、Excel/PDF 分支在 10+ 个方法中重复编写，维护成本极高。 | 抽离为独立导出服务类（Service）或 Trait，通过配置数组驱动。控制器仅负责参数收集与调用。 | `// 控制器内仅保留：<br>$config = $this->getExportConfig('wares_count');<br>$this->exportService->handle($config, $params);` |
| 🟠 警告 | `getWaresCountExport()` 等 | **`json_decode` 未做异常处理**。若前端传入非法 JSON，将返回 `null` 并触发 Warning，后续 `empty()` 判断可能失效。 | 增加 `json_last_error()` 校验或提供默认值，确保类型安全。 | `$exportFields = isset($param['export_fields']) ? json_decode($param['export_fields'], true) : null;<br>if (json_last_error() !== JSON_ERROR_NONE) { $exportFields = array_keys($columnArr); }` |
| 🟡 建议 | 全局方法 | **模型加载分散在各方法中**。CI3 虽会缓存已加载模型，但重复调用降低可读性，且不符合单一职责。 | 在 `__construct()` 中统一加载常用模型，或使用 CI3 的 `autoload.php` 配置。 | `public function __construct() {<br> parent::__construct();<br> $this->load->model(['Export_model', 'Ahead_finance_report_setting_model']);<br>}` |
| 🟡 建议 | `getWaresCountExport()` 等 | **`$this->param` 与 `$_GET` 混用**。框架未明确 `$this->param` 的注入来源，混用易导致参数丢失或过滤策略不一致。 | 统一使用 `$this->input->get()` 或 `$this->input->post()`，并在基类中统一赋值给 `$this->param`。 | `$this->param = $this->input->get(null, true);` |
| 🟡 建议 | 文件末尾 | **代码片段不完整**。`shoppingGuideOrderCountByUserExport()` 方法在数组定义处截断，无法评估完整逻辑。 | 请补充完整代码以便进行全量审查。 | *(无)* |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **安全过滤**：立即替换所有 `$_GET` 为 `$this->input->get(null, true)`，并在模型层强制使用 `$this->db->where()` 或预处理语句，杜绝 SQL 注入。
2. **生命周期规范**：移除所有导出方法中的 `exit()`，确保框架能正常执行 `post_controller` 钩子、记录访问日志及释放数据库连接。
3. **内存与性能**：针对大数据量导出，废弃 `ini_set` 暴力扩容方案。改为数据库层 `SUM/COUNT` 聚合统计，或采用分块查询+流式写入（如 `PHPExcel/PhpSpreadsheet` 的 `setPreCalculateFormulas(false)` 与流式输出）。
### 🛠 后续重构方向
1. **架构解耦（Controller -> Service）**：
当前控制器承担了“参数解析、数据查询、字段映射、格式组装、文件生成”全链路职责。建议引入 `ReportExportService`：
```php
// 示例：服务类调用
$config = [
'title' => '商品套餐销售汇总报表',
'columns' => $columnArr,
'model_method' => 'get_wares_count_list',
'sum_fields' => ['quantity', 'actual_amount_total']
];
$this->reportExportService->export($this->merchant_id, $this->param, $config);
```
2. **统一导出配置管理**：将 `columnArr`、`row_widths`、`header` 等硬编码配置移至 `config/reports.php` 或数据库字典表，实现前端动态配置导出列。
3. **异步导出机制**：对于耗时 > 5s 的报表，建议引入队列（如 Redis + Supervisor 或 CI3 的 `Queue` 扩展），前端轮询或 WebSocket 推送下载链接，彻底解决 HTTP 超时与内存瓶颈。
4. **规范命名与注释**：统一模型命名（如 `AheadManagerDiscountReportModel` 或遵循 CI3 的 `ahead_manager_discount_report_model`），补充 PHPDoc 类型声明（`@param array`, `@return array`），提升 IDE 提示与静态分析能力。
> 💡 **提示**：若需针对具体导出方法（如 `getWaresCountExport`）提供完整的 Service 层重构代码或异步队列方案，可提供完整文件及当前使用的 Excel 库版本（如 `PHPExcel` / `PhpSpreadsheet`），我将输出可直接替换的架构代码。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779776876
|
1779776876
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
597
|
18
|
189
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - Merge branch 'pc-260616&# 🔍 代码审查报告：pc-260616 - Merge branch 'pc-260616' of https://gitea.g-hi.com...
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `e697196740 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `e6971967409fd1bb683322db8428bd85471e594f`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-08 16:37:18
---
## 1. 审查摘要
- **代码质量评分**：5/10 分
- **总体评价**：代码存在明显的语法错误与框架加载反模式，业务参数缺乏基础校验，且将超大型配置数据硬编码至模型中，严重影响系统稳定性与可维护性。需优先修复致命缺陷并规范架构设计。
- **风险等级**：高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_skin_shop_set_model.php` ~L28 | `$fields` 字符串中存在双逗号 `,,`，拼接至 SQL 时将直接引发 `SQLSTATE[42000]: Syntax error`，导致接口 500 崩溃。 | 移除多余逗号，确保字段别名格式正确。 | `$fields = "_skin_type as skin_type, _skin_scene as skin_scene, _skin_key as skin_key";` |
| 🔴 严重 | `Ahead_skin_shop_set_model.php` & `Ahead_community_shop_model.php` L3-4 | 在类外部直接执行 `$CI = &get_instance();` 及 `$CI->load->model()`。该代码会在文件被 `include/require` 时立即执行，而非实例化时，极易导致重复加载、内存泄漏或破坏框架生命周期。 | 移除顶部全局代码。若需加载父类或依赖，应在 `__construct()` 中处理，或直接依赖框架自动加载机制。 | `public function __construct() { parent::__construct(); // 框架通常已处理 }` |
| 🟠 警告 | `Ahead_skin_shop_set_model.php` ~L15 | `get_list()` 接收了 `$merchant_id` 参数但全程未使用；`$order`、`$page`、`$page_size` 未做类型与范围校验，若直接传入查询构造器，存在 SQL 注入或恶意分页耗尽资源的风险。 | 移除无用参数；对排序字段使用白名单校验；对分页参数强制类型转换并设置默认值/上限。 | `$order = in_array($order, ['asc','desc'], true) ? $order : 'desc';`<br>`$page = max(1, (int)$page);`<br>`$page_size = min(100, max(1, (int)$page_size));` |
| 🟠 警告 | `Ahead_skin_shop_set_model.php` ~L19 | `throwError("请选择门店");` 非 PHP 或 CI 标准函数。若未全局定义将触发 `Fatal Error`；若为自定义函数，直接中断流程不利于统一异常捕获与 API 响应格式化。 | 改用 PHP 标准异常 `throw new \InvalidArgumentException('请选择门店');` 或框架内置错误处理函数，并在控制器层统一捕获返回。 | `if (empty($shop_id)) { throw new \InvalidArgumentException('请选择门店'); }` |
| 🟡 建议 | `Ahead_community_shop_model.php` 全文 | 将超大型业务配置数组 `$operational_scene_config` 硬编码在模型文件中。每次加载该模型都会占用大量内存，且修改配置需改代码、重新部署，违反配置与逻辑分离原则。 | 将配置移至 `application/config/` 目录下的独立配置文件，或使用数据库存储+Redis缓存。模型仅负责读取/写入。 | `// config/community_shop_config.php`<br>`return [ 'operational_scene_config' => [...] ];`<br>`// Model中`<br>`$this->config->load('community_shop_config');` |
| 🟡 建议 | `Ahead_skin_shop_set_model.php` ~L8, L30 | `type_arr` 为公开属性且未加类型约束；`foreach ($data['rows'] as &$v)` 使用引用赋值虽可行，但在现代 PHP 中易引发意外副作用。 | 将映射数组改为 `private const TYPE_MAP`；使用 `array_map` 或值拷贝遍历提升安全性。 | `private const TYPE_MAP = [1=>'触摸屏-竖屏', ...];`<br>`$data['rows'] = array_map(function($v){ $v['type_name'] = self::TYPE_MAP[$v['skin_type']] ?? '未知'; return $v; }, $data['rows']);` |
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **修复 SQL 语法错误**：立即修正 `Ahead_skin_shop_set_model.php` 中 `$fields` 的双逗号问题，否则该接口无法正常运行。
2. **清理全局执行代码**：删除两个模型文件顶部的 `$CI = &get_instance();` 及 `load->model()` 调用。在 CI/类 CI 架构中，模型应通过控制器 `$this->load->model()` 加载，或在 `__construct()` 中调用 `parent::__construct()` 完成初始化。
3. **统一错误处理**：将 `throwError` 替换为标准异常或框架响应机制，确保 API 错误码、提示信息格式统一，便于前端解析与日志追踪。
### 🛠 后续重构与优化方向
- **参数校验与安全加固**：所有外部传入参数（尤其是 `$order`、`$page`、`$shop_id`）必须在进入查询前进行类型强制转换、白名单过滤或范围限制。若框架未内置 Query Builder 预处理，请确保底层驱动使用参数绑定。
- **配置数据抽离**：`Ahead_community_shop_model.php` 中的配置数组体积庞大且频繁变更，强烈建议迁移至 `config/` 目录或数据库配置表，并配合缓存层（如 Redis/Memcached）降低数据库压力与内存开销。
- **框架规范对齐**：当前代码结构高度符合 **CodeIgniter 3** 规范，但提示中提及 `phpci` 框架。若 `phpci` 为内部定制框架，请查阅其官方文档确认模型加载、基类继承及错误处理的最佳实践，避免混用不同框架的生命周期钩子。
- **代码片段完整性**：`Ahead_community_shop_model.php` 末尾存在截断（`'config_params` 未闭合）。提交前请确保代码完整，否则将导致 `Parse Error`。
> 💡 **提示**：以上修改建议均基于 PHP 7.4+ 语法与主流 MVC 框架规范编写。若您的运行环境为 PHP 5.x 或特定定制框架，请适当调整类型声明与异常处理方式。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780907838
|
1780907838
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
349
|
18
|
154
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 1
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `eb5b62a472 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `eb5b62a4727dd9dcc9fcdc3423dfd4455db87803`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-26 17:52:33
---
## 1. 审查摘要
- **代码质量评分**：4/10 分
- **总体评价**：该文件主要承载静态业务配置数据结构，但存在严重的框架生命周期违规、职责分离不当及代码截断问题。整体架构不符合现代 PHP 面向对象规范及主流 MVC 框架最佳实践，需进行结构性重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 顶部 1-3 行 | 在类外部直接执行 `$CI = &get_instance();` 与 `$CI->load->model()`。PHP 在 `include/require` 时会立即执行顶层代码，若该文件在框架核心初始化前被加载，将触发 `Fatal Error`；且破坏了模型类的封装性。 | 移除顶层执行代码。依赖加载应移至类的构造函数中，或通过框架自动加载机制（如 `config/autoload.php`）处理。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🟠 警告 | 全文 (数组定义) | 将超大型业务配置数组硬编码在 Model 中。违反单一职责原则（Model 应专注数据交互），每次实例化都会占用大量内存，且不利于多环境配置、热更新与版本控制。 | 将配置抽离至独立配置文件（如 `application/config/scene_config.php`）或存入数据库/Redis。Model 仅负责读取与缓存。 | `// application/config/scene_config.php<br>return [ '1' => [...], '2' => [...] ];` |
| 🟠 警告 | 类定义行 | 框架识别特征（`system/` 目录、`get_instance()`、`$this->load->model()`）明确指向 **CodeIgniter 3**，而非 `phpci`。若为定制分支，请确认生命周期是否一致。类名未遵循 PSR-12 驼峰规范，且未定义构造函数。 | 确认框架基线。若基于 CI3，建议类名与文件名保持一致，并显式继承 `CI_Model` 或基类。 | `class Ahead_community_shop_model extends CI_Model { public function __construct() { parent::__construct(); } }` |
| 🟡 建议 | 数组内部 | 配置项高度重复，魔法数字（如 `'1'`, `'-1'`, `'2'`）缺乏语义化定义，后期维护易出错。 | 使用类常量或枚举定义状态值，提升可读性与类型安全。 | `const STATUS_ENABLE = '1'; const STATUS_DISABLE = '-1';` |
| 🟡 建议 | 文件末尾 | 代码片段在 `'config_params` 处意外截断，无法评估完整逻辑与闭合结构。 | 补充完整文件内容以便进行全量审查。 | 无 |
## 3. 总结与行动建议
### 🔑 优先修复项
1. **移除顶层执行代码**：立即删除文件头部的 `$CI = &get_instance();` 及模型加载语句，将其迁移至 `__construct()` 方法内，避免在文件解析阶段引发致命错误。
2. **配置与逻辑解耦**：将 `$operational_scene_config` 数组迁移至 `application/config/` 目录或配置中心。Model 中仅保留读取方法，例如：
```php
public function get_scene_config($scene_id) {
$config = config_item('scene_config'); // 或从缓存/DB读取
return $config[$scene_id] ?? [];
}
```
3. **补充完整代码**：当前片段截断严重，请提供完整文件以排查潜在的语法闭合错误、数组越界或未定义的键值引用。
### 🛠 后续重构方向
- **引入配置校验层**：针对动态下发的配置，建议在读取时增加类型校验与默认值回退机制（如使用 `filter_var` 或自定义 Validator），防止脏数据导致前端渲染异常。
- **缓存策略优化**：该配置结构变更频率低但读取频繁，建议首次加载后写入 Redis/Memcached，设置合理 TTL，避免每次请求重复解析大数组。
- **框架规范对齐**：若项目确为 `phpci` 定制框架，请查阅官方文档确认模型生命周期与自动加载规范；若实际基于 CodeIgniter 3，建议统一遵循 CI3 官方手册的 Model 编写标准，并逐步向 PSR-12 编码规范靠拢。
> 💡 **注**：受限于提供的代码片段不完整，本次审查主要聚焦于架构设计与框架生命周期规范。补充完整代码后，可进一步针对边界条件处理、数据过滤及具体业务逻辑进行深度审计。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779789153
|
1779789153
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
630
|
18
|
197
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 需求 盘点单增加时间选择
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `f17760cc43 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `f17760cc43e88f2aa1cd9b4f66da9ae98165719a`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-06-09 14:09:54
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑链路基本完整，涵盖了盘点单创建、列表查询、详情计算、报表生成与导出等核心场景。但代码中存在**严重的并发安全隐患**（动态修改单例模型表名）、**隐式类型转换导致的逻辑漏洞**、**事务边界划分不当**以及**模型层直接中断执行**等反模式。整体架构偏向传统 CodeIgniter 3 风格，与现代 PHP 规范及高并发场景要求存在一定差距。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `add_merchant_stocktaking` ~L38 | **时间比较逻辑错误**：`$now_business_date['end_business_day_str']` 为字符串格式，直接与数字 `86400*15` 相加会触发 PHP 隐式类型转换（通常转为 `0` 或截断），导致 `15天限制` 校验完全失效。 | 统一转换为时间戳后再进行数值比较，或使用 `DateTime` 对象。 | ```php<br>$nowTs = strtotime($now_business_date['end_business_day_str']);<br>$lastTs = strtotime($lastTimeBusiness['end_business_day_str']);<br>if ($nowTs > $lastTs + 86400 * 15) {<br> throwError("盘点库存截止时间只能是15天内");<br>}<br>``` |
| 🔴 严重 | `get_stocktaking_detail` ~L118<br>`get_stocktaking_report` ~L168 | **模型单例状态污染**：CI 模型为单例。通过 `set_table_name()` 动态追加别名（如 `table stocktaking`）会修改全局状态。高并发下极易导致请求 A 的别名被请求 B 误用，引发 SQL 语法错误或数据错乱。 | 放弃修改模型属性，改用 Query Builder 的 `from()` 或 `join()` 指定别名，或实例化临时模型对象。 | ```php<br>// 推荐做法：使用 DB 构造器别名<br>$this->db->from($orginalTableName . ' stocktaking');<br>$amountSum = $this->db->get()->row_array();<br>// 或使用临时实例<br>$tempModel = clone $this->Ahead_merchant_stocktaking_infos_model;<br>$tempModel->set_table_name($orginalTableName . ' stocktaking');<br>``` |
| 🔴 严重 | `stocktaking_report_export` ~L208, L213 | **模型层直接 `exit()`**：在 Model 中调用 `exit()` 会直接终止 PHP 进程，导致后续中间件、日志记录、事务回滚或框架生命周期钩子无法执行，且可能引发 `Headers already sent` 错误。 | 改为抛出异常或返回错误数组，由 Controller 层统一处理响应。 | ```php<br>if (empty($exportFields)) {<br> throw new \InvalidArgumentException('导出字段不能为空');<br>}<br>if (!$exportRes['success']) {<br> throw new \RuntimeException($exportRes['msg']);<br>}<br>``` |
| 🟠 警告 | `add_merchant_stocktaking` ~L68 | **事务边界不当**：`$this->db->trans_commit()` 执行后，紧接着调用 `add_data()`。若 `add_data` 失败，主事务已提交，导致 `stocktaking` 主表与 `update_data` 关联表数据不一致。 | 将 `add_data` 移入 `try` 块内，在 `trans_commit()` 前执行；或改为异步队列处理。 | ```php<br>// 移入事务块内<br>$this->db->trans_start();<br>try {<br> // ... insert & update_stock ...<br> if (!empty($stocktaking_update_time)) {<br> $this->Ahead_stocktaking_update_data_model->add_data(...);<br> }<br> $this->db->trans_complete(); // 推荐用 trans_complete()<br>} catch (...) { ... }<br>``` |
| 🟠 警告 | `search_stocktaking_list` ~L95 | **JSON 解析未做容错**：`json_decode($row['operater_names'], true)` 未校验数据合法性。若数据库存储了非法 JSON 字符串，将返回 `null` 并触发 `implode()` 类型警告。 | 增加空值/类型校验，或使用 `json_decode(..., true) ?: []`。 | ```php<br>$names = json_decode($row['operater_names'], true);<br>$row['admin_name'] = is_array($names) ? implode(',', $names) : $row['admin_name'];<br>``` |
| 🟠 警告 | `search_stocktaking_list` ~L78 | **权限过滤潜在越权**：`explode(",", trim($CI->priv_shop_ids, ","))` 当 `priv_shop_ids` 为空字符串时，`explode` 返回 `['']`，传入 `WHERE IN` 可能引发 SQL 异常或意外匹配。 | 增加空值判断，过滤空元素。 | ```php<br>$shopIds = array_filter(explode(",", trim($CI->priv_shop_ids, ",")), 'strlen');<br>if (!empty($shopIds)) {<br> $where['where_in'] = ['_shop_id', $shopIds];<br>}<br>``` |
| 🟡 建议 | 全局 | **命名不符合 PSR-12**：类名 `Ahead_merchant_goods_stocktaking_model` 及方法名 `add_merchant_stocktaking` 使用下划线，不符合现代 PHP 驼峰命名规范，影响自动加载与团队协作。 | 逐步重构为 `AheadMerchantGoodsStocktakingModel` 及 `addMerchantStocktaking`。若为历史遗留项目，可配置 IDE 映射或添加 `@deprecated` 注释过渡。 | `class AheadMerchantGoodsStocktakingModel extends Simple_model` |
| 🟡 建议 | `get_stocktaking_report` ~L145 | **非标准查询语法强耦合**：大量使用 `'_updated_at >= '`、`'where_in' => [...]` 等自定义键名。强依赖 `Simple_model` 内部解析逻辑，降低代码可移植性，且可能绕过框架默认的 SQL 转义。 | 建议统一使用框架原生 Query Builder 语法，或在 `Simple_model` 中明确文档化并增加参数类型校验。 | 无（需结合框架底层实现调整） |
## 3. 总结与行动建议
### 🔑 优先修复项（P0）
1. **修复时间比较漏洞**：立即将字符串时间转换为时间戳后再进行 `15天` 阈值校验，否则业务规则形同虚设。
2. **消除模型单例污染**：全面移除 `set_table_name()` 动态修改表名的写法，改用 `$this->db->from('table alias')` 或 `clone` 模型实例，彻底解决高并发下的数据串扰风险。
3. **规范事务与异常处理**：将 `add_data` 纳入事务范围；将 Model 中的 `exit()` 替换为 `throw new Exception()`，确保框架生命周期完整执行。
### 🛠 后续重构方向
1. **架构解耦**：当前 Model 承担了过多职责（数据查询、业务校验、报表组装、导出逻辑）。建议将 `stocktaking_report_export` 移至 `Service` 层或 `Job` 队列，Model 仅负责数据持久化与基础查询。
2. **查询性能优化**：
- 报表查询涉及多表 `LEFT JOIN` 与 `GROUP BY`，建议在 `ahead_merchant_goods_stocktaking` 表建立复合索引：`INDEX idx_merchant_status_time (_merchant_id, _status, _create_time)`。
- `get_stocktaking_report` 中的 `buying_price` 与 `shop_config` 已做批量查询优化，可考虑引入 Redis 缓存门店配置，减少重复 DB 查询。
3. **框架适配说明**：代码结构高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请重点核对 `Simple_model` 对 `where_in`、`>=` 等自定义语法的 SQL 预处理逻辑，确保已启用参数绑定（Prepared Statements）以防 SQL 注入。
> 💡 **提示**：若需进一步审查 `Simple_model` 底层实现或数据库表结构，可提供相关片段以便进行更精准的索引与查询优化评估。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780985394
|
1780985394
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
340
|
18
|
149
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 自助转房设置
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `f2ac768412 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `f2ac7684120cded91e9bab2cf25b5fcb44f0f52b`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-26 17:11:39
---
## 1. 审查摘要
- **代码质量评分**：4 / 10 分
- **总体评价**：该文件本质上是一个庞大的业务配置数组定义，但存在严重的架构违规（顶层执行代码）、语法截断问题，且将静态配置硬编码在 Model 中，违反了配置与逻辑分离原则。整体可维护性与框架兼容性较差，需进行结构性重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 第 1-3 行 | 在类定义外部直接执行 `$CI = &get_instance();` 和 `$CI->load->model('Simple_model');`。PHP 会在文件被 `include/require` 时立即执行，若框架尚未完成引导将触发 `Fatal Error`，且严重违反 OOP 封装原则。 | 彻底移除顶层执行代码。模型依赖应通过框架自动加载机制或构造函数处理。若 `Simple_model` 为父类，直接 `extends` 即可，无需手动 `load`。 | ```php\n// ❌ 删除以下代码\n$CI = &get_instance();\n$CI->load->model('Simple_model');\n\n// ✅ 正确做法：仅保留类定义\nclass Ahead_community_shop_model extends Simple_model\n{ ... }\n``` |
| 🔴 严重 | 末尾行 | 提供的代码片段不完整，数组定义在 `'config_params` 处突然截断，缺少闭合括号与类结束符，直接导致 `Parse Error`。 | 补全缺失的数组结构闭合符号 `]);`、属性闭合 `];` 及类闭合 `}`，确保语法合法。 | ```php\n ]\n ]\n ]\n ]\n ];\n}\n``` |
| 🟠 警告 | 第 5 行起 | 将超大型静态业务配置数组硬编码在 Model 属性中。每次实例化该模型都会将完整数组载入内存，造成不必要的内存开销，且违反“配置与业务逻辑分离”的最佳实践。 | 将 `$operational_scene_config` 迁移至独立配置文件（如 `application/config/community_shop.php`），或通过框架 Config 组件/缓存层动态读取。 | ```php\n// config/community_shop.php\nreturn [\n 'operational_scene_config' => [ /* 原数组内容 */ ]\n];\n\n// Model 中按需读取\npublic function getSceneConfig($sceneId)\n{\n $config = config('community_shop')['operational_scene_config'];\n return $config[$sceneId] ?? [];\n}\n``` |
| 🟡 建议 | 第 4 行 | 类名 `Ahead_community_shop_model` 采用蛇形命名。虽兼容旧版 CI3 规范，但不符合 PSR-12/PSR-4 现代 PHP 命名标准，不利于自动加载与团队协作。 | 建议重命名为 `AheadCommunityShopModel`，同步修改文件名，并更新项目自动加载映射。 | ```php\nclass AheadCommunityShopModel extends Simple_model\n{\n // ...\n}\n``` |
| 🟡 建议 | 全局 | **框架适配提示**：代码目录结构（`system/helpers/`、`get_instance()`、`load->model()`）高度疑似 CodeIgniter 3。若 `phpci` 为自研/魔改框架，请确认其是否完全兼容 CI3 的单例模式与模型加载机制。 | 查阅 `phpci` 官方文档确认依赖注入或模型加载方式。建议优先使用构造函数初始化或框架提供的 Service Container，避免全局单例滥用。 | ```php\n// 若框架支持 DI 或标准 CI3 规范\npublic function __construct()\n{\n parent::__construct();\n // 框架特定初始化逻辑（如需）\n}\n``` |
## 3. 总结与行动建议
### 🔑 优先修复项
1. **立即移除顶层执行代码**：删除文件开头的 `$CI = &get_instance();` 与 `$CI->load->model()`，避免文件被包含时引发致命错误。
2. **补全语法结构**：修复末尾截断的数组与类定义，确保代码可通过 `php -l` 语法检查。
3. **配置与模型解耦**：将 `$operational_scene_config` 迁移至 `config/` 目录或数据库配置表，模型仅负责读取与校验，不再承担数据定义职责。
### 🛠 后续重构与优化方向
- **引入配置缓存**：该配置数组体积较大且变更频率低，建议结合框架 Cache 组件（如 Redis/File）进行缓存，减少重复解析开销。
- **配置结构抽象化**：若后续配置项持续膨胀，建议将数组结构抽象为 `DTO` 或 `ConfigEntity` 类，配合验证规则（如 `Form_validation` 或自定义 Validator）确保数据完整性。
- **动态化管理**：当前配置为硬编码，不利于运营人员修改。建议后续将此类场景配置持久化至数据库，并提供后台管理界面进行可视化维护。
- **框架规范对齐**：请明确 `phpci` 与 CodeIgniter 的继承关系。若为独立框架，请严格遵循其官方模型加载、配置读取及生命周期规范，避免混用不同框架的 API 导致隐性兼容问题。
> 💡 **注**：本次审查基于提供的代码片段。若实际业务逻辑（如配置读取、校验、入库方法）存在于其他文件中，建议一并提供以便进行完整的链路审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779786699
|
1779786699
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
476
|
18
|
183
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 1
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `f954ecb712 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `f954ecb712f639d10ad639cb3c6491c588f9e5f1`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 18:08:22
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑覆盖较全，报表查询、过滤与导出流程完整。但存在明显的 **N+1 查询性能瓶颈**、**SQL 条件拼接注入风险** 以及 **非面向对象的全局变量调用**。代码风格偏向传统 CodeIgniter 3 写法，部分逻辑依赖隐式类型转换与未声明的全局函数，可维护性与健壮性有待提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | L2-L3 / 全局 | `$CI = &get_instance();` 在类外部直接执行。若文件在框架未完全初始化时被 `include/require`，将触发致命错误。且模型加载不应放在全局作用域。 | 移除全局代码，将依赖模型加载移至类构造函数中，遵循 OOP 规范。 | ```php<br>public function __construct()<br>{<br> parent::__construct();<br> $this->load->model('Report_model');<br> // 其他常用模型可在此预加载<br>}<br>``` |
| 🔴 严重 | `get_community_revenues_list()` ~L158-165 | **SQL 注入风险**：`$pay_platform_where` 使用字符串拼接构造 `OR` 条件，并直接传入 `$where['where'][]`。若底层 Query Builder 未做严格转义，恶意构造的 `pay_platform_arr` 可突破过滤。 | 放弃手动拼接 SQL 字符串，改用框架查询构建器的参数绑定（Prepared Statements）或分组条件方法。 | ```php<br>// 推荐改用框架原生分组查询<br>$this->db->group_start();<br>foreach ($params['pay_platform_arr'] as $item) {<br> $parts = explode('_', $item);<br> $this->db->or_group_start()<br> ->where('a._pay_platform', $parts[0])<br> ->where('a._second_pay_platform', $parts[1] ?? null)<br> ->group_end();<br>}<br>$this->db->group_end();<br>``` |
| 🟠 警告 | `get_community_revenues_list()` ~L130-133 | **N+1 查询性能瓶颈**：在 `foreach ($data as &$v)` 循环中，针对 `order_type == '1'` 的每条记录都执行一次 `ahead_book_order_model->get_one()`。数据量大时将导致数据库连接耗尽与响应超时。 | 采用 **批量查询 + 内存映射** 策略。先收集所有需要查询的 `book_order_id`，一次性查出结果，再在循环中通过键值匹配赋值。 | ```php<br>$book_ids = array_unique(array_filter(array_map(function($v) {<br> return $v['order_type'] == '1' ? preg_replace('/\(退款单号:.*\)$/', '', $v['order_id']) : null;<br>}, $data)));<br>$book_orders = $this->ahead_book_order_model->get_data_by_ids($book_ids, '_id,_shop_name,_arrival_time,_end_time', '_id');<br>// 循环内直接 $book_orders[$book_order_id] 取值<br>``` |
| 🟠 警告 | `get_community_revenues_list()` ~L108 | **分页统计逻辑缺陷**：仅当 `$params['page'] == '1'` 时计算 `$count` 与 `$sum_data`，其他页码变量未定义。虽用 `??` 兜底，但违背分页常规设计，且 `== '1'` 为弱类型比较。 | 统计查询应与分页解耦，或明确约定前端传参逻辑。使用严格类型比较 `(int)$params['page'] === 1`。 | ```php<br>$is_first_page = (int)($params['page'] ?? 1) === 1;<br>if ($is_first_page) {<br> $count = $this->count($where);<br> $sum_data = $this->get_one($where, 'SUM(IF(a._type = 1, a._amount, -a._amount)) as total_amount');<br>}<br>``` |
| 🟠 警告 | `get_community_revenues_list()` ~L145 | **数组越界风险**：`explode('_', $params['pay_platform'])` 后直接访问 `$pay_platform[1]`。若传入值不含 `_`，将触发 `Undefined offset` 警告并导致逻辑错乱。 | 使用 `list()` 解构或 `isset()` 安全取值。 | ```php<br>$parts = explode('_', $params['pay_platform']);<br>$where['a._pay_platform'] = $parts[0];<br>if (isset($parts[1]) && $parts[1] !== '') {<br> $where['a._second_pay_platform'] = $parts[1];<br>}<br>``` |
| 🟡 建议 | L10 | **类名不符合 PSR-12**：使用蛇形命名 `Jh_community_shop_revenues_detail_model`，不符合 PHP 标准类名大驼峰规范。 | 重命名为 `JhCommunityShopRevenuesDetailModel`，并同步更新所有引用处。 | `class JhCommunityShopRevenuesDetailModel extends Report_model` |
| 🟡 建议 | L17-L38 | **常量键类型不一致**：`ORDER_TYPE` 数组键使用字符串 `'1'`、`'2'`，后续逻辑中又与字符串/整型混用比较。 | 统一使用整型键，或在类顶部添加 `declare(strict_types=1);` 强制类型安全。 | `const ORDER_TYPE = [1 => '在线预订', 2 => '卡券核销兑换', ...];` |
| 🟡 建议 | 全局多处 | **依赖未声明的全局函数**：`two_dimensional_arr_sort()`、`filter_emoji()`、`change_number_format()` 等未通过 `use` 或 `require` 显式引入。若未自动加载，将导致 `Fatal Error`。 | 将通用函数封装为静态工具类（如 `App\Utils\StringHelper`）或确保已正确加载对应 Helper。避免污染全局命名空间。 | `use App\Utils\ExcelHelper;`<br>`$v['user_name'] = ExcelHelper::filterSpecialChars($name);` |
> 📌 **框架适配说明**：代码呈现典型的 CodeIgniter 3 语法特征（如 `$CI = &get_instance()`、`$this->load->model()`、数组型 `$where` 构建器）。若 `phpci` 为基于 CI 的定制框架，请核对底层 `DB_driver.php` 是否完整支持该 `$where` 数组语法及自动转义机制；若为独立框架，请严格参照 `phpci` 官方文档调整模型加载、查询构建与分页组件的调用方式。
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **消除 SQL 拼接隐患**：立即重构 `$pay_platform_where` 的拼接逻辑，全面改用框架提供的参数化查询或 Query Builder 分组方法，杜绝注入风险。
2. **解决 N+1 查询**：将循环内的单条 `get_one()` 替换为批量查询，预计可降低 70% 以上的数据库交互耗时，显著提升报表加载速度。
3. **规范模型加载与作用域**：移除文件顶部的全局 `$CI` 调用，统一在 `__construct()` 中加载依赖模型，确保框架生命周期安全。
### 🛠 后续重构与优化方向
- **查询构建器标准化**：当前 `$where` 数组结构（如 `'join' => [...]`、`'where_in' => [...]`）属于高度定制化的封装。建议逐步迁移至框架原生 Query Builder 链式调用，提升代码可读性与跨版本兼容性。
- **输入校验与类型安全**：对 `$params['start_time']`、`$params['end_time']` 增加日期格式校验（如 `DateTime::createFromFormat`），避免 `strtotime()` 返回 `false` 导致查询异常。建议开启 `declare(strict_types=1);` 并规范参数类型声明。
- **配置与常量抽离**：将硬编码的表名（如 `ktv_online.ahead_merchant_room_type`）、支付平台映射、运营场景字典等抽离至配置文件或枚举类中，便于后续多租户/多环境切换。
- **单元测试覆盖**：针对 `get_community_revenues_list()` 的复杂过滤条件与分页逻辑，编写 PHPUnit 测试用例，重点覆盖边界条件（空数组、非法 JSON、跨页查询、退款单号解析等）。
> 💡 **提示**：若当前项目处于快速迭代期，建议先完成 🔴 与 🟠 级别问题的修复，再逐步推进 🟡 级别的重构。所有修改请务必在测试环境验证数据库执行计划（`EXPLAIN`）与内存占用情况。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780394903
|
1780394903
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
629
|
18
|
196
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 前端编译
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `ffb241600a ## 自动代码审查报告
**分支**: pc-260616
**提交**: `ffb241600a5f4bc792f5a1d30759c2b77443372e`
**提交人**: caihongyuchy (1091045324@qq.com)
**时间**: 2026-06-09 13:58:05
---
## 📋 审查摘要
- **变更文件数**: 1
- **严重问题**: 2
- **高危问题**: 4
- **中危问题**: 3
- **建议优化**: 2
## 🐛 发现的问题
### <font color="red">[语法错误] 代码文件不完整/缺少闭合标签</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: 文件末尾
- **问题描述**: 代码在 `getChecksGoodsAfterChangeEndTime` 方法的 `success` 回调中突然截断，缺失了闭合的大括号 `}`、`</script>` 以及 `</template>`。这将直接导致 Vue 单文件组件编译失败或运行时解析错误。
- **修复建议**: 补充完整的方法逻辑、闭合括号及文件结构标签。确保 `methods` 对象和 `export default` 正确闭合。
### <font color="red">[语法错误] 未导入的依赖导致运行时 ReferenceError</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: 全文多处（如 `moment()`, `$()`, `layer.msg()`）
- **问题描述**: 代码中大量使用了 `moment`、`$` (jQuery)、`layer` 等全局变量，但在 `<script>` 顶部未进行 `import` 声明。在现代前端工程化环境（Webpack/Vite）中，若未配置全局注入，将直接抛出 `ReferenceError: moment is not defined` 等错误。
- **修复建议**:
```javascript
import Vue from 'vue';
import XLSX from 'xlsx';
import moment from 'moment';
import $ from 'jquery';
import layer from 'layui/layer'; // 或对应UI库路径
// 确保 daterangepicker, select2, bootstrapTable 等 jQuery 插件已通过全局脚本或 npm 包正确引入
```
### <font color="red">[跨文件调用] 调用了未定义的全局 Vue 原型属性</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: 全文多处（如 `Vue.request_header`, `Vue.ctUrl`, `Vue.version`, `Vue.timeoutfun`）
- **问题描述**: 频繁通过 `Vue.xxx` 调用自定义属性。这些并非 Vue 官方 API，属于项目自定义的全局挂载。若未在 `main.js` 中通过 `Vue.prototype.$xxx = xxx` 正确挂载，或挂载时机晚于组件加载，将导致跨文件调用失败。
- **修复建议**: 确认入口文件已正确挂载。推荐改为实例调用方式以符合 Vue 规范：`this.$request_header`、`this.$ctUrl` 等，或在组件顶部显式引入配置对象。
### [逻辑 BUG] `$.map` 中的 `return` 无法中断循环或阻止后续执行
- **严重程度**: 高危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `saveGoods` 方法 & `getChecksGoods` 方法内
- **问题描述**: 在 `$.map(result, function(row) { ... if(重复) { return; } ... })` 中，`return` 仅表示当前迭代返回 `undefined`，**不会终止循环**，也不会阻止外层函数继续执行。若存在重复商品，`layer.msg` 会连续弹出多次，且重复数据仍可能被错误处理。
- **修复建议**: 改用 `for` 循环或 `Array.prototype.some` 进行前置查重，找到重复项后直接 `return` 终止整个方法：
```javascript
const hasDuplicate = result.some(row =>
_this.checks_goods_list.some(item => item.merchant_goods_id === row.merchant_goods_id)
);
if (hasDuplicate) {
layer.msg("商品已存在");
return;
}
// 后续批量添加逻辑...
```
### [逻辑 BUG] 对象相加导致类型错误与无用代码
- **严重程度**: 高危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `addExport` 方法内 `let sheet3 = sheet1 + sheet2;`
- **问题描述**: `sheet1` 和 `sheet2` 是 `XLSX.utils` 生成的 Worksheet 对象。使用 `+` 运算符会触发隐式类型转换，结果为字符串 `"[object Object][object Object]"`。该变量后续未使用，属于明显的逻辑错误与冗余代码。
- **修复建议**: 直接删除 `let sheet3 = sheet1 + sheet2;` 这行代码。
### [逻辑 BUG] `split()` 无参数调用不符合业务预期
- **严重程度**: 高危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `addExport` 方法内 `"goods_type_name": _this.add_type_txt.split(),`
- **问题描述**: `String.prototype.split()` 不传参数时，会返回包含原字符串的单元素数组 `['原字符串']`，而非按逗号或其他分隔符拆分。若后端期望的是拆分后的数组或纯字符串，将导致数据格式错误。
- **修复建议**: 根据实际分隔符补充参数，例如 `_this.add_type_txt.split(',')`；若只需传字符串，直接传 `_this.add_type_txt` 即可。
### [代码质量] 组件 `name` 与文件名严重不符
- **严重程度**: 中危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `export default { name: 'procurement_return', ... }`
- **问题描述**: 文件名为 `stock_checks.vue`（库存盘点），但组件名称定义为 `'procurement_return'`（采购退货）。这会导致 Vue DevTools 调试信息混乱，影响组件缓存（`keep-alive`）及递归组件调用。
- **修复建议**: 将 `name` 修改为与文件名一致的驼峰命名：`name: 'stock_checks'`。
### [代码质量] 多处方法名拼写错误
- **严重程度**: 中危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `getMunu`, `innitDate`, `getDeteil`
- **问题描述**: 存在明显的英文拼写错误：`getMunu` 应为 `getMenu`，`innitDate` 应为 `initDate`，`getDeteil` 应为 `getDetail`。虽不影响运行，但严重降低代码可读性与专业度。
- **修复建议**: 全局搜索并替换为正确拼写，保持命名规范。
### [安全隐患] 潜在 XSS 风险（DOM 操作与数据渲染）
- **严重程度**: 中危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `initTable` 方法内 `formatter` 及多处 `$.each` 拼接
- **问题描述**: 当前 `formatter` 返回硬编码 HTML 是安全的，但代码中大量使用 jQuery 插件（Select2, BootstrapTable）直接渲染数据。若后续在 `formatter` 中拼接 `row` 字段（如 `row.goods_name`）而未做 HTML 转义，极易引发 XSS 攻击。
- **修复建议**: 优先使用 Vue 的模板语法渲染数据。若必须使用 jQuery 插件的 `formatter`，务必对动态数据进行转义：`return '<span class="text-blue detail">' + Vue.escape(row.number) + '</span>';`。
## ✅ 代码亮点
1. **状态管理清晰**：使用 `watch` 监听 `export_checkModel`、`checks_goods_list` 等数组变化，自动联动 `btnClickable` 和全选状态，交互逻辑闭环较好。
2. **API 请求封装规范**：所有 AJAX 请求统一使用 `Vue.request_header`、`Vue.version` 等标准头信息，并统一处理了 `result_code` 和 `timeoutfun`，便于后端鉴权与会话管理。
3. **用户体验细节**：在 `clearStore` 中保留了滚动条位置 `scrollTop`，避免了表格刷新导致的页面跳动，提升了操作流畅度。
## 📝 总体建议
1. **架构规范**：该文件为 Vue 前端组件，**不涉及 PHP CodeIgniter 的模型/控制器直接调用**。但代码中调用的 API 路由（如 `stock/searchStockTakingList`、`PublicData/api_getShopListByPurview`）符合 CI 框架的 `控制器/方法` 路由规范。请确保后端对应 Controller 已正确实现且权限校验完备。
2. **技术栈冲突**：在 Vue 项目中重度依赖 jQuery 及 jQuery 插件（Select2, BootstrapTable, Daterangepicker）属于**反模式**。Vue 的虚拟 DOM 更新机制与 jQuery 的直接 DOM 操作极易产生状态不同步、内存泄漏或事件绑定失效。强烈建议逐步替换为 Vue 生态组件（如 `element-ui`/`ant-design-vue` 的表格、日期选择器、下拉框）。
3. **工程化改造**：务必补充缺失的 `import` 声明，修复文件截断问题，并统一使用 `this.$nextTick` 包裹所有 jQuery DOM 初始化操作，确保 Vue 渲染完成后再执行插件挂载。
4. **代码整洁度**：清理注释掉的废弃代码块（如 `addExport` 中大段注释的 AJAX 请求），修正拼写错误，统一使用严格相等 `===` 替代 `==`，可大幅提升代码可维护性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780984685
|
1780984685
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
25
|
17
|
3
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - 修改单个文件测试3
|
## 自动代码审查报告
**分支**: pctest
**提交**: `12a4a4ad441e2 ## 自动代码审查报告
**分支**: pctest
**提交**: `12a4a4ad441e2edf19328cb83a4d1532f1985c69`
**时间**: 2026-03-20 15:30:26
**审查模型**: qwen3.5-plus
---
[2026-03-20 15:29:22] 正在调用 Qwen API 进行代码审查...
[2026-03-20 15:30:25] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260320-152922.md
基于您提供的项目结构和 `application/controllers/Book.php` 的文件内容，以下是对该控制器的详细代码分析报告。这份报告涵盖了功能概述、潜在缺陷、安全性问题以及优化建议。
### 1. 功能概述
`Book` 控制器主要处理与**预订单（Booking Orders）**相关的业务逻辑，分为两个主要板块：
1. **微信预订单管理**：包括列表查询 (`getList`)、详情查看 (`getDetail`)、导出报表 (`export`) 和退款处理 (`refund`)。
2. **智慧商家 App 预定**：包括列表查询 (`getBookList`)、详情查看 (`getBookDetail`) 和导出报表 (`exportBook`)。
该控制器继承自 `PcServer`，表明它依赖于父类提供的权限验证、参数处理 (`$this->param`) 和响应格式 (`success_response`, `error_response`) 等基础功能。
### 2. 代码缺陷与潜在 Bug
在代码审查中发现了以下几个明显的问题，建议优先修复：
* **数组键名拼写错误 (高危)**
* **位置**: `getList()` 方法，第 23 行。
* **代码**: `$page_size = isset($param['page_size's]) ? ...`
* **问题**: 键名写成了 `'page_size's`（多了一个 `s` 且在字符串内），导致无法正确获取前端传递的分页大小参数，始终默认值为 20。
* **修复**: 应改为 `$param['page_size']`。
* **变量名不一致 (中危)**
* **位置**: `getList()` 方法，第 16 行。
* **代码**: `$merchant_id = $this->merchant_id ? $this->merchant_ids : ...`
* **问题**: 判断使用的是 `$this->merchant_id`，但赋值使用的是 `$this->merchant_ids`（复数）。如果父类 `PcServer` 中只定义了 `merchant_id`，这里会导致 `$merchant_id` 为空或报错。
* **修复**: 确认父类属性名，通常应统一为 `$this->merchant_id`。
* **响应流中断 (中危)**
* **位置**: `export()` 和 `exportBook()` 方法。
* **代码**: `exit('导出字段不能为空');` 和 `exit($exportRes['msg']);`
* **问题**: 直接使用 `exit` 会输出纯文本，破坏了项目统一的 JSON 响应格式。前端接收到的将是非 JSON 数据，导致解析错误。
* **修复**: 应使用 `$this->error_response('...')` 或类似的标准错误处理方法。
* **模型加载大小写不一致**
* **位置**: `export()` 方法。
* **代码**: `$this->load->model('Export_model');` vs `$this->load->model('export_model');` (在 `exportBook` 中)
* **问题**: 虽然 Linux 文件系统区分大小写，但为了规范性和兼容性，建议统一模型命名的大小写（通常首字母大写或全小写需与文件名一致）。
### 3. 安全性分析
* **超全局变量直接使用**
* **位置**: `export()` 和 `exportBook()` 方法。
* **代码**: `$param = $_GET;`
* **风险**: `getList()` 使用了 `$this->param`（通常经过过滤），但导出方法直接使用 `$_GET`。这可能绕过父类中的参数清洗或安全验证逻辑，存在 SQL 注入或 XSS 风险（尽管模型层可能做了防护，但控制器层应保持一致）。
* **建议**: 统一使用 `$this->param` 或经过验证的输入变量。
* **文件导出路径与权限**
* **位置**: `export()` 方法末尾。
* **代码**: `$objPHPExcel->saveFile($filename);`
* **风险**: 文件保存路径未明确显示（取决于 `saveFile` 实现）。如果文件保存在 Web 根目录且未加访问限制，可能导致敏感数据泄露。
* **建议**: 确保导出文件存储在非公开目录，或通过脚本流式输出后删除，避免遗留敏感文件。
* **硬编码的 Excel 列索引**
* **位置**: `export()` 方法。
* **代码**: `$objPHPExcel->getActiveSheet()->setCellValue('H' . $rowCount, ...)`
* **风险**: 硬编码列号（如 'H', 'I'）。如果 `fields` 数组顺序发生变化，合计金额可能会错位到错误的列。
* **建议**: 动态计算列索引。
### 4. 依赖关系分析
根据代码内容，该控制器强依赖以下模型和库：
| 依赖项 | 类型 | 用途 | 备注 |
| :--- | :--- | :--- | :--- |
| `PcServer` | 控制器父类 | 基础权限、响应格式 | 手动 include 引入 |
| `ahead_book_order_model` | 模型 | 微信预订单 CRUD、退款、统计 | 核心业务逻辑 |
| `ahead_book_model` | 模型 | 智慧商家 App 预订单逻辑 | 核心业务逻辑 |
| `Export_model` / `export_model` | 模型 | 生成 Excel/PDF 文件 | 注意大小写一致性 |
| `Ahead_finance_report_setting_model` | 模型 | 保存导出配置记录 | 审计用途 |
| `ahead_setting_clean_data_model` | 模型 | 数据清洗日期策略 | 合规/数据保留策略 |
| `PHPExcel` | 第三方库 | Excel 文件操作 | **已废弃**，建议迁移至 PhpSpreadsheet |
### 5. 优化与重构建议
1. **修复致命 Bug**: 立即修复 `getList` 中的 `'page_size's` 拼写错误和 `$merchant_ids` 变量名问题。
2. **统一出口规范**: 移除所有 `exit()` 调用，改为返回标准 JSON 错误响应。
3. **代码复用**: `export()` 和 `exportBook()` 中有大量重复的 Excel 生成逻辑（表头设置、样式、保存）。建议提取到一个公共 Helper 方法或父类方法中。
4. **库升级**: `PHPExcel` 库已不再维护，存在安全漏洞且不支持新版 PHP。建议迁移至 `PhpSpreadsheet`。
5. **参数验证**: 在控制器层增加对 `$param` 的严格类型验证（如 `shop_id` 必须为整数），减轻模型层压力。
6. **权限检查**: 确保 `refund()` 方法也进行了严格的权限校验（当前代码直接调用模型，未见明显的权限检查逻辑，依赖模型内部实现可能存在风险）。
### 6. 总结
`Book.php` 是一个业务逻辑密集的控制器，涵盖了查询、导出和退款功能。虽然功能完整，但存在明显的拼写错误和不规范的代码实践（如直接 `exit`、混用 `$_GET`）。建议在进行下一步开发前，先完成上述的 Bug 修复和代码规范化工作，以确保系统的稳定性和安全性。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773991826
|
1773991826
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
34
|
17
|
4
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - ce1
|
## 自动代码审查报告
**分支**: pctest
**提交**: `6a1850c0a6b2b ## 自动代码审查报告
**分支**: pctest
**提交**: `6a1850c0a6b2b9a5c6f910af5ddecbf2d99c8a96`
**时间**: 2026-03-25 09:56:34
**审查模型**: qwen3.5-plus
---
[2026-03-25 09:55:38] 正在调用 Qwen API 进行代码审查...
[2026-03-25 09:56:34] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260325-095538.md
### alilog.php 中的引用:
1. **外部 SDK 依赖**:
* **文件路径**: `/mnt/data/www/CommonClass/aliyun-log-php-sdk-master/Log_Autoload.php`
* **说明**: 通过 `require_once` 引入了阿里云日志服务（SLS）PHP SDK 的自动加载文件。这是一个位于项目目录结构之外（`/mnt/data/www/CommonClass/`）的公共类库依赖。
2. **依赖的类 (来自阿里云 SDK)**:
该文件大量实例化或使用了以下 SDK 类，均依赖于上述引入的 `Log_Autoload.php`：
* `Aliyun_Log_Client`: 核心客户端类，用于建立与阿里云日志服务的连接。
* `Aliyun_Log_Models_LogItem`: 用于构建单条日志内容。
* `Aliyun_Log_Models_PutLogsRequest`: 写入日志的请求对象。
* `Aliyun_Log_Exception`: 捕获 SDK 抛出的特定异常。
* `Aliyun_Log_Models_ListLogstoresRequest`: 列举 Logstore 的请求对象。
* `Aliyun_Log_Models_ListTopicsRequest`: 列举 Topic 的请求对象。
* `Aliyun_Log_Models_GetLogsRequest`: 查询普通日志的请求对象。
* `Aliyun_Log_Models_LogStoreSqlRequest`: 执行 Logstore SQL 查询的请求对象。
* `Aliyun_Log_Models_GetProjectLogsRequest`: 执行项目级日志查询的请求对象。
* `Aliyun_Log_Models_ProjectSqlRequest`: 执行项目级 SQL 查询的请求对象。
* `Aliyun_Log_Models_GetHistogramsRequest`: 获取日志直方图的请求对象。
* `Aliyun_Log_Models_ListShardsRequest`: 列举 Shard 分片的请求对象。
* `Aliyun_Log_Models_GetCursorRequest`: 获取 Shard 游标的请求对象。
* `Aliyun_Log_Models_BatchGetLogsRequest`: 批量拉取日志的请求对象。
* `Aliyun_Log_Models_MergeShardsRequest`: 合并 Shard 的请求对象。
* `Aliyun_Log_Models_SplitShardRequest`: 分裂 Shard 的请求对象。
3. **项目内部文件引用**:
* **无**。该文件未引用项目结构列表中的任何文件（如 `application/config/config.php`, `application/controllers/*` 等）。
* 它没有使用项目的配置管理类来获取数据库连接、Redis 配置或阿里云密钥，而是直接在脚本底部硬编码了配置信息。
4. **配置耦合**:
* **硬编码凭证**: 文件中直接写死了 `endpoint`, `accessKeyId`, `accessKey`, `project`, `logstore` 等敏感配置信息。
* **影响**: 这导致该文件与环境强耦合，无法通过项目的配置文件（如 `application/config/` 下的文件）动态切换环境（开发/生产），且存在安全风险。
5. **执行模式**:
* 文件末尾包含直接执行代码（实例化 `Aliyun_Log_Client` 并调用 `putLogs`），表明该文件不仅是一个函数库，被访问时还会立即执行日志写入操作。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774403794
|
1774403794
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
24
|
17
|
2
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - 修改单个文件测试2
|
## 自动代码审查报告
**分支**: pctest
**提交**: `7fbec3ef95a38 ## 自动代码审查报告
**分支**: pctest
**提交**: `7fbec3ef95a3856eef25ceabf266030d5ca445e4`
**时间**: 2026-03-20 15:24:52
**审查模型**: qwen3.5-plus
---
[2026-03-20 15:23:33] 正在调用 Qwen API 进行代码审查...
[2026-03-20 15:24:52] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260320-152333.md
基于您提供的项目结构、`Book.php` 控制器代码以及跨文件引用分析，以下是对该文件的详细分析报告。这份报告补充了“项目中定义的类和方法”部分，并对代码质量、潜在风险及优化建议进行了审查。
## 1. 文件功能概述
**文件路径**: `application/controllers/Book.php`
**类名**: `Book`
**继承**: `PcServer`
**主要功能**:
该控制器主要用于处理**微信预订单**和**智慧商家 APP 预定**相关的业务逻辑。功能涵盖预订单的列表查询、详情查看、数据导出（Excel/PDF）以及退款操作。它区分了两种主要的业务场景：
1. **微信预订单** (`getList`, `getDetail`, `export`, `refund`)：基于 `ahead_book_order_model`。
2. **智慧商家 APP 预定** (`getBookList`, `getBookDetail`, `exportBook`)：基于 `ahead_book_model`。
## 2. 项目中定义的类和方法 (补充)
根据提供的 `Book.php` 内容，补充该文件中定义的类和方法信息：
### PHP 类:
| 类名 | 继承自 | 文件路径 | 描述 |
| :--- | :--- | :--- | :--- |
| `Book` | `PcServer` | `application/controllers/Book.php` | 预订单管理控制器，处理列表、详情、导出及退款逻辑 |
### 公共方法列表:
| 方法名 | 对应业务 | 描述 |
| :--- | :--- | :--- |
| `getList()` | 微信预订单 | 获取微信预订单列表，支持多条件筛选、分页及总金额统计 |
| `getDetail()` | 微信预订单 | 获取单个微信预订单的详细信息 |
| `export()` | 微信预订单 | 导出微信预订单报表 (Excel/PDF) |
| `refund()` | 微信预订单 | 执行预订单退款操作 |
| `getBookList()` | 智慧商家 APP | 获取商家端预订单列表 |
| `getBookDetail()` | 智慧商家 APP | 获取商家端预订单详情 |
| `exportBook()` | 智慧商家 APP | 导出商家端预订单报表 (Excel/PDF) |
## 3. 依赖项分析
### 引用的模型 (Models)
| 模型名 | 用途 | 备注 |
| :--- | :--- | :--- |
| `ahead_setting_clean_data_model` | 数据清洗/日期处理 | 仅在 `getList` 中用于处理清理日期 |
| `ahead_book_order_model` | 微信预订单核心逻辑 | 用于列表、详情、导出、退款 |
| `ahead_book_model` | 商家 APP 预订单核心逻辑 | 用于商家端列表、详情、导出 |
| `Export_model` / `export_model` | 通用导出逻辑 | **注意：存在大小写不一致** |
| `Ahead_finance_report_setting_model` | 导出配置保存 | 用于记录导出设置 |
### 引用的类库 (Libraries)
| 类库路径 | 用途 |
| :--- | :--- |
| `PHPExcel/Classes/MyPHPExcel.php` | 手动生成 Excel 文件 | 仅在 `export` 方法后半段使用 |
### 引用的助手函数 (Helpers)
| 助手名 | 用途 |
| :--- | :--- |
| `check` | 权限或数据校验 | 在 `getList` 和 `export` 中加载 |
## 4. 代码审查与潜在问题 (Code Review)
在分析 `Book.php` 代码后，发现以下几个值得注意的问题和优化建议：
### 4.1 严重逻辑冗余 (Critical)
**位置**: `export()` 方法 (第 220 行 - 第 270 行)
**问题**: 该方法中存在**两段独立的 Excel 生成逻辑**。
1. 第一段 (约 220 行)：调��� `$this->Export_model->export2Excel` 生成对象并保存文件。
2. 第二段 (约 237 行)：重新加载 `MyPHPExcel` 库，实例化新对象，手动设置单元格内容，再次保存文件。
**风险**:
* 第二段逻辑覆盖了第一段的文件名生成规则（第一段包含 `merchant_id` 和 `admin_id`，第二段没有）。
* 造成性能浪费，且第二段手动设置单元格的逻辑可能与第一段 `Export_model` 处理的数据不一致。
* **建议**: 删除第二段手动生成 Excel 的代码，统一使用 `Export_model` 封装好的方法，或确认是否为遗留代码未清理。
### 4.2 命名规范不一致 (Consistency)
**位置**: 模型加载
**问题**:
* 在 `export()` 方法中：`$this->load->model('Export_model');` (大写 E)
* 在 `exportBook()` 方法中：`$this->load->model('export_model');` (小写 e)
**风险**: 虽然 PHP 类名通常不区分大小写，但在文件系统区分大小写的环境（如 Linux）下可能导致加载失败。
**建议**: 统一模型命名规范，建议全部使用小写或遵循 PSR 标准。
### 4.3 输入参数获取方式不统一 (Input Handling)
**位置**: 方法参数获取
**问题**:
* `getList()`: 使用 `$this->param` (通常来自框架封装的输入处理)。
* `export()` & `exportBook()`: 直接使用 `$_GET`。
**风险**:
* 直接使用 `$_GET` 绕过了框架可能提供的安全过滤或统一参数处理机制。
* 导致代码风格不一致，维护困难。
**建议**: 统一使用 `$this->param` 或框架推荐的输入获取方式。
### 4.4 错误处理方式不统一 (Error Handling)
**位置**: `export()` 方法
**问题**:
* 大部分方法使用 `$this->error_response()` 返回标准 JSON 错误。
* `export()` 方法中多处使用 `exit('...')` 直接终止脚本。
**风险**: 直接 `exit` 会导致前端无法接收到标准的错误响应格式，难以进行统一的错误提示处理。
**建议**: 统一使用 `$this->error_response()`。
### 4.5 代码复用性 (DRY Principle)
**位置**: `getList()` 与 `export()`
**问题**: 这两个方法中关于参数解析、权限校验、`where` 条件构建的逻辑高度重复（约 80% 代码相似）。
**风险**: 如果业务规则变更（例如增加一个筛选条件），需要同时修改两处，容易遗漏。
**建议**: 将构建查询条件的逻辑提取为私有方法（例如 `_buildBookQueryWhere()`），供 `getList` 和 `export` 共同调用。
### 4.6 硬编码 (Hardcoding)
**位置**: `getList()`
**问题**: `$user_type = $this->user_type ? $this->user_type : '2';`
**风险**: 默认用户类型硬编码为 '2'，建议通过配置文件或常量管理，以便后续扩展。
## 5. 总结
`Book.php` 是一个功能丰富的控制器，涵盖了预订单管理的核心流程。但代码中存在明显的**重复逻辑**和**不一致的编码风格**，特别是在 `export` 方法中存在疑似冗余的 Excel 生成代码，建议优先清理和优化，以提高代码的可维护性和稳定性。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773991492
|
1773991492
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
23
|
17
|
1
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - 修改单个文件测试
|
## 自动代码审查报告
**分支**: pctest
**提交**: `b23272882d68e ## 自动代码审查报告
**分支**: pctest
**提交**: `b23272882d68e0b61ab119ff0f670c4dbbe7a652`
**时间**: 2026-03-20 15:06:01
**审查模型**: qwen3.5-plus
---
[2026-03-20 15:04:58] 正在调用 Qwen API 进行代码审查...
[2026-03-20 15:06:01] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260320-150458.md
基于您提供的项目结构和代码变更内容，以下是对 `application/controllers/AdMachine.php` 文件的代码分析及潜在问题审查：
### 1. 文件概述
* **功能**: 该控制器主要用于管理广告机（AdMachine）的屏保内容，提供了获取 (`getScreensaver`) 和设置 (`setScreensaver`) 屏保内容的接口。
* **框架**: 基于 CodeIgniter 框架（通过 `BASEPATH`, `FCPATH`, `load->model` 等特征判断）。
* **继承关系**: 继承自 `PcServer` 控制器 (`application/controllers/PcServer.php`)。
### 2. 代码审查与潜在风险
#### ⚠️ 高风险：模型名称拼写可疑
在 `getScreensaver` 方法中，加载了两个非常相似的模型：
1. `Ahead_shop_ad_machine_set_modelsss` (注意末尾的三个 's')
2. `Ahead_shop_ad_machine_set_model`
```php
$this->load->model('Ahead_shop_ad_machine_set_modelsss');
$this->Ahead_shop_ad_machine_set_modelsss->set("dw"); // 疑似 typo
$this->load->model('Ahead_shop_ad_machine_set_model');
$res = $this->Ahead_shop_ad_machine_set_model->get_datas(...);
```
* **问题**: `Ahead_shop_ad_machine_set_modelsss` 极大概率是 `Ahead_shop_ad_machine_set_model` 的**拼写错误**。
* **影响**: 如果项目中不存在 `Ahead_shop_ad_machine_set_modelsss.php` 模型文件，代码运行时会报错。如果存在，需确认其用途（是否用于设置某种特定状态 "dw"？），否则应予以修正以减少维护困惑。
#### ⚠️ 架构规范：控制器继承控制器
* **现状**: `AdMachine` 继承自 `PcServer`，且在文件头部手动 `include` 了父控制器文件。
* **建议**: 在 CodeIgniter 标准实践中，通常建议创建一个基类控制器放在 `application/core/` 目录下（例如 `MY_Controller.php`），而不是直接在 `controllers` 目录下让业务控制器相互继承。
* **依赖耦合**: 该类强依赖于父类 `PcServer` 中定义的属性（`$this->merchant_id`, `$this->param`, `$this->stream`）和方法（`success_response()`）。如果父类逻辑变更，此类极易受影响。
#### ⚠️ 模型加载方式
* **现状**: 模型在方法内部按需加载。
* **建议**: 如果这两个模型在该控制器的多个方法中都会用到，建议在 `__construct` 构造函数中统一加载，以提高代码整洁度。
### 3. 跨文件引用总结
根据提供的变更内容，该文件依赖以下外部资源：
| 类型 | 名称 | 路径推测 | 用途 |
| :--- | :--- | :--- | :--- |
| **父类控制器** | `PcServer` | `application/controllers/PcServer.php` | 提供基础请求处理、参数解析 (`$this->param`) 及响应格式 (`success_response`) |
| **模型 (疑似)** | `Ahead_shop_ad_machine_set_modelsss` | `application/models/` | 用于调用 `set("dw")` 方法 (需确认是否存在) |
| **模型** | `Ahead_shop_ad_machine_set_model` | `application/models/` | 用于获取 (`get_datas`) 和设置 (`set_data`) 屏保数据 |
### 4. 优化建议代码示例
如果确认 `modelsss` 是笔误，且希望规范代码，建议修改如下：
```php
<?php
defined('BASEPATH') OR exit('No direct script access allowed');
// 建���：确保 PcServer 作为基类被正确 autoload 或放在 core 目录，避免手动 include
// include FCPATH . 'application' . DIRECTORY_SEPARATOR . 'controllers' . DIRECTORY_SEPARATOR . 'PcServer.php';
class AdMachine extends PcServer
{
public function __construct()
{
parent::__construct();
// 建议：在构造函数中加载常用模型
$this->load->model('Ahead_shop_ad_machine_set_model');
// 如果 modelsss 确实存在且必要，也在此加载，否则删除
// $this->load->model('Ahead_shop_ad_machine_set_modelsss');
}
public function getScreensaver()
{
// 修正潜在的拼写错误，确认 set("dw") 是否需要在主模型上调用
// $this->Ahead_shop_ad_machine_set_model->set("dw");
$res = $this->Ahead_shop_ad_machine_set_model->get_datas($this->merchant_id, $this->param);
$this->stream['request']['result'] = $res;
$this->success_response();
}
public function setScreensaver()
{
$this->Ahead_shop_ad_machine_set_model->set_data($this->merchant_id, $this->param);
$this->success_response();
}
}
```
### 5. 待确认事项
1. **确认模型文件是否存在**: 请检查 `application/models/` 目录下是否有 `Ahead_shop_ad_machine_set_modelsss.php`。
2. **确认 "dw" 参数含义**: 确认 `$this->Ahead_shop_ad_machine_set_modelsss->set("dw")` 的具体业务逻辑，是否可合并到主模型中。
3. **确认父类依赖**: 确保 `PcServer.php` 中已正确初始化 `$this->merchant_id` 和 `$this->param`，否则此类方法会运行失败。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773990362
|
1773990362
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
35
|
17
|
5
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - ce12
|
## 自动代码审查报告
**分支**: pctest
**提交**: `e03027962aed5 ## 自动代码审查报告
**分支**: pctest
**提交**: `e03027962aed59b0c87ce41b201b6905760521d6`
**时间**: 2026-03-25 10:24:19
**审查模型**: qwen3.5-plus
---
[2026-03-25 10:23:14] 正在调用 Qwen API 进行代码审查...
[2026-03-25 10:24:19] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260325-102314.md
### alilog.php 中的引用:
1. **外部依赖库**:
* **文件路径**: `/mnt/data/www/CommonClass/aliyun-log-php-sdk-master/Log_Autoload.php`
* **说明**: 该文件位于项目根目录之外（`/mnt/data/www/CommonClass/...`），引入了阿里云日志服务 PHP SDK 的自动加载文件。这意味着项目运行环境依赖于此特定路径下的 SDK 代码。
2. **使用的 SDK 类**:
代码中实例化或引用了以下阿里云 SDK 类（均通过 `Log_Autoload.php` 加载）：
* `Aliyun_Log_Client`: 客户端主类，用于发起请求。
* `Aliyun_Log_Models_LogItem`: 日志项模型。
* `Aliyun_Log_Models_PutLogsRequest`: 写入日志请求模型。
* `Aliyun_Log_Exception`: SDK 异常类。
* `Aliyun_Log_Models_ListLogstoresRequest`: 列举 Logstore 请求。
* `Aliyun_Log_Models_ListTopicsRequest`: 列举 Topic 请求。
* `Aliyun_Log_Models_GetLogsRequest`: 查询日志请求。
* `Aliyun_Log_Models_LogStoreSqlRequest`: LogStore SQL 查询请求。
* `Aliyun_Log_Models_GetProjectLogsRequest`: 项目级日志查询请求。
* `Aliyun_Log_Models_ProjectSqlRequest`: 项目级 SQL 查询请求。
* `Aliyun_Log_Models_GetHistogramsRequest`: 直方图查询请求。
* `Aliyun_Log_Models_ListShardsRequest`: 列举 Shard 请求。
* `Aliyun_Log_Models_GetCursorRequest`: 获取游标请求。
* `Aliyun_Log_Models_BatchGetLogsRequest`: 批量获取日志请求。
* `Aliyun_Log_Models_MergeShardsRequest`: 合并 Shard 请求。
* `Aliyun_Log_Models_SplitShardRequest`: 分裂 Shard 请求。
3. **项目内部引用**:
* **无**: 该文件未引用项目 `application/` 目录下的任何控制器、模型、配置文件或辅助函数。
* **配置独立**: 脚本底部硬编码了阿里云的 `endpoint`, `accessKeyId`, `accessKey`, `project`, `logstore` 等信息，未使用 `application/config/` 下的配置文件（如 `config.php` 或 `redis.php` 等）。
* **独立性**: 该文件看起来像是一个独立的测试脚本或工具脚本，尚未集成到项目的 MVC 流程中。
4. **代码问题与风险提示**:
* **严重安全风险**: 文件中明文硬编码了阿里云 AccessKey ID (`LTAInvzKfeu0j7KN`) 和 AccessKey Secret (`RH94sjMSAPx72l6tTq2NtMnl7uuwLM`)。**这属于严重的安全泄露，建议立即在阿里云控制台禁用该密钥并轮换。**
* **语法错误/残留代码**:
* 第 12 行：`function putLogs(...) { sss1111` 中存在无意义的字符串 `sss1111`，会导致 PHP 解析错误。
* 第 336 行：`print(..., nextCursor, ...)` 中变量 `nextCursor` 缺少 `$` 符号，应为 `$nextCursor`，否则会导致未定义变量警告或错误。
* **调试代码泄露**: 文件中包含大量 `var_dump`, `print`, `logVarDump` 调试输出，生产环境中应移除或改为日志记录。
* **路径依赖**: 依赖绝对路径 `/mnt/data/www/...`，降低了代码的可移植性，建议改为相对路径或通过配置文件管理 SDK 路径。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码���照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774405459
|
1774405459
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
691
|
50
|
1
|
5
|
|
0
|
fix(web): update Lyra product labels
|
<!--
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，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `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` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1783327279
|
1783431501
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
692
|
57
|
1
|
5
|
|
0
|
feat(material-factory): 图像模型真实推理对接 + 双后端渲染管线闭环
|
<!--
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，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `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` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1783381428
|
1783381594
|
1783381594
|
0
|
0
|
0
|
Edit
Delete
|
|
720
|
67
|
1
|
5
|
|
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，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `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` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1785285580
|
1785285593
|
1785285593
|
0
|
0
|
0
|
Edit
Delete
|
|
731
|
57
|
2
|
5
|
|
0
|
chore: add material factory model weights
|
<!--
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，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `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` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1786603028
|
1786603028
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
760
|
85
|
5
|
4
|
|
0
|
[Web] 按省市统计设备月度新增上线数量报表
|
> 子任务，属于主 Issue #8「按省市统计设备月度新增上线数量报表」
## 关联 De > 子任务，属于主 Issue #8「按省市统计设备月度新增上线数量报表」
## 关联 Design PR
[PR #3 — Design: 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/pulls/3)
## 工作范围
负责 Web 前端页面实现：
- 数据报表 → 设备上线分布 页面
- 月份筛选器 + 当月总数卡片 + 省份横向柱状图
- 点击省份下钻到城市视图，支持面包屑返回
## 产物
- plan/device-online-stats-by-region-web.md（实现前提交，Tech Lead 审核后开始 Build）
## 参考
- 需求说明：[design/device-online-stats-by-region.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/design/device-online-stats-by-region/design/device-online-stats-by-region.md)
- 在线原型：https://claude.ai/code/artifact/b50eaa98-689b-4742-8024-cdf74f95d5c8...
|
1
|
0
|
1
|
0
|
3
|
|
0
|
1787909556
|
1789380193
|
1789380193
|
0
|
1
|
0
|
Edit
Delete
|
|
759
|
85
|
4
|
4
|
|
0
|
[后台] 按省市统计设备月度新增上线数量报表
|
> 子任务，属于主 Issue #8「按省市统计设备月度新增上线数量报表」
## 关联 De > 子任务，属于主 Issue #8「按省市统计设备月度新增上线数量报表」
## 关联 Design PR
[PR #3 — Design: 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/pulls/3)
## 工作范围
负责后台接口实现：
- 新增查询接口，按月份统计各省份新增上线设备数量
- 支持下钻查询：指定省份，返回城市级别数据
- 按权限过滤：超管返回全国数据，运营商返回自己区域
## 产物
- （实现前提交，Tech Lead 审核后开始 Build）
## 参考
- 需求说明：[design/device-online-stats-by-region.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/design/device-online-stats-by-region/design/device-online-stats-by-region.md)
- 在线原型：https://claude.ai/code/artifact/b50eaa98-689b-4742-8024-cdf74f95d5c8...
|
1
|
0
|
1
|
0
|
2
|
|
0
|
1787909526
|
1789380193
|
1789380193
|
0
|
1
|
0
|
Edit
Delete
|
|
134
|
6
|
8
|
5
|
|
0
|
test(governance): AI Agent/获客域过时 E2E 选择器对齐 canonic test(governance): AI Agent/获客域过时 E2E 选择器对齐 canonical...
|
AI Agent 员工中心 / 获客中心域审计验收后残留 E2E 过时选择器对齐 canonical AI Agent 员工中心 / 获客中心域审计验收后残留 E2E 过时选择器对齐 canonical（curl/页面实证），两 spec 单跑+合并复跑 14/14 全绿，未放宽断言或 skip。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1779070186
|
1779328857
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
758
|
85
|
3
|
4
|
|
0
|
[Design] 按省市统计设备月度新增上线数量报表
|
Related to #8
## 关联 Intent
[PR #1 — 按省市统计设备月度新增上 Related to #8
## 关联 Intent
[PR #1 — 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/pulls/1)
---
## PM 确认原型
**需求说明文档：**
[design/device-online-stats-by-region.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/design/device-online-stats-by-region/design/device-online-stats-by-region.md)
**在线原型（点击直接打开，可交互，无需登录）：**
https://claude.ai/code/artifact/b50eaa98-689b-4742-8024-cdf74f95d5c8
**页面状态：**
- 全国省份视图 — 月份筛选器 + 当月总数卡片 + 各省份横向柱状图
- 城市下钻视图 — 点击省份柱子进入，面包屑「全国 › 广东省」+ 城市排行
**交互流程：** 点击省份柱子进入城市视图，点击面包屑「全国」或「返回全国」按钮返回。
**审核 checklist：**
- [ ] 页面布局和组件符合预期？
- [ ] 交互路径（下钻、返回）正确？
- [ ] 数据定义（新增上线 = 首次上线，归属 = 门店注册地址）理解一致？
- [ ] 权限范围（超管看全国，运营商看自己区域）正确？
**确认方式：在本 PR 评论「原型确认 ✅」，然后 Approve + Merge**...
|
1
|
0
|
1
|
1
|
0
|
|
0
|
1787829439
|
1789379899
|
1787908921
|
0
|
7
|
0
|
Edit
Delete
|
|
768
|
85
|
6
|
4
|
|
0
|
[Plan] 按省市统计设备月度新增上线数量报表 (后台管理)
|
Related to #8
## 关联 Issue
[#4 — [后台] 按省市统计设备月度新增 Related to #8
## 关联 Issue
[#4 — [后台] 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/issues/4)
## 说明
补交 Plan 阶段产物（此前误提交到已合并关闭的 design 分支，未能进入 main，见此 PR 重新提交）。
代码已实现并提交：[laiqiaojie/admin#1](https://gitea.g-hi.com/laiqiaojie/admin/pulls/1)
按 REVIEW.md「跳过评审的路径」，`plan/` 产物文件不需要 Tech Lead 单独审核，随此 PR 直接入库。...
|
0
|
0
|
1
|
1
|
1
|
|
0
|
1788245555
|
1789379899
|
1788245966
|
0
|
1
|
0
|
Edit
Delete
|
|
770
|
85
|
7
|
4
|
|
0
|
[Skill] build-complete 新增接口文档归档步骤
|
Related to #8
## 说明
在 `build-complete` skill 里新增 Related to #8
## 说明
在 `build-complete` skill 里新增一步：Plan 阶段确定接口设计后、代码开发完成时，如涉及新增/修改接口，需要生成并归档接口文档：
- Apifox（规范化 OpenAPI 文档，Claude 通过 API 增量导入）
- `docs/frontend-specs/<功能名>.md`（面向其他端的对接说明文档，固定五章结构）
背景：issue #4 实现完成后发现接口文档一直没有归档，靠事后补做；这次把它固化成流程里的正式一步，插在"提交 plan.md"和"开代码 PR"之间。
同时补充了代码仓库与 jh-project 分离时的注意事项（milestone/label 可能不存在、Close #N 跨仓库不生效）。...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1788247386
|
1789379900
|
1788247940
|
0
|
1
|
0
|
Edit
Delete
|
|
756
|
85
|
1
|
4
|
|
0
|
[Intent] 按省市统计设备月度新增上线数量报表
|
Related to #8
## 需求摘要
在「数据报表 > 设备分析」下新增报表，按月筛 Related to #8
## 需求摘要
在「数据报表 > 设备分析」下新增报表，按月筛选展示各省份新激活上线设备数量柱状图排行，支持下钻到城市维度。
## 核心决策
- 统计口径：当月**新激活**上线设备（首次上线，取 ahead_authenticate_log 表的 create_time 字段）
- 地区归属：取设备绑定**门店地址**的省/市字段
- 交互方式：点击省份**下钻**到城市排行（非 Tab 切换）
- 权限：跟随现有 agent_id 角色体系，超管看全国，运营商看自己区域
## 不在范围内
- 不做 Excel 导出
- 不做地图热力图
- 不做区/县级别
- 不做跨月趋势
## 产物文件
[intent/device-online-stats-by-region.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/intent/device-online-stats-by-region/intent/device-online-stats-by-region.md)
## PM 审核清单（参考 REVIEW.md）
- [ ] 需求背景和问题描述是否准确反映了原始诉求？
- [ ] 期望结果是否可验证？
- [ ] 不在范围内的排除项是否合理？
- [ ] 受影响系统判断是否正确（admin / ahead_authenticate_log）？
- [ ] 本次是否接受排期，进入 Design 阶段？...
|
1
|
0
|
1
|
1
|
2
|
|
0
|
1787817232
|
1789379898
|
1787823622
|
0
|
5
|
0
|
Edit
Delete
|
|
130
|
6
|
5
|
5
|
|
0
|
feat: 培训管理种子 + MQL 评分规则 + Gitea CI 迁移
|
对应 GitHub PR #52。
- backend/prisma/seeds/22-hr-tr 对应 GitHub PR #52。
- backend/prisma/seeds/22-hr-training-content.ts: 培训管理 12 张表种子
- backend/prisma/seeds/23-lead-scoring-rules.ts: MQL 默认评分规则集(修复 /lead-qualification 评估 400)
- .gitea/workflows/ci.yml: Gitea Actions CI(取代账户计费阻塞的 GitHub Actions)
- frontend course-marketplace.ts: array-guard 修复
- 已合并 origin/main 至 97a07d598,冲突全部解决
GitHub Actions 已仓库级禁用,CI 迁移到本 Gitea PR 验证。...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1778885087
|
1778891427
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
743
|
57
|
3
|
5
|
|
0
|
feat(script): 接入大语言模型——歌词/文案 → 拍摄脚本 → 角色确认 → 定格图
|
本仓此前零 LLM 接入（全仓只有 DashScope 的图像调用），「歌词成图」屏是纯前端
原型 本仓此前零 LLM 接入（全仓只有 DashScope 的图像调用），「歌词成图」屏是纯前端
原型（procedural.ts 画色块 + setTimeout 假装推理，不落库不接后端）。本轮把这条
动线升级为真链路 + 真持久化。
契约单源（packages/contracts/src/script.ts）
- 角色确认状态机 scriptRoleMachine、脚本状态机 shootingScriptMachine
- 景别/运镜用枚举而非自由文本：LLM 一旦自由发挥，下游 prompt 与统计就没有稳定口径
- scriptDraftSchema 是 LLM 输出的唯一判据；prompt 构造器单源，两后端禁止各拼一份
- evaluateRoleReadiness：本域核心不变量，双后端硬拦截 + 前端同源即时反馈
- 前端私有的 findBannedHits 上收为 contracts 的 findBannedWords（后端也要逐镜标记，
两份并行就是双真源）
LLM 不是权威
- sidecar /v1/script 只保证「返回长得像脚本的 JSON」，两个后端各自再 parse 一次才落库
- 模型引用未声明角色键 → 422 SCRIPT_ROLE_KEYS_DANGLING（幻觉护栏）
- 连续给不出合法 JSON → 502 LLM_OUTPUT_UNPARSEABLE，绝不返回占位脚本
- parse_json_object 只剥围栏、截首尾花括号，不做字段猜测或补齐
角色必须先确认才允许出定格图
- 判据 = status CONFIRMED 且有 referenceAssetId（没有参考图就没有一致性锚点）
- CONFIRMED --reset--> DRAFT 会解除参考图绑定：人确认的是那一张图，不是那个名字
- 出图时按 roleKeys 顺序喂参考图，与指令里「第 N 张参考图」编号严格对齐
写链纪律与物料链路同构：tenant 全覆盖、updateMany 带状态前置条件（0 行即 409）、
终态与 outbox 同 tx、能力不可用 → job BLOCKED + 诚实 reason，禁止静默降级。
品牌准绳只逐镜标记不拒绝整份脚本（脚本属创作内容），硬拦截点在出定格图这个受控出口，
显式放行需带参数且落审计事件。
顺带修掉三个既有缺陷
1. 合并遗留：sidecarEdit 用 sidecarGenerateResponseSchema 校验响应（要求 steps/
loadSeconds），但 run_hosted_edit 两个字段都不返回 —— 托管改字会在客户端边界
直接 Zod 抛错（本地路径返回这两个字段，所以一直没暴露）
2. check-migrations 的列检查只扫 CREATE TABLE，凡 ALTER TABLE ADD COLUMN 追加的
@map 列一律误报 drift（此前唯一的 ALTER 列 brand_kits.profile 没 @map 才没现形）
3. ktv-poster-master.png 血统在合并两侧打架，git log --all 证明该文件自初始化提交后
从未重新生成，取诚实标注 prototype-demo-master
验收
- 新增 pnpm check:script → reports/script-acceptance.latest.json（双后端各 8 passed
+ 1 skipped-live）；own-tests floor 10→14，棘轮已收紧
- pnpm check / check:runtime 均 exit 0
- 浏览器实测：未确认角色 → 按钮 disabled 且提示缺哪个角色；绕过前端直打后端 → 422
ROLE_NOT_CONFIRMED；跨租户 404；无凭证生成 → UI 显示 503 且不产生占位脚本
诚实边界（已写入 CLAUDE.md）
- liveLlmAsserted=false：本机无 DASHSCOPE_API_KEY，真调 LLM 的 live 断言被 skip，
门禁成立 ≠ LLM 出脚本已验证
- check:inference 在本机为 failed（动手前即为 failed）：本工作副本的模型权重是
git-lfs 指针存根，非本轮改动导致；基线表该行已从 GREEN 改为 OPEN
- ui-acceptance 报告早于本屏，未覆盖 script 屏，已标 STALE
- 新开缺口 G16（LLM 只做结构校验、无事实性校验）、G17（同步 HTTP 挂分钟级推理 /
render_jobs 索引非 CONCURRENTLY / 本地多图 I2I 未在本机验证）
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
<!--
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，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `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` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1787029791
|
1787032807
|
1787032807
|
0
|
0
|
0
|
Edit
Delete
|
|
21
|
2
|
21
|
1
|
|
0
|
11111
|
测试
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1773910676
|
1773910796
|
1773910698
|
0
|
0
|
0
|
Edit
Delete
|
|
790
|
116
|
1
|
5
|
|
0
|
fix(contracts): 注册中心规则补失败关闭与摘要规范化——三个入口此前抛 TypeErr fix(contracts): 注册中心规则补失败关闭与摘要规范化——三个入口此前抛 TypeError，同一摘要的两种写法此前分叉成两个复核键...
|
蓝图 docs/domain/registry-snapshot-blueprint.md 从迁入起 蓝图 docs/domain/registry-snapshot-blueprint.md 从迁入起就写着「失败关闭」与
「review key 仅用于人工复核幂等」两条不变量，实测三处不成立。本次按蓝图自己的口径补齐，
改动只在 contracts/src/domain/application-contract-registry/：不新增写入点、不新增宿主、
不动 Catalog 与 Schema，每条输出仍固定 activationAllowed=false / registryWriteAllowed=false /
humanApprovalRequired=true。
1. 失败关闭：assessRegistrySnapshot / assessRegistryTransition /
reviewCandidateSnapshotUpgrade 对畸形输入抛 TypeError（快照缺字段、now 不是 Date、
decisions 整个缺失都会抛），四个入口里只有 LocalManifestAdmissionPlanner.plan 是
失败关闭的。现四个入口一致返回 REJECT + 原因码。这不是洁癖：check-fixtures 把抛出当
配置错误处理，实测一次 TypeError 会让整套 15 例逐例判定塌成一条 configuration 失败。
2. 十六进制引用规范化：digest / commit 的正则带 /i 收大小写两种写法，比较与入键却
大小写敏感——同一摘要写成大写会得到另一个 reviewIdempotencyKey，同版本大小写翻转被判成
SAME_VERSION_DIGEST_CHANGED，大写回滚 pin 被判成 ROLLBACK_PIN_MUST_MATCH_CURRENT。
现一律规范化为小写后再比较、再入键，ACCEPT 回吐规范化后的值。
3. 版本文法收敛：一个模块三套（快照流与清单准入用 \d+\.\d+\.\d+，升级复核用严格 SemVer），
新增 registry-primitives.ts 作单一来源，含预发布序；DEC-009—012 此前在两份文件各写一份，
一并收敛。snapshot-upgrade-review.ts 因此净减约 60 行重复实现。
两处语义变化（蓝图新增一节已如实登记）：
- 收紧：1.02.0 这类前导零版本此前被放行，现在拒；assessRegistrySnapshot 另开始拒 latest
这类可变标签——此前它能过体检却必然过不了迁移规则。
- 放宽：1.0.0-rc.1 这类预发布版本此前被快照流整条挡在外面，现在按 SemVer 序参与迁移。
平台自己发的就是 1.0.0-rc.N，升级复核那一侧本来就这么判。风险面有限：本模块所有出口
都不激活、不写 Registry，多进人工复核队列不等于多放行制品。
证据（本分支隔离工作树，clean）：注册中心单测 44/44（原 34 + 加固 10）；
contracts check:local 302/302；check:fixtures 九个契约域套件 100 例 0 失败，
其中本套件 15 例（正 4 / 反 11，原 8 例）。篡改必红实测两次：sameHexRef 退回大小写敏感
→ 单测 1 红 + 夹具 P03 红；去掉快照入口守卫 → 整套塌成 0 例 1 配置失败。
仍为仓内候选 sdk_e2，不代表正式 Registry、Snapshot 发布或跨仓 Required Check 已上线。
清单登记的 C01.04 / C01.06 是「接线」型缺口且前置 Q01 运行宿主未裁，本次未动。
reports/fixtures.latest.json 未回绑：它是 17 套件的聚合报告，只跑 9 个套件去覆盖会缩小结论面，
应在整批（含并行会话的 public-file / IM / 跨域流程改动）落定后统一重跑回绑。
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1789742918
|
1789744064
|
1789742925
|
0
|
0
|
0
|
Edit
Delete
|