JavaScript嵌套map修改数组对象属性未生效问题

• 对Array.map()的用法理解存在偏差：map本身不会修改原数组，核心作用是根据遍历逻辑返回一个和原数组长度一致的新数组。你写的内层两层person.map()、p.sports.map()的返回值没有被任何变量接收，遍历产生的结果直接被丢弃，属于无效计算。
• 匹配到的修改结果没有向外传递：你在最内层循环里判断匹配后return的{...st, found: true}，只会出现在最内层sports.map生成的临时数组里，外层遍历sportTag的回调最后直接return st，返回的还是最开始传入的原始未修改对象，found字段自然不会更新为true。
• 遍历逻辑存在性能浪费：只要找到任意一个匹配的运动项，当前sportTag的found就可以直接设为true，不需要遍历完所有人员、所有运动项，你原来的三层全量map遍历会做很多无用功。

推荐先聚合所有运动项再匹配的写法，时间复杂度更低，逻辑也更清晰：

// 第一步：收集所有人员参与的运动，统一转大写存入Set，后续查询时间复杂度为O(1)
const existedSports = new Set()
person.forEach(user => {
  user.sports.forEach(sport => {
    existedSports.add(sport.toLocaleUpperCase())
  })
})

// 第二步：遍历标签直接判断是否存在匹配项
const result = sportTag.map(tag => ({
  ...tag,
  found: existedSports.has(tag.name.toLocaleUpperCase())
}))

如果偏好嵌套判断的语义化写法，可以用Array.some()替代map，它会在找到第一个匹配项时自动终止遍历，完全匹配业务逻辑：

const result = sportTag.map(tag => {
  const hasMatched = person.some(user => 
    user.sports.some(sport => 
      sport.toLocaleUpperCase() === tag.name.toLocaleUpperCase()
    )
  )
  return { ...tag, found: hasMatched }
})

两种写法运行后都会得到预期输出：

[
  { id: 1, name: 'FOOTBALL', found: true },
  { id: 2, name: 'TENNIS', found: true },
  { id: 3, name: 'BASKET', found: false }
]

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