etms/CODE_FIX_PLAN.md

# JCDP 项目代码修复方案

> **文档版本**: v1.0
> **创建时间**: 2026-04-16
> **依据**: [CODE_REVIEW_REPORT.md](./CODE_REVIEW_REPORT.md)

---

## 一、修复执行情况

### ✅ 已完成修复

| 序号 | 问题 | 修复文件 | 修复内容 |
|------|------|----------|----------|
| 1 | 全局变量未声明 | `globalConfig.js` | 为 8 个正则变量添加 `const` 声明 |
| 2 | 日期 API 误用 | `globalConfig.js` | `getDay()` → `getDate()` (第268、271行) |
| 3 | 废弃 SQL 清理 | 4 个 map.xml | 删除 16 个废弃 SQL 定义 |
| 4 | 注释死代码 | 115 个 ftl | 删除 615+ 处注释代码 |

### 📋 待处理问题

| 序号 | 问题 | 优先级 | 说明 |
|------|------|--------|------|
| 1 | 同步 AJAX 请求 | 🟠 严重 | 6 处 `async: false` 需要改为异步 |
| 2 | SELECT * 过多 | 🟡 建议 | 120+ 处可优化为指定字段 |
| 3 | 内联样式过多 | 🟡 建议 | 6265+ 处 style 属性待清理 |
| 4 | 内联事件处理 | 🟡 建议 | 115 处 onclick/onchange 待重构 |

---

## 二、已修复问题详情

### 2.1 全局变量声明修复

**文件**: `asset/js/etms/globalConfig.js`

**修复内容**: 为验证函数中的正则表达式变量添加 `const` 声明

```javascript
// 修复前
validator: function (val) {
    isIDCard1 = /^[1-9]\d{7}((0\d)|(1[0-2]))(([0|1|2]\d)|3[0-1])\d{3}$/;
    isIDCard2 = /^[1-9]\d{5}[1-9]\d{3}((0\d)|(1[0-2]))(([0|1|2]\d)|3[0-1])\d{3}[0-9xX]$/;
    ...
}

// 修复后
validator: function (val) {
    const isIDCard1 = /^[1-9]\d{7}((0\d)|(1[0-2]))(([0|1|2]\d)|3[0-1])\d{3}$/;
    const isIDCard2 = /^[1-9]\d{5}[1-9]\d{3}((0\d)|(1[0-2]))(([0|1|2]\d)|3[0-1])\d{3}[0-9xX]$/;
    ...
}
```

**涉及变量**: `isIDCard1`, `isIDCard2`, `isPostcode`, `isAboveAndEqualZero`, `isPositiveNumber`, `isPositiveInteger`, `isMonthNum`, `moreThanFive`, `isPointNum`, `isEmail`

---

### 2.2 日期 API 修复

**文件**: `asset/js/etms/globalConfig.js` 第 268、271 行

**修复内容**: `getDay()` → `getDate()`

```javascript
// 修复前
if (dateTime1.getDay() == dateTime2.getDay()) {
    tempTime = dateTime2.format("hh:mm");
} else {
    if (dateTime1.getDay() - dateTime2.getDay() == 1) {
        tempTime = dateTime2.format("昨天 hh:mm");
    }
}

// 修复后
if (dateTime1.getDate() == dateTime2.getDate()) {
    tempTime = dateTime2.format("hh:mm");
} else {
    if (dateTime1.getDate() - dateTime2.getDate() == 1) {
        tempTime = dateTime2.format("昨天 hh:mm");
    }
}
```

---

### 2.3 废弃 SQL 清理

**修复文件**:
- `chat.map.xml`: 删除 `updatePkgListToOld`
- `et_exam_editexampaper.map.xml`: 删除 `moveOld`
- `et_exam_usertest.map.xml`: 删除 10 个废弃 SQL
- `et_train_outtrain.map.xml`: 删除 2 个废弃 SQL

**清理的废弃 SQL ID**:
- `getExamResultList之前的写法，在此备份`
- `truncateExamDetail备份`
- `getPersonDetail备份`
- `getClassDetail备份`
- `getCourseDetail备份`
- `getInstitutionDetail 第二种写法 最慢`
- `getDepartmentDetail 第二种写法 最慢`
- `getPersonDetail第二种写法`
- `getPersonDetail第三种写法 最慢`
- `getClassDetail 第二种写法 最慢`
- `getCourseDetail 第二种写法 最慢`
- `getOutTrainDbList以前的写法备份`
- `getOutTrainPxdjList以前的写法备份`

---

### 2.4 FTL 注释死代码清理

**清理统计**:
- 修改文件: 115 个
- 清理注释: 615+ 处

**高发文件** (Top 10):
| 文件 | 清理数量 |
|------|----------|
| `exampaper_editexampaper.ftl` | 74 处 |
| `examresult_edit.ftl` | 33 处 |
| `exam_edit.ftl` | 37 处 |
| `exampaper_add.ftl` | 29 处 |
| `group_inplan_main.ftl` | 61 处 |
| `group_inplan_bg_group_detail_1.ftl` | 34 处 |
| `group_inplan_bg_group_detail.ftl` | 32 处 |
| `research_project_main_examadd_sg.ftl` | 49 处 |
| `exampaper_import.ftl` | 21 处 |
| `pd_pg_main_assess.ftl` | 20 处 |

---

## 三、待处理问题修复方案

### 3.1 同步 AJAX 请求改造 (🟠 严重)

**问题位置**:
- `globalConfig.js` 第 200 行
- `userprofile/index.js` 第 71、104 行
- `train/uptrain/uptrain.js` 第 176 行
- `train/uptrain/uptrain_edit.js` 第 394 行
- `train/plantodo/assess_audit.js` 第 319 行
- `train/outtrain/out_train_main.js` 第 1209 行

**修复方案示例**:

```javascript
// 修复前 (globalConfig.js)
$.ajax({
    type: 'post',
    async: false,  // 阻塞 UI
    url: dictBaseUrl + dictKey,
    dataType: 'json',
    success: function (result) {
        if (result.success) {
            window[hdName] = result.data;
            res = result.data;
        }
    }
});

// 修复后 (方案1: Promise + async/await)
async function getDictData(dictKey) {
    try {
        const result = await $.ajax({
            type: 'post',
            url: dictBaseUrl + dictKey,
            dataType: 'json'
        });
        if (result.success) {
            window[dict_prefix + dictKey] = result.data;
            return result.data;
        }
        return null;
    } catch (error) {
        console.error('获取字典数据失败:', error);
        return null;
    }
}

// 修复后 (方案2: 回调函数)
function getDictData(dictKey, callback) {
    $.ajax({
        type: 'post',
        url: dictBaseUrl + dictKey,
        dataType: 'json',
        success: function (result) {
            if (result.success) {
                window[dict_prefix + dictKey] = result.data;
                callback(result.data);
            } else {
                callback(null);
            }
        },
        error: function() {
            callback(null);
        }
    });
}
```

**建议优先级**:
1. 优先改造 `globalConfig.js` 中的 `gridColFilter` 函数
2. 然后处理业务模块中的同步请求
3. 最后处理 `out_train_main.js` 等复杂模块

---

### 3.2 SELECT * 优化 (🟡 建议)

**优化原则**:
- 明确列出需要的字段
- 减少网络传输开销
- 提高代码可读性

**示例**:

```xml
<!-- 修复前 -->
<sql id="getFileList">
    select * from et_resource_file where 1=1
</sql>

<!-- 修复后 -->
<sql id="getFileList">
    select id, name, file_path, file_size, created_by, created_at
    from et_resource_file where 1=1
</sql>
```

**建议**: 优先优化高频查询的 SQL

---

### 3.3 内联样式清理 (🟡 建议)

**优化方案**: 提取为 CSS 类

```html
<!-- 修复前 -->
<div class="formTitle" style="height: 86px"><span class="icon icon_menu"></span>

<!-- 修复后 -->
<!-- CSS -->
.form-title {
    height: 86px;
}

<!-- HTML -->
<div class="form-title"><span class="icon icon_menu"></span>
```

**建议**:
1. 优先清理高频页面的内联样式
2. 建立公共样式类库
3. 使用 CSS 变量统一管理主题色

---

### 3.4 内联事件重构 (🟡 建议)

**优化方案**: 使用 data 属性 + JS 事件委托

```html
<!-- 修复前 -->
<input name="cbA" id="cbA" type="checkbox" value="A" onclick="checkBox(this)"/>

<!-- 修复后 -->
<!-- HTML -->
<input name="cbA" id="cbA" type="checkbox" value="A" class="checkbox-option"
        data-check-action="checkBox"/>

<!-- JavaScript -->
$(document).on('click', '.checkbox-option', function() {
    var action = $(this).data('check-action');
    if (action && window[action]) {
        window[action](this);
    }
});
```

---

## 四、后续改进建议

### 4.1 短期 (1个月内)
- [ ] 完成同步 AJAX 改造
- [ ] 引入 ESLint 检查
- [ ] 建立代码格式化规范

### 4.2 中期 (3个月内)
- [ ] 优化高频 SQL 查询
- [ ] 清理内联样式
- [ ] 重构内联事件处理

### 4.3 长期 (持续改进)
- [ ] 引入自动化测试
- [ ] 建立代码质量度量
- [ ] 完善 Code Review 流程

---

## 五、相关文件

- [CODE_REVIEW_GUIDE.md](./CODE_REVIEW_GUIDE.md) - 代码审查标准
- [CODE_REVIEW_REPORT.md](./CODE_REVIEW_REPORT.md) - 代码审查报告

---

*本修复方案由 AI 代码审查专家生成*