有没有更优雅的方式提取Firestore返回的PromiseResult数据？

你的代码已经实现了功能，但可以通过简化异步逻辑、移除冗余变量、增加健壮性来让写法更优雅，以下是几种优化方案：

核心优化思路 #

• 用async/await替代then链，让异步代码更接近同步逻辑，可读性更强
• 移除不必要的全局变量，避免状态混乱
• 增加文档存在性检查与错误捕获，避免潜在报错

方案1：直接封装获取指定字段的函数 #

如果只需要提取Description字段，可以直接把逻辑封装到函数里，一步到位：

async function getProductDescription(specificProduct) {
  try {
    // 获取文档引用
    const docRef = doc(db, "MyProductList", specificProduct);
    // 获取文档快照
    const docSnap = await getDoc(docRef);
    
    if (docSnap.exists()) {
      // 直接返回目标字段
      return docSnap.data().Description;
    } else {
      console.log("未找到指定产品文档");
      return null;
    }
  } catch (error) {
    console.error("获取产品数据失败：", error);
    return null;
  }
}

// 使用方式
async function logProductDesc() {
  const desc = await getProductDescription("someProduct");
  if (desc) {
    console.log(desc);
  }
}

logProductDesc();

方案2：通用获取产品数据函数 #

如果需要访问产品的多个字段，先获取完整数据再按需提取：

async function getProduct(specificProduct) {
  try {
    const docSnap = await getDoc(doc(db, "MyProductList", specificProduct));
    // 存在则返回数据，否则返回null
    return docSnap.exists() ? docSnap.data() : null;
  } catch (error) {
    console.error("获取产品数据失败：", error);
    return null;
  }
}

// 使用时直接提取所需字段
async function logProductDetails() {
  const product = await getProduct("someProduct");
  if (product) {
    console.log(product.Description);
    // 可直接访问其他字段，比如 product.Price、product.Name 等
  } else {
    console.log("未找到指定产品");
  }
}

logProductDetails();

对比原代码的改进点 #

1. 移除了全局变量price1，所有数据都在函数内部处理，避免状态污染
2. 用async/await替代then链，逻辑更线性，不需要拆分多个回调
3. 增加了文档存在性检查和错误捕获，避免文档不存在或网络错误导致的崩溃
4. 函数职责更清晰：getProduct专注于获取数据，调用方负责处理数据使用

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