Firestore中forEach循环内使用doc函数报错问题求助

你遇到的错误核心原因是forEach循环的参数名doc和Firebase SDK提供的doc()函数重名了。在循环内部，doc会被优先解析为当前遍历到的文档快照对象，而非你导入的用于创建文档引用的doc方法，因此调用doc(targetCollectionRef, docId)时会触发“不是函数”的报错。

修正后的代码示例 #

首先确保已正确导入Firestore的doc函数（比如从firebase/firestore中导入），然后修改循环参数名避免冲突：

// Import necessary Firebase libraries and initialize Firebase app
// 示例导入：import { getFirestore, collection, getDocs, doc, setDoc } from "firebase/firestore";

async function moveData() {
  const db = getFirestore(app);
  const sourceCollectionRef = collection(db, 'sourceCollection');
  const targetCollectionRef = collection(db, 'targetCollection');

  const querySnapshot = await getDocs(sourceCollectionRef);

  // 将循环参数名改为docSnapshot，避免与Firebase的doc函数重名
  querySnapshot.forEach(async (docSnapshot) => {
    const docId = docSnapshot.id;
    // 现在可以正常调用Firebase的doc函数生成目标文档引用
    const targetDocRef = doc(targetCollectionRef, docId);

    // 数据处理逻辑示例
    const originalData = docSnapshot.data();
    const modifiedData = { ...originalData, migratedAt: new Date() };
    
    await setDoc(targetDocRef, modifiedData);
  });
}

额外优化建议 #

forEach循环中使用async回调会导致所有异步操作并行执行，若数据量较大可能触发Firestore的速率限制。更稳妥的方式是使用for...of循环串行执行，或用Promise.all控制并发数：

// 串行执行版本（适合数据量较大的场景）
async function moveData() {
  const db = getFirestore(app);
  const sourceCollectionRef = collection(db, 'sourceCollection');
  const targetCollectionRef = collection(db, 'targetCollection');

  const querySnapshot = await getDocs(sourceCollectionRef);

  for (const docSnapshot of querySnapshot.docs) {
    const docId = docSnapshot.id;
    const targetDocRef = doc(targetCollectionRef, docId);
    
    const originalData = docSnapshot.data();
    const modifiedData = { ...originalData, migratedAt: new Date() };
    
    await setDoc(targetDocRef, modifiedData);
  }
}

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