为什么 javascript forEach 循环不使用 splice 迭代完整数组

Question

我在玩 javascript.splice()和.forEach()方法，代码工作正常，但没有按预期提供输出。

问题是.forEach()运行半阵列然后退出。

我期望的输出是

[['Apple'],['Banana'],['Mango'],['Pomegranate']]

但输出是

[['Apple'],['Banana']]

我无法弄清楚它为什么会这样。从mdn

它说该 .splice()方法更新数组的长度属性。那么它应该可以正常工作。

有什么办法可以让它像预期的那样工作。

出于某种原因，我只是不想使用.map()或任何其他功能。

const fruits = ['Apple', 'Banana', 'Mango', 'Pomegranate']

const fruitsList = []

function removeArrayItem(item,index){

    let fruit = fruits.splice(0,1)

    fruitsList.push(fruit)

}

fruits.forEach(removeArrayItem)

console.log(fruitsList)

Answer 1

正如MDN 文档中非常清楚地描述的那样，该.forEach()方法在迭代开始之前确定将涉及的元素。如果元素被添加到数组中，它们将不会被包含在迭代中。如果删除元素，则将跳过后续元素。基本上有一个用于选择每个元素的内部索引，因此如果数组变短，则无法相应地调整该索引。最重要的是，您编写的代码使得.forEach()回调忽略它的参数。它总是只去掉fruits数组的第一个元素。因此，在第一次迭代时，您会得到“Apple”，而数组现在只有三个元素。下一个迭代的元素将是“Mango”，而不是“Banana”，但因为代码没有注意item你得到源数组的第一个新元素。之后，数组只有两个项目，因此迭代终止。您将看到与此代码完全相同的效果，这可能更清晰，因为该机制是可见的：const fruits = ['Apple', 'Banana', 'Mango', 'Pomegranate']const fruitsList = []for (let index = 0, len = fruits.length; index < len; index++) {&nbsp; if (index in fruits) {&nbsp; &nbsp; &nbsp; let fruit = fruits.splice(0,1)&nbsp; &nbsp; &nbsp; fruitsList.push(fruit)&nbsp; }}该if循环中的测试说明了一个重要的.forEach()行为：它跳过空数组的索引（即从未初始化为值的索引）。从 中.slice()删除元素的调用fruits，因此在前两次迭代之后，它甚至不会再尝试执行任何.slice()调用，因为索引 2 和 3 将为空。这样做的正确方法，如果它是真正的代码，将是const fruits = ['Apple', 'Banana', 'Mango', 'Pomegranate']const fruitsList = fruits.map(function(fruit) {&nbsp; return [fruit];});fruits = []; // if you really want to empty it out（而且=>函数会更简洁。）

Answer 2

Splice 修改源数组。splice() 方法通过删除或替换现有元素和/或在适当位置添加新元素来更改数组的内容。来源：https&nbsp;:&nbsp;//developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/splice这意味着当splice()应用于forEach()正在迭代的数组时，它被修改（变短），因此一段时间后它没有元素可以迭代。

Answer 3

如果您看到每次迭代后所拥有的是从水果中删除的数组，并且您将该数组推送到新列表中，那么您正在以错误的方式改变原始数组。使用这种方法，您将保持索引完好无损，在拼接突变期间不会发生变化。const fruits = ['Apple', 'Banana', 'Mango', 'Pomegranate']const fruitsList = []function removeArrayItem(item,index){&nbsp; &nbsp; let fruit = fruits.splice(index,1,'')&nbsp; &nbsp; fruitsList.push(fruit)}fruits.forEach(removeArrayItem)console.log(fruitsList)https://codepen.io/psyNiiM/pen/JjjNRaJT