Redux中通过||运算符分发selectedMovie/selectedSerie id是否可行

你当前用||运算符的写法无法稳定实现二选一分发id的需求，存在以下明确问题：

• 存在假值误判风险：JavaScript里||会把所有假值（包括0、''这类完全合法的id值）判定为无效，自动走到后面的兜底逻辑。举个例子，如果SelectedMovie[0].id是0（部分后端会用0作为特殊内容的id），哪怕当前选中的确实是电影，代码也会错误取SelectedSerie?.id的值。
• 没有处理边界异常：如果两个id都不存在，最终会给dispatch传undefined，很容易引发Redux侧的取值报错。
• 依赖项缺失：当前useEffect的依赖数组没有加入dt、SelectedMovie、SelectedSerie三个用到的变量，会触发闭包陷阱，这几个值更新后effect不会重新执行，永远拿第一次渲染时的旧值。
• 没有实现严格互斥：如果业务上要求同一时间只能选中电影/剧集其中一个，||的写法不会做校验，万一两个值同时存在，会永远优先取电影id，不符合二选一的规则。

不要靠||的假值判断做兜底，先基于业务规则明确判断当前选中的内容类型，再取对应id，参考代码如下：

useEffect(() => {
  // 提前return减少嵌套
  if (!isModalVisible) return
  if (!Array.isArray(dt) || dt.length === 0) return

  let contentId = null
  // 替换成你项目里判断内容类型的真实条件，比如结合当前选中tab、路由参数判断，比单纯判断id存在更严谨
  const hasSelectedMovie = SelectedMovie?.length > 0
  const hasSelectedSerie = !!SelectedSerie?.id

  if (hasSelectedMovie) {
    contentId = SelectedMovie[0].id
  } else if (hasSelectedSerie) {
    contentId = SelectedSerie.id
  } else {
    // 两个id都不存在的场景加兜底，避免传undefined
    console.error('未获取到有效选中内容的ID')
    return
  }

  dt.forEach(el => {
    dispatch(synt(language, el, contentId))
  })
// 补全所有依赖项，避免闭包问题
}, [dispatch, language, isModalVisible, dt, SelectedMovie, SelectedSerie])

• 前面已经通过Array.isArray(dt)确认dt是数组的前提下，不需要再加dt?.forEach的可选链，冗余判断反而会掩盖类型异常。
• 如果你的项目里电影和剧集的选中态本身是互斥的（选中电影时会自动清空SelectedSerie，选中剧集时自动清空SelectedMovie），上述判断逻辑可以完全覆盖需求；如果存在两个值同时存在的场景，建议结合页面上的类型标识（比如当前激活的tab key、页面路由类型）做判断，可靠性更高。

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