etms/CODE_FIX_PLAN.md

JCDP 项目代码修复方案

文档版本: v1.0 创建时间: 2026-04-16 依据: 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 声明

// 修复前
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()

// 修复前
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 行

修复方案示例:

// 修复前 (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 * 优化 (🟡 建议)

优化原则:

• 明确列出需要的字段
• 减少网络传输开销
• 提高代码可读性

示例:

<!-- 修复前 -->
<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 类

<!-- 修复前 -->
<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 事件委托

<!-- 修复前 -->
<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_REPORT.md - 代码审查报告

---

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