JavaScript实现表格行累加计算时多行参与运算结果错误问题排查

问题根因 #

你的代码存在3个核心错误，直接导致多行计算结果不符合预期：

• 逻辑遗漏：仅判断了每行mins+i字段非空，计算逻辑完全没有用到该字段的数值，所有行都复用了全局的time字段值，如果你的需求是按每行单独的时长计算，该逻辑完全不匹配
• 运算符优先级错误：如果你的业务规则是**(系数+固定值)时长*，现有写法t*a1+a2会先执行乘法再执行加法，运算顺序和预期不符，结果必然出错
• 边界容错缺失：空值、非数字输入转换为Number类型时会得到NaN，累加后最终结果会变成NaN，你的代码没有处理这类异常场景

修复后代码 #

var total = 0;
// 统一存储配置系数，避免重复冗余的变量定义
var insConfig = {
    o1: {
        rate: Number(this.getField("o1i").value) || 0,
        fixed: Number(this.getField("o1iother").value) || 0
    },
    o2: {
        rate: Number(this.getField("o2i").value) || 0,
        fixed: Number(this.getField("o2iother").value) || 0
    },
    o3: {
        rate: Number(this.getField("o3i").value) || 0,
        fixed: Number(this.getField("o3iother").value) || 0
    },
    o4: {
        rate: Number(this.getField("o4i").value) || 0,
        fixed: Number(this.getField("o4iother").value) || 0
    },
    o5: {
        rate: Number(this.getField("o5i").value) || 0,
        fixed: Number(this.getField("o5iother").value) || 0
    }
};
// 若要使用全局time字段保留该行，要使用每行单独的mins值就删除该行
var timeVal = Number(this.getField("time").value) || 0;

for(var i=1; i<=15; i++){
    var currentIns = this.getField("ins."+i).valueAsString;
    var currentMins = this.getField("mins"+i).valueAsString;
    // 过滤无效行
    if(!currentIns || !currentMins || !insConfig[currentIns]) continue;
    // 若要使用每行的mins值，把timeVal替换为 Number(currentMins)||0
    var rowResult = timeVal * insConfig[currentIns].rate + insConfig[currentIns].fixed;
    // 如果业务规则是时长*(系数+固定值)，就改用下面这行代码
    // var rowResult = timeVal * (insConfig[currentIns].rate + insConfig[currentIns].fixed);
    total += rowResult;
}
event.value = total || 0;

调整说明 #

• 用配置表替代多分支if判断，逻辑更清晰，后续新增设备类型时只需要修改配置即可
• 所有数值转换都增加了容错处理，异常输入默认取0，避免出现NaN无效结果
• 标注了两处可调整的逻辑位置，你可以根据实际业务规则灵活修改

内容的提问来源于stack exchange，提问作者missy miss