これらの約束事を正しく取り扱っていますか？

Question

ここですべてのエラーを単に捕まえるには良い方法がありますか、複数の要求を持つこの約束のチェーンには複数のレベルがあるので、他のオプションはありませんか？

 return RequesterService.get('peakBlockIndex', request) 
      .then(function(res) { 
       rangeData = res.data; 
       rangeData.intervals = []; 
       if (!rangeData.intervals || rangeData.intervals <= 0) { 
        return $q.reject({ message: 'Request returned no interval data.' }); 
       } 

       return SiteService.getSite(rangeData.site.id) 
        .then(function(site) { 
         if (!site.zoneId) { 
          return $q.reject({ message: 'Request for zoneId failed for the given site.' }); 
         } 

         return getDayAheadData(site.zoneId, start, end) 
          .then(function(data) { 
           return handleBlockRange(rangeData, data[0].data, data[1].data); 
          }) 
          .catch(function(err) { 
           return $q.reject({ message: 'Request for cleared & forecasted data failed for the given site.' }); 
          }) 
        }) 
        .catch(function(err) { 
         return $q.reject({ message: 'Request for zoneId failed for the given site.' }); 
        }) 
      }) 
      .catch(function(err){ 
       return $q.reject({ message: 'Request for block-range data failed.' }); 
      }); 
     } 

Answer 1

一見すると、次のように入れ子を完全に削除する必要があります。エラーが発生した場合は、コードに似たメッセージRequest for block-range data failed.が返されます。理由は、$q.reject()を使用すると、次のcatch()ハンドラまでプロミスチェーンの実行がスキップされるためです。コードの最後にこのようなハンドラを置くと、すべてのエラーがここで捕捉されます。 async/await新しい構文を使用して

return RequesterService.get('peakBlockIndex', request) 
     .then(function(res) { 
      rangeData = res.data; 
      rangeData.intervals = []; 
      if (!rangeData.intervals || rangeData.intervals <= 0) { 
       return $q.reject({ message: 'Request returned no interval data.' }); 
      } 

      return SiteService.getSite(rangeData.site.id) 
     }) 
     .then(function(site) { 
      if (!site.zoneId) { 
       return $q.reject({ message: 'Request for zoneId failed for the given site.' }); 
      } 

      return getDayAheadData(site.zoneId, start, end); 
     }) 
     .then(function(data) { 
      return handleBlockRange(rangeData, data[0].data, data[1].data); 
     }) 
     .catch(function(err){ 
      return $q.reject({ message: 'Request for block-range data failed.' }); 
     }); 

、1は、読みやすさを向上させることができます。今回は、実際に一致するエラーメッセージが返されます。 （上の構文を使用して特定のエラーメッセージを返すこともできます）。

async function someRequest(request){ 

    let res; 
    try{ 
    res = await RequesterService.get('peakBlockIndex', request); 
    } catch(e){ 
    return $q.reject({ message: 'Request for block-range data failed.' }); 
    } 

    rangeData = res.data; 
    rangeData.intervals = []; 
    if (!rangeData.intervals || rangeData.intervals <= 0) { 
    return $q.reject({ message: 'Request returned no interval data.' }); 
    } 

    let site; 
    try { 
    site = await SiteService.getSite(rangeData.site.id); 
    } catch(e) { 
    return $q.reject({ message: 'Request for zoneId failed for the given site.' }); 
    } 

    if (!site.zoneId) { 
    return $q.reject({ message: 'Request for zoneId failed for the given site.' }); 
    } 

    let data; 
    try{ 
    data = getDayAheadData(site.zoneId, start, end); 
    } catch(e){ 
    return $q.reject({ message: 'Request for cleared & forecasted data failed for the given site.' }); 
    } 

    return handleBlockRange(rangeData, data[0].data, data[1].data); 

}