IF..ELSE条件异常排查：MxStat=1时页面崩溃问题求助

看起来你遇到的问题是当MxStat等于1时页面崩溃，其他分支却能正常运行。我来帮你拆解可能的问题点，以及对应的修复思路：

1. 检查mxCost的类型与值合法性 #

当MxStat == 1时，你直接将mxCost赋值给mxAdj，但mxCost来自表格文本，可能存在以下隐患：

• 如果表格中对应的td:eq(6)文本为空，或者解析后不是有效数字，mxCost可能是空字符串或者非数字格式，后续计算ReturnVal = (mxAdj * InflationRate1) + mxAdj时会产生NaN，进而导致后续的Math.pow、Math.floor等操作抛出异常。
• 你对mxCost做了substring(1)的截取（应该是去掉货币符号？），但如果截取后的字符串无法转为数字（比如包含非数字字符），也会导致后续计算失败。

修复建议：
强制将mxCost转为数字类型，避免字符串参与数值计算：

// 替换原代码中处理mxCost的部分
if (mxCost) {
  // 截取后转为数字，转换失败则设为0
  mxCost = parseFloat(mxCost.substring(1, mxCost.length)) || 0;
} else {
  mxCost = 0;
}

2. 检查MxStat的计算逻辑 #

你通过MxStat = MxStat/100转换值，但如果document.getElementById("MxStat").value不是有效数字（比如空值、非数字字符串），MxStat会变成NaN。虽然NaN == 1是false，但如果用户输入的是100，MxStat会是1，这时候要确保mxCost是有效数字。

额外验证：
在使用MxStat前增加合法性检查：

var MxStat = parseFloat(document.getElementById("MxStat").value);
if (isNaN(MxStat)) {
  MxStat = 0; // 或者根据业务需求给出用户提示
} else {
  MxStat = MxStat / 100;
}

3. 排查未定义变量periods #

你的代码中使用了periods变量，但没有看到它的定义：

var ReturnPV = Math.floor(ReturnCalc/Math.pow(1+rate3,periods));

如果periods未定义，Math.pow会返回NaN，导致ReturnPV为NaN，当赋值给ReturnCondition时可能引发页面崩溃。其他分支正常可能是巧合（比如其他分支下某些计算暂时掩盖了这个问题），但MxStat=1时这个问题暴露了出来。

修复建议：
确保periods有合法的数值，比如增加检查：

if (typeof periods === 'undefined' || isNaN(periods)) {
  periods = 0; // 或者根据业务逻辑设置合理默认值
}
var ReturnPV = Math.floor(ReturnCalc/Math.pow(1+rate3,periods));

4. 分支逻辑简化（可选） #

你的else if (MxStat != 1)可以简化为else，因为只有两种互斥情况，避免冗余判断：

if (MxStat == 1){
  mxAdj = mxCost;
} else {
  var TotalCost = mxCost * 2;
  mxAdj = (TotalCost * MxStat) - mxCost;
}

调试小技巧 #

你可以在代码中添加console.log来快速定位问题，比如在if (MxStat == 1)分支前打印关键变量：

console.log('MxStat:', MxStat);
console.log('mxCost:', mxCost, typeof mxCost);
console.log('periods:', periods);

打开浏览器开发者工具的控制台，触发崩溃场景时，查看这些变量的值，就能快速锁定是哪个变量出了问题。

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