Eckige bedingte Versprechen

Question

Für ein eckiges Projekt muss ich Versprechungen verschachteln und ich stoße auf Fälle, in denen ich nicht sicher bin, was ich tue. Hier ist eine meiner Code:

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, Action2, Action3 und Action4 arbeiten Versprechen Funktionen. Es ist eine Menge Versprechungen und Aktionen hängen von den Bedingungen ab. Kann ich das tun und sicher sein, dass meine Hauptfunktion immer gelöst oder zurückgewiesen wird?

Ich lese, dass wir Versprechen innerhalb der Auflösungsfunktion übergeben können. Kann ich das und das ist das gleiche wie oben:

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

Nein, ist es nicht. Ihre erste Funktion würde für immer ausstehend bleiben, wenn einer von Action2(), Action3() oder Action4() "werfen" würde, und das Versprechen $q.all(…) zurückweisen - Ihr Deferred wird dann nie gelöst. Dies ist der häufigste Fehler des deferred antipattern, den Sie hier verwendet haben.

Ihre zweite Funktion mildert dies, ist aber immer noch unnötig kompliziert. Sie brauchen hier überhaupt kein Deferred! Einfach das ausge Versprechen direkt und verwenden $q.reject:

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

Oder, wie dies in einem then Handler geschieht, können Sie auch throw "error_code" verwenden.

Answer 2

Danke für Ihre Antwort, ich kann meinen Fehler auf der ersten Code-Version sehen. Ich denke, es ist der q.all, der mich stört.

Ich las das verzögerte Antipattern. Es heißt, dass wir keine verzögerten Objekte ohne Grund erstellen müssen. diese

Der einfache Fall ist:

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

Aber aufgrund der if (data.condition) Ich kann es nicht tun. Ist mein zweiter Code der einzige Weg, es zu tun? Bin ich in einem Fall oder muss ich aufschieben?

Es spricht von "Promistification", aber mit Angular weiß ich nicht, ob es eine gute Sache ist (libs scheinen nicht gepflegt).

Prost,