node.js - Bluebird promise 乘坐快车/ Mongoose 返回太早

Question

我对 promise 很陌生，想知道为什么以下内容不起作用。我之所以这么假设，是因为我需要以特定的方式设置自定义方法来使用 Promise。

我使用 bluebird 作为我的 Promise 库。我试图通过在这些各自的模型(即讨论和用户模型)上使用 Mongoose 静态和方法来获取给定“用户”用户名的“讨论”对象列表。

下面是我在 Express 中的请求处理程序。现在只是尝试获取 JSON 并将其作为响应发送:

/* /routes/users.js */

var Promise = require('bluebird');
var User = require('../models/').User

/* ... express inclusions, other routes ... */

/* Get list of discussions for user */
router.get('/:username/discussions', function(req, res) {
    User.findOne({username: req.params.username}).then(function(user) {
        if (user) {
            return user.getDiscussions();
        } else {
            res.send('no user found');
        }
    }).then(function(discussions) {
        res.send(discussions);
    }).catch(function(error) {
        console.log('ERROR: ' + error);
        res.send('ERROR: ' + error);
    });
});

下面是 user.getDiscussions():

/* /models/user.js */

var Promise = require('bluebird');
var mongoose = Promise.promisifyAll(require('mongoose'));
var Discussion = require('./discussion').Discussion;
var Schema = mongoose.Schema;

/* ... UserSchema defined here ... */

UserSchema.methods.getDiscussions = Promise.method(function() {
    var self = this;
    self.model('User').findById(self.id).then(function(user) {
        if (!user) {
            Promise.reject('User \'' + self.id + '\' does not exist.');
        } else {
            return Discussion.getDiscussionsForUser(self.username);
        }
    });
});

最后，Discussion.getDiscussionsForUser:

/* /models/discussion.js */

var Promise = require('bluebird');
var mongoose = Promise.promisifyAll(require('mongoose'));
var Schema = mongoose.Schema;

/* ... DiscussionSchema defined here ... */

DiscussionSchema.statics.getDiscussionsForUser = Promise.method(function(username) {
    var self = this;
    self.model('User').findOne({username: username}, 'discussions').then(function(user) {
        if (!user) {
            return Promise.reject('User with username \'' + username + '\' does not exist.');
        } else {
            var discussions = [];
            return self.model('Discussion').find({
                '_id': { $in : user.discussions }
            });
        }
    });
});

看来路由中的请求提前完成并在响应中发送未定义的内容。知道为什么讨论模型中从 Mongoose 获得的讨论没有冒泡到路由吗？

Answer 1

getDiscussions() 和 getDiscussionsForUser() 未正确返回 promise 。按如下方式更改它们:

UserSchema.methods.getDiscussions = Promise.method(function() {
    var self = this;
    // *** added a return to this next line
    return self.model('User').findById(self.id).then(function(user) {
        if (!user) {
            // *** added a return to this next line
            return Promise.reject('User \'' + self.id + '\' does not exist.');
        } else {
            return Discussion.getDiscussionsForUser(self.username);
        }
    });
});

与 getDiscussionsForUser() 相同，应该是这样的:

DiscussionSchema.statics.getDiscussionsForUser = Promise.method(function(username) {
    var self = this;
    // *** added a return to the next line
    return self.model('User').findOne({username: username}, 'discussions').then(function(user) {
        if (!user) {
            return Promise.reject('User with username \'' + username + '\' does not exist.');
        } else {
            var discussions = [];
            return self.model('Discussion').find({
                '_id': { $in : user.discussions }
            });
        }
    });
});

为了使调用者能够使用 Promise 协调函数内异步操作的完成，通过该函数的所有代码路径必须始终从该函数返回 Promise。进入 .then() 处理程序后，如果存在进一步嵌套的异步操作，则必须返回一个 Promise，或者如果您希望该值成为您所在 Promise 的值，则可以只返回一个值。