移除冗余代码后JavaScript计算两个最小正整数之和功能异常问题排查

你遇到的问题其实是JavaScript数组方法的一个常见陷阱——sort()是原地排序，会直接修改原数组，而你删除const m后，在filter的回调里多次调用sort()，导致数组在遍历过程中被反复修改，最终逻辑彻底混乱。

原代码的执行逻辑（有const m时） #

原代码里的const m = numbers.sort( (a, b) => a - b )[2];这行看似冗余，其实做了关键的前置操作：

• 它先对numbers数组执行原地升序排序，此时numbers已经变成了从小到大排列的数组。
• 之后你在filter的回调里再次调用numbers.sort(...)，但这时候数组已经是有序的了，sort()不会对它做任何修改（或者说修改可以忽略），所以每次回调里的numbers.sort(...)[2]都是同一个值——排序后数组的第三个元素。
• 这时候filter遍历的是已经排好序的数组，筛选出所有小于第三个元素的数（也就是前两个最小的数），求和结果自然正确。

删除const m后的执行逻辑（出错的核心原因） #

当你删除const m后，sort()第一次被调用是在filter的回调函数里，而filter会遍历数组的每个元素，每次回调执行都会调用一次sort()：

1. 假设原数组是[5,3,1,2,4]，filter开始遍历第一个元素（初始是5），此时回调里执行numbers.sort(...)，数组被原地改成[1,2,3,4,5]，然后取索引2的元素3，判断5 < 3？结果是false，5被排除。
2. 接下来filter遍历第二个元素，但此时数组已经被修改成排序后的数组了，第二个元素变成了2，回调里再次调用sort()（但数组已经有序，无变化），判断2 < 3？结果是true，2被加入结果数组。
3. 后续遍历的元素是3、4、5，都不满足v < 3的条件，最终结果数组只有[2]，求和得到2，自然和预期结果不符。

简单来说，删除const m后，sort()在filter遍历过程中修改了原数组，导致filter遍历的元素和你预期的完全不一样，逻辑直接乱了。

更简洁高效的修正方案 #

其实你完全不需要多次调用sort()，只需要排序一次，然后取前两个元素求和即可，这才是最合理的写法：

function sumTwoSmallestNumbers(numbers) {
  // 先排序，若不想修改原数组可以先拷贝一份
  const sorted = [...numbers].sort((a, b) => a - b);
  return sorted[0] + sorted[1];
}

如果允许修改原数组，也可以直接排序后取值：

function sumTwoSmallestNumbers(numbers) {
  numbers.sort((a, b) => a - b);
  return numbers[0] + numbers[1];
}

这样既避免了冗余代码，又不会出现逻辑错误，效率也更高（只排序一次）。

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