javascript - Loop、DNS 和 Bluebird Js - Promise 仍然异步工作

Question

我的函数有一个小问题，该函数应该检查具有特定名称的域在几个顶级域下是否可用。当我像 whois("example434"); 这样调用这个函数时它返回 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

您的主要问题是:

1. dns.resolve4 看起来不像返回 Promise，因为它使用 NodeJS 回调模式，由于它不返回 Promise，Promise.map 不会等待异步代码完成
2. 为什么将 Promise.map 包装在 Promise.resolve 中 - 没有意义，因为 Promise.map 返回一个 Promise
3. 函数 whois 永远不会返回值，因此您会得到 undefined
4. typeof fields 永远不可能是 Object，您的意思是 object - 即使这不是确定 vlaue 是否是一个的好方法数组

你的标题包含这样一句话: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
}

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

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

此外，您希望针对不同的成功结果采取不同的解决方案，因此，最好“手动” promise

使用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));
}