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 202 from issue
id
202
repo_id
21
index
20
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260519 - 14003服务费判断临时授权。余额同步修改。
content
## 自动代码审查报告 **分支**: pay-260519 **提交**: `9ecd6a01b
## 自动代码审查报告 **分支**: pay-260519 **提交**: `9ecd6a01b44fa05433ae81813f037b665e7efdf2` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-05-19 15:43:42 --- ## 1. 审查摘要 - **代码质量评分**：5/10 分 - **总体评价**：代码实现了门店账户余额查询、扣减、日志记录及第三方接口上报等核心业务，但存在**严重的并发竞态条件、浮点数精度隐患、未定义变量引用及 SQL 拼接风险**。部分方法存在死代码，且大量重复逻辑未做抽象，整体可维护性与财务安全性不足。 - **风险等级**：🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `_check_consume_deduction_balance` 方法内 | 方法参数为 `$merchantId, $shopId`，但内部直接判断 `if (empty($shopInfo))`。`$shopInfo` 未定义，将触发 PHP Notice 且逻辑永远走 DB 查询，属于明显 BUG。 | 移除无意义的 `if (empty($shopInfo))` 判断，直接执行查询逻辑。 | `public function _check_consume_deduction_balance($merchantId, $shopId) { $shopWhere = [...]; $shopInfo = $this->get_one(...); ... }` | | 🔴 严重 | `modify_shop_account` / `modify_shop_all_account` | 采用“先 `get_one` 判断是否存在，再 `insert` 或 `up`”的模式。高并发下会产生**竞态条件(Race Condition)**，导致重复插入或余额超扣/负数。 | 使用数据库原子更新或事务。插入使用 `INSERT ... ON DUPLICATE KEY UPDATE`，更新时增加余额校验条件。 | `$this->db->set('_balance', '_balance + ' . (float)$amount, FALSE)->where('_id', $id)->where('_balance >=', $amount)->update($table);` | | 🔴 严重 | 全局金额计算与比较 | 使用 `float` 进行金额计算（如 `0.08 * count`）和比较（`$balance <= 0`）。PHP 浮点数精度丢失会导致财务对账错误（如 `0.1+0.2 != 0.3`）。 | 财务字段统一使用数据库 `DECIMAL` 类型，PHP 层转为**“分”(整数)**计算，或使用 `bcmath` 扩展。 | `$amount_cents = (int)round($amount * 100);` 比较时：`if ($balance_cents <= 0)` | | 🟠 警告 | `modify_shop_*` 系列方法 | SQL 更新语句使用字符串拼接：`$up = '_balance=_balance+' . $amount`。若 `$amount` 来源不可控，存在 **SQL 注入风险**，且不符合框架安全规范。 | 使用框架 Query Builder 或参数绑定，严禁直接拼接变量。若底层 `up()` 仅支持字符串，必须强制类型转换。 | `$amount = (float)$amount; $up = "_balance=_balance+{$amount}";`（临时方案）<br>推荐：使用 `$this->db->set()` 链式调用。 | | 🟠 警告 | `sent_cavca_open_room_order` | 方法首行直接 `return true;`，导致后续所有业务逻辑（加载模型、计算价格、请求 API）成为**死代码**。 | 确认是否为调试遗留。若需保留逻辑，删除首行 `return true;`；若已废弃，直接删除该方法。 | 移除 `return true;` 或添加注释说明废弃原因。 | | 🟠 警告 | `check_shop_balance` | `$shopInfo` 可能为 `null`，但后续直接访问 `$shopInfo['_start_time']` 和 `$shopInfo['_temp_auth_expire_time']`，未做空值保护，可能触发 `Undefined array key` 警告。 | 在访问数组键前增加 `isset()` 或空合并运算符 `??` 保护。 | `$startTime = $shopInfo['_start_time'] ?? 0;`<br>`$tempAuth = $shopInfo['_temp_auth_expire_time'] ?? 0;` | | 🟡 建议 | 全局多处 | 频繁在方法内部调用 `$this->load->model()` 和 `$this->config->load()`，增加 I/O 开销且违反依赖注入原则。 | 将依赖的 Model 和 Config 统一在 `__construct()` 中加载，或使用框架自动加载机制。 | `public function __construct() { parent::__construct(); $this->load->model(['Ahead_agent_authenticate_model', 'ahead_shop_account_log_model']); }` | | 🟡 建议 | 命名与规范 | 变量命名风格混用（`$merchantId` vs `$merchant_id`），缺乏 PHP 7+ 类型声明，注释与代码实际行为不符（如 `consume_deduction_account` 注释说金额由 `$data` 传入，实际被硬编码覆盖）。 | 统一命名规范（建议驼峰），添加类型声明，修正注释。遵循 PSR-12。 | `public function consume_deduction_account(int $merchantId, int $shopId, array $data, array $shopInfo = []): bool` | ## 3. 总结与行动建议 ### 🚨 优先修复的关键问题 1. **修复未定义变量 BUG**：立即修正 `_check_consume_deduction_balance` 中的 `$shopInfo` 作用域问题。 2. **消除并发竞态条件**：所有涉及余额增减的操作必须改为**原子 SQL 更新**或包裹在数据库事务中（`$this->db->trans_start()` / `$this->db->trans_complete()`），确保扣款与日志写入的 ACID 特性。 3. **统一金额精度处理**：财务系统严禁使用 `float` 直接计算。建议全链路改为“分”为单位整数，或引入 `bcmath` 函数族进行安全运算。 4. **清理死代码与 SQL 拼接**：移除 `sent_cavca_open_room_order` 的无效 `return`；将 `$up` 字符串拼接替换为框架安全的 Query Builder 或严格类型转换。 ### 🛠 后续重构与优化方向 - **抽象财务操作服务层**：当前 Model 承担了查询、扣减、日志、报表、第三方 API 调用等多重职责。建议拆分出 `ShopAccountService`，遵循单一职责原则（SRP），Model 仅负责数据持久化。 - **引入数据库事务机制**：在 `update_shop_account_by_sms`、`consume_deduction_account` 等方法中，将余额更新与日志插入包裹在事务内，失败时自动回滚，避免“钱扣了但没日志”的脏数据。 - **规范异常处理**：替换自定义 `throwError()`，改用标准 `throw new \InvalidArgumentException()` 或框架异常类，并在 Controller 层统一捕获返回 JSON 错误码，避免敏感堆栈信息泄露。 - **框架适配提示**：代码语法与 **CodeIgniter 3** 高度一致。若 `phpci` 为内部定制框架，请务必查阅官方文档确认： - `up()`、`get_one()` 是否支持预处理参数绑定（Prepared Statements）。 - 框架是否提供内置的 `trans_start()` 事务管理。 - 若底层不支持参数绑定，请在所有拼接处强制使用 `(int)` 或 `(float)` 过滤，并开启框架的 `db_debug` 进行安全审计。 > 💡 **审查局限性说明**：本次审查基于提供的单文件代码。由于未提供 `Simple_model` 父类实现、数据库表结构及全局配置，部分底层方法（如 `up()`、`get_one()`）的安全性与事务支持需结合实际框架源码进一步验证。建议结合完整项目上下文进行集成测试。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779176622
updated_unix
1779176622
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel