我有以下代码,它创建了一个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);
};