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 325 from issue
id
325
repo_id
18
index
145
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pc-260616 - 需求 赠时报表 16382
content
## 自动代码审查报告 **分支**: pc-260616 **提交**: `e59f808813
## 自动代码审查报告 **分支**: pc-260616 **提交**: `e59f80881314e0b44bbb1e846529fd54e49b9105` **提交人**: chenjunfeng (developer.jeff.c@gmail.com) **时间**: 2026-05-26 14:27:56 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：控制器承载了过多业务逻辑与导出模板配置，功能实现完整但架构臃肿。存在明显的参数过滤缺失、内存管理不当及大量重复代码。若数据量增长，极易引发 OOM 或响应超时。需进行服务层抽离与安全加固。 - **风险等级**：🔴 高 > 📌 **框架说明**：代码中出现的 `BASEPATH`、`$this->load->model()`、`$this->param`、`$this->success_response()` 等特征高度符合 **CodeIgniter 3** 规范。若项目实际使用 `phpci`（可能为 CI 二次开发或定制版），请结合其官方文档核对生命周期与输入过滤机制。以下审查基于 CI3 最佳实践。 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | 全局导出方法 | **直接使用 `$_GET` 传入模型，未做安全过滤**。若底层模型未使用 Query Builder 或预处理语句，将导致 SQL 注入。 | 统一使用框架输入类获取参数，并在控制器层进行类型/范围校验。模型层必须使用参数绑定。 | `$params = $this->input->get(null, true); // 启用 XSS 过滤`<br>`$params['start_date'] = $this->input->get('start_date', true);` | | 🔴 严重 | `managerDiscountExport()` 等 | **滥用 `exit()` 终止脚本**。绕过框架输出生命周期，导致响应头丢失、日志未记录、数据库连接未释放，且不利于后续中间件/钩子执行。 | 移除 `exit()`，改用框架标准响应输出或让模型直接输出文件流后由控制器返回。 | `// 移除 exit();<br>// 若模型已输出文件流，可返回空响应：<br>$this->output->set_status_header(200);` | | 🟠 警告 | `getWaresCountExport()` 等 | **硬编码 `ini_set("memory_limit", "500M")` 应对大数据导出**。治标不治本，易引发服务器资源争抢，且未恢复原值，影响后续请求。 | 采用分块查询（`LIMIT/OFFSET` 或游标）、生成器（`yield`）或数据库层聚合。若必须调高内存，应在方法结束前恢复。 | `$oldLimit = ini_get('memory_limit');<br>ini_set('memory_limit', '500M');<br>// ... 业务逻辑 ...<br>ini_set('memory_limit', $oldLimit);` | | 🟠 警告 | 全局导出方法 | **导出逻辑高度重复（DRY 原则违反）**。列定义、字段过滤、表头配置、Excel/PDF 分支在 10+ 个方法中重复编写，维护成本极高。 | 抽离为独立导出服务类（Service）或 Trait，通过配置数组驱动。控制器仅负责参数收集与调用。 | `// 控制器内仅保留：<br>$config = $this->getExportConfig('wares_count');<br>$this->exportService->handle($config, $params);` | | 🟠 警告 | `getWaresCountExport()` 等 | **`json_decode` 未做异常处理**。若前端传入非法 JSON，将返回 `null` 并触发 Warning，后续 `empty()` 判断可能失效。 | 增加 `json_last_error()` 校验或提供默认值，确保类型安全。 | `$exportFields = isset($param['export_fields']) ? json_decode($param['export_fields'], true) : null;<br>if (json_last_error() !== JSON_ERROR_NONE) { $exportFields = array_keys($columnArr); }` | | 🟡 建议 | 全局方法 | **模型加载分散在各方法中**。CI3 虽会缓存已加载模型，但重复调用降低可读性，且不符合单一职责。 | 在 `__construct()` 中统一加载常用模型，或使用 CI3 的 `autoload.php` 配置。 | `public function __construct() {<br> parent::__construct();<br> $this->load->model(['Export_model', 'Ahead_finance_report_setting_model']);<br>}` | | 🟡 建议 | `getWaresCountExport()` 等 | **`$this->param` 与 `$_GET` 混用**。框架未明确 `$this->param` 的注入来源，混用易导致参数丢失或过滤策略不一致。 | 统一使用 `$this->input->get()` 或 `$this->input->post()`，并在基类中统一赋值给 `$this->param`。 | `$this->param = $this->input->get(null, true);` | | 🟡 建议 | 文件末尾 | **代码片段不完整**。`shoppingGuideOrderCountByUserExport()` 方法在数组定义处截断，无法评估完整逻辑。 | 请补充完整代码以便进行全量审查。 | *(无)* | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **安全过滤**：立即替换所有 `$_GET` 为 `$this->input->get(null, true)`，并在模型层强制使用 `$this->db->where()` 或预处理语句，杜绝 SQL 注入。 2. **生命周期规范**：移除所有导出方法中的 `exit()`，确保框架能正常执行 `post_controller` 钩子、记录访问日志及释放数据库连接。 3. **内存与性能**：针对大数据量导出，废弃 `ini_set` 暴力扩容方案。改为数据库层 `SUM/COUNT` 聚合统计，或采用分块查询+流式写入（如 `PHPExcel/PhpSpreadsheet` 的 `setPreCalculateFormulas(false)` 与流式输出）。 ### 🛠 后续重构方向 1. **架构解耦（Controller -> Service）**： 当前控制器承担了“参数解析、数据查询、字段映射、格式组装、文件生成”全链路职责。建议引入 `ReportExportService`： ```php // 示例：服务类调用 $config = [ 'title' => '商品套餐销售汇总报表', 'columns' => $columnArr, 'model_method' => 'get_wares_count_list', 'sum_fields' => ['quantity', 'actual_amount_total'] ]; $this->reportExportService->export($this->merchant_id, $this->param, $config); ``` 2. **统一导出配置管理**：将 `columnArr`、`row_widths`、`header` 等硬编码配置移至 `config/reports.php` 或数据库字典表，实现前端动态配置导出列。 3. **异步导出机制**：对于耗时 > 5s 的报表，建议引入队列（如 Redis + Supervisor 或 CI3 的 `Queue` 扩展），前端轮询或 WebSocket 推送下载链接，彻底解决 HTTP 超时与内存瓶颈。 4. **规范命名与注释**：统一模型命名（如 `AheadManagerDiscountReportModel` 或遵循 CI3 的 `ahead_manager_discount_report_model`），补充 PHPDoc 类型声明（`@param array`, `@return array`），提升 IDE 提示与静态分析能力。 > 💡 **提示**：若需针对具体导出方法（如 `getWaresCountExport`）提供完整的 Service 层重构代码或异步队列方案，可提供完整文件及当前使用的 Excel 库版本（如 `PHPExcel` / `PhpSpreadsheet`），我将输出可直接替换的架构代码。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779776876
updated_unix
1779776876
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel