如何在Firestore中校验商品存在性后新增或更新购物车商品数量？

现有实现存在的问题 #

你的代码在低并发、单用户操作的场景下可以正常运行，但存在几个明显缺陷：

• 没有异常处理逻辑：所有Firestore读写操作都是异步Promise，你没有编写catch回调捕获异常，遇到网络错误、权限不足、参数非法等问题时，没有任何错误反馈，调用方无法感知操作结果。
• 存在并发竞态风险：如果短时间内多次触发同一商品的添加操作，多个请求可能同时读取到相同的旧数量值，各自加1后写入，最终导致计数比实际应有的数值小，计数不准。
• 函数无异步结果返回：addToCart函数没有返回内部的Promise，外部调用方无法等待操作执行完成，也无法判断执行状态。
• 字段硬编码维护成本高：set操作时手动列出所有商品字段，后续商品新增属性时需要同步修改这段代码，容易漏改出错。

优化方案 #

我们可以利用Firestore自带的原子增量操作 + set的merge配置，大幅简化代码，同时解决上面所有问题：

优化后代码（Web V8 版本） #

// 从Firebase SDK引入原子操作工具类，不同引入方式写法略有差异
const FieldValue = firebase.firestore.FieldValue;

async function addToCart(item) {
  // 入参校验，避免生成无效数据
  if (!item?.id) {
    throw new Error('商品ID不能为空');
  }
  const cartItemRef = db.collection('cart-items').doc(item.id);
  try {
    // merge设为true：文档不存在就插入全量数据，存在就仅更新传入的字段
    // increment是服务端原子操作，不会出现并发计数错误
    await cartItemRef.set({
      ...item,
      quantity: FieldValue.increment(1)
    }, { merge: true });
    return true;
  } catch (err) {
    console.error('加入购物车失败：', err);
    // 抛出错误让外层可以自定义处理逻辑，比如弹出失败提示
    throw err;
  }
}

优化后代码（Web V9 模块化版本） #

如果你用的是新版模块化SDK，写法如下：

import { doc, setDoc, increment } from 'firebase/firestore';
// 替换为你自己的firebase初始化文件路径
import { db } from './firebaseInit';

async function addToCart(item) {
  if (!item?.id) throw new Error('商品ID不能为空');
  const cartItemRef = doc(db, 'cart-items', item.id);
  try {
    await setDoc(cartItemRef, {
      ...item,
      quantity: increment(1)
    }, { merge: true });
    return true;
  } catch (err) {
    console.error('加入购物车失败：', err);
    throw err;
  }
}

优化点说明 #

• 消除竞态风险：增量操作是Firestore服务端原子执行的，不管多少并发请求，计数都不会出错。
• 性能更好：省去了先查询文档是否存在的读请求，减少了一次数据库交互，也节省了读写配额。
• 维护成本更低：用扩展运算符自动带入所有商品属性，不需要手动枚举字段，后续商品加属性不需要修改这段代码。
• 可维护性更高：用async/await代替嵌套then，代码可读性更高，返回Promise方便外层做loading状态管理、成功/失败提示。

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