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 189 from issue
id
189
repo_id
18
index
123
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告:pc-260519 - 需求 批量更新套餐价格 16243
content
## 自动代码审查报告 **分支**: pc-260519 **提交**: `55de0793ef
## 自动代码审查报告 **分支**: pc-260519 **提交**: `55de0793ef7f1c5520e3c190f96c5d79af71b675` **提交人**: chenjunfeng (developer.jeff.c@gmail.com) **时间**: 2026-05-19 14:03:59 --- ## 1. 审查摘要 - **代码质量评分**:4.5 / 10 分 - **总体评价**:该模型类承载了核心套餐价格设置与查询逻辑,业务覆盖较全,但存在多处严重的安全漏洞(SQL注入)、逻辑缺陷(权限过滤被覆盖)、性能瓶颈(N+1查询)及框架反模式(全局实例化、事务混用)。代码整体可维护性较低,需优先进行安全加固与架构规范化重构。 - **风险等级**:🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `mult_set_room_package_service_charge_rate` & `batch_update` | **SQL注入漏洞**:直接将 `$merchantId`、`$params['goods_type']`、`$shop_id` 等用户可控参数拼接到原生 SQL 中,未做任何转义或参数绑定。 | 废弃原生 SQL 拼接,全面改用 CI 查询构建器(Query Builder)或 `$this->db->query($sql, $bindings)` 参数绑定。 | `$this->db->set('_service_charge_rate', $serviceChargeRate)<br>->where('info._merchant_id', $merchantId)<br>->where('info._status', 1)<br>->update($this->table_name . ' info');` | | 🔴 严重 | `get_package_price_list` 约第 130-145 行 | **权限过滤失效**:`$where['where_in']` 在权限校验时被赋值,随后在门店名称搜索逻辑中被**直接覆盖**。若同时满足两个条件,权限校验将完全失效,导致越权访问。 | 使用 `array_intersect` 合并权限门店ID与搜索门店ID,或构建独立的 `where_in` 数组条件。 | `$finalShopIds = !empty($permissionShopIds) ? array_intersect($permissionShopIds, $filterPermissionIds) : $filterPermissionIds;<br>$where['where_in'] = ['info._shop_id', $finalShopIds];` | | 🔴 严重 | 文件顶部 (1-4行) | **框架反模式/资源浪费**:`$CI = &get_instance();` 在类外部执行,每次 `load->model()` 都会触发一次全局实例化,且 `$CI->load->model('Simple_model');` 属于冗余调用(继承关系已自动加载)。 | 移除文件顶部的全局实例化代码。在模型内部直接使用 `$this->load` 或 `$this->db`。 | `// 删除以下两行<br>$CI = &get_instance();<br>$CI->load->model('Simple_model');` | | 🟠 警告 | `get_package_price_list` & `get_price_set_detail` | **N+1 查询性能瓶颈**:在 `foreach` 循环中逐条调用 `select()` 和 `get_list_for_search()` 获取关联商品与门店数据。分页为 20 时,将额外产生 40+ 次数据库查询。 | 提取所有 `package_id` 后批量查询,使用 `array_column` 在内存中映射组装,将 O(N) 查询降为 O(1)。 | `$packageIds = array_column($list['rows'], 'package_id');<br>$allGoods = $this->ahead_room_package_goods_model->select(['where_in' => ['_package_id', $packageIds]], $goodFields);<br>// 后续通过 $packageId 映射到对应行` | | 🟠 警告 | `set_package_price` 约第 105-115 行 | **事务状态异常风险**:混用 `trans_start()`、手动 `trans_rollback()` 和 `trans_complete()`。在 CI 中,`trans_start()` 内部已开启事务,手动回滚后再次调用 `trans_complete()` 可能抛出 `Transaction already completed` 异常。 | 统一使用 `trans_begin()` / `trans_commit()` / `trans_rollback()` 显式控制,或完全依赖 `trans_start()` + `trans_complete()` 的自动提交/回滚机制。 | `try {<br> $this->db->trans_begin();<br> // ... 业务逻辑 ...<br> $this->db->trans_commit();<br>} catch (Exception $e) {<br> $this->db->trans_rollback();<br> return FALSE;<br>}` | | 🟠 警告 | `set_package_price` 第 15 行 | **无效模型加载**:`$this->load->model('');` 传入空字符串,CI 会忽略或抛出 Warning,属于无效代码。 | 直接删除该行。若需加载其他模型,请明确指定名称。 | `// 删除 $this->load->model('');` | | 🟡 建议 | 全局类定义 | **命名规范不符 PSR-12**:类名 `Ahead_room_package_infos_model` 使用下划线,不符合现代 PHP 规范。 | 建议重构为 `PascalCase`(如 `AheadRoomPackageInfosModel`)。若受历史框架限制,至少保持内部方法命名统一。 | `class AheadRoomPackageInfosModel extends Simple_model` | | 🟡 建议 | `set_package_price` 约第 65-69 行 | **价格校验逻辑低效**:遍历整个 `$baseData` 数组并使用 `strpos` 匹配 `_price`,易误判非价格字段且性能不佳。 | 明确定义价格字段白名单进行校验,提升可读性与执行效率。 | `$priceFields = ['_price', '_actual_price', '_time_cost_price', '_vip_price'];<br>foreach ($priceFields as $field) {<br> if (isset($baseData[$field]) && $baseData[$field] < 0) throwError('...');<br>}` | | 🟡 建议 | `update_with_link` | **隐式依赖与递归调用风险**:`$this->ahead_room_package_infos_model->update_v2()` 依赖外部动态加载的模型实例,且 `update_v2` 未在当前类定义,易导致 `Call to undefined method`。 | 明确调用 `$this->update()` 或 `$this->db->update()`,避免依赖未声明的动态属性。 | `// 替换为<br>$result = $this->db->where(['_shop_id' => $shop_id, '_link_id' => $link_id])<br> ->update($this->table_name, $updateData);` | ## 3. 总结与行动建议 ### 🚨 优先修复的关键问题 1. **立即修复 SQL 注入**:`mult_set_room_package_service_charge_rate` 和 `batch_update` 中的原生 SQL 拼接必须替换为查询构建器或参数绑定。这是最高优先级的安全红线。 2. **修复权限越权漏洞**:`get_package_price_list` 中的 `$where['where_in']` 覆盖问题会导致数据隔离失效,必须使用 `array_intersect` 或独立条件合并逻辑。 3. **规范事务控制**:统一使用 `trans_begin()` / `trans_commit()` / `trans_rollback()` 显式事务流,避免框架自动机制与手动回滚冲突导致的数据不一致。 ### 🛠 后续重构与优化方向 1. **消除 N+1 查询**:将 `get_package_price_list` 和 `get_price_set_detail` 中的循环查询重构为**批量预加载(Eager Loading)**模式。提取主键集合 → 一次性查询关联表 → 内存中 `key => value` 映射组装。预计可提升列表接口响应速度 60% 以上。 2. **清理框架反模式**:移除文件顶部的 `$CI = &get_instance();`,统一在模型内部使用 `$this->load`。检查并删除 `$this->load->model('');` 等无效调用。 3. **输入校验与类型安全**:当前代码大量依赖 `??` 默认值,缺乏对核心参数(如价格、时间、门店ID)的类型与范围校验。建议引入独立的 `Validator` 类或使用 CI 的 `Form_validation` 库在 Controller 层拦截非法请求。 4. **代码规范对齐**:逐步将类名、方法名向 PSR-12 靠拢。若 `phpci` 为内部定制框架,请确认其是否支持自动加载与命名空间,以便后续引入现代 PHP 特性(如类型声明、属性注入等)。 > 💡 **框架适配说明**:代码结构高度符合 **CodeIgniter 3** 规范。若 `phpci` 为基于 CI3 的二次开发框架,上述建议完全适用;若为独立架构,请重点核对事务驱动与查询构建器的底层实现差异。建议查阅 `phpci` 官方文档中关于 `Database Transactions` 与 `Query Builder` 的章节以确认最佳实践。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779170639
updated_unix
1779170639
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel