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 195 from issue
id
195
repo_id
22
index
14
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：app-260616 - Merge pull request '1
🔍 代码审查报告：app-260616 - Merge pull request '1' (#11) from app into app-260
...
content
## 自动代码审查报告 **分支**: app-260616 **提交**: `ca8685bf1
## 自动代码审查报告 **分支**: app-260616 **提交**: `ca8685bf1e6e2f4b02ea62d3657c59200d3693bf` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-05-19 14:13:37 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：代码具备完整的业务闭环能力，但存在严重的架构与规范问题。核心逻辑高度耦合在 Model 层，大量使用字符串拼接构造 SQL，事务与外部支付 API 混用导致资金一致性风险极高，且存在典型的 N+1 查询性能瓶颈。整体可维护性与安全性亟待重构。 - **风险等级**：🔴 高（涉及资金安全、SQL 注入、数据一致性） > 📌 **框架说明**：代码语法特征（如 `$CI =& get_instance()`、`$this->load->model()`、`$this->db->trans_start()`）高度符合 **CodeIgniter 3** 规范。若 `phpci` 为贵司内部定制框架，请确认其底层 DB 驱动与事务机制是否与 CI 兼容。以下审查基于标准 PHP 及 CI 最佳实践。 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `Ahead_pay_log_model.php`<br>`Ahead_book_order_model.php` | **SQL 注入风险**：多处使用字符串拼接构造 `WHERE` 与 `UPDATE` 语句（如 `$log_where = '_relation_id="' . $order_data['_id'] . '"...'`），若上游数据未严格过滤，将导致严重注入漏洞。 | 全面改用 CI 查询构造器（Query Builder）或参数绑定，禁止手动拼接 SQL 片段。 | `$this->db->where('_relation_id', $order_data['_id'])`<br>`->where_in('_type', [5, 13])`<br>`->update('table', ['_status' => 4]);` | | 🔴 严重 | `Ahead_book_order_model.php`<br>`invalid_book()` / `refund_by_notify()` | **事务与外部 API 混用**：在数据库事务内直接调用微信/银联退款 API。若 API 成功但后续 DB 操作失败触发回滚，将导致“钱已退、库未改”的资金不一致事故。 | 采用**最终一致性**设计：1. 先更新本地状态为 `退款中` 并提交事务；2. 异步/同步调用 API；3. 根据回调结果更新最终状态。失败则走补偿/重试队列。 | 见下方重构建议 | | 🔴 严重 | `Ahead_book_order_model.php`<br>`get_list()` / `get_detail()` | **N+1 查询性能瓶颈**：在 `foreach` 循环中调用 `$this->ahead_yc_order_model->get_one()` 获取关联数据，数据量稍大时将引发严重性能雪崩。 | 收集所有关联 ID，使用 `where_in` 一次性批量查询，在内存中通过键值映射组装数据。 | `$ids = array_column($order_info, 'relation_order_id');`<br>`$orders = $this->ahead_yc_order_model->where_in('_id', $ids)->get()->result_array();`<br>`$map = array_column($orders, null, '_id');` | | 🟠 警告 | `Ahead_preorder_order_model.php`<br>`get_list()` | **空数据抛出异常**：`throwError("没有更多数据了")` 将正常的业务空状态当作致命异常抛出，破坏调用方控制流，且不符合 RESTful 规范。 | 返回空数组 `[]`，由 Controller 层负责分页提示或前端展示逻辑。 | `return $data ?: [];` | | 🟠 警告 | `Ahead_book_order_model.php`<br>`openRoomByCommunityOrder()` | **上帝方法（God Method）**：单方法超 300 行，承担订单创建、库存扣减、支付流水、外部通知、打印机调用等职责，违反单一职责原则（SRP），极难测试与维护。 | 引入 **Service 层** 拆分职责：`OrderCreationService`、`InventoryService`、`PaymentService`、`NotificationService`。Model 仅负责数据存取。 | 架构级重构，建议按业务域拆分 | | 🟠 警告 | 全局多处 | **事务管理不规范**：混用 `trans_start()`、`trans_rollback()`、`trans_complete()` 与 `try-catch`。CI 的 `trans_complete()` 会自动检测状态并回滚，手动干预易导致事务状态机混乱。 | 统一使用 CI 标准事务流或 PHP 原生 `try-catch` 配合 `trans_begin()`。确保异常捕获后正确回滚。 | `try { $this->db->trans_begin(); ... $this->db->trans_commit(); } catch (\Exception $e) { $this->db->trans_rollback(); throw $e; }` | | 🟡 建议 | 全局 | **代码规范与可读性**：命名不一致（`setTablename` vs `set_table_name`）、混用 `array()` 与 `[]`、缺乏 PHP 7+ 类型声明、遗留调试注释（如 `// 20210712版本，hlj`）。 | 严格遵循 PSR-12，统一短数组语法，添加参数/返回值类型声明，清理历史注释，使用 IDE 格式化。 | `public function get_list(int $merchant_id, array $param): array` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题（P0/P1） 1. **立即修复 SQL 拼接漏洞**：全局搜索 `up(`、`where = '`、`$log_where` 等字符串拼接 SQL 的位置，替换为 CI Query Builder 或 PDO 预处理。这是资金系统的红线。 2. **解耦事务与支付 API**：将 `refund_by_notify` 和 `mobile_refund` 中的第三方退款调用移出当前事务。改为“本地状态机 + 消息队列/定时任务补偿”模式，确保资金流水绝对可追溯。 3. **消除 N+1 查询**：对 `get_list`、`get_detail`、`get_bill_pay_log` 等列表/详情接口实施批量查询优化，预计可降低 60%~80% 的数据库 IO 耗时。 ### 🛠 后续重构与优化方向 1. **架构分层（Model → Service → Controller）**： - `Model` 层仅保留 CRUD、基础查询与数据校验。 - 将 `openRoomByCommunityOrder`、`invalid_book` 等复杂业务逻辑抽离至 `Service` 类，便于单元测试与复用。 - 使用依赖注入（DI）替代 `$this->load->model()` 动态加载，提升可测试性。 2. **统一异常处理机制**： - 废弃全局 `throwError()` 函数，改用 PHP 标准 `throw new \RuntimeException()` 或自定义业务异常类（如 `BusinessException`）。 - 在框架入口或中间件统一捕获异常，返回标准化 JSON 响应。 3. **财务与状态机规范化**： - 订单状态流转建议引入 **状态机模式（State Machine）**，明确定义合法流转路径，防止非法状态跃迁。 - 所有涉及金额的计算强制使用 `bcmath` 扩展（如 `bcadd`, `bcsub`），避免浮点数精度丢失。 4. **补充自动化测试**： - 针对退款、开房、支付回调等核心链路编写 PHPUnit 单元测试与集成测试，覆盖正常流、异常流、网络超时、并发重复提交等场景。 > 💡 **局限性说明**：本次审查基于提供的 Model 层代码片段。由于未提供 `Simple_model` 基类实现、全局 Helper 函数（如 `throwError`, `get_business_date`）及 Controller 调用上下文，部分边界条件（如并发锁、幂等性设计）需结合完整项目链路进一步评估。建议优先落地 P0 安全与事务修复，再逐步推进架构重构。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779171217
updated_unix
1779171217
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel