Node.js过滤文本文件时for循环在首次匹配后退出是什么原因？

问题原因 #

• 核心错误是Array.splice()方法的参数使用错误：splice()接收的第一个参数是删除起始下标，第二个参数是要删除的元素数量。你代码里只传了起始下标index，没有传删除数量，此时会默认删除从起始下标开始到数组末尾的所有元素，所以第一次匹配到下标74的元素时，直接把下标74到末尾的所有元素都删了，数组长度直接变为74，此时循环变量i自增后变成75，不再满足i < list.length的条件，循环直接终止。
• 额外冗余问题：let index = list.indexOf(list[i])完全没有必要，当前遍历的下标就是i，如果数组内有重复的行内容，indexOf会返回第一个匹配内容的下标，反而会导致错删其他位置的元素。
• 正向遍历删除数组元素还存在下标偏移问题：就算你修正了splice的参数写splice(i, 1)，每删一个元素后后面的元素都会往前移一位，下一次循环i自增会跳过紧邻的下一个元素，导致漏删。

修复方案 #

推荐直接用数组的filter方法来过滤，逻辑更简洁也不会出现下标问题：

const fs = require('fs')

const list = fs.readFileSync('./newmR.txt','utf-8').split('\r\n')
console.log(list.length)

// 直接过滤出不符合删除条件的行
const filteredList = list.filter(line => {
  const shouldDelete = line.includes('has met failed') || line.includes('get captcha')
  if (shouldDelete) console.log(list.indexOf(line))
  return !shouldDelete
})

fs.writeFileSync('./newmRfiltered.txt', filteredList.join('\r\n'))

如果一定要用for循环实现，推荐用反向遍历（从数组末尾往开头遍历），可以避免下标偏移问题：

const fs = require('fs')

let list = fs.readFileSync('./newmR.txt','utf-8').split('\r\n')
console.log(list.length)

// 反向遍历
for(let i = list.length - 1; i >= 0; i--){ 
   if(list[i].includes('has met failed') || list[i].includes('get captcha')){
       console.log(i)
       list.splice(i, 1) // 明确只删当前下标对应的1个元素
   }
}

fs.writeFileSync('./newmRfiltered.txt',list.join('\r\n'))

内容的提问来源于stack exchange，提问作者Tiến Nguyễn