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 528 from issue
id
528
repo_id
21
index
215
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260616 - 卡券续费
content
## 自动代码审查报告 **分支**: pay-260616 **提交**: `eed16689b
## 自动代码审查报告 **分支**: pay-260616 **提交**: `eed16689b3505c2ac579ca169344eb7c9f8acfba` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-04 16:19:31 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：业务逻辑覆盖较全，但存在**高危安全漏洞（SQL注入）**、**资金计算精度隐患**、**严重性能瓶颈（循环内加载模型/N+1查询）**及**大量调试代码残留**。方法体过长违反单一职责原则，框架使用存在反模式。需优先进行安全加固与核心计费/账单逻辑重构。 - **风险等级**：🔴 高（涉及资金安全、数据泄露、SQL注入） > 📌 **框架说明**：代码特征（`BASEPATH`、`get_instance()`、`$this->load->model()` 等）明确指向 **CodeIgniter 3 (CI3)** 架构。若实际为 `phpci` 定制框架，请对照其官方文档调整组件调用方式。以下建议基于 CI3 最佳实践与通用 PHP 规范。 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `Ahead_yc_order_model.php`<br>`get_bill_goods_info()` 方法内 | **SQL 注入风险**：使用字符串拼接构造 SQL 条件 `$sql = '_unique_key="' . $unique_key . '" AND ...'`，未进行参数绑定或转义。 | 使用 CI3 Query Builder 或预处理语句，彻底杜绝拼接。 | `$this->db->where('_unique_key', $unique_key);`<br>`$this->db->where_in('_status', [1, 4]);`<br>`$this->db->or_where(...);` | | 🔴 严重 | `Neworderservice.php`<br>约第 250 行 | **调试代码残留**：`echo $vip_upgrade_data_actual_pay;` 未删除，会导致 API 响应格式破坏（JSON 解析失败）。 | 立即删除或替换为框架标准日志记录。 | `// 删除该行 echo`<br>`log_message('debug', 'vip_upgrade_actual_pay: ' . $vip_upgrade_data_actual_pay);` | | 🔴 严重 | `Order.php`<br>`buyRenewalPackage()` / `createOrder()` | **敏感信息泄露**：`do_log(var_export($chinaums_set, 1)...)` 可能将支付密钥、商户号等敏感配置明文写入日志。 | 脱敏后记录，或仅记录关键业务标识。 | `do_log('Chinaums Pay Initiated. ShopID: ' . $chinaums_set['_shop_id'], 'buyRenewalPackage');` | | 🟠 警告 | `Ahead_yc_order_model.php`<br>`get_list()` / `get_bill_goods_info()` | **N+1 查询与循环内加载模型**：在 `foreach` 中频繁调用 `$this->load->model()` 及单条查询，严重拖慢响应。 | 将模型加载移至类属性或构造函数；使用 `where_in` 批量查询后内存映射。 | `// 批量查询替代循环`<br>`$ids = array_column($order_info, 'package_id');`<br>`$packages = $this->ahead_room_package_model->select(['where_in' => ['_id', $ids]]);` | | 🟠 警告 | `Neworderservice.php`<br>`getOrderTypeInfo()` 及多处 | **浮点数精度丢失**：金额计算直接使用 `float` 运算（如 `$goods_actual_pay = $v[$price_key] * $v['_quantity']`），易产生 `0.1+0.2=0.30000000000000004` 问题。 | 金额统一使用**分（整数）**存储与计算，或使用 `bcmath` 扩展。 | `$actual_pay = bcmul($price_key, $quantity, 2);`<br>`$actual_pay = bcadd($actual_pay, $new_val, 2);` | | 🟠 警告 | `Order.php`<br>`buyRenewalPackage()` | **框架反模式**：通过 `$CI =& get_instance(); $CI->room_id = $this->room_id;` 向库传递状态，破坏封装性且线程不安全。 | 通过方法参数或构造函数注入传递上下文。 | `$order_add_res = $this->neworderservice->createOrderWeb($param, $this->room_id, $this->pay_scene);` | | 🟠 警告 | `Index.php`<br>`public_getIndex()` | **输入未校验**：`$this->param['longitude']`、`latitude`、`family_server_id` 直接参与业务逻辑，未做类型/范围校验。 | 增加基础校验，防止非法坐标或越权查询。 | `$lat = filter_var($this->param['latitude'] ?? 0, FILTER_VALIDATE_FLOAT);`<br>`if ($lat === false || $lat < -90 || $lat > 90) throwError('坐标非法');` | | 🟡 建议 | 全局多个文件 | **违反单一职责 & 方法过长**：`public_getIndex()`、`getOrderTypeInfo()`、`get_bill_goods_info()` 均超 200 行，逻辑耦合严重。 | 按功能拆分为独立私有方法或 Service 类（如 `ShopInfoService`、`VipPriceCalculator`）。 | 提取：`private function buildIndexPageFields($res)`<br>`private function calculateVipDiscount($goods, $vipLevel)` | | 🟡 建议 | `Index.php` / `Order.php` | **模型命名与加载不一致**：混用 `ahead_xxx_model` 与 `Ahead_xxx_model`，且同一请求中重复 `$this->load->model()`。 | 统一使用 PascalCase 命名，在 `__construct()` 中预加载高频模型。 | `// 构造函数中统一加载`<br>`$this->load->model(['Ahead_merchant_wx_min_set_model', 'Ahead_yc_shop_model']);` | | 🟡 建议 | 全局 | **PSR-12 规范缺失**：缩进混用、魔法数字/字符串泛滥、注释不完整、控制流嵌套过深。 | 接入 `PHP_CodeSniffer` 或 `PHP-CS-Fixer` 自动化格式化；提取常量配置。 | `const PAY_PLATFORM_WECHAT = 1;`<br>`const ORDER_TYPE_RENEWAL = 4;` | ## 3. 总结与行动建议 ### 🚨 优先修复（P0） 1. **修复 SQL 注入**：立即将 `Ahead_yc_order_model::get_bill_goods_info()` 中的字符串拼接替换为 CI3 Query Builder (`$this->db->where()`, `$this->db->or_where()`)。 2. **清除调试代码**：全局搜索 `echo`、`var_dump`、未脱敏的 `do_log(var_export(...))` 并移除或替换为结构化日志。 3. **资金计算精度**：所有涉及金额加减乘除的逻辑，必须切换为 `bcmath` 函数或统一转为“分”进行整数运算，避免财务对账差异。 ### 🛠 重构与优化方向（P1-P2） 1. **拆分巨型方法**：将 `Index::public_getIndex()`、`Neworderservice::getOrderTypeInfo()` 按业务域拆分为独立 Service 类。例如： - `MiniIndexDataService`（首页数据聚合） - `RoomStatusService`（包厢状态与倒计时） - `OrderPricingService`（订单计价、会员折扣、服务费计算） 2. **消除 N+1 查询**： - 将 `foreach` 内的 `$this->load->model()` 和单条 `get_one()` 改为批量查询。 - 使用 `array_column()` 提取 ID 集合，通过 `where_in` 一次性获取数据，再在内存中通过 `array_combine()` 映射。 3. **规范框架使用**： - 停止使用 `$CI->xxx = $value` 传递状态，改用方法参数或依赖注入。 - 统一模型命名规范，在控制器构造函数中集中加载模型。 - 对 `$this->param` 增加统一校验层（可封装为 `BaseController::validateParams()`）。 4. **引入自动化检查**：配置 `PHP_CodeSniffer` (PSR-12) 与 `PHPStan` 静态分析，在 CI/CD 流水线中拦截低级错误与类型隐患。 > 💡 **后续建议**：当前代码处于“业务快速迭代”状态，技术债累积较多。建议在下一个迭代周期预留 **20%~30% 时间** 用于核心计费链路与账单查询模块的重构，并补充单元测试（PHPUnit）覆盖计价逻辑，确保后续迭代的安全性与可维护性。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780561171
updated_unix
1780561171
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel