JavaScript测验计数中success变量莫名变为NaN问题求助

看起来你遇到的问题是触发.gotoresult操作时，success变量意外变成了NaN，导致结果判断失效。结合你描述的前几步弹窗计数正常的情况，我来帮你分析可能的原因和修复方案：

最可能的原因：变量作用域冲突或非数字操作 #

1. 局部变量覆盖闭包内变量 #

检查你的.gotoresult点击事件处理代码，是否重新声明了success变量（比如写了var success或者let success）。如果是这样，这个局部变量会覆盖你在$(document).ready开头定义的success，初始值为undefined，后续对它进行++或其他运算就会得到NaN。

错误示例：

$('.gotoresult').on('click', function() {
  var success; // 重新声明局部变量，覆盖外层的success
  alert(success); // 弹出NaN
});

2. 对success进行了非数字运算 #

如果在.gotoresult的逻辑里，你把success和非数字值（比如undefined、无效字符串、null）进行运算，也会导致NaN。比如：

// 假设correct属性获取失败返回undefined
success += $(someElement).attr('correct') === 'true' ? 1 : undefined;

修复步骤 #

步骤1：排查.gotoresult的代码逻辑 #

• 删除.gotoresult事件里任何重新声明success的代码，确保使用的是$(document).ready开头定义的变量。
• 在.gotoresult事件最开头添加调试代码，确认变量状态：

  $('.gotoresult').on('click', function() {
    console.log('当前success值:', success);
    console.log('当前success类型:', typeof success);
    // 后续逻辑...
  });
  

通过控制台输出确认触发时success的状态，排除是否在进入这个事件前变量就已被污染。

步骤2：确保所有修改success的操作都是数字运算 #

在所有判断correct并累加的地方，显式保证累加的是数字，避免类型转换错误：

// 替换原来的success++写法，确保类型安全
if ($(this).attr('correct') === 'true') {
  success = Number(success) + 1; // 显式转换为数字后累加
}

这样即使success意外变成字符串，也能转换回数字进行运算。

步骤3：统一变量初始化和维护 #

可以把success变量的初始化放在更清晰的位置，全程避免非数字赋值：

$(document).ready(function() {
  let success = 0; // 使用let更清晰，避免变量提升问题

  $('.gotostep1').on('click', function() {
    success = 0; // 重置计数
    if ($(this).attr('correct') === 'true') {
      success += 1;
    }
    alert(success);
    $(this).parent().hide();
    $('.step1').show();
  });

  // 其他步骤的事件处理...

  $('.gotoresult').on('click', function() {
    console.log('最终success计数:', success);
    if (success >= 3) { // 假设及格线是3
      $('.win').show();
    } else {
      $('.lost').show();
    }
  });
});

额外建议 #

• 尽量使用console.log()代替alert()调试，不会打断流程，还能看到更详细的变量信息。
• 可以添加一个类型检查函数，确保success始终是数字：

  function ensureNumber(num) {
    return typeof num === 'number' && !isNaN(num) ? num : 0;
  }
  // 使用时：
  success = ensureNumber(success) + 1;
  

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