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 593 from issue
id
593
repo_id
22
index
73
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：app-260616 - 自助商家小程序兑换开房，续费的订单卡券要退回给用户
content
## 自动代码审查报告 **分支**: app-260616 **提交**: `5999c22d1
## 自动代码审查报告 **分支**: app-260616 **提交**: `5999c22d1ad94b48eae8ff43caaefa11d27932c6` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-08 15:21:04 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：该 Model 承担了支付流水记录、回调处理、账单查询、营业报表及核心退款等大量业务逻辑。整体功能完整，但存在明显的架构越界（Model 层混杂业务逻辑与第三方 API 调用）、SQL 拼接安全隐患、循环内 N+1 查询性能瓶颈以及事务管理不规范等问题。代码风格偏向传统 PHP 开发，缺乏现代 PHP 类型约束与 PSR-12 规范。 - **风险等级**：🔴 高（存在 SQL 注入隐患、事务回滚不一致、硬编码敏感信息） > 📌 **框架说明**：代码中大量使用 `$CI = &get_instance()`、`$this->load->model()`、`$this->db->trans_begin()` 等特征，实际为 **CodeIgniter 3** 架构，而非 `phpci`（PHP 持续集成服务器）。以下审查基于 CI3 及现代 PHP 最佳实践进行。若为自研封装框架，请确保底层查询构建器支持参数化绑定。 --- ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `update_refund_amount` / `refund` 多处 | **SQL 注入风险**：直接使用字符串拼接构造 `$where` 和 `$up` 条件（如 `'_relation_id="' . $relation_id . '"...'`），未使用预处理或查询构建器，极易被恶意参数注入。 | 全面改用 CI3 查询构建器或参数化查询，禁止手动拼接 SQL 片段。 | `$this->db->where('_relation_id', $relation_id)->where('_type', $type)->where_in('_status', [1,4]);`<br>`$this->db->set('_status', 4)->set('_refund_amount', '_refund_amount+' . $refund_amount, FALSE)->update($this->table_name);` | | 🔴 严重 | `refund` 方法 | **事务回滚逻辑缺陷**：在多处 `if (!$res)` 中重复调用 `$this->db->trans_rollback()`，且依赖自定义 `throwError()`。若 `throwError` 未正确抛出异常或终止执行，会导致事务状态不一致或数据库锁死。 | 使用 `try...catch` 统一包裹事务，仅在 `catch` 中执行一次回滚，并抛出标准 `Exception`。 | `try { $this->db->trans_begin(); /* 业务逻辑 */ $this->db->trans_commit(); } catch (\Exception $e) { $this->db->trans_rollback(); throw $e; }` | | 🔴 严重 | `mobile_refund` 方法 | **硬编码敏感密钥**：`md5()` 中直接写死 `'1441600902'` 和 `'2017040606573534'`，疑似支付网关盐值或签名密钥，存在代码泄露导致资金安全风险。 | 将密钥移至 `config/payment.php` 或环境变量中，通过 `config_item()` 读取。 | `$salt = config_item('refund_sign_salt'); $data['refund_key'] = md5($order_info['_id'] . $order_info['_trade_no'] . $salt);` | | 🟠 警告 | `get_bill_pay_log` / `get_refundable_pay_log` | **N+1 查询性能瓶颈**：在 `foreach` 循环内频繁调用 `get_one()`、加载模型及查询配置。当 `$log_data` 数据量较大时，将产生数十次甚至上百次 DB 请求，严重拖慢接口响应。 | 提前收集所有关联 ID，使用 `WHERE IN` 批量查询，在内存中构建映射表后循环赋值。 | `$ids = array_column($log_data, 'relation_id'); $exts = $this->ahead_yc_order_extension_model->select(['where_in' => ['_order_id', $ids]]); $extMap = turn_array_key($exts, '_order_id');` | | 🟠 警告 | `add_order_pay_log` | **循环内修改传入参数**：`$params` 在循环中被直接覆盖（如 `$params['_pay_platform'] = $platform`），导致后续迭代使用被污染的数据，且 `$actual_pay` 累加逻辑可能失真。 | 使用局部变量或克隆参数数组，避免副作用。明确多支付金额计算边界。 | `foreach ($payPlatformArr as $platform) { $currentParams = $params; if ($payPlatform == 14) { $currentParams['_pay_platform'] = $platform; ... } }` | | 🟠 警告 | `update_after_pay_call_back` | **类型比较不一致**：`$newPayLogData['_type'] == 1` 与后续 `== '1'` 混用，PHP 弱类型比较可能导致分支误判。常量键名为字符串 `'1'`，但传入值可能为整型。 | 统一使用严格比较 `===`，或在方法入口处进行类型标准化。 | `if ((string)$newPayLogData['_type'] === '1')` | | 🟡 建议 | 全局 | **违反单一职责原则 (SRP)**：Model 层包含退款计算、第三方 API 调用、报表聚合、Redis 队列推送等大量业务逻辑，导致类臃肿且难以测试。 | 抽离业务逻辑至 `Service` 或 `Library` 层，Model 仅保留数据存取与基础查询。 | 创建 `RefundService::processRefund()` 处理退款流程，Model 仅提供 `updateStatus()`、`getById()` 等方法。 | | 🟡 建议 | 全局 | **魔法数字泛滥**：大量硬编码的平台/状态码（如 `1, 2, 11, 15, 17-28`），可读性差且易遗漏维护。 | 充分利用已定义的 `const` 常量，或建立统一的 `PaymentPlatform` 枚举/配置类。 | `if ($platform == self::VIP_PRESENT_PAY)` 替代 `== 15` | | 🟡 建议 | 全局 | **缺乏现代 PHP 规范**：无 `declare(strict_types=1);`，无参数/返回值类型声明，注释含口语化历史遗留说明（如 `//之前不知道为啥被注释掉...`）。 | 遵循 PSR-12，补充类型提示，清理无效注释，使用 PHPDoc 规范方法签名。 | `public function add_order_pay_log(int $merchantId, int $adminUid, string $type, int $relationId, array $params, array $multPayIndexArr = []): array` | --- ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **修复 SQL 注入隐患**：立即替换所有字符串拼接的 `$where` 和 `$up` 逻辑，改用框架提供的查询构建器或参数化绑定。这是资金安全与数据完整性的底线。 2. **重构事务管理**：将 `refund()` 方法中的 `trans_begin/commit/rollback` 包裹在 `try...catch` 中，移除重复的 `trans_rollback()` 调用，确保异常发生时事务能安全回滚且不阻塞连接。 3. **移除硬编码密钥**：将 `mobile_refund` 中的签名盐值迁移至配置文件或环境变量，禁止在代码库中明文存储支付相关密钥。 ### 🛠 后续重构与优化方向 1. **架构分层（MVC -> MVCS）**： - 将 `refund()`、`update_after_pay_call_back()` 中的业务编排逻辑抽离至 `application/services/PaymentService.php`。 - Model 仅负责 `insert()`, `update()`, `select()`, `getOne()` 等数据操作。 - 第三方 API 调用（微信/支付宝退款）封装至 `application/libraries/PaymentGateway.php`。 2. **性能优化（消除 N+1）**： - 对 `get_bill_pay_log` 和 `get_refundable_pay_log` 实施“批量预加载”策略。使用 `WHERE IN` 一次性拉取关联表数据，通过 `array_column` + `array_combine` 构建内存映射，将 DB 查询次数从 `O(N)` 降至 `O(1)`。 3. **规范化与可维护性**： - 引入 PHP 7.4+ 类型声明与严格模式。 - 建立统一的 `PaymentPlatform` 与 `PayLogStatus` 常量/枚举类，彻底消除魔法数字。 - 清理历史注释，使用 PHPDoc 补充 `@param`、`@return`、`@throws` 说明。 > ⚠️ **局限性说明**：提供的代码在 `mobile_refund` 方法末尾被截断，未能完整审查支付宝退款回调处理及后续逻辑。建议补充完整代码以便进行全链路安全与异常流审查。若需针对特定 CI3 版本或自定义底层封装进行适配调整，请提供框架核心加载器或 DB 驱动实现细节。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780903264
updated_unix
1780903264
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel