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
**提交**: `8ea0707645d983022f654ab4f72675f90830a302`
**提交人**: linyangrui (yangruilin888@gmail.com)
**时间**: 2026-05-19 15:04:41

---

## 📋 审查摘要
- **变更文件数**: 1
- **严重问题**: 0
- **高危问题**: 4
- **中危问题**: 3
- **建议优化**: 2

## 🐛 发现的问题

### [跨文件调用] 方法名拼写错误可能导致调用失败
- **严重程度**: 高危
- **文件**: web/Hi-Zan/Hi-Zan/pages/community-reserve/shop-detail/shop-detail.js
- **行号**: 约 738 行 (`openPriceRulePop` 方法内)
- **问题描述**: 调用了 `reserveModel.getRoomPackgeTimePriceInfo`，其中 `Packge` 明显为 `Package` 的拼写错误。若 `ReserveModel` 中实际定义的方法名为 `getRoomPackageTimePriceInfo`，运行时将直接抛出 `TypeError: reserveModel.getRoomPackgeTimePriceInfo is not a function`，导致价格规则弹窗无法打开。
- **修复建议**: 核对模型定义，修正拼写：`reserveModel.getRoomPackageTimePriceInfo(...)`

### [跨文件调用] 引用的前端模型/工具类文件未在提供的项目结构中定义
- **严重程度**: 高危
- **文件**: web/Hi-Zan/Hi-Zan/pages/community-reserve/shop-detail/shop-detail.js
- **行号**: 第 2-8 行
- **问题描述**: 代码顶部通过 `import` 引用了 `config.js`, `util.js`, `reserve.js`, `imageClickHandler.js`, `user.js`, `reward.js`, `room.js`。但提供的「项目结构」仅包含 PHP CodeIgniter 系统文件，完全缺失前端 JS 目录。若这些文件实际不存在、路径错误或未正确导出对应类/方法，页面加载时将直接白屏崩溃。
- **修复建议**: 确认前端工程目录结构，确保 `../../../models/` 和 `../../../utils/` 路径下存在对应文件且导出名称一致。建议后续审查提供完整的前端文件树以便进行静态依赖分析。

### [逻辑 BUG] 动态方法调用未传递事件对象导致运行时崩溃
- **严重程度**: 高危
- **文件**: web/Hi-Zan/Hi-Zan/pages/community-reserve/shop-detail/shop-detail.js
- **行号**: 约 668 行 (`onUserInfoUpdated` 方法)
- **问题描述**: `this[doing]()` 动态调用了 `handleBookRoomClick` 或 `handlePackageClick`。这两个方法内部强依赖事件参数 `e`（例如 `e.currentTarget.dataset.room`）。在无参调用时 `e` 为 `undefined`，执行到 `e.currentTarget` 时将抛出 `Cannot read properties of undefined` 错误，阻断后续业务。
- **修复建议**: 将核心业务逻辑抽离为独立函数（如 `executeBookFlow(room, packageItem)`），或在调用时构造符合预期的模拟事件对象：
  ```javascript
  if (doing === 'handleBookRoomClick') {
    this.handleBookRoomClick({ currentTarget: { dataset: { room: this.data.room } } });
  } else if (doing === 'handlePackageClick') {
    this.handlePackageClick({ currentTarget: { dataset: { room: this.data.room, package: this.data.packageItem } } });
  }
  ```

### [逻辑 BUG] 跨方法调用时状态数据未同步导致支付参数错误
- **严重程度**: 高危
- **文件**: web/Hi-Zan/Hi-Zan/pages/community-reserve/shop-detail/shop-detail.js
- **行号**: 约 715 行 (`getRoomNotCleanTime` 方法)
- **问题描述**: 该方法调用 `this.openRoomCheckPackageTime(this.data.room_id, null)`。但在 `openRoomCheckPackageTime` 内部，跳转支付页的 URL 拼接使用了 `this.data.room.room_id` 和 `this.data.packageItem.id`。由于 `room` 和 `packageItem` 仅在 `handlePackageClick` 中被赋值，此处调用时它们仍为初始空对象 `{}`，导致传入支付页的 `room_id` 和 `package_id` 为 `undefined`，引发下游接口报错。
- **修复建议**: 修改 `openRoomCheckPackageTime` 方法签名，直接接收 `room_id` 和 `package_id` 作为参数，并在方法内部使用传入参数拼接 URL，避免强依赖 `this.data` 的临时状态。

### [代码质量] 直接修改 this.data 违反小程序最佳实践
- **严重程度**: 中危
- **文件**: web/Hi-Zan/Hi-Zan/pages/community-reserve/shop-detail/shop-detail.js
- **行号**: 约 498 行 (`toggleRuleInfo` 方法)
- **问题描述**: `this.data.coupons[index].openRule = !this.data.coupons[index].openRule` 直接修改了 `data` 属性。微信小程序框架中，直接修改 `this.data` 不会触发视图层更新，且可能导致数据流混乱或后续 `setData` 覆盖失效。
- **修复建议**: 使用 `setData` 的路径语法进行精准更新：
  ```javascript
  this.setData({
    [`coupons[${index}].openRule`]: !this.data.coupons[index].openRule
  })
  ```

### [逻辑 BUG] Promise 链缺少 catch 处理可能导致未捕获异常与 Loading 常驻
- **严重程度**: 中危
- **文件**: web/Hi-Zan/Hi-Zan/pages/community-reserve/shop-detail/shop-detail.js
- **行号**: 约 385 行 (`toExchangeCoupon`) & 约 465 行 (`exchange`)
- **问题描述**: `this.getTuanGouBookMethod(...).then(...)` 未提供 `.catch()` 回调。若底层模型请求失败触发 `reject`，将抛出 `Unhandled Promise Rejection`。同时，`wx.showLoading()` 仅在 `.then()` 中调用 `wx.hideLoading()`，异常路径下 Loading 遮罩将无法关闭，导致页面假死。
- **修复建议**: 补充完整的错误处理链：
  ```javascript
  .then((methodResult) => { /* ... */ })
  .catch((err) => {
    wx.hideLoading();
    wx.showToast({ title: '网络异常，请重试', icon: 'none' });
    console.error('getTuanGouBookMethod failed:', err);
  })
  ```

### [逻辑 BUG] onUnload 中直接修改上一页 options 不可靠
- **严重程度**: 中危
- **文件**: web/Hi-Zan/Hi-Zan/pages/community-reserve/shop-detail/shop-detail.js
- **行号**: 约 795 行 (`onUnload` 方法)
- **问题描述**: `prevPage.options.needRefresh = 'true';` 试图通过修改页面栈中上一页的 `options` 对象来传递刷新标记。微信小程序的 `options` 是页面加载时的只读快照，直接修改通常不会触发上一页的 `onShow` 或数据响应式更新，导致返回上一页时刷新逻辑失效。
- **修复建议**: 推荐通过调用上一页的自定义方法或使用全局状态管理：
  ```javascript
  if (prevPage && typeof prevPage.onShow === 'function') {
    prevPage.onShow(); // 或 prevPage.refreshData?.()
  }
  // 或使用 wx.setStorageSync('needRefreshShopList', true) 配合上一页 onShow 检查
  ```

## ✅ 代码亮点
1. **状态管理清晰**：页面 `data` 字段划分明确，注释详细，便于后续维护。
2. **防抖/节流意识**：`onPageScroll` 中使用了局部变量 `updateData` 收集变更，仅在数据真正变化时调用 `setData`，有效减少了渲染开销。
3. **Promise 封装合理**：将回调风格的模型方法封装为 `Promise`（如 `getTuangouCouponInfoPromise`），便于使用 `Promise.all` 进行并行请求，提升了首屏加载性能。
4. **权限处理规范**：`getLocation` 中完整处理了 `wx.getSetting`、`wx.authorize` 及失败引导，符合微信小程序合规要求。

## 📝 总体建议
1. **统一模型方法命名规范**：代码中存在 `Packge` 等拼写错误，建议在团队内引入 ESLint + TypeScript 或 JSDoc 类型检查，在编译期拦截此类拼写错误。
2. **解耦 UI 与业务逻辑**：`this[doing]()` 动态调用及 `e.currentTarget.dataset` 强依赖暴露了 UI 事件与业务逻辑耦合过深的问题。建议将“选择包厢/套餐 -> 校验 -> 跳转”的核心流程抽离为纯函数，仅接收业务参数，提升可测试性与复用性。
3. **完善异常边界处理**：当前大量模型回调仅处理 `success`，未处理网络超时、服务端返回非 200 状态码等情况。建议封装统一的请求拦截器，或在关键路径补充 `fail` 回调与用户提示。
4. **URL 参数安全编码**：多处 `wx.navigateTo` 使用字符串拼接传递参数（如 `shop_id`, `reward_id`）。若参数值包含特殊字符（如 `&`, `=`, `+`），可能导致路由解析错误。建议统一使用 `encodeURIComponent()` 包裹参数值。

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