为什么JavaScript中for of循环统计数组唯一元素时出现异常?
代码问题原因分析
你给出的测试数组[9, 8, 7, 6, 5, 4, 3, 2, 1, 10]所有元素均为唯一值,按统计唯一元素的预期应该返回10,但你的代码实际返回0,不符合预期的核心原因如下:
- 核心逻辑和目标需求完全背离:你要统计数组的唯一元素数量,但现有代码的
sum仅在「当前元素在数组的counter索引之后还存在相同值」的时候才累加,统计的是重复出现的元素次数,而非唯一元素数量。 counter变量的增量逻辑错误:counter仅在满足if条件的时候才会+1,你给出的测试数组中没有任何重复元素,if条件始终不成立,counter会一直停留在初始值1,所有arr.includes(a, counter)判断都会因为从索引1开始找不到对应遍历值返回false,最终sum始终为0。- for of循环本身不存在问题:你说移除for of后运行正常,大概率是你更换循环写法的时候同时修改了内部逻辑,遍历方式本身不会导致逻辑错误。
正确实现参考
如果要统计数组唯一元素数量,最简单的实现是直接使用ES6的Set:
function countIdentic(arr) { return new Set(arr).size }
如果需要手动遍历实现,逻辑应该是记录已出现的元素,遇到未出现过的元素就累加计数:
function countIdentic(arr) { let sum = 0 const seen = new Set() for (let a of arr) { if (!seen.has(a)) { sum++ seen.add(a) } } return sum }
内容的提问来源于stack exchange,提问作者Carl Parker
相关产品推荐
相关产品推荐

