Node.js使用Firebase SDK上传多图到Firestore时URL收集不全

你的代码逻辑存在两个核心错误：

1. for循环是同步执行的，会瞬间触发所有上传请求，但不会等待任何一个上传的异步回调执行完成。.then()仅会等待当前单个上传Promise的状态变更，不会自动等待循环中其他未完成的上传任务。你在回调里判断images.length == x+1，仅代表这是循环中最后一个被触发的上传请求，不代表这是最后一个完成的上传请求——异步任务的完成顺序受文件大小、网络波动影响，和触发顺序没有必然关系，很可能最后一个触发的任务先完成，此时你直接执行写入Firestore的逻辑，其余还在上传的任务对应的URL自然不会被收集到列表里。
2. 原代码中admin.storage.bucket()写法有误，缺少实例化的调用括号，正确写法为admin.storage().bucket()。

优先用Promise.all实现并行上传，代码简洁且执行效率最高，等所有上传任务全部完成后再统一收集URL、写入Firestore：

// 批量构造上传任务数组
const uploadTasks = images.map(async (imageItem) => {
  // 替换为实际的文件路径、上传配置
  const [uploadedFile] = await admin.storage().bucket().upload(/* 对应imageItem的文件路径 */, {
    // 你的原有上传配置，例如存储路径、文件元数据等
  });
  // 替换为你实际使用的获取下载链接的逻辑，以下是签名URL示例
  const [downloadUrl] = await uploadedFile.getSignedUrl({
    action: 'read',
    expires: '2999-01-01' // 按业务需求设置过期时间即可
  });
  return downloadUrl;
});

// 等待所有上传任务完成，拿到完整的URL列表
const imageUrls = await Promise.all(uploadTasks);

// 所有URL收集完毕后，再写入Firestore
await admin.firestore().collection('你的目标集合').doc(/* 目标文档ID */).set({
  // 其他业务字段
  imageUrls
});

如果单批次上传图片数量较多，担心并行并发过高触发存储限流，可以改用for...of加await实现串行上传：

const imageUrls = [];
for (const imageItem of images) {
  const [uploadedFile] = await admin.storage().bucket().upload(/* 对应文件路径 */, {
    // 上传配置
  });
  const [downloadUrl] = await uploadedFile.getSignedUrl({
    action: 'read',
    expires: '2999-01-01'
  });
  imageUrls.push(downloadUrl);
}

// 所有上传完成后写入Firestore
await admin.firestore().collection('你的目标集合').doc(/* 目标文档ID */).set({
  // 其他业务字段
  imageUrls
});

• 永远不要靠循环索引、回调触发顺序判断批量异步任务是否全部完成，这类写法在网络波动时必然出现数据不全的问题，统一使用Promise.all或者循环内await的方式等待所有异步任务执行完毕再做后续处理。
• 如果需要处理单张图片上传失败的场景，可以给单个上传任务加try/catch，避免单张图失败导致整个批次写入中断。

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