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 391 from issue
id
391
repo_id
18
index
169
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pc-260519 - 111
content
## 自动代码审查报告 **分支**: pc-260519 **提交**: `fde349cbf4
## 自动代码审查报告 **分支**: pc-260519 **提交**: `fde349cbf4c2b674b76e8f58bc3005ad22e1e4a5` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-05-28 10:50:22 --- ## 1. 审查摘要 - **代码质量评分**：5/10 分 - **总体评价**：该 Model 承载了大量订单查询、统计、格式化与导出逻辑，业务覆盖较全。但存在明显的架构反模式：SQL 拼接未做参数化、全局作用域执行框架实例、循环内重复加载模型、方法严重超长且职责混杂。整体可维护性与安全性较低，需进行系统性重构。 - **风险等级**：🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `get_bill_goods_info` (~L248) | **SQL 注入风险**：`$sql = '_unique_key="' . $unique_key . '" AND ...'` 直接拼接外部参数，未使用查询构造器或参数绑定，极易被恶意注入。 | 使用框架查询构造器或参数化查询，严禁手动拼接 SQL 条件。 | `$this->db->where('_unique_key', $unique_key)->where_in('_status', [1, 4])->or_where(['_pay_platform' => 10, '_status' => -1]);` | | 🔴 严重 | 文件顶部 (~L4-L5) | **全局作用域执行 `get_instance()`**：`$CI = &get_instance();` 在类外执行，每次 `require/include` 该文件都会触发，可能导致未初始化报错或内存泄漏。 | 移至类的 `__construct()` 中，或依赖框架自动加载机制。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` | | 🟠 警告 | `get_list_export_v2` (~L290-L310) | **循环内重复加载模型**：在分页循环中反复调用 `$this->load->model()`，虽 CI 有缓存机制，但严重拖慢执行速度且违反规范。 | 统一在方法开头或构造函数中加载一次，后续直接调用。 | `$this->load->model(['ahead_yc_order_extension_model', 'ahead_shop_config_model']);` | | 🟠 警告 | `get_list_export` (~L198-L205) | **导出逻辑缺陷与内存风险**：分页循环被注释，直接全量查询 `$res = $this->select(...)`。若数据量大将导致 OOM；且手动 `+=` 累加统计值效率极低。 | 恢复分页导出逻辑，或改用数据库 `SUM()` 聚合查询替代 PHP 循环计算。 | `$totals = $this->db->select('SUM(_prime_actual_pay) as total_amount, SUM(_actual_pay) as total_actual_pay...')->get()->row();` | | 🟠 警告 | `get_detail`, `get_list` | **N+1 查询隐患**：多次独立调用 `get_one` 获取关联数据（如导购、房间、扩展表），未使用 `JOIN` 或批量 `where_in`，数据库交互频繁。 | 优先使用 `JOIN` 一次性拉取，或收集 ID 后批量查询。 | `$this->db->join('ahead_yc_order_extension ext', 'ext._order_id = order._id', 'left')->get()->result_array();` | | 🟠 警告 | `set_table_name` 调用处 | **原始 SQL 注入表名**：`$this->table_name . ' `order`' . ' force index(_merchant_id)'` 直接拼接 `force index`，若框架未做严格转义可能引发语法错误或注入。 | 确认 `Simple_model` 是否支持原生索引提示。若不支持，建议通过查询构造器的 `force_index()` 方法或配置索引优化器处理。 | *(需查阅 phpci/CI 文档确认是否支持 `$this->db->force_index('_merchant_id')`)* | | 🟡 建议 | 全局多处 | **违反 PSR-12 与单一职责**：`get_bill_goods_info` 超 300 行，混合数据查询、金额计算、格式化、业务分支判断。数组语法混用 `array()` 与 `[]`。 | 拆分方法，将计算与格式化逻辑抽离至 Service/Helper 层；统一使用短数组语法 `[]`。 | 提取 `calculateOrderTotals()`, `formatGoodsList()`, `resolvePayPlatform()` 等方法。 | | 🟡 建议 | 数组定义处 | **键类型不一致**：`$pay_platform_arr` 使用字符串键 `'1' => ...`，但 `in_array` 使用整数数组 `[17, 18...]`，依赖 PHP 弱类型比较，易引发隐蔽 Bug。 | 统一使用整数键，或开启严格类型比较 `===`。 | `const PAY_PLATFORM_ARR = [1 => '微信支付', 2 => '支付宝', ...];` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **立即修复 SQL 注入**：将 `get_bill_goods_info` 中的 `$sql` 拼接替换为查询构造器或参数绑定。这是最高优先级的安全红线。 2. **移除全局 `$CI` 实例化**：将 `$CI = &get_instance();` 移入 `__construct()`，避免文件加载时的副作用。 3. **修复导出内存泄漏**：`get_list_export` 必须恢复分页循环或改用流式导出（如生成器 `yield`），避免一次性加载全量数据到内存。 ### 🛠 后续重构与优化方向 1. **职责分离（SRP）**：当前 Model 承担了“数据存取 + 业务计算 + 视图格式化”三重职责。建议： - **Model 层**：仅保留 `get_detail_raw()`, `get_list_raw()`, `get_bill_raw()` 等纯净数据查询方法。 - **Service/Helper 层**：新建 `OrderFormatterService` 处理折扣计算、支付平台映射、时长格式化、团购信息组装等逻辑。 2. **查询性能优化**： - 将分散的 `get_one` 合并为 `JOIN` 查询或 `where_in` 批量查询。 - 统计金额（`total_amount` 等）务必下沉至数据库层使用 `SUM()`，避免 PHP 循环累加。 3. **规范与可维护性**： - 全面启用 PSR-12 规范，统一使用 `[]` 数组语法、严格比较 `===`、类型声明（PHP 7.4+ 支持属性类型声明）。 - 消除魔法数字：将 `17, 18, 19...` 等硬编码支付平台 ID 提取为类常量或配置数组。 - 添加完整 PHPDoc 类型提示（`@param`, `@return`），便于 IDE 静态分析与团队协作。 > 📌 **框架适配说明**：代码结构高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请重点核对 `$this->load->model()` 的缓存机制、`$this->select()` 的底层实现是否支持参数绑定，以及是否提供原生的 `force_index()` 查询构造器方法。建议查阅 `phpci` 官方文档确认差异。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779936622
updated_unix
1779936622
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel