SQLite Web: gitea.db

id

Primary key. INTEGER NOT NULL

repo_id

INTEGER

index

INTEGER

poster_id

INTEGER

original_author

TEXT

original_author_id

INTEGER

name

TEXT

content

## 自动代码审查报告

**分支**: pay-260616
**提交**: `f32131c65de891d3fc755a6e9511b716d7eec12d`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-01 18:07:49

---

## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了较为完整的小程序首页、包厢预订、订单支付等核心业务链路，但存在明显的架构耦合、安全漏洞与逻辑隐患。大量硬编码、全局实例属性污染、未严格校验的输入参数以及原始 SQL 拼接降低了系统的可维护性与安全性。整体处于“能跑但脆弱”的状态，亟需安全加固与规范化重构。
- **风险等级**：🔴 高

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `Ahead_yc_order_model.php` `get_bill_goods_info` 方法内 | **SQL 注入漏洞**：使用字符串拼接构造查询条件 `$sql = '_unique_key="' . $unique_key . '" ...'`，若 `$unique_key` 未严格过滤，将导致数据库被恶意查询或篡改。 | 废弃手动拼接 SQL，全面改用 CI Query Builder 或预处理语句。 | `$this->db->where('_unique_key', $unique_key)->where_in('_status', [1,4])->get($this->table_name)->result_array();` |
| 🔴 严重 | `Index.php` `getRoomInfo` 方法 | **逻辑分支未终止**：`if (empty($open_room_log))` 块内调用 `$this->success_response()` 后未 `return` 或 `exit`。若响应函数未中断脚本，后续代码将尝试访问空数组键值导致 `Fatal Error`。 | 在条件分支末尾显式添加 `return;`，确保控制流正确终止。 | `if (empty($open_room_log)) { ... $this->success_response(); return; }` |
| 🔴 严重 | `Index.php` `__construct` 方法 | **横向越权风险 (IDOR)**：直接信任客户端传入的 `$this->param['spe_merchant_id']` 并赋值给核心业务变量，未校验当前用户是否有权访问该商家数据。 | 增加权限校验中间件或模型层校验，确保 `spe_merchant_id` 与当前登录用户/会话绑定。 | `if (!$this->auth->check_merchant_access($this->spe_merchant_id)) { throwError('无权访问该商家'); }` |
| 🟠 警告 | `Index.php` `getConfig` 方法 | **重复加载模型**：构造函数已加载 `ahead_merchant_wx_min_set_model`，方法内再次 `$this->load->model()` 造成冗余调用（虽 CI 会缓存，但违背规范）。 | 移除方法内的重复加载，直接使用构造函数已加载的实例。 | 删除 `$this->load->model('Ahead_merchant_wx_min_set_model');` |
| 🟠 警告 | `Ahead_billiards_model.php` `check_room` 方法 | **全局实例属性污染**：直接修改 `$CI->merchant_id`、`$CI->operational_scene` 等全局属性，破坏封装性，易引发并发请求下的状态串扰。 | 将状态数据封装至 Model 自身属性或通过方法参数/返回值传递，避免污染 CI 超全局实例。 | `private $merchant_id; private $operational_scene; // 使用 $this->merchant_id 替代 $CI->xxx` |
| 🟠 警告 | `Ahead_shop_book_time_info_model.php` 多处静态属性 | **静态缓存生命周期风险**：大量使用 `self::$book_days_info`、`self::$shop_data` 等静态变量缓存数据。在 PHP-FPM 长连接或 CLI 模式下可能导致跨请求脏数据，且难以清理。 | 改用 CI 内置缓存组件 `$this->cache->save()` 或实例属性，并在请求结束时重置。 | `if (empty($this->cache->get('book_days_'.$merchant_id))) { ... $this->cache->save(...); }` |
| 🟡 建议 | 全局多处 | **魔法数字/硬编码泛滥**：大量使用 `'-1'`、`'1'`、`'2'`、`33424` 等字面量，业务含义不透明，后期维护成本极高。 | 提取为类常量或独立配置类，统一命名管理。 | `class OrderStatus { const PENDING = -1; const PAID = 1; }` |
| 🟡 建议 | `Index.php` / `Ahead_billiards_model.php` | **命名规范不一致**：方法名混用驼峰 (`getRoomInfo`)、下划线 (`get_room_info`) 及自定义前缀 (`public_getIndex`)，不符合 PSR-12。 | 统一采用驼峰命名法，路由前缀建议通过路由配置文件处理，而非硬编码在方法名中。 | `public function getIndex()`, `public function getRoomInfo()` |
| 🟡 建议 | 全局 | **框架适配说明**：代码特征（`get_instance()`, `$this->load->model()`, `BASEPATH`）高度符合 **CodeIgniter 3** 架构。若 `phpci` 为内部定制框架或笔误，请确认其生命周期与 CI3 是否一致。 | 建议查阅 `phpci` 官方文档确认模型加载、路由解析及请求生命周期差异，必要时适配对应框架的依赖注入容器。 | 无 |

## 3. 总结与行动建议

### 🔑 优先修复的关键问题
1. **立即修复 SQL 注入**：替换 `Ahead_yc_order_model.php` 中所有手动拼接的 `$sql` 字符串，全面启用 CI Query Builder (`$this->db->where()`, `$this->db->select()`) 或 PDO 预处理。
2. **阻断逻辑穿透**：在 `Index.php::getRoomInfo` 的空结果判断块末尾补充 `return;`，防止空指针异常。
3. **实施商家权限校验**：在控制器层或中间件层增加 `spe_merchant_id` 的归属权校验，严禁直接信任前端传入的商户标识。

### 🛠 后续重构与优化方向
1. **解耦全局状态**：将 `Ahead_billiards_model` 中对 `$CI` 实例属性的直接读写改为依赖注入或方法参数传递。Model 应保持无状态或仅维护自身实例属性，提升单元测试可行性。
2. **统一配置与常量管理**：建立 `config/constants.php` 或独立 `Enum` 类，集中管理订单状态、支付场景、运营场景等魔法值。例如：`const SCENE_KTV = '1'; const SCENE_BILLIARDS = '2';`。
3. **规范模型加载与缓存策略**：
   - 将频繁使用的 Model 移至控制器 `__construct` 中加载。
   - 替换 `self::$` 静态缓存为 CI 原生缓存驱动（Redis/Memcached/File），并设置合理的 TTL，避免内存泄漏或脏读。
4. **强化输入验证层**：在控制器入口统一使用 CI 的 `form_validation` 或自定义 `RequestValidator` 对 `$this->param` 进行类型转换、范围校验与 XSS 过滤，避免业务代码中散落 `intval()`、`empty()` 判断。
5. **框架适配确认**：若项目确为 `phpci` 框架，请核对其是否兼容 CI3 的 `load` 机制。若为自研框架，建议逐步迁移至现代 PHP 架构（如 PSR-4 自动加载、依赖注入容器、中间件管道），以提升长期可维护性。

> 💡 **提示**：本次审查基于提供的代码片段。若涉及核心支付、用户鉴权或高并发预订场景，建议补充单元测试覆盖率，并引入静态分析工具（如 PHPStan / Psalm）进行自动化质量门禁。

---
*此 Issue 由代码审查服务自动创建*

TEXT

milestone_id

INTEGER

priority

INTEGER

is_closed

INTEGER

is_pull

INTEGER

num_comments

INTEGER

ref

TEXT

deadline_unix

INTEGER

created_unix

INTEGER

updated_unix

INTEGER

closed_unix

INTEGER

is_locked

INTEGER NOT NULL (default 0

content_version

INTEGER NOT NULL (default 0

time_estimate

INTEGER NOT NULL (default 0

Update row 437 in issue