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 381 from issue
id
381
repo_id
18
index
168
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pc-260616 - 合并
content
## 自动代码审查报告 **分支**: pc-260616 **提交**: `707cdc2a2a
## 自动代码审查报告 **分支**: pc-260616 **提交**: `707cdc2a2a4b53b6b899b8354b822a611e80eb60` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-05-27 16:39:55 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：代码整体实现了计时开房、套餐设置、包断价格等核心业务逻辑，具备基础的事务控制与参数校验意识。但存在多处**变量拼写错误导致的功能失效**、**事务异常捕获掩盖真实错误**、**循环内 N+1 查询性能瓶颈**以及**原生 SQL 拼接安全隐患**。类命名与 PSR-12 规范不符，部分业务逻辑耦合较重，可维护性有待提升。 - **风险等级**：🔴 高（存在数据写入异常、SQL注入风险及严重性能隐患） > 📌 **框架适配说明**：从 `defined('BASEPATH')`、`$this->load->model()`、`$this->db->trans_start()` 等特征判断，当前代码基于 **CodeIgniter 3**（或其深度定制版），而非 `phpci`。以下审查建议将基于 CI3 架构与通用 PHP 最佳实践给出。若为内部定制框架，请结合其底层实现微调。 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `Ahead_room_timing_bd_model.php` / `add_bd_prise_set()`<br>`Ahead_room_timing_detail_model.php` / `add_room_timing_detail()` | **变量名拼写错误**：VIP 价格循环中使用了未定义的 `$param` 而非 `$params`，导致 PHP Notice/Warning，且 VIP 级别价格始终写入 `0`。 | 统一修正为 `$params`，并在开发环境开启 `error_reporting(E_ALL)` 提前拦截。 | `$addData['_bd_vip_level'.$i.'_price'] = $params['bd_vip_level'.$i.'_price'] ?? 0;` | | 🔴 严重 | `RoomTiming.php` / `edit()` | **事务异常捕获掩盖业务错误**：`throwError()` 抛出异常被 `catch` 捕获后，统一返回“操作失败”，丢失具体校验提示；且手动 `trans_rollback()` 后未调用 `trans_complete()` 重置事务状态，可能影响后续请求。 | 移除 `try-catch` 包裹核心逻辑，或捕获后记录日志并重新抛出；交由 CI 的 `trans_complete()` 自动处理提交/回滚。 | 见下方重构示例 | | 🔴 严重 | `Ahead_room_package_infos_model.php` / `mult_set_room_package_service_charge_rate()` | **原生 SQL 拼接存在注入风险**：`$whereStr` 直接拼接 `$merchantId`、`$goods_type` 等参数，未使用查询构建器或参数绑定。 | 改用 CI 查询构建器或 `$this->db->query($sql, $binds)` 进行参数化查询。 | `$this->db->set('_service_charge_rate', $serviceChargeRate)->where('_merchant_id', $merchantId)->update($this->table_name);` | | 🟠 警告 | `Ahead_book_order_model.php` / `get_list()` | **N+1 查询与循环内更新**：在 `foreach` 中频繁调用 `get_one()` 和 `update_book_mobile()`，数据量大时会导致严重性能下降与数据库连接压力。 | 批量收集缺失手机号，统一查询后批量更新；或直接在主查询中 `LEFT JOIN` 用户表。 | 见下方优化思路 | | 🟠 警告 | `RoomTiming.php` / `getList()` | **空数组导致 SQL 语法错误**：`$where['where_in'] = array('_shop_id', $shop_ids_arr);` 若 `$shop_ids_arr` 为空，可能生成 `WHERE _shop_id IN ()` 非法 SQL。 | 增加空数组判断，或改用 CI 原生 `where_in` 安全写法。 | `if (!empty($shop_ids_arr)) { $this->db->where_in('_shop_id', $shop_ids_arr); }` | | 🟠 警告 | `RoomTiming.php` / `edit()` | **价格校验逻辑位置不当**：在组装 `$update_data`/`$add_data` 后遍历校验，失败时抛出异常会中断事务流程，且校验逻辑重复。 | 提取独立校验方法，在 `trans_start()` 之前执行，失败直接返回错误，不进入事务。 | `private function validatePrices(array $data): void { ... }` | | 🟡 建议 | `RoomTiming.php` / 类定义 | **违反 PSR-12 命名规范**：`class roomTiming` 首字母未大写，不符合 PHP 类名 PascalCase 规范。 | 修改为 `class RoomTiming extends PcServer`，并同步更新路由/自动加载配置。 | `class RoomTiming extends PcServer` | | 🟡 建议 | `Ahead_room_package_infos_model.php` / `set_package_price()` | **无效代码残留**：`$this->load->model('');` 参数为空，无实际作用且可能触发底层警告。 | 直接删除该行。 | - | | 🟡 建议 | 全局多处 | **魔法数字硬编码**：`86400`、`24 * 3600`、`2145888000` 等时间常量散落各处，可读性差且易出错。 | 在基类或配置文件中定义常量，如 `const SECONDS_PER_DAY = 86400;`。 | `if ($start_time >= $end_time) { $end_time += self::SECONDS_PER_DAY; }` | ### 🔧 关键代码重构示例 **1. 事务与异常处理优化 (`RoomTiming.php::edit`)** ```php // 优化前：try-catch 掩盖错误，事务状态混乱 // 优化后：前置校验 + CI 自动事务管理 public function edit() { $param = $this->param; $this->load->helper('common'); $this->load->model('Ahead_room_timing_detail_model'); // 1. 前置参数校验（失败直接返回，不进入事务） $this->validateTimingParams($param); $this->db->trans_start(); // 开启事务 // 2. 组装数据 $data = $this->assembleTimingData($param); // 3. 执行 DB 操作 $res = $id ? $this->ahead_room_timing_model->update($data, ['_id' => $id]) : $this->ahead_room_timing_model->insert($data); // 4. CI 自动根据执行状态决定 commit 或 rollback $this->db->trans_complete(); if ($this->db->trans_status() === FALSE || $res === false) { $this->error_response('操作失败'); } $this->success_response('操作成功'); } private function validateTimingParams(array $param): void { // 提取价格校验逻辑，失败直接 throwError 或 error_response foreach ($param as $key => $value) { if (strpos($key, 'price') !== false && strpos($key, 'bd_') === false && $value < 0) { throwError('所有的价格设置不能为负数'); } } } ``` **2. N+1 查询优化 (`Ahead_book_order_model.php::get_list`)** ```php // 优化思路：批量获取缺失手机号，避免循环内查询与更新 public function get_list($where, $page = '', $page_size = '') { // ... 原有查询逻辑 ... $order_info = $this->select($where, $fields, '_use_status ASC,_arrival_time DESC', $page, $page_size); // 收集需要补全手机号的订单 $missingMobileOrders = []; foreach ($order_info as $v) { if (empty($v['book_mobile'])) { $missingMobileOrders[$v['id']] = $v['ahead_user_id']; } } if (!empty($missingMobileOrders)) { // 批量查询用户信息 $userIds = array_unique($missingMobileOrders); $users = $this->ahead_user_model->select(['where_in' => ['_id', $userIds]], '_id,_mobile'); $userMap = array_column($users, '_mobile', '_id'); // 批量更新预订单手机号（使用 insert_batch 或 update_batch 替代循环） $updateBatch = []; foreach ($missingMobileOrders as $orderId => $userId) { if (isset($userMap[$userId])) { $updateBatch[] = ['_id' => $orderId, '_book_mobile' => $userMap[$userId]]; } } if (!empty($updateBatch)) { $this->update_batch($updateBatch, '_id'); // 假设 Simple_model 支持 } } // ... 后续处理 ... } ``` ## 3. 总结与行动建议 ### 🚨 优先修复项（P0） 1. **修正变量拼写错误**：全局搜索 `$param` 与 `$params` 混用场景，重点修复 `Ahead_room_timing_bd_model` 与 `Ahead_room_timing_detail_model` 中的 VIP 价格写入逻辑。 2. **修复事务异常吞没问题**：移除 `RoomTiming::edit()` 中包裹业务逻辑的 `try-catch`，或确保 `catch` 中记录完整异常堆栈并正确重置事务状态。 3. **消除 SQL 注入隐患**：将 `Ahead_room_package_infos_model::mult_set_room_package_service_charge_rate()` 中的原生 SQL 替换为 CI 查询构建器或参数绑定查询。 ### 🛠 后续重构方向 1. **统一数据校验层**：将散落在 Controller/Model 中的 `if (empty(...)) throwError(...)` 抽取为独立的 `Validator` 类或使用 CI 的 `Form_validation` 库，实现校验规则与业务逻辑解耦。 2. **性能架构优化**： - 列表查询严禁在 `foreach` 中执行单条 `SELECT/UPDATE`，统一改为 `JOIN` 或 `批量查询+批量更新`。 - 引入缓存机制（如 Redis）存储门店、包厢类型、VIP等级等高频只读字典数据，减少重复 DB 查询。 3. **规范与可维护性**： - 严格遵循 PSR-12：类名 PascalCase，方法名 camelCase，常量 UPPER_SNAKE_CASE。 - 替换魔法数字为语义化常量（如 `DAY_SECONDS`、`MAX_TIMESTAMP`）。 - 补充关键方法的 PHPDoc 注释，明确参数类型、返回值及异常抛出条件。 > 💡 **提示**：若当前项目为遗留系统，建议采用**渐进式重构**策略。优先修复 🔴 严重问题并补充单元测试，再逐步对 Model 层进行查询优化与规范对齐。如需针对 `Simple_model` 底层封装逻辑进行深度适配审查，可提供该基类代码以便进一步分析。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779871195
updated_unix
1779871195
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel