使用键值相同的对象配合switch语句是否属于冗余无意义代码？

原写法的问题 #

你之前写的valStore完全是冗余代码——键和值完全相同的对象没有任何实际意义，而且switch里的case valStore[val]逻辑等价于判断val是否是valStore的键，写法绕了一圈，既增加代码量又降低可读性，完全可以替换。

几种优化方案 #

1. 贯穿式switch（小场景首选） #

就是你提到的多case并列写法，代码直观无冗余，5个值的场景下可读性拉满：

switch (val) {
  case "val4":
    function();
    break;
  case "val5":
    coolFunction();
    break;
  case "val1":
  case "val2":
  case "val3":
    otherFunction(val);
    break;
  // 其他分支
}

• 优势：代码一目了然，不需要额外维护集合或对象，小场景下零学习成本。
• 劣势：如果后续需要添加大量同逻辑的case，会显得冗长，但当前5个值的场景完全不存在这个问题。

2. Set + else if（适合动态维护场景） #

用Set存储需要统一处理的取值，通过has()方法判断，后续调整取值时不用动条件分支：

const targetValues = new Set(["val1", "val2", "val3"]);

if (val === "val4") {
  function();
} else if (val === "val5") {
  coolFunction();
} else if (targetValues.has(val)) {
  otherFunction(val);
}
// 其他分支

• 优势：新增/删除同逻辑的取值时，只需要修改Set内容，扩展性更好。
• 劣势：多了一个Set的定义，小场景下和switch比优势不明显胜在灵活。

3. 对象映射（适合逻辑集中管理） #

把所有取值对应的处理函数映射到一个对象中，逻辑和判断完全分离：

const handlerMap = {
  val4: () => function(), // 若原函数可直接调用，可简化为function
  val5: coolFunction,
  val1: otherFunction,
  val2: otherFunction,
  val3: otherFunction
};

if (handlerMap[val]) {
  handlerMap[val](val); // 根据函数是否需要传参调整调用方式
}
// 其他分支

• 优势：所有逻辑集中在一个对象里，后续新增case只需要添加键值对，适合分支较多或需要统一管理的场景。
• 劣势：如果不同函数的参数差异大，处理起来会稍显麻烦，小场景下有点“大材小用”。

要不要替换？ #

完全值得替换，原valStore写法纯粹是冗余代码，既增加维护成本又没有实际价值。具体选哪种方案看你的场景：

• 固定5个值且短期不变：选贯穿式switch，最省心；
• 后续可能频繁调整同逻辑的取值：选Set+else if；
• 想把所有逻辑集中管理：选对象映射。

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