array.splice从剩余元素中删除值

Question

我遇到了array.splice的这种奇怪的副作用,并将代码提炼到了重新创建所需的最小值.是的,其中大部分可以在array.filter的一行上完成,但我对是否犯了错误或是否还有其他事情感兴趣.

var array = [];

for (var i = 0; i < 10; i++) {
  array.push({
    value: i
  });
}

array.forEach(function(item, index, intArray) {
  if (item.value % 2 == 1) {
    item.odd = true;
  } else {
    item.odd = false;
  }

  if (item.odd) {
    console.log("Removing " + item.value);
    intArray.splice(index, 1);
  }

});

console.log(array);

运行在奇数元素此JavaScript结果如预期被去除,但它也除去item.odd值6项2,4和8卸下intArray.splice线带回奇数阵列元件,但也带来了返回所有元素的item.odd值.

我在FF和Chrome中测试了这个.即使只将项传递给回调,该行为仍然存在,索引通过array.indexOf计算,并从循环外部引用该数组.

Answer 1

我认为当你在每个奇数处拼接数组时,forEach最终跳过下一个项目,这是一个偶数.所以这些项目根本没有修改.

var array = [];

for (var i = 0; i < 10; i++) {
  array.push({
    value: i
  });
}

array.forEach(function(item, index, intArray) {
  console.log(item); // only prints out 0, 1, 3, 5, 7, 9

  if (item.value % 2 == 1) {
    item.odd = true;
  } else {
    item.odd = false;
  }

  if (item.odd) {
    console.log("Removing " + item.value);
    intArray.splice(index, 1);
  }

});

console.log(array);

换句话说,forEach只访问每个索引一次.所以说它到达项目1,它在索引1处.它删除项目1.项目2现在在索引1处.但是索引1已经被访问过,所以它转移到索引2处的项目,现在是项目3.