Переписывая эту цепочку обещаний frankenstein

Question

Итак, я вызвал к жизни эту мерзость, и я не мог за свою жизнь задуматься о том, как ее оптимизировать таким образом, чтобы я мог правильно управлять этой цепочкой с помощью Promise.all/Promise.join.

Кто-нибудь может указать мне в правильном направлении? Возможно, сначала следует отделить эти методы.

Любое понимание оценено.

getOpenIDConf: function() { 
 
    return client 
 
    .getAsync('openId') 
 
    .then(
 
     function(result) { 
 
     if (!result) { 
 
      return request 
 
      .getAsync({ 
 
       url: 'https://accounts.google.com/.well-known/openid-configuration', 
 
       json: true 
 
      }).spread(
 
       function(response, body) { 
 
       var result = JSON 
 
        .stringify(body); 
 
       client.setAsync('openId', 
 
        result).then(
 
        function() { 
 
        return result; 
 
        }); 
 
       }); 
 
     } else { 
 
      return result; 
 
     } 
 
     }); 
 

 
},

[EDIT] Чтобы уточнить, я использую Bluebird

Answer 1

Рефакторинг немного и изменение стиля кода дает это.

getOpenIDConf:() => client.getAsync('openId').then(result => 
 
    result || request.getAsync({ 
 
     url: 'https://accounts.google.com/.well-known/openid-configuration', 
 
     json: true 
 
    }).get(1).then(JSON.stringify).then(result => 
 
     client.setAsync('openId', result).return(result); 
 
    ) 
 
) 
 
},

Answer 2

Несколько особенностей хорошей библиотеки обещание (не уверен, какой из них вы используете) является то, что вы можете цепи обетования, как так:

doSomething(function(result) { 
    return doSomethingElse(); 
}).then(function(result2) { 
    return doSomethingElseAgain(); 
}).then(function(result3) { 
    // It all worked! 
}).catch(function() { 
    // Something went wrong 
}); 

Или вы можете ждать набора их завершения:

var promiseArray = []; 
promiseArray.push(doSomething()); 
promiseArray.push(doSomethingElse()); 
promiseArray.push(doSomethingElseAgain()); 

Promise.all(promiseArray).then(function() { 
    // It all worked! 
}).catch(function() { 
    // Something went wrong 
}); 

Надеюсь, что это информативно.