使用Mongoose更新MongoDB文档时车辆状态修改未生效问题

你没有把修改后的数据持久化写入MongoDB，只修改了Node.js进程内存里的Mongoose文档对象：

• 执行await Car.findById(req.body.car)拿到的car是Mongoose实例化的内存对象，直接给car.status赋值只会修改内存里的属性值，不会自动同步到数据库，所以你控制台打印内存对象能看到修改后的值，但数据库里的原始数据完全没被改动。
• 你的代码还存在三个隐藏问题：
  • 变量名冲突：开头从req.body解构了car字段，后面又用let car声明查询到的车辆对象，容易引发不可预期的逻辑错误。
  • 错误判断逻辑不严谨：if (!user || !car)的判断分支复用了同一条错误提示，用户不存在时也会返回「车辆不可用」的错误信息，和实际问题不符。
  • 没有校验车辆当前状态：如果车辆已经是unavailable状态，依然会重复创建预订。

修改文档属性后必须调用save()方法才能把变更写入数据库，修正后的核心代码如下：

exports.Booking = async (req, res) => {
    // 重命名解构字段，避免和后续查询的car对象变量名冲突
    const { car: carId, bookingDate, returnDate, location } = req.body;
    try {
        const userId = req.user;
        const user = await User.findById(userId);
        const car = await Car.findById(carId);

        // 分开校验不同错误场景，返回对应提示
        if (!user) {
            return res.status(400).json('Invalid user');
        }
        if (!car || car.status === 'unavailable') {
            return res.status(400).json('This car is unavailable...');
        }

        const booking = await Booking.create({ 
            user: user._id, 
            car: car._id, 
            bookingDate, 
            returnDate, 
            location 
        });
        if (!booking) {
            return res.status(404).json({ message: 'failed to create booking' });
        }

        // 修改状态后调用save()持久化到数据库，这是你之前漏掉的关键步骤
        car.status = 'unavailable';
        await car.save();
        
        return res.status(202).json({ message: 'Success', booking });
    } catch (error) {
        return res.status(500).json(error.message);
    }
}

优化建议：如果不需要对车辆文档做其他额外操作，可以直接用更新语句跳过查询步骤，性能更好：

await Car.findByIdAndUpdate(carId, { status: 'unavailable' });

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