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 561 from issue
id
561
repo_id
21
index
247
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260616 - 测试
content
## 自动代码审查报告 **分支**: pay-260616 **提交**: `229aad9e0
## 自动代码审查报告 **分支**: pay-260616 **提交**: `229aad9e0cc30eaad4e9498784b3b6fd12cfbcb3` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-05 14:20:10 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：该模型文件承载了大量核心业务逻辑（套餐查询、价格计算、触屏端适配、微信预订等），功能覆盖较全。但存在明显的 **SQL 注入风险**、**数组遍历删除逻辑错误**、**静态缓存跨请求污染** 以及 **框架使用不规范** 等问题。代码风格混杂，魔法数字较多，部分折扣计算逻辑存在歧义，整体可维护性与安全性亟待提升。 - **风险等级**：🔴 高（存在直接的安全漏洞与核心业务逻辑缺陷） > 📌 **框架说明**：根据目录结构（`system/`、`application/models/`）及 `$this->load->model()`、`get_instance()` 等特征，推断该项目基于 **CodeIgniter 3 (CI3)**。若 `phpci` 为内部定制框架，请结合其官方文档核对模型生命周期差异。 > ⚠️ **局限性说明**：提供的代码在 `get_book_package_list_group_by_shop` 方法处被截断，后续逻辑无法评估。以下审查基于已提供内容。 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `get_book_package_list` / `get_hot_sale_top5` | **SQL 注入漏洞**：直接将 `$param['special_merchant_id']` 和 `$shop_name` 拼接到原生 SQL 中，未做任何转义或参数绑定。 | 废弃原生 SQL 拼接，全面改用 CI 查询构造器（Query Builder），或至少使用 `$this->db->escape()`。 | `$this->db->where('shop._merchant_id', $param['special_merchant_id']);`<br>`$this->db->like('shop._name', $shop_name, 'both');` | | 🔴 严重 | `get_package_price_list` / `get_screen_list` | **逻辑缺陷：`unset($row)` 无效**：在 `foreach ($list as &$row)` 中使用 `unset($row)` 仅销毁了引用变量，**并未从原数组中移除元素**，导致过滤失效。 | 改用键值遍历或 `array_filter` 进行安全移除。 | `foreach ($list as $key => &$row) {`<br>` if ($book_arrival_time < $order_end_time) {`<br>` unset($list[$key]);`<br>` continue;`<br>` }`<br>`}` | | 🟠 警告 | 文件顶部 (第 4-5 行) | **框架反模式：顶层 `get_instance()`**：在模型文件顶层执行 `$CI = &get_instance();` 会在每次 `include` 时触发，且 CI 模型中 `$this` 本身已继承超对象，此写法冗余且易引发上下文混乱。 | 删除顶层代码。在方法内部按需使用 `$this->load->model()` 或 `$this->config->load()`。 | `// 删除以下两行`<br>`$CI = &get_instance();`<br>`$CI->load->model('Simple_model');` | | 🟠 警告 | `$_shop_id_arr` 等静态属性 | **静态属性跨请求污染**：PHP-FPM 环境下 `static` 属性会在多个 HTTP 请求间保留。不同城市/商户的查询会命中缓存，导致数据串扰。 | 改为实例属性（`protected $shop_id_arr = [];`），或在每次请求开始时显式清空。 | `protected $shop_id_arr = [];`<br>`// 方法内使用 $this->shop_id_arr` | | 🟠 警告 | `get_package_price_list` (折扣计算段) | **折扣率计算逻辑混乱**：`$goods_discount_rate / 10` 连续执行两次，且硬编码 `10`。若传入 `95`（95折），结果变为 `0.95`，但逻辑意图不清晰，易引发价格计算偏差。 | 明确折扣转换规则，提取为常量或独立方法，避免隐式除法。 | `const DISCOUNT_DIVISOR = 100;`<br>`$discount_factor = $goods_discount_rate / self::DISCOUNT_DIVISOR;` | | 🟠 警告 | `check_use_package` / `get_package_price_list` | **星期周期判断不一致**：一处使用 `LIKE '%'.$w.'%'`，另一处使用 `explode(",", $data['_week_cycle'])`。若数据库存储格式不统一，将导致套餐可用性判断错误。 | 统一星期数据存储格式（建议存为 `1,2,3` 逗号分隔），并在查询/校验时统一使用 `FIND_IN_SET` 或 `in_array`。 | `// 统一使用 FIND_IN_SET`<br>`$where['FIND_IN_SET('.$w.', info._week_cycle) >'] = 0;` | | 🟡 建议 | 全局方法命名 | **命名规范不一致**：混用驼峰命名（`getPackageInfoByIds`）与下划线命名（`get_package_price_list`），违反 CI 框架惯例及 PSR-12。 | 统一采用下划线命名法（snake_case），符合 CI3 模型规范。 | `public function get_package_info_by_ids(array $ids, ...)` | | 🟡 建议 | 多处硬编码 | **魔法数字泛滥**：`4`、`2`、`100`、`86400`、`3` 等直接写死在业务逻辑中，降低可读性与后期维护效率。 | 提取为类常量或配置文件，并添加语义化注释。 | `const PACKAGE_TYPE_GROUP_BUY = 4;`<br>`const SECONDS_PER_DAY = 86400;` | | 🟡 建议 | `get_price_set_detail` / `get_package_price_list` | **重复模型加载**：同一方法内多次 `$this->load->model()`，且部分模型（如 `Ahead_vip_model`）在多个方法中重复加载。 | 建议在 `__construct()` 中统一加载高频模型，或使用 CI 的自动加载配置。 | `public function __construct() {`<br>` parent::__construct();`<br>` $this->load->model('Ahead_vip_model');`<br>`}` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **立即修复 SQL 注入**：`get_book_package_list` 与 `get_hot_sale_top5` 中的原生 SQL 拼接必须替换为查询构造器或预处理语句。这是最高优先级的安全红线。 2. **修正数组过滤逻辑**：全局搜索 `foreach ($list as &$row)` 配合 `unset($row)` 的代码块，全部改为 `unset($list[$key])` 或使用 `array_filter`，否则套餐过滤功能形同虚设。 3. **清理静态缓存污染**：将 `self::$_shop_id_arr` 等静态属性改为实例属性，或在请求入口（如 Controller 的 `__construct`）中重置，避免多租户数据交叉。 ### 🛠 后续重构与优化方向 1. **架构解耦与职责单一**：当前 Model 承担了过多职责（价格计算、时间校验、坐标转换、商品组装）。建议： - 将 **价格计算逻辑** 抽离至独立的 `PriceCalculator` 服务类。 - 将 **时间/星期校验逻辑** 封装为 `PackageTimeValidator`。 - Model 仅保留数据查询与基础映射，符合单一职责原则（SRP）。 2. **统一查询规范**：全面弃用 `$this->db->query($sql)` 拼接，改用 CI Query Builder。不仅提升安全性，还能自动处理表前缀、字段转义，并便于后续切换数据库驱动。 3. **规范代码风格**： - 使用 `php-cs-fixer` 或 IDE 自动格式化，严格遵循 PSR-12。 - 统一方法命名、注释格式（建议采用 PHPDoc 标准），移除过期的 `//edit by nan` 注释，改用 Git 提交记录追溯。 4. **性能优化**： - `get_package_by_fields` 中存在明显的 N+1 查询隐患（先查套餐，再循环查商品）。建议改用 `JOIN` 一次性拉取，或使用 `WHERE IN` 批量查询后在 PHP 层组装。 - 高频调用的 `date()`、`strtotime()`、`hourToTime()` 应在方法入口处缓存为局部变量，避免重复计算。 > 💡 **提示**：由于代码末尾被截断，`get_book_package_list_group_by_shop` 的完整逻辑未纳入审查。建议在补全代码后，重点检查其分页逻辑、排序缓存及坐标转换函数的性能表现。如需针对特定方法进行深度重构，可提供完整片段。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780640410
updated_unix
1780640410
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel