SQLite Web: gitea.db

id

Primary key. INTEGER NOT NULL

repo_id

INTEGER

index

INTEGER

poster_id

INTEGER

original_author

TEXT

original_author_id

INTEGER

name

TEXT

content

## 自动代码审查报告

**分支**: pc-260519
**提交**: `4e7306a6a6644d1331592babacf48ec477325c22`
**提交人**: caihongyuchy (1091045324@qq.com)
**时间**: 2026-05-19 10:06:32

---

## 📋 审查摘要
- **变更文件数**: 1
- **严重问题**: 0
- **高危问题**: 2
- **中危问题**: 2
- **建议优化**: 3

## 🐛 发现的问题

### [逻辑 BUG] `toDecimal2` 方法中变量重复声明与参数误用导致精度计算异常
- **严重程度**: 高危
- **文件**: web/youc_business_operate_pc/src/main.js
- **行号**: 约 65-68 行
- **问题描述**: 函数内先声明 `var f = parseFloat(x);`，随后又声明 `var f = Math.round(x * 100) / 100;`。此处不仅重复声明了变量 `f`，且第二处计算错误地使用了原始参数 `x` 而非已转换的浮点数 `f`。若传入非数字字符串，`Math.round(x * 100)` 会返回 `NaN`，导致后续逻辑失效。此外，JS 中浮点数直接乘除存在精度丢失风险。
- **修复建议**: 移除重复声明，修正变量引用，并推荐使用原生 `Number.prototype.toFixed()` 保证精度与兼容性。
  ```javascript
  Vue.toDecimal2 = function(x) {
    var f = parseFloat(x);
    if (isNaN(f)) return false;
    return Number(f).toFixed(2);
  }
  ```

### [安全隐患] Axios `transformRequest` 全局强制序列化可能破坏文件上传与特殊请求
- **严重程度**: 高危
- **文件**: web/youc_business_operate_pc/src/main.js
- **行号**: 约 118 行
- **问题描述**: `axios.defaults.transformRequest` 被全局重写为 `return JSON.stringify(data)`。当业务需要上传文件（`FormData`）或发送 URL 编码数据时，此配置会将 `FormData` 对象强制序列化为 `"[object Object]"` 字符串，导致后端无法解析，引发请求失败或数据损坏。
- **修复建议**: 删除全局 `transformRequest` 配置，依赖 Axios 默认的序列化机制。若需特定接口使用 JSON，应在具体请求中配置 `headers: { 'Content-Type': 'application/json' }`，或通过 Axios 拦截器按需处理。

### [代码质量] 全局工具函数直接挂载到 `Vue` 构造函数违反 Vue 2 规范
- **严重程度**: 中危
- **文件**: web/youc_business_operate_pc/src/main.js
- **行号**: 约 45-75 行
- **问题描述**: `Vue.ctUrl`、`Vue.request_header`、`Vue.timeoutfun`、`Vue.accMul`、`Vue.toDecimal2` 直接作为静态属性挂载在 `Vue` 构造函数上。这不符合 Vue 2 的插件/原型扩展规范，会造成全局命名空间污染。在组件内调用时需使用 `Vue.xxx` 而非 `this.$xxx`，增加维护成本且不利于单元测试。
- **修复建议**: 将纯函数工具移至 `src/utils/math.js` 独立导出。若需在组件实例中全局使用，应挂载到 `Vue.prototype`（如 `Vue.prototype.$accMul = function...`），或通过 Vue Plugin/Mixin 规范注入。

### [逻辑 BUG] `request_header.merchant_id` 初始化可能为 `undefined` 引发后续请求异常
- **严重程度**: 中危
- **文件**: web/youc_business_operate_pc/src/main.js
- **行号**: 约 24 行
- **问题描述**: `Vue.request_header.merchant_id = store.state.mercid` 在应用启动时同步读取 Vuex 状态。若 `store` 初始化存在异步逻辑，或 `merc` 字段依赖接口返回尚未就绪，此处将赋值为 `undefined`。后续所有请求头将携带非法值，可能导致后端鉴权失败。
- **修复建议**: 使用 Axios 请求拦截器动态读取 `store.state.mercid`，确保每次请求都能获取最新状态，避免初始化竞态条件。

## ✅ 代码亮点
- 采用了按需引入 Element UI 组件的方式，有效减小了 Webpack 打包体积。
- 使用 `VueAxios` 插件将 Axios 集成到 Vue 实例中，便于组件内统一调用。
- 针对本地开发环境 (`localhost`) 和生产环境做了 URL 动态切换处理，提升了多环境部署的适配性。
- `Vue.accMul` 实现了基础的浮点数乘法精度补偿逻辑，体现了对前端数值计算痛点的关注。

## 📝 总体建议
1. **架构职责分离**：当前 `main.js` 承担了过多职责（URL 路由解析、全局配置、数学工具、UI 库注册、网络库配置）。建议拆分为独立模块：`src/config/env.js`（环境配置）、`src/utils/math.js`（工具函数）、`src/plugins/element.js`（UI 注册）、`src/plugins/axios.js`（网络实例），保持入口文件仅负责应用挂载。
2. **跨文件引用验证说明**：提供的「项目结构」仅包含 PHP CodeIgniter 后端系统文件，**未包含前端 Vue 项目目录结构**。因此无法验证 `./App.vue`、`./router`、`./store` 及 `./components/global` 的实际存在性与导出规范。请确保这些文件路径符合 Vue CLI 标准，且 `store` 中正确初始化并导出了 `merc` 状态字段。
3. **状态与请求头解耦**：全局请求头中的动态字段（如 `merchant_id`）强烈建议通过 Axios 拦截器注入，而非在 `main.js` 初始化时静态赋值。这能彻底解决状态不同步、多账号切换未刷新请求头等问题。
4. **类型安全与规范**：建议引入 TypeScript 或完善 JSDoc 注释，对 `Vue.timeoutfun(datas, _this)` 等回调函数的参数类型进行约束，避免 `_this` 上下文传递错误或 `datas` 类型不匹配导致的运行时异常。

---
*此 Issue 由代码审查服务自动创建*

TEXT

milestone_id

INTEGER

priority

INTEGER

is_closed

INTEGER

is_pull

INTEGER

num_comments

INTEGER

ref

TEXT

deadline_unix

INTEGER

created_unix

INTEGER

updated_unix

INTEGER

closed_unix

INTEGER

is_locked

INTEGER NOT NULL (default 0

content_version

INTEGER NOT NULL (default 0

time_estimate

INTEGER NOT NULL (default 0

Update row 165 in issue