sqlite-web 0.7.2
gitea.db
issue
Create
Query
access
access_token
action
action_artifact
action_run
action_run_index
action_run_job
action_runner
action_runner_token
action_schedule
action_schedule_spec
action_task
action_task_output
action_task_step
action_tasks_version
action_variable
app_state
attachment
auth_token
badge
branch
collaboration
comment
commit_status
commit_status_index
commit_status_summary
commit_sync_log
commit_sync_status
dbfs_data
dbfs_meta
deploy_key
email_address
email_hash
external_login_user
follow
gpg_key
gpg_key_import
hook_task
issue
issue_assignees
issue_content_history
issue_dependency
issue_index
issue_label
issue_pin
issue_user
issue_watch
label
language_stat
lfs_lock
lfs_meta_object
login_source
milestone
mirror
notice
notification
oauth2_application
oauth2_authorization_code
oauth2_grant
org_user
package
package_blob
package_blob_upload
package_cleanup_rule
package_file
package_property
package_version
project
project_board
project_issue
protected_branch
protected_tag
public_key
pull_auto_merge
pull_request
push_mirror
reaction
release
renamed_branch
repo_archiver
repo_hidden_file
repo_indexer_status
repo_license
repo_redirect
repo_topic
repo_transfer
repo_unit
repository
review
review_state
secret
session
sqlite_sequence
star
stopwatch
system_setting
task
team
team_invite
team_repo
team_unit
team_user
topic
tracked_time
two_factor
upload
user
user_badge
user_blocking
user_open_id
user_redirect
user_setting
version
watch
webauthn_credential
webhook
Toggle helper tables
Structure
Content
Query
Insert
Drop
Import
Export
Delete row 163 from issue
id
163
repo_id
18
index
113
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pc - Merge pull request '合并0519最新'
🔍 代码审查报告：pc - Merge pull request '合并0519最新' (#111) from pc-26051
...
content
## 自动代码审查报告 **分支**: pc **提交**: `cf63e3da1114aa4af
## 自动代码审查报告 **分支**: pc **提交**: `cf63e3da1114aa4af755cbe9dc0f5263d88e0075` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-05-19 09:42:07 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：代码实现了预订单列表、详情与退款的基础业务逻辑，但存在**致命 SQL 语法错误**、**循环内数据库操作（N+1 查询）**、**资金操作缺乏事务保护**等严重问题。前端入口文件存在硬编码测试地址与 Axios 配置隐患。整体架构偏向“过程式”堆砌，未充分利用框架生命周期与面向对象设计原则，需进行结构性优化。 - **风险等级**：🔴 高（涉及资金退款、数据一致性、SQL 报错阻断） --- ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `Ahead_book_order_model.php` ~L28 | `get_list` 的 `$fields` 中使用了表别名 `e` (`e._bill_no`, `e._status`)，但 `$where['join']` 中未定义 `e` 表的关联条件，执行时将直接抛出 SQL 语法错误。 | 补充 `e` 表的 `JOIN` 条件，或移除未关联的字段。若 `e` 表为订单主表，需确认关联键。 | `$where['join'][] = ['your_order_table e', 'a._id=e._book_order_id', 'left'];` | | 🔴 严重 | `Ahead_book_order_model.php` ~L48 | `get_list` 循环内执行 `ahead_user_model->get_one()` 与 `update_book_mobile()`。分页数据量大时将引发 **N+1 查询风暴**，且更新操作无事务保护，极易导致数据库连接耗尽与数据不一致。 | 改为**批量查询**获取手机号，更新操作移至循环外或使用事务包裹。避免在查询方法中执行写操作。 | 见下方 `3. 总结与行动建议` 中的重构示例 | | 🟠 警告 | `Ahead_book_order_model.php` ~L108 | `get_detail` 中 `$refund_admin` 仅在 `if (!empty($order_refund['_admin_id']))` 内赋值，但在后续 `foreach` 中直接使用。若条件不满足，PHP 8+ 将抛出 `Undefined variable` 致命错误。 | 在循环前初始化变量，或使用空合并运算符安全访问。 | `$refund_admin = $refund_admin ?? [];`<br>`$v['operator'] = $refund_admin['_name'] ?? '';` | | 🟠 警告 | `Ahead_book_order_model.php` ~L1-L3 | 文件顶部直接使用 `$CI = &get_instance();` 加载模型。该代码会在**每次 HTTP 请求时执行**（即使未实例化该模型），严重浪费资源且违反框架生命周期规范。 | 移除顶部代码，将依赖加载移至 `__construct()` 中，或交由框架自动加载机制处理。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` | | 🟠 警告 | `Ahead_book_order_model.php` ~L138 | `refund` 方法直接调用全局函数 `bookOrderRefund()`，未使用数据库事务。若退款接口调用成功但本地状态更新失败，将导致**资金与订单状态不一致**。 | 使用框架事务机制包裹核心逻辑，并增加 `try-catch` 异常捕获。 | `$this->db->trans_start();`<br>`// 业务逻辑`<br>`$this->db->trans_complete();` | | 🟡 建议 | `Ahead_book_order_model.php` ~L15 | `$book_status` 键为字符串 `'-1'`，而数据库返回的 `status` 通常为整型。PHP 弱类型虽可隐式匹配，但易引发类型混淆与静态分析警告。 | 统一使用整型键，或在映射时显式转换类型。 | `public $book_status = [-1 => '已作废', 1 => '未使用', 2 => '已使用'];` | | 🟡 建议 | `main.js` ~L10 | 硬编码测试环境 URL `https://test-pc.g-hi.com/pc-260331/`，未做环境隔离。发布生产环境时极易导致请求错乱或敏感测试接口暴露。 | 使用构建工具环境变量（如 `process.env.VUE_APP_BASE_URL`）动态注入。 | `Vue.ctUrl = process.env.VUE_APP_BASE_URL || window.location.origin;` | | 🟡 建议 | `main.js` ~L68 | `axios.defaults.transformRequest` 强制 `JSON.stringify(data)`，未判断数据类型。若组件提交 `FormData`（如文件上传），序列化将导致请求失败。 | 增加类型守卫，仅对普通对象进行序列化。 | `if (data instanceof FormData) return data; return JSON.stringify(data);` | --- ## 3. 总结与行动建议 ### 🚨 优先修复的关键问题 1. **修复 SQL 报错**：立即补充 `get_list` 中缺失的 `e` 表 `JOIN` 条件，否则该接口将直接 500 报错。 2. **消除 N+1 查询与循环写操作**：`get_list` 中的手机号补全逻辑必须改为批量查询。模型层原则上**不应包含写操作**，建议将 `update_book_mobile` 移至 Service 层或 Controller 层，并配合定时任务/异步队列处理数据清洗。 3. **资金操作事务化**：`refund` 与 `update_book_mobile` 涉及多表更新与外部退款接口调用，必须使用 `$this->db->trans_start()` / `$this->db->trans_complete()` 确保原子性，失败时自动回滚。 4. **清理全局反模式**：移除模型文件顶部的 `$CI = &get_instance();`，改用构造函数加载依赖，符合框架生命周期。 ### 🛠 后续重构与优化方向 #### 🔹 PHP 后端重构示例（`get_list` 性能优化） ```php public function get_list($where, $page = '', $page_size = '') { // 1. 补充缺失的 JOIN (假设 e 表为订单主表) $where['join'][] = ['your_order_table e', 'a._id=e._book_order_id', 'left']; // ... 原有查询逻辑 ... $order_info = $this->select($where, $fields, '_use_status ASC,_arrival_time DESC', $page, $page_size); if (empty($order_info)) return []; // 2. 批量获取缺失手机号的用户 (避免循环查库) $missing_user_ids = []; foreach ($order_info as $v) { if (empty($v['book_mobile'])) { $missing_user_ids[] = $v['ahead_user_id']; } } $user_map = []; if (!empty($missing_user_ids)) { // 假设 Simple_model 支持 IN 查询或需自行拼接 $users = $this->db->where_in('_id', array_unique($missing_user_ids)) ->get('ahead_user') ->result_array(); foreach ($users as $u) { $user_map[$u['_id']] = $u['_mobile']; } } // 3. 数据映射与格式化 $pay_scene = ['5'=>'微信预订', '6'=>'会员app预订', '7'=>'员工预订', '8'=>'嗨赞app预订', '9'=>'嗨赞小程序预订', '10'=>'抖音小程序预订']; foreach ($order_info as &$v) { $v['use_status'] = $this->book_status[(int)$v['status']] ?? ''; $v['pay_scene'] = $pay_scene[(string)$v['pay_scene']] ?? ''; if ($v['refund_amount'] > 0) { $v['actual_pay'] = number_format((float)$v['actual_pay'] - (float)$v['refund_amount'], 2, '.', ''); } // 仅做数据填充，不在此处执行 UPDATE if (empty($v['book_mobile']) && isset($user_map[$v['ahead_user_id']])) { $v['book_mobile'] = $user_map[$v['ahead_user_id']]; } } unset($v); return $order_info; } ``` #### 🔹 架构与规范建议 - **模型职责单一化**：Model 仅负责数据读写，业务逻辑（如退款状态流转、手机号同步策略）应抽离至 `Service` 层。 - **统一异常处理**：避免使用全局 `throwError()`，建议抛出标准 `Exception` 或在 Controller 层统一捕获并返回 JSON 格式错误码。 - **前端配置解耦**：`main.js` 中的 `Vue.request_header` 全局状态易被意外篡改。建议改用 Axios 拦截器动态注入 Header，并移除全局 `Vue.timeoutfun` 的路由硬跳转逻辑。 - **框架适配说明**：当前代码结构高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请确认其是否支持 PSR-4 自动加载与依赖注入。若支持，建议逐步迁移至现代 PHP 标准（PHP 8.1+、类型声明、构造函数属性提升）。 > 💡 **注**：若需对 `Simple_model` 的底层实现、`bookOrderRefund` 全局函数或数据库表结构进行深度审查，请提供相关代码片段，以便进一步评估事务边界与数据一致性策略。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779154927
updated_unix
1779154927
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel