Firestore事务中批量递减queuePosition字段失败问题排查

那个报错的核心原因很明确：你在Promise.all()里用了users.forEach()，但forEach方法不会返回任何值（默认返回undefined），而Promise.all()必须接收一个可迭代的Promise数组。当它尝试读取undefined的迭代器时，就抛出了TypeError: Cannot read property 'Symbol(Symbol.iterator)' of undefined这个错误。

修复代码 #

只需要把forEach换成map就行——map会返回一个由回调里的Promise组成的新数组，正好符合Promise.all()的要求。另外我还加了文档存在性检查，避免因为某个用户的队列文档不存在导致整个事务失败：

function decrementPosition(users, eventId) {
  return db.runTransaction(transaction => {
    // 用map替代forEach，生成Promise数组
    return Promise.all(users.map(user => {
      const queuerRef = firebase.db.collection("events")
        .doc(eventId)
        .collection("queue")
        .doc(user.id);
      return transaction.get(queuerRef)
        .then(doc => {
          // 先确认文档存在，防止后续读取data()报错
          if (!doc.exists) {
            throw new Error(`用户${user.id}的队列文档不存在`);
          }
          const newPosition = doc.data().queuePosition - 1;
          return transaction.update(queuerRef, { queuePosition: newPosition });
        });
    }));
  });
}

额外提醒 #

1. 事务里的所有读写操作一定要通过transaction对象执行，你的代码这部分是正确的，继续保持就好
2. 调用这个函数的时候记得加错误处理，比如用try/catch包裹：

try {
  await decrementPosition(targetUsers, "your-event-id");
  console.log("队列位置批量递减成功");
} catch (error) {
  console.error("事务执行失败:", error);
}

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