約束で複数のリクエストを使用する

Question

request-promiseモジュールを使用して複数のWebサイトをチェックしようとしています。デザインごとにPromise.allを使用した場合、約束は最初の却下で返されます。複数の要求タスクを実行し、すべての要求が満たされても拒否されても完了するのを待つ適切な方法は何ですか？私は次の2つの機能を思いついた。

CheckSitesV1は、約束を拒否したため例外を返します。しかし、CheckSitesV2は、それらが満たされているか拒否されているかに関係なく、すべての約束が完了するのを待ちます。私が書いたコードが合理的であるかどうかをコメントすることができれば幸いです。私は

const sitesArray = ['http://www.example.com','https://doesnt-really-exist.org','http://www.httpbin.org']; 

async function CheckSitesV1() { 
    let ps = []; 
    for (let i = 0; i < sitesArray.length; i++) { 
     let ops = { 
      method: 'GET', 
      uri:sitesArray[i], 
     }; 
     const resp = await rp.get(ops); 
     ps.push(resp); 
    } 
    return Promise.all(ps) 
} 

function CheckSitesV2() { 
    let ps = []; 
    for (let i = 0; i < sitesArray.length; i++) { 
     let ops = { 
      method: 'GET', 
      uri:sitesArray[i], 
     }; 
     ps.push(rp.get(ops)); 
    } 
    return Promise.all(ps.map(p => p.catch(e => e))) 
} 

CheckSitesV1().then(function (result) { 
    console.log(result); 
}).catch(function (e) { 
    console.log('Exception: ' + e); 
}); 

CheckSitesV2().then(function (result) { 
    console.log(result); 
}).catch(function (e) { 
    console.log('Exception: ' + e); 
}); 

Answer 1

あなた

function CheckSitesV2() { 
    let ps = []; 
    for (let i = 0; i < sitesArray.length; i++) { 
     let ops = { 
      method: 'GET', 
      uri:sitesArray[i], 
     }; 
     ps.push(rp.get(ops)); 
    } 
    return Promise.all(ps.map(p => p.catch(e => e))) 
} 

は完全に罰金ですNodeJSのv7.9.0を使用しています。私は読みやすさのためにビットをリファクタリング提案することができます：

function CheckSitesV2() { 
 
    let ps = sitesArray 
 
    .map(site => rp.get({method: 'GET', uri: site}).catch(e => e)); 
 

 
    return Promise.all(ps); 
 
}

非同期が "正しい" 方法が正確にない方法でこの

async function CheckSitesV1() { 
 
    let results = []; 
 
    for (let i = 0; i < sitesArray.length; i++) { 
 
     let opts = { 
 
      method: "GET", 
 
      uri: sitesArray[i] 
 
     }; 
 
     const resp = await rp.get(opts).catch(e => e); 
 
     results.push(resp); 
 
    } 
 
    return results; 
 
} 
 

 
CheckSitesV1().then(console.log)