TypeScript如何精简参数空值组合判断并优化构造对象的代码？

首先你原代码存在一个逻辑bug：单边界判断分支里，对象的动态键错误使用了range.from/range.to，应该改为range.key，否则返回的过滤条件结构完全不符合预期。

以下是更紧凑、易维护的重构实现：

protected temporalFilter({ range }: Filter): W | null {
  const temporalKeys = ['created', 'bootedSince', 'firstComunication'] as const;
  // 反向判断提前返回，减少嵌套层级
  if (!temporalKeys.includes(range.key)) return null;

  const conditions: Record<string, unknown>[] = [];
  // 拆分两个边界的判断逻辑，互不依赖，减少重复代码
  if (range.from) conditions.push({ [range.key as keyof T]: { gte: range.from } });
  if (range.to) conditions.push({ [range.key as keyof T]: { lte: range.to } });

  // 根据条件数量返回对应结构
  return (
    conditions.length === 0 ? {} :
    conditions.length === 1 ? conditions[0] :
    { AND: conditions }
  ) as unknown as W;
}

主要优化点 #

• 提前做非法场景返回，消除多层嵌套的if-else，代码可读性更高
• 把两个边界的判断拆成独立逻辑，避免多分支重复写相似的结构，后续如果要新增其他过滤规则（比如不等于、开区间判断）扩展成本极低
• 常量temporalKey改为复数形式temporalKeys更符合语义，添加as const可以让TS做更严格的类型校验
• 总代码行数减少近40%，逻辑更紧凑，没有冗余分支

可选进阶优化 #

• 可以提取公共的TemporalKey类型：type TemporalKey = typeof temporalKeys[number]，给Filter的range.key加上对应类型约束，就可以省去后续不必要的as keyof T、as unknown as W类型断言，提升类型安全性
• 如果业务场景中range可能为空，可以在函数开头新增空值判断if (!range) return null，避免运行时报错

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