Array.some()在第二个数组为空时无法向空数组push元素问题排查

问题原因 #

• 你对Array.some()的用法不符合设计逻辑：some()会遍历数组直到找到第一个符合条件的元素就返回true停止遍历，若遍历的数组为空，回调函数完全不会执行，这就是第二个数组为空时没有元素被push到新数组的直接原因。
• 现有核心逻辑完全错误：你当前在some()回调中无论ID是否匹配都会推送groupContacts到结果数组，完全达不到「筛选第一个数组中不存在于第二个数组的元素」的需求。
• 误用Array.filter()：filter()本身会返回符合条件的新数组，你没有使用它的返回值，仅将其作为遍历方法使用，属于方法滥用。

修复方案 #

我们可以先将群组已有的联系人ID转为Set结构（查找效率远高于数组遍历，时间复杂度从O(n)降为O(1)），再直接筛选出选中联系人中不存在于该Set的ID即可，逻辑清晰且性能更好：

const addContactsToGroup = async (req, res) => {
  const { groupId, selectedContacts } = req.body
  const group = await Group.findById(groupId)
  // 把群组已有联系人转成Set，方便快速查找
  const groupContactSet = new Set(group.contacts)
  // 直接筛选出选中联系人里不在群组中的ID
  const toAddContacts = selectedContacts.filter(contact => !groupContactSet.has(contact))
  // 最终的toAddContacts就是你需要的结果，原ID都是唯一的无需额外去重
  console.log(toAddContacts)
  // 后续可直接把toAddContacts加到group.contacts里保存即可
}

如果坚持要用some()实现，也可以按正确逻辑编写：

const toAddContacts = selectedContacts.filter(contact => {
  // some返回true说明ID在群组里存在，filter返回false过滤该元素；返回false说明不存在，保留该元素
  return !group.contacts.some(groupContact => groupContact === contact)
})

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