| content |
## 自动代码审查报告
**分支**: pay-260616
**提交**: `8bccacc25 ## 自动代码审查报告
**分支**: pay-260616
**提交**: `8bccacc2593ec81377a40498adce4fbc00e21239`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-05 13:43:13
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：代码完整实现了商户礼品/卡券的核心业务逻辑，数据流转清晰。但存在明显的架构与规范问题：类外部直接调用框架实例、核心方法严重违反 DRY 原则、SQL 拼接存在注入隐患、日期格式化不规范。整体可维护性较低，需优先进行安全加固与结构重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 全局/第3行 | `$CI = &get_instance();` 在类外部定义。若文件在框架未完全初始化时被 `include/require`，将直接触发 Fatal Error。且模型内部应直接使用 `$this`。 | 移除全局 `$CI` 赋值。若需加载其他模型，应在 `__construct()` 中调用或按需延迟加载。 | `public function __construct() { parent::__construct(); }` |
| 🔴 严重 | `get_sold_coupons_list` / `buying_gift_check_status` | **SQL 注入风险**：`$shop_id` 未经严格类型校验直接拼接至 `where_str`。攻击者可传入恶意字符串破坏查询逻辑或越权访问。 | 强制类型转换 `(int)$shop_id`，并优先使用框架查询构建器的参数绑定机制。若 `Simple_model` 不支持占位符，至少需做严格过滤。 | `$shop_id = (int)$shop_id;`<br>`$where_str[] = "(gift._shop_id = {$shop_id} OR FIND_IN_SET({$shop_id}, gift._satisfy_shop_ids))";` |
| 🟠 警告 | `get_gift_data` / `create_gift_data` | **日期格式化隐患**：`date('Ymd 8:00:00', ...)` 格式非标准，依赖 PHP 宽松解析。在部分 PHP 版本或严格模式下可能返回 `false` 或错误时间戳。 | 统一使用标准格式 `Y-m-d 08:00:00`，并增加空值/异常保护。 | `date('Y-m-d 08:00:00', strtotime("+$validity day", $excute_time))` |
| 🟠 警告 | `create_gift_data` | **严重违反 DRY 原则**：类型 1/2/3/4/5-8 的字段映射、时间计算、消息拼接逻辑高度重复（超 300 行）。新增券类型需修改多处，极易遗漏。 | 抽取私有方法 `build_coupon_payload($base, $type, $relation_data)` 统一处理公共逻辑，使用策略模式或配置数组驱动差异字段。 | *(见下方重构建议)* |
| 🟠 警告 | 多处查询 | **性能瓶颈**：频繁使用 `FIND_IN_SET` 匹配逗号分隔的 `_satisfy_shop_ids`。该函数无法利用 B-Tree 索引，数据量增长后将导致全表扫描。 | 建议拆分为关联表 `merchant_gift_shop`（`gift_id`, `shop_id`）。若暂无法改表，可考虑 MySQL 5.7+ 的 JSON 字段 + `JSON_CONTAINS`。 | 架构级优化，需 DBA 配合 |
| 🟡 建议 | 全文 | **魔法数字/字符串泛滥**：如 `'1'`, `'4'`, `86400`, `'当天生效'` 等硬编码散落各处，降低可读性与后期维护效率。 | 使用类常量集中管理业务状态与配置，如 `const TYPE_ROOM_PACKAGE = 4; const SECONDS_PER_DAY = 86400;`。 | `const STATUS_ACTIVE = 1;`<br>`const VALIDITY_TODAY = 1;` |
| 🟡 建议 | `get_coupon_pay_page` | **依赖隐式全局函数**：`throwError()`, `timeToHour()`, `returnWeek()` 等未声明依赖，破坏类的内聚性与单元测试可行性。 | 将全局函数封装为服务类注入，或统一使用标准 `throw new \Exception()` / CI 的 `show_error()`。 | `if (!in_array($pay_platform, ['1','3','22'])) { throw new \InvalidArgumentException('非法支付方式'); }` |
| 🟡 建议 | 全文 | **PSR-12 规范不符**：部分 `if/else` 缺少大括号、缩进混用（Tab/Space）、`array()` 与 `[]` 混用、超长行未换行。 | 配置 `PHP_CodeSniffer` 或 `PHP-CS-Fixer` 自动格式化，统一团队编码风格。 | `if (empty($fields)) { $fields = '...'; }` |
## 3. 总结与行动建议
### 🚨 优先修复项（P0）
1. **移除全局 `$CI` 实例化**：改为在 `__construct()` 中初始化，或直接使用 `$this->load->model()`。
2. **修复 SQL 拼接漏洞**：对所有外部传入的 ID/参数执行 `(int)` 强转或白名单校验，杜绝直接字符串拼接。
3. **修正日期计算逻辑**：将 `date('Ymd 8:00:00', ...)` 替换为 `date('Y-m-d 08:00:00', ...)`，避免时间解析异常导致券过期时间错误。
### 🛠 后续重构方向
1. **DRY 重构 `create_gift_data`**：
该方法是本次审查的核心痛点。建议将重复的“时间计算+字段映射”抽取为独立方法：
```php
private function build_coupon_data(array $base, array $relation, int $type): array
{
$time = time();
$excute = ($base['_start_time'] == 1) ? $time : $time + self::SECONDS_PER_DAY;
$validity = intval($base['_validity'] ?? 0) + 1;
$expire = strtotime(date('Y-m-d 08:00:00', strtotime("+$validity day", $excute)));
// 统一基础字段
$data = [
'_excute_time' => $excute,
'_expire_time' => $expire,
'_relation_id' => $relation['_id'] ?? $base['_relation_id'],
'_name' => $base['_name'],
'_quantity' => $base['_quantity'],
// ... 其他公共字段
];
// 按类型注入差异化字段
if ($type === self::GOODS_TYPE) {
$data['_value'] = $relation['_cost_price'];
}
// ... 其他类型处理
return $data;
}
```
2. **数据库设计优化**：评估将 `_satisfy_shop_ids` 从逗号分隔字符串迁移至独立关联表。若业务强依赖当前结构，可考虑在应用层缓存门店映射关系，减少 `FIND_IN_SET` 查询频率。
3. **框架适配说明**：代码结构高度契合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请确认 `Simple_model::select()` 是否支持预处理参数绑定（如 `?` 占位符）。若不支持，建议在框架底层升级查询构建器，或严格使用 `(int)`/`$this->db->escape()` 进行防御。
> 💡 **审查局限性提示**：本次审查仅基于提供的单一 Model 文件。未包含 `Simple_model` 基类实现、全局辅助函数（如 `throwError`, `turn_array_key`）及 Controller 层调用上下文。建议结合完整调用链进行集成测试，重点验证并发场景下的券库存扣减与时间边界条件。
---
*此 Issue 由代码审查服务自动创建*... |