Loop、DNS 和 Bluebird Js - promise 仍然可以异步工作

Question

我的函数有一个小问题，它应该检查具有特定名称的域是否在几个顶级域下可用。当我像 whois("example434"); 这样调用这个函数时它 returns undefined 而不是数组 ["example424.org", ..., "example424.com"] 。如何修改函数来解决这个问题？我哪里弄错了？

function whois (name, domains = '*') {
    if ( typeof domains == 'Object' && domains.length > 0 ) {
      var TLD = domains;
    } else {
      var TLD = ['.com','.net','.org','.edu','.gov', '.biz', '.info', '.mobi', '.me', '.tv', '.info', '.io'];
    }

    var available = [];
    Promise.resolve(
      Promise.map(TLD, function(domain) {
        return dns.resolve4(name + domain, (err, addresses) => {
          if (!addresses || addresses.length <= 0 ){
            console.log(name + domain);
            available.push(name + domain);
          }
        });
      })
    ).then(()=>{
      console.log(available);
      return available;
    });
}

Answer 1

您的主要问题是：

dns.resolve4 看起来不像是 return 承诺，因为它使用 nodejs 回调模式，因为它不是 return 承诺，Promise.map 不会等待异步代码完成
为什么要在 Promise.resolve 中包装 Promise.map - 没有意义，因为 Promise.map return 是一个承诺
函数 whois 从来没有 return 值，因此你得到 undefined
typeof domains 永远不可能是 Object，你的意思是 object - 即使这不是确定值是否为 Array[=42 的好方法=]

您的标题包含以下短语：promise 仍然可以异步工作 - 嗯，是的，您还期待其他东西吗？异步代码就是异步的，什么都不会改变

无论如何，您的代码中有问题注释：

function whois(name, domains = '*') {
    // Issue 4
    if (typeof domains == 'Object' && domains.length > 0) {
        var TLD = domains;
    } else {
        var TLD = ['.com', '.net', '.org', '.edu', '.gov', '.biz', '.info', '.mobi', '.me', '.tv', '.info', '.io'];
    }

    var available = [];
    // Issue 2
    Promise.resolve(
        Promise.map(TLD, function(domain) {
            // Issue 1
            return dns.resolve4(name + domain, (err, addresses) => {
                if (!addresses || addresses.length <= 0) {
                    console.log(name + domain);
                    available.push(name + domain);
                }
                // possible error: if there's an error (err is not falsey) does that really mean the domain is available?
            });
        })
    ).then(() => {
        console.log(available);
        return available;
    });
    // Issue 3
}

您需要在下面的代码中承诺 dns.resolve4 函数，const resolve4 =。

我知道 bluebirdjs 和后来的 nodejs 已经内置了 promisify 方法，但是这些方法的逻辑意味着 err 拒绝了 promise 你可能不想要。

此外，您希望针对不同的成功结果进行不同的解析，因此，最好按原样"by hand"承诺

使用Array.isArray确定TLD是否为数组

function whois (name, TLD) {
    // if TLD isn't an array, use the default array
    if (!Array.isArray(TLD)) {
        TLD = ['.com','.net','.org','.edu','.gov', '.biz', '.info', '.mobi', '.me', '.tv', '.info', '.io'];
    }

    // prepare the list of full domains to check
    const domains = TLD.map(domain => name + domain);

    const resolve4 = domain => new Promise(resolve => {
        dns.resolve4(domain, (err, addresses) => {
            if (addresses && addresses.length) {
                // domain resolves - so it's not available
                resolve(false);
            }
            // uncomment the next two lines if you want an error to indicate the domain is NOT available
            //else if (err) {
            //    resolve(false);
            } else {
                // it's available
                resolve(domain);
            }
        });
    });

    return Promise.map(domains, resolve4)
    // filter for available domains (i.e. return an array of non-falsey results)
    .then(results => results.filter(domain => domain));
}

Loop、DNS 和 Bluebird Js - promise 仍然可以异步工作

Loop, DNS and Bluebirdjs - promise still works asynchronouse

javascript

promise

bluebird