CodeWars练习中JavaScript函数致Chrome崩溃，请求排查原因

嘿，我一眼就揪出问题根源了——你的代码里藏着一个无限循环，这就是浏览器崩溃、完全没输出的罪魁祸首！

问题出在哪？ #

看你循环里的这行代码：

for (var i = 0; i < str.length; i + 2)

这里的i + 2只是一个计算表达式，根本没有改变变量i的值！也就是说，i会一直停留在0，循环条件i < str.length永远成立，循环会无休止地跑下去，直到浏览器把内存耗光，直接崩溃。

而且你原代码里的奇偶分支逻辑还有冗余，甚至会导致重复添加字符的问题（不过无限循环已经让这个问题没机会暴露了）。

修正后的代码 #

我帮你简化并修复了逻辑，同时解决了所有问题：

function solution(str) {
  var result = [];
  // 统一处理所有情况，不用分奇偶写两个循环
  for (var i = 0; i < str.length; i += 2) {
    // 每次从当前位置取2个字符，若只剩1个则只取到1个
    var currentPair = str.substr(i, 2);
    // 如果是奇数长度的最后一对，补下划线
    if (currentPair.length === 1) {
      currentPair += '_';
    }
    result.push(currentPair);
  }
  return result;
}

关键改动说明 #

1. 修复循环递增逻辑：把i + 2改成i += 2，这才是让i每次增加2的正确赋值操作，循环才能正常结束。
2. 简化分支逻辑：不用分开处理奇偶长度，一次循环就能搞定——每次取2个字符，最后如果只剩1个字符，直接补下划线即可。
3. 避免重复添加：原奇数分支里会额外push一次最后一个字符加下划线，现在合并逻辑后不会出现重复问题。

测试验证 #

• 输入"abc"，返回["ab", "c_"]，符合要求
• 输入"abcd"，返回["ab", "cd"]，正确无误

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