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-260519
**提交**: `eb0f6c752ec531d38f10fc74a1315f68591405a9`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-27 16:10:04

---

## 1. 审查摘要
- **代码质量评分**：5.5 / 10 分
- **总体评价**：代码实现了复杂的业务逻辑（时段预订计算、评价发券、触摸屏设备交互），但存在明显的架构与安全缺陷。模型层承担了过多服务层与控制器职责，存在 SQL 注入隐患、`goto` 滥用、异常静默吞没、异步状态不同步等问题。整体耦合度高，可维护性与扩展性较弱。
- **风险等级**：🔴 高

## 2. 问题详情

| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `ScreenApi.php` / `case "2001"` | 使用 `goto famail_close;` 破坏控制流，且标签拼写为 `famail`（应为 `family`），极易引发逻辑跳跃与维护灾难。 | 提取关房后置逻辑为独立私有方法，使用条件分支或 `return` 替代 `goto`。 | `if ($family_data['_status'] == 1) { return $this->_handle_auto_close($family_data, $open_room_data); }` |
| 🔴 严重 | `Ahead_shop_comment_model.php` / `checkWriteable()` | `$this->db->where($openLogWhereStr)` 直接拼接外部参数 `$uniqueKey` 与 `$uid`，存在 SQL 注入风险。 | 使用 CI3 查询构造器或参数绑定，避免原生字符串拼接。 | `$this->db->where('_unique_key', $uniqueKey)->where("FIND_IN_SET($uid, _join_customer) > 0");` |
| 🔴 严重 | `ScreenApi.php` / `case "2001"` | `$upOpenLogStr` 直接拼接 `$cost` 到 SQL 更新语句中，违反安全编码规范，且易因类型转换引发语法错误。 | 使用 Query Builder 的 `set()` 方法，并强制类型转换。 | `$this->db->set('_unpaid_amount', '_unpaid_amount + ' . floatval($cost), FALSE)->update('table', $where);` |
| 🔴 严重 | `ScreenApi.php` / 文件头部 | `header("Access-Control-Allow-Origin:*");` 允许任意域名跨域，易被恶意站点利用发起 CSRF 或数据窃取。 | 根据业务需求配置可信域名白名单，或动态读取 `HTTP_ORIGIN` 校验后返回。 | `header("Access-Control-Allow-Origin: " . ($allowed_origin ?: 'https://yourdomain.com'));` |
| 🟠 警告 | `Ahead_shop_book_time_info_model.php` / `get_book_day_time_info()` | `array_intersect(...array_values($all_room_book_time))` 当数组元素不足 2 个时，PHP 8+ 会抛出 `ArgumentCountError` 致命错误。 | 增加数量校验，或使用兼容写法。 | `if (count($all_room_book_time) > 1) { $un_book_time = array_intersect(...array_values($all_room_book_time)); }` |
| 🟠 警告 | `Ahead_shop_comment_model.php` / `add_shop_comment()` | `catch (Exception $e)` 仅返回“评价失败”，未记录异常堆栈，导致线上故障无法定位。 | 捕获后写入日志，并返回脱敏提示。 | `log_message('error', '评价提交异常: ' . $e->getMessage()); return ['success'=>false, 'msg'=>'系统繁忙，请稍后重试'];` |
| 🟠 警告 | `service-receipt.js` / `addService()` | 图片上传为异步操作，但提交时直接读取 `this.data.imageList`，未等待上传完成，可能导致提交空数组或部分失败数据。 | 使用 `Promise.all` 聚合上传任务，全部成功后再调用提交接口。 | `Promise.all(uploadTasks).then(urls => this.submitComment(urls)).catch(err => wx.showToast({title:'上传失败'}))` |
| 🟠 警告 | `Ahead_shop_book_time_info_model.php` / 多处 | 频繁在方法内部调用 `$CI = &get_instance()`，且部分逻辑重复加载 Model。 | 在构造函数中统一获取并赋值给实例属性，后续直接复用。 | `protected $ci; public function __construct(){ parent::__construct(); $this->ci =& get_instance(); }` |
| 🟡 建议 | `ScreenApi.php` / 全局 | 控制器过于臃肿（God Object），`switch` 包含数百行业务逻辑，违反单一职责与开闭原则。 | 按功能域拆分为独立 Service 类，控制器仅负责路由分发、参数校验与响应格式化。 | 引入策略模式：`$handler = ScreenHandlerFactory::create($function); $handler->handle($params);` |
| 🟡 建议 | `Ahead_shop_book_time_info_model.php` / 多处 | 大量使用 `self::$xxx` 静态变量缓存数据。在长连接/CLI 环境下易导致内存泄漏或请求间数据污染。 | 改用实例属性缓存，或接入 Redis 等外部缓存并设置 TTL。 | `private $requestCache = [];` 替代 `public static $xxx = [];` |
| 🟡 建议 | 全局 | 代码风格未严格遵循 PSR-12（缩进不一致、大括号位置混用、命名规范不统一）。 | 配置 `PHP-CS-Fixer` 或 `PHP_CodeSniffer` 进行自动化格式化与静态检查。 | `php-cs-fixer fix . --rules=@PSR12` |

## 3. 总结与行动建议

### 🔑 优先修复的关键问题
1. **安全漏洞修复**：立即替换 `checkWriteable()` 与关房逻辑中的原生 SQL 拼接，全面改用 CI3 Query Builder 或预处理语句；限制 CORS 白名单。
2. **控制流重构**：彻底移除 `goto` 语句，将关房后的状态更新、消息推送、任务生成等逻辑封装为独立方法，通过 `return` 或条件判断流转。
3. **致命错误防御**：修复 `array_intersect` 空数组展开问题，增加 `json_decode` 后的类型校验（如 `is_array()`），避免线上 500 错误。
4. **前端异步治理**：小程序端 `addService` 需引入 `Promise.all` 等待图片上传完成，并增加 loading 状态与失败重试机制。

### 🛠 后续重构与优化方向
1. **架构分层解耦**：当前 Model 层严重越权（承担了 Service、Helper、甚至部分 Controller 职责）。建议引入 `Service` 层处理复杂业务（如时段计算、发券逻辑、微信推送），Model 仅保留数据存取与基础查询。
2. **控制器瘦身**：`ScreenApi.php` 建议采用 **命令模式（Command Pattern）** 或 **策略模式（Strategy Pattern）** 替代巨型 `switch`。每个 `function` 对应一个独立 Handler 类，便于单元测试与后续扩展。
3. **缓存策略升级**：静态变量缓存改为请求级缓存（实例属性）或 Redis 分布式缓存。对于高频查询的门店配置、包厢状态，建议增加缓存失效机制（如发布订阅或版本号控制）。
4. **框架适配说明**：
   > ⚠️ 注：提交代码实际基于 **CodeIgniter 3** 架构（特征：`get_instance()`、`load->model()`、`BASEPATH`、`$this->db`）。若 `phpci` 为内部定制框架，请对照其官方生命周期文档调整组件加载方式。CI3 已停止官方维护，若项目处于长期迭代期，建议评估平滑迁移至 CI4 或现代 PHP 框架（如 Laravel/Symfony）的可行性，以获得更好的类型安全、依赖注入与中间件支持。
5. **质量保障**：引入 PHPStan 或 Psalm 进行静态类型分析；为核心时段计算逻辑补充 PHPUnit 单元测试，覆盖边界条件（如跨天、24小时营业、最低时长限制、并发预订冲突）。

---
*此 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 366 in issue