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 601 from issue
id
601
repo_id
21
index
274
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260616 - 1
content
## 自动代码审查报告 **分支**: pay-260616 **提交**: `83c89ed41
## 自动代码审查报告 **分支**: pay-260616 **提交**: `83c89ed414e6697b8666f2aa438a02c8c575bdc3` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-08 19:29:15 --- ## 1. 审查摘要 - **代码质量评分**：6.5 / 10 分 - **总体评价**：代码实现了完整的预订、支付回调、退款及门店配置读取流程，业务逻辑覆盖较全面。但存在明显的 SQL 拼接隐患、事务管理不规范、魔法数字泛滥、方法职责过重等问题。整体偏向传统 CodeIgniter 3 风格，部分写法未充分利用现代 PHP 特性与框架最佳实践。末尾 `create_community_shop_book_order` 代码截断，无法评估完整逻辑。 - **风险等级**：🔴 高（主要源于 SQL 注入隐患、支付状态误判及退款单号并发冲突风险） ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `Ahead_book_order_model.php`<br>`refund_by_notify` 方法内 | `$log_where` 使用字符串直接拼接订单 ID 构造查询条件，若 `_id` 未严格过滤或包含特殊字符，将导致 **SQL 注入** 或语法错误。 | 废弃字符串拼接，全面改用 CI 查询构建器（Query Builder）或参数绑定。 | `$this->db->where('_relation_id', $order_data['_id'])<br>->where('_status', 1)<br>->where_in('_type', [5, 13]);` | | 🔴 严重 | `Ahead_book_order_model.php`<br>`check_notify` 方法内 | 状态判断逻辑缺陷：当订单状态为 `3`（退款申请中）时，直接返回 `'支付成功'`。支付网关回调可能误认为交易完成，导致重复发货或资金对账混乱。 | 明确状态机流转。退款中应拦截回调或返回特定业务码，交由退款流程处理。 | `if (in_array($order_data['_status'], [1, 4])) { return ['status' => true, 'msg' => '支付成功']; }<br>if ($order_data['_status'] == 3) { return ['status' => false, 'msg' => '订单退款处理中', 'return' => 500]; }` | | 🟠 警告 | `Ahead_book_order_model.php`<br>`refund_by_notify` 方法内 | 微信退款单号 `Out_refund_no` 仅使用 `date('YmdHis')` 生成。在高并发退款场景下极易重复，触发微信 `OUT_TRADE_NO_USED` 错误导致退款失败。 | 结合订单号、时间戳与随机数/微秒保证全局唯一性。 | `$input->SetOut_refund_no('RF' . $order_data['_id'] . '_' . time() . '_' . mt_rand(1000, 9999));` | | 🟠 警告 | `Ahead_shop_config_second_model.php`<br>`get_shop_setting` 方法内 | 使用 `self::$shop_config[$shop_id]` 静态缓存配置，未区分 `merchant_id`，且无缓存失效机制。多租户或后台修改配置后，PHP-FPM 进程内将读取到脏数据。 | 缓存键需包含商户 ID，或提供 `clearCache()` 方法；建议接入框架 Cache 驱动（如 Redis/Memcached）替代静态变量。 | `$cache_key = "{$merchant_id}_{$shop_id}";<br>if (empty(self::$shop_config[$cache_key])) { ... }` | | 🟠 警告 | `Ahead_book_order_model.php`<br>`check_notify` 方法内 | 事务管理混用：`$this->db->trans_start()` 配合 `try-catch` 手动 `trans_rollback()`。CI 框架在严格模式下会自动回滚，手动干预易引发事务状态不一致或死锁。 | 统一使用 `$this->db->trans_begin()` 与 `$this->db->trans_complete()`，移除 `catch` 中的手动回滚，依赖框架自动处理。 | `// 移除 try-catch 中的 $this->db->trans_rollback();<br>// 确保开启 $this->db->trans_strict(TRUE);` | | 🟡 建议 | `Ahead_book_order_model.php` 全文 | 大量硬编码“魔法数字”（如状态 `-1,1,2,3,4,5`，支付平台 `1,3,8,9,14`，模板索引 `14`）。可读性差，后期维护极易出错。 | 提取为类常量或独立配置文件，使用语义化命名。 | `const STATUS_PENDING = -1; const STATUS_PAID = 1; const PAY_SCENE_WECHAT = '5';` | | 🟡 建议 | `Ahead_book_order_model.php`<br>`refund_by_notify` 方法 | 方法体超过 250 行，混合了支付网关调用、状态更新、日志记录、消息推送、财务流水等逻辑，严重违反单一职责原则（SRP）。 | 拆分为独立 Service 类或私有方法，如 `processGatewayRefund()`, `updateOrderStatus()`, `notifyUserAfterRefund()`。 | 将支付网关逻辑抽离至 `PaymentRefundService::refund()`，模型仅负责数据持久化。 | | 🟡 建议 | `Ahead_book_order_model.php`<br>文件顶部 | 全局作用域调用 `$CI = &get_instance();` 违反 CI 模型规范，易导致上下文污染、内存泄漏及单元测试困难。 | 删除文件顶部全局调用。在方法内部按需获取，或在模型构造函数中初始化。 | `// 删除顶部 $CI = &get_instance();<br>public function __construct() { parent::__construct(); $this->ci =& get_instance(); }` | | 🟡 建议 | `Ahead_book_order_model.php`<br>`get_detail` 方法内 | 变量拼写错误 `$rooom_data`；日志函数命名不统一（`doLog` 与 `do_log` 混用）；数组语法混用 `array()` 与 `[]`。 | 修正拼写，统一日志函数命名；全面采用短数组语法 `[]` 以符合 PSR-12。 | `$room_data = ...; do_log(...);` | | 🟡 建议 | `Ahead_shop_config_second_model.php`<br>`get_shop_setting` 方法 | 庞大的 `switch` 语句中直接加载模型并查询数据库，导致模型层职责越界，且每次调用都会触发重复查询。 | 将字段映射与业务逻辑解耦。模型仅返回原始数据，字段转换逻辑移至 Helper 或 Service 层。 | `$raw = $this->get_one(...);<br>return ConfigTransformer::transform($raw, $field);` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **修复 SQL 注入隐患**：立即将 `refund_by_notify` 中的 `$log_where` 和 `$log_up` 字符串拼接替换为 CI Query Builder 链式调用或预处理语句。 2. **修正支付回调状态机**：重新梳理 `check_notify` 的状态拦截逻辑，确保 `退款申请中(3)` 不会误报为支付成功，避免资金与订单状态不一致。 3. **保证退款单号唯一性**：修改 `Out_refund_no` 生成规则，引入订单 ID 或唯一标识符，防止高并发下微信/银联退款接口报错。 ### 🛠 后续重构与优化方向 1. **架构分层与职责拆分**：当前 Model 承担了过多业务逻辑（支付网关、消息推送、财务流水）。建议引入 `Service` 层处理复杂业务流程，Model 仅负责数据读写与基础校验。 2. **统一配置与常量管理**：将散落在各处的状态码、支付平台标识、微信模板 ID 提取至 `config/constants.php` 或独立配置类，提升代码可维护性。 3. **缓存策略升级**：`self::$shop_config` 静态缓存在 PHP-FPM 环境下存在生命周期与脏读问题。建议迁移至 Redis/Memcached，并设置合理的 TTL 与主动失效机制。 4. **规范事务与异常处理**：全面采用 `$this->db->trans_begin()` / `$this->db->trans_complete()` 标准范式。移除冗余的 `try-catch` 手动回滚，利用框架的 `trans_strict` 特性保证数据一致性。 5. **补充缺失代码审查**：`create_community_shop_book_order` 方法末尾被截断，请补充完整代码以便进行闭环逻辑与事务安全性的二次审查。 > 💡 **框架适配说明**：代码呈现典型的 CodeIgniter 3 架构特征。若 `phpci` 为基于 CI 二次开发的内部框架，上述建议完全兼容。若为独立框架，请确保 `$this->db` 事务驱动与模型加载机制与 CI 保持一致，或查阅官方文档替换为对应框架的 ORM/Query Builder 语法。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780918155
updated_unix
1780918155
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel