js关于foreach循环中使用splice问题

生活中我发现很多喜欢用foreach去循环数组，然后会看到在循环中做各种各样的操作来满足我们的日常需求，最近就遇到一个比较常见的但是又有很多人没有避免的一个小坑~ 首先在这里澄清一下，没有不好的代码只有适合的场景~

• 场景是这样的:

1、循环arr将item为1的元素从数组中删除。

let arr = [1,1,2];
arr.forEach((item,index,arr) =>{
  if(item == 1 ) {
   arr.splice(index,1)
}
})
console.log(arr); //[1,2]

2、结果：循环之后打印[1,2]，还有一个1无法删除。

3、原因分析：

1. 第一次forEach循环，arr是[1, 1, 2]，index是0，item是1，if条件成立，使用splice删除了item1，arr变成[1, 2]
2. 第二次forEach循环，arr是[1, 2]，index是1，item是2，if条件不成立，使用splice无法删除了第二个重复的1

原因是使用splice容易使数组的index乱序。

解决方法

• 使用filter()方法筛选条件符合条件的元素，去除不符合条件的元素。

1、filter()应用到数组上（修改上面的例子，去除为1的元素）。

let arr = [1,1,2];
arr = arr.filter(item => {
    return item !=1;
})
console.log(arr); //[2]

2、filter()应用到数组对象上（去除对象数组num为0的产品）。

let arr = [
      {
        productId: 1,
        productName: '电视机',
        num: 20
      }, {
        productId: 2,
        productName: '洗衣机',
        num: 0
      }, {
        productId: 3,
        productName: '冰箱',
        num: 50
      }, {
        productId: 4,
        productName: '空调',
        num: 0
      },
    ]
    arr = arr.filter(item => {
      return item.num != 0
    })
    console.log(arr) 
    // [
    // {productId: 1, productName: "电视机", num: 20}, 
    // {productId: 3, productName: "冰箱", num: 50}
    // ]

总结：

请记住：
在使用map、forEach等链式调用的方式去遍历数组集合时,不要在循环体中移除数组集合的成员,简单来说就是不要减少数组的长度,否则数组的指针会指向错误！！！