Угловые условные обещания

Question

Для углового проекта мне нужно выполнить обещания, и я сталкиваюсь с ситуациями, когда я не уверен, что делаю. Вот один мой код:

return Action1().then(function (data) { 
    var defer = $q.defer(); 
    if (data.condition) { 
     $q.all([Action2(), Action3(), Action4()]).then(function() { 
      defer.resolve(); 
     }); 
    } else { 
     defer.reject("error_code"); 
    } 
    return defer.promise; 
}); 

Action1, Мотор2, действий3 и действий4 работают обещания функции. Много обещаний и действий зависит от условий. Могу ли я это сделать и быть уверенным, что моя основная функция будет всегда решена или отвергнута?

Я прочитал, что мы можем передать обещание внутри функции разрешения. Могу ли я сделать это, и это то же самое, что и выше:

return Action1().then(function (data) { 
    var defer = $q.defer(); 
    if (data.condition) { 
     defer.resolve($q.all([Action2(), Action3(), Action4()]); 
    } else { 
     defer.reject("error_code"); 
    } 
    return defer.promise; 
}); 

Answer 1

Нет, это не так. Ваша первая функция останется навсегда, если один из Action2(), Action3() или Action4() сделал «бросок» и отклонил обещание $q.all(…) - ваш отложенный никогда не разрешается. Это наиболее распространенная ошибка deferred antipattern, которую вы использовали здесь.

Ваша вторая функция смягчает это, но все еще слишком сложна. Вам вообще не нужно откладывать! Просто вернуть обещание прямо, и использовать $q.reject:

return Action1().then(function (data) { 
    if (data.condition) { 
     return $q.all([Action2(), Action3(), Action4()]); 
    } else { 
     return $q.reject("error_code"); 
    } 
}); 

Или, как это происходит внутри then обработчика, вы можете также использовать throw "error_code".

Answer 2

Спасибо за ваш ответ, я вижу свою ошибку в первой версии кода. Я думаю, что это q.all, что возмущает меня.

Я прочитал отложенный антипаттерн. Он сказал, что нам не нужно создавать отложенные объекты без всякой причины.

Простой случай это:

return Action1().then(function() { 
    return $q.all([Action2(),Action3(), Action4()]);   
}); 

Но из-за, если (data.condition) Я не могу это сделать. Является ли мой второй код единственным способом сделать это? Я в каком-то случае, или я должен использовать отсрочку?

Это говорит о «обещании», но с Угловым Я не знаю, хорошо ли это (libs кажутся неподдерживаемыми).

Cheers,