那些遇到过的问题

FindAndUpdate 如何检查文档是否真的更新了

ggd*_*ras 4 javascript mongoose mongodb node.js mongodb-query

想象一下以下模型：

var Office = 
{
    id: 1,
    name: "My Office",
    branches: 
    [
      {
        adddress: "Some street, that avenue",
        isPrincipal: true,
      },
      {
        adddress: "Another address",
        isPrincipal: false,
      },      
    ]
}
Run Code Online (Sandbox Code Playgroud)

我想删除一个分支，但我们不能让用户从办公室删除主要分支。所以这是我的功能：

remove: function(body)
{
  return new Promise(function(resolve, reject)
  {
    return Office.findByIdAndUpdate(1, { $pull: {'branches': {_id: body.branch.id}}}, { new: true })
    .then(function(updatedOffice){
      resolve(updatedOffice)
    })
    .catch(function(error){
      reject(error);
    });

  })
}
Run Code Online (Sandbox Code Playgroud)

我在这里有一些疑问：

  1. 正如您所看到的，我没有在 isPrincipal 属性中包含另一个 WHERE，这是因为我不知道如何确定 office 对象是否已实际更改。因为对象总是会被检索但是......我怎么能确定呢？
  2. 考虑到我们不能让用户删除主体分支，FindByIdAndUpdate 是最好的方法，如果他试图这样做，我们必须显示警告。

Nei*_*unn 7

查看是否对诸如 a 之类的$pull内容应用了更新的唯一真正可靠的方法是基本上检查返回的文档并查看您想要的数据是否$pull仍在那里。

这适用于"findAndUpdate"各种操作中的任何一种，并且有一个正当的理由，并且还有一个情况，即一个平原.update()实际上会“可靠地”告诉您是否确实进行了修改。

浏览案例：

检查返回的内容

这基本上涉及查看返回文档中的数组，以查看我们要求删除的内容是否确实存在：

var pullId = "5961de06ea264532c684611a";

Office.findByIdAndUpdate(1,
  { "$pull": { "branches": { "_id": pullId } } },
  { "new": true }
).then(office => {
  // Check if the supplied value is still in the array
  console.log(
    "Still there?: %s",
    (office.branches.find( b => b._id.toHexString() === pullId))
      ? true : false
  );
}).catch(err => console.error(err))
Run Code Online (Sandbox Code Playgroud)

我们使用.toHexString()是为了比较来自 an 的实际值，ObjectId因为 JavaScript 只是不与“对象”“相等”。如果提供已经“强制转换”为ObjectId值的内容，您将同时检查“左”和“右” ，但在这种情况下，我们知道另一个输入是“字符串”。

只需使用 .update()，“它是可靠的”

这里要考虑的另一种情况是您是否“确实需要”返回的修改数据。因为该.update()方法将可靠地返回一个结果，告诉您是否实际修改了任何内容：

Office.update(
  { "_id": 1 },
  { "$pull": { "branches": { "_id": pullId } } },
).then(result => {
  log(result);
}).catch(err => console.error(err))
Run Code Online (Sandbox Code Playgroud)

其中result，这里将是这样的：

{
  "n": 1,
  "nModified": 1,        // <--- This always tells the truth, and cannot lie!
  "opTime": {
    "ts": "6440673063762657282",
    "t": 4
  },
  "electionId": "7fffffff0000000000000004",
  "ok": 1
}
Run Code Online (Sandbox Code Playgroud)

其中nModified是一个“真实”指标，表明某些内容是否“实际更新”。因此，如果它1当时$pull确实有效果，但是0实际上没有从数组中删除任何内容并且没有修改任何内容。

这是因为该方法实际上使用了更新的 API，该 API 确实具有指示实际修改的可靠结果。这同样适用于 a 之类的东西$set，它实际上并没有改变值，因为提供的值等于文档中已经存在的值。

发现并修改谎言！

在仔细查看文档时，您可能会想到的另一种情况是实际检查“原始结果”并查看文档是否被修改。规范中实际上有一个指标。

问题是（以及需要更多的 Promises 工作）结果实际上并不真实：

var bogusId = "5961de06ea264532c684611a"; // We know this is not there!

Promise((resolve,reject) => {
  Office.findByIdAndUpdate(1,
    { "$pull": { "branches": { "_id": bogusId } } },
    { "new": true, "passRawResult" },
    (err,result,raw) => {        // We cannot pass multiple results to a Promise
      if (err) reject(err);
      resolve({ result, raw });   // So we wrap it!
    }
  )
})
.then(response => log(response.raw))
.catch(err => console.error(err));
Run Code Online (Sandbox Code Playgroud)

这里的问题是，即使我们“知道”这不应该修改，响应却另有说明：

{
  "lastErrorObject": {
    "updatedExisting": true,
    "n": 1                     // <--- LIES! IT'S ALL LIES!!!
  },
  "value": {
    "_id": 1,
    "name": "My Office",
    "branches": [
      {
        "address": "Third address",
        "isPrincipal": false,
        "_id": "5961de06ea264532c6846118"
      }
    ],
    "__v": 0
  },
  "ok": 1,
  "_kareemIgnore": true
}
Run Code Online (Sandbox Code Playgroud)

因此，即使在从回调响应中获取“第三个”参数的所有工作之后，我们仍然没有被告知有关更新的正确信息。

总结

因此，如果您想通过单个请求“可靠地”执行此操作（并且您无法通过多个请求可靠地执行此操作，因为文档可能会在两者之间更改！），那么您的两个选项是：

  1. 检查返回的文档以查看您要删除的数据是否仍然存在。

  2. 忘记返回一个.update()总是告诉你“真相”的文件和信任；）

您使用哪一种取决于应用程序使用模式，但这是返回“可靠”结果的两种不同方式。

清单的一点点

所以可以肯定的是，这里列出了所有示例并演示了它们实际返回的内容：

const async = require('async'),
      mongoose = require('mongoose'),
      Schema = mongoose.Schema;

mongoose.Promise = global.Promise;
mongoose.set('debug',true);

mongoose.connect('mongodb://localhost/test');

const branchesSchema = new Schema({
  address: String,
  isPrincipal: Boolean
});

const officeSchema = new Schema({
  _id: Number,
  name: String,
  branches: [branchesSchema]
},{ _id: false });

const Office = mongoose.model('Office', officeSchema);

function log(data) {
  console.log(JSON.stringify(data,undefined,2))
}

const testId = "5961a56d3ffd3d5e19c61610";

async.series(
  [
    // Clean data
    (callback) =>
      async.each(mongoose.models,(model,callback) =>
        model.remove({},callback),callback),

    // Insert some data and pull
    (callback) =>
      async.waterfall(
        [
          // Create and demonstrate
          (callback) =>
            Office.create({
              _id: 1,
              name: "My Office",
              branches: [
                {
                  address: "Some street, that avenue",
                  isPrincipal: true
                },
                {
                  address: "Another address",
                  isPrincipal: false
                },
                {
                  address: "Third address",
                  isPrincipal: false
                }
              ]
            },callback),

          // Demo Alternates
          (office,callback) =>
            async.mapSeries(
              [true,false].map((t,i) => ({ t, branch: office.branches[i] })),
              (test,callback) =>
                (test.t)
                  ? Office.findByIdAndUpdate(office._id,
                      { "$pull": { "branches": { "_id": test.branch._id } } },
                      { "new": true , "passRawResult": true },
                      (err,result,raw) => {
                        if (err) callback(err);
                        log(result);
                        log(raw);
                        callback();
                      })
                  : Office.findByIdAndUpdate(office._id,
                      { "$pull": { "branches": { "_id": test.branch._id } } },
                      { "new": true } // false here
                    ).then(result => {
                      log(result);
                      console.log(
                        "Present %s",
                        (result.branches.find( b =>
                          b._id.toHexString() === test.branch._id.toHexString() ))
                          ? true : false
                      );
                      callback();
                    }).catch(err => callback(err)),
              callback
            )
        ],
        callback
      ),

    // Find and demonstate fails
    (callback) =>
      async.waterfall(
        [
          (callback) => Office.findOne({},callback),

          (office,callback) =>
            async.eachSeries([true,false],(item,callback) =>
              (item)
                ? Office.findByIdAndUpdate(office._id,
                    { "$pull": { "branches": { "_id": testId } } },
                    { "new": true, "passRawResult": true },
                    (err,result,raw) => {
                      if (err) callback(err);
                      log(result);
                      log(raw);
                      callback();
                    }
                  )
                : Office.findByIdAndUpdate(office._id,
                    { "$pull": { "branches": { "_id": testId } } },
                    { "new": true }
                  ).then(result => {
                    console.log(result);
                    console.log(
                      "Present %s",
                      (result.branches.find( b =>
                        b._id.toHexString() === office.branches[0]._id.toHexString()))
                        ? true : false
                    );
                    callback();
                  })
                  .catch(err => callback(err)),
              callback)

        ],
        callback
      ),

    // Demonstrate update() modified shows 0
    (callback) =>
      Office.update(
        {},
        { "$pull": { "branches": { "_id": testId } } }
      ).then(result => {
        log(result);
        callback();
      })
      .catch(err => callback(err)),

    // Demonstrate wrapped promise
    (callback) =>
      Office.findOne()
        .then(office => {
          return new Promise((resolve,reject) => {
            Office.findByIdAndUpdate(office._id,
              { "$pull": { "branches": { "_id": testId } } },
              { "new": true, "passRawResult": true },
              (err,result,raw) => {
                if (err) reject(err);
                resolve(raw)
              }
            );
          })
        })
        .then(office => {
          log(office);
          callback();
        })
        .catch(err => callback(err))

  ],
  (err) => {
    if (err) throw err;
    mongoose.disconnect();
  }
);
Run Code Online (Sandbox Code Playgroud)

它产生的输出：

Mongoose: offices.remove({}, {})
Mongoose: offices.insert({ _id: 1, name: 'My Office', branches: [ { address: 'Some street, that avenue', isPrincipal: true, _id: ObjectId("5961e5211a73e8331b44d74b") }, { address: 'Another address', isPrincipal: false, _id: ObjectId("5961e5211a73e8331b44d74a") }, { address: 'Third address', isPrincipal: false, _id: ObjectId("5961e5211a73e8331b44d749") } ], __v: 0 })
Mongoose: offices.findAndModify({ _id: 1 }, [], { '$pull': { branches: { _id: ObjectId("5961e5211a73e8331b44d74b") } } }, { new: true, passRawResult: true, upsert: false, remove: false, fields: {} })
{
  "_id": 1,
  "name": "My Office",
  "__v": 0,
  "branches": [
    {
      "address": "Another address",
      "isPrincipal": false,
      "_id": "5961e5211a73e8331b44d74a"
    },
    {
      "address": "Third address",
      "isPrincipal": false,
      "_id": "5961e5211a73e8331b44d749"
    }
  ]
}
{
  "lastErrorObject": {
    "updatedExisting": true,
    "n": 1
  },
  "value": {
    "_id": 1,
    "name": "My Office",
    "branches": [
      {
        "address": "Another address",
        "isPrincipal": false,
        "_id": "5961e5211a73e8331b44d74a"
      },
      {
        "address": "Third address",
        "isPrincipal": false,
        "_id": "5961e5211a73e8331b44d749"
      }
    ],
    "__v": 0
  },
  "ok": 1,
  "_kareemIgnore": true
}
Mongoose: offices.findAndModify({ _id: 1 }, [], { '$pull': { branches: { _id: ObjectId("5961e5211a73e8331b44d74a") } } }, { new: true, upsert: false, remove: false, fields: {} })
{
  "_id": 1,
  "name": "My Office",
  "__v": 0,
  "branches": [
    {
      "address": "Third address",
      "isPrincipal": false,
      "_id": "5961e5211a73e8331b44d749"
    }
  ]
}
Present false
Mongoose: offices.findOne({}, { fields: {} })
Mongoose: offices.findAndModify({ _id: 1 }, [], { '$pull': { branches: { _id: ObjectId("5961a56d3ffd3d5e19c61610") } } }, { new: true, passRawResult: true, upsert: false, remove: false, fields: {} })
{
  "_id": 1,
  "name": "My Office",
  "__v": 0,
  "branches": [
    {
      "address": "Third address",
      "isPrincipal": false,
      "_id": "5961e5211a73e8331b44d749"
    }
  ]
}
{
  "lastErrorObject": {
    "updatedExisting": true,
    "n": 1
  },
  "value": {
    "_id": 1,
    "name": "My Office",
    "branches": [
      {
        "address": "Third address",
        "isPrincipal": false,
        "_id": "5961e5211a73e8331b44d749"
      }
    ],
    "__v": 0
  },
  "ok": 1,
  "_kareemIgnore": true
}
Mongoose: offices.findAndModify({ _id: 1 }, [], { '$pull': { branches: { _id: ObjectId("5961a56d3ffd3d5e19c61610") } } }, { new: true, upsert: false, remove: false, fields: {} })
{ _id: 1,
  name: 'My Office',
  __v: 0,
  branches:
   [ { address: 'Third address',
       isPrincipal: false,
       _id: 5961e5211a73e8331b44d749 } ] }
Present true
Mongoose: offices.update({}, { '$pull': { branches: { _id: ObjectId("5961a56d3ffd3d5e19c61610") } } }, {})
{
  "n": 1,
  "nModified": 0,
  "opTime": {
    "ts": "6440680872013201413",
    "t": 4
  },
  "electionId": "7fffffff0000000000000004",
  "ok": 1
}
Mongoose: offices.findOne({}, { fields: {} })
Mongoose: offices.findAndModify({ _id: 1 }, [], { '$pull': { branches: { _id: ObjectId("5961a56d3ffd3d5e19c61610") } } }, { new: true, passRawResult: true, upsert: false, remove: false, fields: {} })
{
  "lastErrorObject": {
    "updatedExisting": true,
    "n": 1
  },
  "value": {
    "_id": 1,
    "name": "My Office",
    "branches": [
      {
        "address": "Third address",
        "isPrincipal": false,
        "_id": "5961e5211a73e8331b44d749"
      }
    ],
    "__v": 0
  },
  "ok": 1,
  "_kareemIgnore": true
}
Run Code Online (Sandbox Code Playgroud)

归档时间：

查看次数：

3324 次

最近记录：

8 年，4 月 前
相关归档
<script type ="text/template"> ... </ script>的说明 446
JavaScript:将参数传递给回调函数 271
如何从JavaScript中的字符串中提取基本URL？ 163
您可以通过脚本确定Chrome是否处于隐身模式吗？ 107
快递路线参数条件 65
.NET中是否可以使用非阻塞,单线程,异步Web服务器(如Node.js)？ 57
假"单击"以激活oncl​​ick方法 46
Mongoose pre.save()异步中间件无法按预期工作 19
npm WARN bootstrap@4.2.1需要jquery@1.9.1-3的对等节点，但未安装 11
在MongoDB中,将帖子的所有评论保存在一个文档中是否实用？ 8
难疑归档
可以在JSON中使用注释吗？ 7104
在JavaScript中深度克隆对象的最有效方法是什么？ 5181
CSS三角形如何工作？ 1791
静态类和单例模式之间的区别？ 1708
从JSON文件中解析值？ 1400
如何在JavaScript正则表达式中访问匹配的组？ 1277
如何在正则表达式中使用变量？ 1250
如何从另一个分支中获取一个文件 1154
如何从git存储库中删除目录？ 1138
按ID删除元素 1085