猫鼬和es6的承诺没有如预期的那样发挥作用



我有以下代码,它创建了一个promise数组来保存一些数字,然后生成promise(使用co-library(并打印出结果。然而,我不明白的是,当它打印输出时,它会打印同一条记录10次。

这是代码:

'use strict'
const Promise = require('bluebird');
const co = require('co');
const _ = require('lodash');
const mongoose = require('mongoose');
// plug in the bluebird promise library for mongoose
mongoose.Promise = Promise;
mongoose.connect('mongodb://localhost:27017/nodejs_testing');
const numSchema = new mongoose.Schema({
  num: { type: Number, required: true }
});
const Num = mongoose.model('Num', numSchema);
let promises = [];
let x;
// create an array of promises to save some numbers
for (let i = 0; i < 10; ++i) {
  let p = new Promise((resolve,reject) => {
    x = Num();
    x.num = i;
    x.save((err) => {
      if (err) {
        reject(err);
      } else {
        resolve(x);
      }
    });
  });
  promises.push(p);
};
// yield all the promises, then print out the results
co(function * () {
  let res = yield Promise.all(promises);
  _.each(res, item => {
    console.log(JSON.stringify(item));
  });
  mongoose.disconnect();
});

这是输出:

/tmp/test$ node m
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}
{"__v":0,"num":9,"_id":"57d1931037a370055f51977c"}

请注意,如果我在Promise中声明变量x,那么我会得到预期的结果(例如,输出中的10个不同数字(。换句话说,如果我做了这个改变(见下文(,它会像预期的那样工作:

  let p = new Promise((resolve,reject) => {
    let x = Num(); // <--- declare x inside the promise
    .
    .
  });

我的问题是,为什么代码会这样运行?请注意,如果我使用mongodb/mongoose重复完全相同类型的测试而不是,并且只打印一些数字,即使在Promise之外声明了x,它也能按预期工作。下面的示例代码:

'use strict'
const Promise = require('bluebird');
const co = require('co');
const _ = require('lodash');
class Number {
  constructor(num) {
    this.num = num;
  }
};
let x;
let promises = [];
for (let i = 0; i < 10; ++i) {
  let p = new Promise((resolve,reject) => {
    setTimeout(() => {
      x = new Number(i);
      resolve(x);
    }, 300);
  });
  promises.push(p);
};
co(function * () {
  let res = yield Promise.all(promises);
  _.each(res, item => {
    console.log(JSON.stringify(item));
  });
});

输出:

/tmp/test$ node t
{"num":0}
{"num":1}
{"num":2}
{"num":3}
{"num":4}
{"num":5}
{"num":6}
{"num":7}
{"num":8}
{"num":9}

区别不在于Mongoose与非Mongoose。您的代码正在执行不同的操作。

在您的第一个示例中,您有(参见***注释(:

let p = new Promise((resolve,reject) => {
  x = Num();               // *** A
  x.num = i;
  x.save((err) => {
    if (err) {
      reject(err);
    } else {
      resolve(x);          // *** B
    }
   });
});

其中x在代码所在的循环之外声明,因此所有迭代都重用该变量。

请注意,上面标记为A和B的语句彼此异步发生。到B发生时,所有迭代都已完成A;由于B看到分配给x最后一个值,这就是它用来解析的,并且它们都是用相同的值解析的。

与您的第二个示例相比:

let p = new Promise((resolve,reject) => {
  setTimeout(() => {
    x = new Number(i);     // *** A
    resolve(x);            // *** B
  }, 300);
});

请注意,这两者现在是同步发生的;CCD_ 9每次进行解析时都使用CCD_ 10的当前值。

这就是两者行为差异的原因。

从根本上讲,x应该声明得更接近它的使用位置,的promise init回调中:

//let x;                         // *** Not here
// create an array of promises to save some numbers
for (let i = 0; i < 10; ++i) {
  let p = new Promise((resolve,reject) => {
    let x = Num();               // *** Here
    x.num = i;
    x.save((err) => {
      if (err) {
        reject(err);
      } else {
        resolve(x);
      }
     });
  });
}

记住规则是:总是在尽可能小的范围内声明。

发生这种情况的原因是x不在for循环的范围内。当您运行for循环时,您不会更改x的其他变量的实例化,而是重新分配原始x的值。实际情况是,x的最终值是Num的值在其他值保存到Mongo之前为9,而数组中的其他promise在x设置为9之前未解析。

如果你想得到正确的输出,只需在for循环中放置x即可:

// create an array of promises to save some numbers
for (let i = 0; i < 10; ++i) {
  let x;
  let p = new Promise((resolve,reject) => {
    x = Num();
    x.num = i;
    x.save((err) => {
      if (err) {
        reject(err);
      } else {
        resolve(x);
      }
    });
  });
  promises.push(p);
};

最新更新