蓝鸟承诺变量：'未定义不是函数'

Question

我有一个 nodejs express 服务器，我正在使用 bluebird Promises 来同步所有异步内容。

使用 AWS RDS MySQL 数据库在 localhost 上一切正常，但是当我将服务器上传到我的 AWS EC2 实例时，我发现此功能存在问题：

var Promise = require('bluebird');
var db      = require('./db');

exports.matchValue = function (params) {
    return new Promise(function(resolve, reject) {
        var findValue = new Promise(); 
        if (params.find.includes(".")) {
            var aux = params.find.split(".");
            var matchBy = {};
            if (aux[0]) matchBy.a = aux[0];
            if (aux[1]) matchBy.b = aux[1];
            findValue = db.getValues1(params.limit,params.page,matchBy);
        }
        else {
            findValue = db.getValues2(params.limit,params.page,params.find);
        }
        findValue
        .then(function(result) {
            resolve(result);
        })
        .catch(function(err) {
            reject(err);
        });
    }); 
}

我已将变量 findValue 声明为新的 Promise，因为根据 if 条件，它将接收不同数据库查询函数的值（此函数返回一个 Promise）。

当我调用这个函数时，结果如下："undefined is not a function".

我了解这种行为的发生是因为它首先执行 findValue.then() 而不是 if/else 块代码，并且由于变量未定义，它可以是一个函数。

我以为将一个变量声明为一个新的 Promise 它会等到分配给这个变量的函数的返回完成，但实际上并没有发生。

我做错了什么？有人能帮我吗？

谢谢你的建议！！

Answer 1

我已将变量 findValue 声明为新的 Promise，因为根据 if 条件，它将接收不同数据库查询函数的值（此函数返回 Promise）。

在 JavaScript 中，您不能将变量声明为任何类型。这一行：

var findValue = new Promise();

声明一个变量 (var findValue) 并尝试创建一个承诺 (= new Promise())。 Promise 构造函数要求你向它传递一个函数。它尝试将其第一个参数用作函数，并且由于您没有向它传递任何参数，因此它会为第一个参数获取undefined - 因此出现错误。

你可以把它改成

var findValue;

...但是您的代码正在成为不必要的 Promise 创建谬误的牺牲品：只需使用您已经拥有的 Promise：

exports.matchValue = function (params) {
    var findValue;
    if (params.find.includes(".")) {
        var aux = params.find.split(".");
        var matchBy = {};
        if (aux[0]) matchBy.a = aux[0];
        if (aux[1]) matchBy.b = aux[1];
        findValue = db.getValues1(params.limit,params.page,matchBy);
    }
    else {
        findValue = db.getValues2(params.limit,params.page,params.find);
    }
    return findValue;
}

Answer 2

假设您的 db.getValues1/db.getValues2 函数返回承诺（它们似乎不接受回调，我拒绝假设您在生产中使用同步数据库驱动程序），请尝试以下版本的发布代码：

var Promise = require('bluebird');
var db      = require('./db');

exports.matchValue = function (params) {
    var findValue;

    if (params.find.includes(".")) {
        var aux = params.find.split(".");
        var matchBy = {};
        if (aux[0]) matchBy.a = aux[0];
        if (aux[1]) matchBy.b = aux[1];
        findValue = db.getValues1(params.limit,params.page,matchBy);
    }
    else {
        findValue = db.getValues2(params.limit,params.page,params.find);
    }

    return findValue;
}

如果它有效，那就太好了。以下是一些需要考虑的事项：

1. 您正在创建一个新的 Promise。不要那样做。使用 Bluebird 的 promisification 例程 Promisify 标准的、符合节点的函数/库。

2. Bluebird 2.x 和 3.x 将抛出 var findValue = new Promise()。正如已经指出的，构造函数需要一个函数作为其参数。我怀疑您解释了您实际收到的错误消息。如果是这样，请不要这样做，这会让其他人很难找出问题所在。

3. 下次您发布到 SO 时，请将您的示例代码缩减为任何人都可以运行并轻松重现您的问题的代码。您可能会发现在解决您自己的问题的最小复制过程中。如果您不这样做，这将使这里的人们能够更有效地帮助您。

Answer 3

我遇到了类似的问题，巧合的是我也在使用 Sequelize。

这里是重现问题的单行代码：

Promise.resolve({ a: 1 }).then(([x]) => {});

输出：

未处理的拒绝类型错误：未定义不是函数

这个错误的原因是 JavaScript 试图在解析的值上执行 [Symbol.iterator]()，但由于它是一个对象，它没有 [Symbol.iterator] - 错误消息中的 undefined 是 value[Symbol.iterator] .

更好的错误信息应该是 value is not iterable 行中的内容。

至于你为什么会出现这种情况，那一定是 Sequelize 的 bug。如果您仍然遇到问题，请提出问题。

Answer 4

您的 .matchValue() 函数应简化为：

exports.matchValue = function(params) {
    var aux = params.find.split('.');
    return (aux.length < 2) ? db.getValues2(params.limit, params.page, params.find) : db.getValues1(params.limit, params.page, { 'a': aux[0], 'b': aux[1] });
};

有了这个，.then(function(result) 肯定不会抛出，因为那行已经消失了，但是函数的调用者很可能会抛出。如果是这样，那么您必须（再次）怀疑 db.getValues1() 或 db.getValues2() 没有在 EC2 服务器上返回承诺；尝试记录返回的值/对象以查看它是什么。