Mongo Shell中$pull操作生效，Node.js(Express)代码执行无效问题

首先，先明确你的场景：

• 你有一个doctors集合，文档结构如下：

{ 
  "_id" : ObjectId("5b02df7a45b504151cb42c40"), 
  "email" : "micro@gmail.com", 
  "appointments" : [ 
    { "patient_name" : "patient_1", "patient_id" : 1.0, "time" : 1528915447.0 }, 
    { "patient_name" : "patient_2", "patient_id" : 2.0, "time" : 1529915447.0 } 
  ]
}

• 在Mongo Shell中执行$pull操作可以成功删除目标数组元素，但在Express接口中使用Mongoose的findByIdAndUpdate时，接口返回{success: true}，却没有实际删除数据。

可能的原因及解决方案 #

1. 数据类型不匹配（最常见的原因）

看你的数据库文档，patient_id和time字段都是浮点数类型（比如2.0、1529915447.0），但你的请求体中传的是整数（patient_id: 2、time: 1529915447）。虽然MongoDB原生会认为2和2.0相等，但Mongoose在做严格匹配时，可能因为类型差异导致$pull无法匹配到目标元素。

解决方案：
在构造$pull条件时，将请求中的数字转换为浮点数：

$pull: { 
  appointments: {
    patient_name: req.body.patient_name,
    patient_id: parseFloat(req.body.patient_id),
    time: parseFloat(req.body.time)
  }
}

2. findByIdAndUpdate返回的是旧文档，导致你误以为未更新

Mongoose的findByIdAndUpdate默认返回更新前的文档，所以你console.log(doc.appointments)看到的是更新前的数组，并不是实际数据库中的最新状态。

解决方案：
修改findByIdAndUpdate的选项，添加new: true来返回更新后的文档，同时去掉不必要的upsert: true（你是更新已存在的医生文档，不需要插入新文档）：

Doctor.findByIdAndUpdate(
  req.user.id,
  {$pull: { appointments: {/* 你的条件 */} }},
  {new: true, safe: true}, // 添加new: true，移除upsert: true
  function(err, doc) {
    if(err){
      console.log(err);
      res.json({ success:false }).end();
    }else{
      console.log(doc.appointments); // 现在打印的是更新后的数组
      res.json({ success:true }).end();
    }
  }
);

3. Mongoose Schema定义不匹配

如果你的Doctor Schema中，appointments数组的字段类型定义不正确，也会导致匹配失败。比如如果patient_id定义为String，而数据库里是数字类型，就会匹配不到。

检查并修正Schema：
确保你的Schema定义和数据库字段类型一致：

const DoctorSchema = new mongoose.Schema({
  email: { type: String, required: true },
  appointments: [{
    patient_name: String,
    patient_id: Number, // 和数据库的数字类型对应
    time: Number // 和数据库的数字类型对应
  }]
});

const Doctor = mongoose.model('Doctor', DoctorSchema);

4. 尝试手动过滤数组并保存（更直观的调试方式）

如果上面的方法都不行，可以先查询文档，手动过滤掉目标元素后再保存，这种方式更容易调试：

app.post('/secure/doctor/reject_appointment', function(req, res){
  console.log(req.body);
  if(!req.user){
    res.json({success: false,message:"Unauthorized"});
    return;
  }

  Doctor.findById(req.user.id, function(err, doctor) {
    if(err){
      console.log(err);
      return res.json({ success:false }).end();
    }
    // 过滤掉要删除的预约
    doctor.appointments = doctor.appointments.filter(apt => {
      return !(apt.patient_name === req.body.patient_name && 
              apt.patient_id === parseFloat(req.body.patient_id) && 
              apt.time === parseFloat(req.body.time));
    });
    // 保存修改
    doctor.save(function(saveErr) {
      if(saveErr){
        console.log(saveErr);
        res.json({ success:false }).end();
      }else{
        console.log(doctor.appointments);
        res.json({ success:true }).end();
      }
    });
  });
});

调试小技巧 #

在执行更新操作前，打印出你要匹配的条件，对比数据库中的文档，确保值和类型完全一致：

const pullCondition = {
  appointments: {
    patient_name: req.body.patient_name,
    patient_id: parseFloat(req.body.patient_id),
    time: parseFloat(req.body.time)
  }
};
console.log("Pull Condition:", pullCondition);
// 然后执行findByIdAndUpdate

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