我刚开始尝试在我的 Node.js 应用程序上实现 Promises。现在我正在检查用户和密码是否存在，然后使用 mongodb 查找它们，如果没有找到用户，它会设置 promise.reject() 但它太早地返回承诺并且它仍处于挂起状态。如果有人可以帮助我或给我有关如何重构的想法，将不胜感激。
【问题讨论】：
当它仍处于待处理状态时，它返回承诺是预期的行为。
您应该使用then() 来检查承诺解决方案。
另外一点是你在开始使用promise的时候不应该使用传统的mongo接口，而是promisifyCollection原型的所有方法，这样你就可以返回mongo find方法创建的Promise及其链可以的。否则我看不到使用承诺的意义。您这样做的方式不会减少您必须编写的代码量。
顺便说一句，恕我直言，您应该使用 bluebird 而不是 Q，因为它是目前唯一一个不是特别慢的 Promise 库。
例子：
在 db.js 中
var mongoClient = require('mongodb').MongoClient;
var collection = require('mongodb').Collection;
var Promise = require('bluebird');
// We promisify everything. Bluebird add ***Async function which return promises.
Promise.promisifyAll(collection.prototype);
Promise.promisifyAll(mongoClient);
//In mongodb cursor is not built from protoype I use to promisify it each time. Not necessary if you don't use cursors.
collection.prototype._find = collection.prototype.find;
collection.prototype.find = function() {
var cursor = this._find.apply(this, arguments);
cursor.toArrayAsync = Promise.promisify(cursor.toArray, cursor);
return cursor;
};
//then you connect to the DB and exports your collections...
在其他地方，以你为例：
this.loginUser = function(user) {
var userid = user.userid,
var password = (user.password) ?
crypto.createHash("md5").update(user.password).digest("hex"):
undefined
var rememberme = user.rememberme;
if(userid && password) {
// we directly return the promise from mongodb, that we chain with a then
return db.users.findOneAsync({ email: userid, password: password }) // return a promise that we chain
.then(function(user) { // that we chain
if(user) {
var logincode = uuid.v4(),
var token = jwt.encode({email: userid, password: password}, secret);
if(rememberme) {
res.cookie("clogincode", logincode, { magAge: 900000 } );
}
return user; // return in a then callback will just resolve the promise
} else {
throw new Error('User not found'); // throwing in a then callback will just reject the promise
}
}); // end of the then, then return a promise that we return (with the return keyword above)
} else {
return Promise.reject("Username or Password was not entered"); // this is the only case where we have to actually create a promise ourself
}
}
【讨论】：