查找文件数组内重名文件的JS代码条件判断错误如何修正

现有代码错误原因 #

• 数组相等判断逻辑错误：split('.',1) 执行后返回的是数组类型，JavaScript中数组用===判断比较的是内存引用地址，就算两个数组内容完全一致也会返回false，需要取数组的第一个元素（即去掉后缀的文件名）做字符串比较，应该改成split('.',1)[0]。
• 循环逻辑冗余错误：外层已经写了map遍历，内层又嵌套了两层for循环，会出现大量重复判断（比如i=0,j=1和i=1,j=0是同一组对比），还会出现自己和自己对比的误判，整体时间复杂度高达O(n²)，性能极低。
• 方法误用：map方法的作用是生成对应的新数组，你这里没有返回值也没有用到map的返回结果，属于无意义调用，应该直接去掉外层的map。
• 打印输出错误：直接把文件对象拼接到字符串中，输出结果只会是[object Object]，无法看到实际文件信息，需要取对象的具体属性或者用JSON.stringify转成字符串打印。

优化实现方案 #

使用分组统计的方式实现需求，时间复杂度只有O(n)，效率更高，直接就能得到所有同文件名的分组，更适配后续单独上传的场景：

const filesForStorage = [
  {id: 12323, name: 'name.png'},
  {id: 3123, name: 'abc.jpg'},
  {id: 3213, name: 'name.jpeg'},
  {id: 4562, name: 'abc.gif'}
]

// 用来存储分组结果，key是去掉后缀的文件名，value是对应文件数组
const nameGroupMap = {}

filesForStorage.forEach(file => {
  // 取无后缀的文件名
  const pureName = file.name.split('.', 1)[0]
  // 分组不存在就先初始化
  if (!nameGroupMap[pureName]) {
    nameGroupMap[pureName] = []
  }
  nameGroupMap[pureName].push(file)
})

// 筛选出有多个文件的同名分组，就是你需要的结果
const sameNameFileGroups = Object.values(nameGroupMap).filter(group => group.length > 1)

// 打印验证
sameNameFileGroups.forEach(group => {
  console.log('----同名文件组：', group.map(item => item.name))
})

内容的提问来源于stack exchange，提问作者Jacopo