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 540 from issue
id
540
repo_id
21
index
227
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260616 - 1
content
## 自动代码审查报告 **分支**: pay-260616 **提交**: `16cb202ae
## 自动代码审查报告 **分支**: pay-260616 **提交**: `16cb202ae7a397766a9904b49881ecfd2a39951d` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-04 19:18:00 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：业务链路完整，覆盖了订单创建、支付路由、优惠券校验等核心场景。但代码存在较多历史包袱，**严重违反框架生命周期规范**，存在明显的 SQL 注入隐患、全局状态污染及并发安全风险。时间计算逻辑脆弱，命名与注释未遵循现代 PHP 规范，可维护性与扩展性较差。 - **风险等级**：🔴 高 > 📌 **框架说明**：根据目录结构（`system/`, `application/`, `BASEPATH`, `get_instance()`, `$this->load->model()` 等特征），该代码实际基于 **CodeIgniter 3.x** 架构。以下审查基于 CI3 最佳实践。若 `phpci` 为内部定制框架，请对照其特定文档调整。 ## 2. 问题详情 | 严重程度 | 文件/位置 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `Order.php` / `RoomPackage.php` 顶部 | 控制器直接 `include` 父类控制器文件 | 违反 CI 路由与自动加载机制，易导致类重复声明或生命周期错乱。应通过 `application/core/MY_Controller.php` 统一继承基类。 | `// 删除 include 语句<br>class Order extends MY_Controller { ... }` | | 🔴 严重 | `Order.php`<br>`buyRenewalPackage()` / `createOrder()` | 直接修改 CI 超全局对象属性 (`$CI->room_id = ...`) | 在 PHP-FPM 环境下，`get_instance()` 返回单例。直接赋值会导致**并发请求数据串扰**，引发严重业务错乱。应通过参数传递或使用 Session/Config。 | `$order_params['room_id'] = $this->room_id;<br>$res = $this->neworderservice->createOrderWeb($order_params);` | | 🔴 严重 | `Order.php`<br>`buyRenewalPackage()` | 支付配置日志记录敏感信息 | `do_log(var_export($chinaums_set, 1))` 会完整打印商户密钥、AppID、签名串等，存在**敏感数据泄露**风险。 | `do_log('chinaums_pay: order_id=' . $chinaums_order['order_id'] . ', type=' . ($chinaums_set['_pay_type'] ?? ''), 'buyRenewalPackage');` | | 🔴 严重 | `Ahead_shop_group_buying_coupon_model.php`<br>`Ahead_user_reward_model.php` 顶部 | 文件顶层执行 `$CI = &get_instance();` 并加载模型 | 违反面向对象原则。文件被 `include/require` 时即执行，破坏框架实例化生命周期，且在 CLI/单元测试中会直接报错。应移至 `__construct()`。 | `public function __construct() {<br> parent::__construct();<br> $this->load->model('Simple_model');<br>}` | | 🔴 严重 | `Ahead_shop_group_buying_coupon_model.php`<br>`get_gift_data()` | SQL 拼接未转义 (`FIND_IN_SET`) | `$shop_id` 直接拼接进 SQL 字符串，若未严格过滤将导致 **SQL 注入**。应使用 CI Query Builder 或 `$this->db->escape()`。 | `$shop_id_escaped = $this->db->escape($shop_id);<br>$this->db->where("(_shop_id = $shop_id_escaped OR FIND_IN_SET($shop_id_escaped, _satisfy_shop_ids))");` | | 🟠 警告 | `Ahead_user_reward_model.php`<br>`get_my_reward_list()` | 使用 `REGEXP` 进行门店匹配 | `REGEXP` 无法命中索引，导致全表扫描，数据量增长后性能急剧下降。建议改用 `FIND_IN_SET` 或规范化关联表设计。 | `// 改用 FIND_IN_SET 或 JSON_CONTAINS<br>$this->db->where("FIND_IN_SET('{$this->db->escape($shop_id)}', _satisfy_shop_ids)");` | | 🟠 警告 | `Order.php` / `RoomPackage.php` | 松散类型比较 (`==`, `in_array` 未严格匹配) | PHP 弱类型易受类型欺骗（如 `'1' == 1` 为真）。应使用严格比较 `===` 及 `in_array($val, $arr, true)`。 | `if (!in_array($from, ['1', '2', '3'], true)) { ... }<br>if ($order_type === '1') { ... }` | | 🟠 警告 | `RoomPackage.php`<br>`timePackagePayPage()` | 时间精度丢失计算 (`strtotime(date('Ymd', ...))`) | 丢失时分秒，在跨日临界点（如 `23:59:59`）易产生逻辑错误。建议使用 `DateTime` 或数学取整。 | `$start_of_day = floor($timestamp / 86400) * 86400;` | | 🟡 建议 | 全局 | 方法命名风格不统一 (驼峰 vs 下划线) | 违反 PSR-12。CI 传统推荐下划线，现代 PHP 推荐驼峰。需统一规范（建议统一为 `snake_case` 或 `camelCase`）。 | `buyRenewalPackage` → `buy_renewal_package`<br>`getActualPay` → `get_actual_pay` | | 🟡 建议 | 全局 | 魔法数字泛滥 (`1`, `2`, `3`, `4`, `-1`, `10`, `14`) | 降低可读性与可维护性。应提取为类常量或配置文件，并添加明确注释。 | `const PAY_PLATFORM_WECHAT = 1;<br>const ORDER_TYPE_RENEWAL = 4;` | > ⚠️ **局限性说明**：`Ahead_user_reward_model.php` 代码在末尾被截断，部分逻辑（如 `get_valid_coupon` 完整实现、异常捕获闭环）无法全面评估。建议补充完整代码后二次审查。 ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **消除全局状态污染**：立即移除 `$CI->xxx = $this->xxx` 的赋值方式。所有上下文数据必须通过方法参数、返回值或 Session 传递，确保请求隔离。 2. **修复 SQL 注入漏洞**：全面排查 `FIND_IN_SET`、`REGEXP`、`LIKE` 等动态拼接处，统一替换为 CI Query Builder 的 `where()`、`like()` 或 `$this->db->escape()`。 3. **规范模型生命周期**：将模型文件顶部的 `$CI = &get_instance();` 及 `$this->load->model()` 全部移入 `__construct()` 方法中。 4. **支付日志脱敏**：禁止在生产环境记录完整支付配置数组，仅保留订单号、流水号、状态码等必要追踪字段。 ### 🛠 后续重构与优化方向 - **架构解耦**：将 `Order` 与 `RoomPackage` 中重复的支付路由、金额计算、包厢状态校验逻辑抽离至独立的 `Service` 层（如 `OrderService`, `PaymentRouterService`），控制器仅负责参数接收与响应格式化。 - **数据库设计优化**：`_satisfy_shop_ids` 使用逗号分隔字符串存储严重违反第一范式。建议拆分为关联表（如 `reward_shop_relation`），或使用 MySQL `JSON` 类型配合 `JSON_CONTAINS` 查询，彻底解决 `REGEXP`/`FIND_IN_SET` 的性能瓶颈。 - **统一类型与常量**：建立 `OrderConstants.php`、`PayConstants.php` 等配置类，替换所有魔法数字。在入口层使用 CI 的 `Form_validation` 或自定义 DTO 对 `$this->param` 进行强类型校验。 - **时间处理标准化**：引入 `Carbon` 或 PHP 原生 `DateTimeImmutable` 处理跨天、时段计算，避免 `strtotime(date())` 带来的精度丢失与时区隐患。 - **代码规范落地**：配置 `PHP_CodeSniffer` (PSR-12) 与 `PHPStan` 到 CI/CD 流水线，强制拦截命名不规范、未定义变量、类型不匹配等问题。 如需针对特定模块（如支付路由逻辑、优惠券校验算法）进行深度重构设计，可提供完整上下文，我将输出详细的重构方案与类图。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780571880
updated_unix
1780571880
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel