Firestore带.where的仅更新查询异常：为何更新全部文档？

问题根源 #

你遇到的问题核心在于文档引用的构建方式，虽然你的where条件逻辑上是对的，但更新时获取文档引用的方法出了问题，导致原本的筛选限制失效（或者说原代码里的updateRef.doc(doc.id)虽然路径看似正确，但实际执行时没有正确关联到筛选后的文档集合）。另外原代码还有两个容易踩的坑：普通函数回调导致this上下文丢失，以及没有正确处理批量更新的异步流程。

具体分析与修复步骤 #

1. 错误的文档引用获取 #

原代码里你用updateRef.doc(doc.id)来创建要更新的文档引用，但updateRef是一个带where条件的Query对象，不是集合引用。虽然Firestore SDK理论上允许通过Query对象的doc()方法生成文档引用，但这种方式很容易出现上下文偏差，不如直接用查询结果里doc对象自带的ref属性来得可靠——这个属性就是当前文档的直接引用，绝对不会出错。

比如把原代码里的更新部分改成：

querySnapshot.forEach(function (doc) {
  if (doc.exists) {
    // 直接用doc.ref获取正确的文档引用
    return doc.ref.update({ selectedDate: new Date().toLocaleDateString() });
  }
});

2. 修复上下文与异步流程问题 #

原代码用普通函数作为then的回调，会导致this指向不再是你的组件实例，后续调用this.setState必然报错。同时，forEach里的update是异步操作，原代码没有等待所有更新完成就想更新状态，这会导致加载状态提前结束，用户体验不好。

优化后的完整代码 #

结合所有修复点，更严谨的代码应该是这样：

onEndPress() {
  this.setState({ isLoading: true });
  const userId = firebase.auth().currentUser.uid;
  // 先提前获取正确的集合引用
  const bleedingCollection = firebase.firestore()
    .collection('users')
    .doc(userId)
    .collection('MyBleeding');
  
  bleedingCollection.where('selectedDate', '==', '-')
    .get()
    .then((querySnapshot) => {
      // 收集所有更新操作的Promise，等待批量完成
      const updateTasks = [];
      querySnapshot.forEach((doc) => {
        if (doc.exists) {
          updateTasks.push(
            doc.ref.update({ selectedDate: new Date().toLocaleDateString() })
          );
        }
      });
      return Promise.all(updateTasks);
    })
    .then(() => {
      // 所有更新完成后再更新组件状态
      this.setState({ selectedDate: '', isLoading: false });
    })
    .catch((error) => {
      console.error("更新文档出错: ", error);
      this.setState({ isLoading: false });
    });
}

为什么这样改就对了？ #

• doc.ref直接绑定查询到的符合条件的文档，确保只更新你筛选出来的那些selectedDate为"-"的文档
• Promise.all等待所有更新操作完成再更新状态，避免加载状态提前消失
• 箭头函数保持了组件的this上下文，setState可以正常工作
• 完善的错误处理能在更新失败时及时重置加载状态，避免用户一直看到加载中

内容的提问来源于stack exchange，提问作者Karl Oskar Nilsson