如何正确使用try/catch、Promise catch及async函数？上传函数错误处理咨询

嘿，这个问题其实挺常见的——当你在async函数里混用Promise链式catch和try/catch时，确实容易搞不清逻辑。咱们把这个问题拆透：

在async函数里，await会把Promise的异步结果同步化，这时候用try/catch是最直观、最易维护的错误捕获方式。它能统一捕获所有错误：包括client.mutate()返回的Promise拒绝，还有函数内部的同步错误（比如变量未定义、类型错误等）。

给你修正后的代码示例：

// 推荐的写法：用try/catch统一处理所有错误
export default async function upload(client) {
  try {
    // 用await等待mutate的Promise完成
    const mutationResult = await client.mutate({
      mutation: YOUR_UPLOAD_MUTATION, // 替换成你的上传Mutation
      variables: { /* 你的上传变量，比如file、metadata等 */ }
    });
    
    // 处理上传成功后的逻辑，比如返回结果或更新状态
    return mutationResult.data.upload;
  } catch (error) {
    // 这里捕获所有错误：网络错误、GraphQL错误、同步错误都能到这里
    console.error('上传失败详情:', error);
    
    // 你可以在这里做错误转换、用户提示，或者重新抛出给上层处理
    if (error.networkError) {
      throw new Error('网络连接失败，请检查网络后重试');
    } else if (error.graphQLErrors?.length) {
      throw new Error(`上传失败：${error.graphQLErrors[0].message}`);
    } else {
      throw new Error(`未知错误：${error.message}`);
    }
  }
}

如果在await后面链式调用.catch()，比如await client.mutate(...).catch(err => {})，会导致错误被链式catch“消化”，除非你在链式catch里重新抛出错误，否则try/catch的catch块不会触发。这种分散的错误处理会让逻辑变得混乱，不利于后续维护。

举个反例（不推荐）：

// 不推荐：分散的错误处理，逻辑割裂
export default async function upload(client) {
  try {
    const result = await client.mutate({
      mutation: YOUR_UPLOAD_MUTATION
    }).catch(err => {
      console.log('链式catch捕获到错误:', err);
      // 这里如果不抛出，try/catch的catch块永远不会触发
      // return null; // 错误被“吞掉”，上层拿到null而非错误
    });
    return result;
  } catch (error) {
    // 只有当链式catch里重新抛出，或者有其他同步错误时才会执行
    console.log('try/catch捕获到错误:', error);
  }
}

如果你的业务需要分阶段处理特定错误（比如先单独处理Mutation的GraphQL错误，再处理其他通用错误），可以嵌套try/catch，这比链式catch更清晰：

export default async function upload(client) {
  try {
    let mutationResult;
    // 嵌套try/catch专门处理Mutation的错误
    try {
      mutationResult = await client.mutate({
        mutation: YOUR_UPLOAD_MUTATION
      });
    } catch (mutateError) {
      // 专门处理Mutation相关的错误
      if (mutateError.graphQLErrors?.some(err => err.extensions.code === 'FILE_TOO_LARGE')) {
        throw new Error('文件过大，请上传小于10MB的文件');
      }
      // 其他Mutation错误重新抛出，交给外层catch处理
      throw mutateError;
    }
    
    // 其他业务逻辑
    return mutationResult.data.upload;
  } catch (generalError) {
    // 处理所有最终抛出的错误
    console.error('上传失败:', generalError);
    throw generalError;
  }
}

1. 在async函数中，优先使用try/catch，它是最符合同步思维的错误处理方式，逻辑统一且易读。
2. 不需要同时保留Promise链式catch和try/catch，这会导致逻辑混乱。
3. 如果用了链式catch，记得要重新抛出错误，才能让上层（包括try/catch）捕获到。

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