函数 args 在 return new Promise(function(resolve, reject) 内部不可见

Question

参数et 和instance，传递给promisified 的函数在return new promise 块中不可见（被视为null）。 注意：

rp = require('request-promise');
Promise = require('bluebird');

.. 和我自己的一些模块，如 urlLib，在顶部是“必需的”。 rp 库是承诺的请求库...如果我在 return 语句中明确设置 et 和实例变量，它就可以工作。 （参见注释的 et 和实例声明）。

module.exports.getEntityInstance = function (et, instance) {

return new Promise(function(resolve, reject) {

    // explicit setting, to test, works fine.
    // var et = "device", instance = "33";

    if ((!et) || (!instance)) {
        // ****** FAILING HERE *******
        reject(new Error(utilLib.errorAsJSON ("Invalid et/instance arg")));
    }
    OPTS.url = base_url + et + "/" + instance;
    rp(OPTS)
        .then(function (data) {
            resolve(data)
        })
    .catch(function (err) {
            reject(err);
        });
});
}

这是纯正的代码：

module.exports.em7GetEntityInstance = function (et, instance) {

log.info("----- et before promise:(" + et + ")");
log.info("----- instance before promise:(" + instance + ")");

return new Promise(function(resolve, reject) {
    // var et = "device";
    // var instance = "33";

    // if null or undefined...
    if (!et || !instance) {
        log.error("---- et and/or instance args null.  rejecting");
        reject(new Error(utilLib.errorAsJSON ("getEntityInstace():Invalid entity type argument")));
    }
    var et = et.toLowerCase().trim();
    var url = baseurl + et + "/" + instance
    log.info("url:(" + url + ")");
    OPTS.url = url;
    rp(OPTS)
        .then(function (data) {
            log.debug ("success. invoking resolve()");
            resolve(data)
        })
    .catch(function (err) {
            log.error("+++ rp failed:(" + err  + ")");
            reject(err);
        });
});

}

还有日志语句（它是一个服务器进程（没有 console.log）。使用 winston 日志模块。

2016-05-05T15:44:06.866Z - info: ----- et before promise:(device)
2016-05-05T15:44:06.867Z - info: ----- instance before promise:(33)
2016-05-05T15:44:06.867Z - error: ---- et and/or instance args null.  rejecting

Answer 1

您的内部函数中的var et 确实会从外部范围中隐藏具有相同名称的变量。评估测试时是undefined。

避免使用var，使用不同的变量名，或者只是这样做

var url = baseurl + et.toLowerCase().trim() + "/" + instance;

完全没有中间变量。

---

但是you shouldn't use an inner function at all，宁可这样做

module.exports.em7GetEntityInstance = function(et, instance) {
    log.info("----- et before promise:(" + et + ")");
    log.info("----- instance before promise:(" + instance + ")");
    // if null or undefined...
    if (et==null || instance==null) {
        log.error("---- et and/or instance args null.  rejecting");
        return Promise.reject(new Error(utilLib.errorAsJSON ("getEntityInstace():Invalid entity type argument")));
    }
    var url = baseurl + et.toLowerCase().trim() + "/" + instance;
    log.info("url:(" + url + ")");
    OPTS.url = url;
    return rp(OPTS).then(function(data) {
        log.debug ("success. invoking resolve()");
        return data;
    }, function(err) {
        log.error("+++ rp failed:(" + err  + ")");
        throw err;
    });
};