不可拆分的多动作POST API 代码优化及最佳设计方案推荐

• 缺乏基础入参校验：未校验request.body.action是否存在、是否属于合法取值范围，非法请求也会返回done，调用方无法感知参数错误。
• 未处理执行异常：doProcessA/doProcessB/doProcessC执行抛出错误时没有捕获逻辑，会直接抛出服务异常，且没有任何错误反馈给调用方。
• 异步逻辑兼容问题：如果处理函数是异步逻辑，现有代码没有等待执行完成就返回done，会出现响应返回后业务逻辑还在执行的时序问题，业务正确性无法保障。
• 扩展性差：后续新增动作时需要不断追加switch分支，违反开闭原则，代码会随着业务迭代越来越臃肿。
• 类型声明不严谨：request、response使用any类型，无法利用TS静态校验能力提前发现问题。

使用对象映射替代switch分支，同时补全校验、异常处理、异步兼容逻辑，优化后代码示例如下：

// 先定义动作枚举和处理函数映射
const ACTION_HANDLERS: Record<'a' | 'b' | 'c', () => Promise<void> | void> = {
  a: doProcessA,
  b: doProcessB,
  c: doProcessC
}
const SUPPORTED_ACTIONS = Object.keys(ACTION_HANDLERS) as Array<'a' | 'b' | 'c'>;

exports.myFunction = functions.https.onRequest(async (request: functions.https.Request, response: functions.Response) => {
  try {
    const action = request.body.action;
    // 入参合法性校验
    if (!action || !SUPPORTED_ACTIONS.includes(action)) {
      return response.status(400).send({
        code: 'INVALID_ACTION',
        message: `合法动作取值为：${SUPPORTED_ACTIONS.join(',')}`
      })
    }
    // 执行对应处理逻辑，兼容同步/异步
    await ACTION_HANDLERS[action]();
    response.status(200).send({
      code: 'SUCCESS',
      message: '处理完成'
    })
  } catch (err) {
    // 全局异常捕获，返回错误信息
    response.status(500).send({
      code: 'PROCESS_FAILED',
      message: err instanceof Error ? err.message : '处理失败'
    })
  }
});

• 符合开闭原则：后续新增动作仅需要在ACTION_HANDLERS中追加映射关系即可，不需要修改主流程逻辑。
• 逻辑可测性更强：每个动作的处理函数和主流程完全解耦，可以单独编写单元测试。
• 业务正确性有保障：补全了异步等待逻辑，确保处理逻辑执行完成后再返回响应。
• 调用方友好：不同场景返回不同的状态码和明确的错误提示，便于调用方排查问题。
• 类型更严谨：使用框架提供的原生Request/Response类型替换any，静态阶段即可发现非法操作。

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