如何正确使用Mongoose与Express的Patch请求更新订单状态

现有代码的可行性与存在的问题 #

首先明确：你的现有代码逻辑可以正常运行，Mongoose支持查询到文档实例后直接修改属性，再调用save()方法同步到数据库，这个过程Mongoose内部会自动生成对应的更新操作符，所以你没有手动写$set也能生效。

但这种实现方式存在两个明显的缺陷：

• 存在竞态条件风险：查询订单和保存订单是两个非原子操作，如果同一时间有多个请求修改同一个订单的状态，可能会出现两个请求都判断没有终态、重复插入状态的问题，甚至会出现已经插入终态后还插入其他状态的逻辑错误。
• 性能开销更高：需要先做一次全文档查询，再做一次更新，两次数据库请求，比原子更新多了一次查询开销，订单文档越大开销越高。

是否需要使用$set类操作符 #

你当前的场景是给数组追加元素，不需要用$set，更适合用MongoDB原生的数组更新操作符$push，配合findOneAndUpdate实现原子性的单请求更新，把状态校验和更新合并为一个原子操作，从根源上避免竞态问题，同时提升性能。

优化后的实现代码 #

module.exports.updateStatus = async (req, res) => {
    const { orderid } = req.params;
    const { heading } = req.body;

    // 原子操作：查询+更新一步完成
    const updateResult = await Order.findOneAndUpdate(
        // 匹配条件：订单ID存在 + 状态数组中没有终态项
        {
            _id: orderid,
            'status.heading': { $ne: 'Arrived At Location' }
        },
        // 更新操作：给status数组追加新状态
        {
            $push: { status: { heading, date: new Date() } }
        }
    );

    // 返回值为空说明要么订单不存在，要么已经是终态
    if (!updateResult) {
        req.flash('error', 'Order is already marked as fulfilled.');
        return res.redirect(`/orders/${orderid}`);
    }

    res.redirect(`/orders/${orderid}`);
}

补充说明 #

如果你的业务需要在更新前做更复杂的校验（比如校验订单所属用户、校验其他字段状态等），必须先查询文档再修改的话，可以给Order模型加乐观锁版本号，避免并发修改问题，比无锁先查后改的方式更安全。

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