DRY up Versprechen in Javascript

Question

Ich habe den folgenden Code in Knoten v6.3.0 läuft auf einem API, das zwei separate Versprechungen abhängig von einer Bedingung, ob ein Param existiert in der POST-Anfrage läuft.

if (paramExists) { 
    // call database with this query 
    User.filter(/* do something with param */) 
     .then(function (user) { 
      Data.filter(/* same in both conditions */) 
       .then(function (data) { 
        // join data and user 
        res.send(joinedData); 
       }) // omit catch for clarity 
     }) // omit catch for clarity 
} else { 
    // call database with this query 
    User.filter(/* do something with header */) 
     .then(function (user) { 
      Data.filter(/* same in both conditions */) 
       .then(function (data) { 
        // join data and user 
        res.send(joinedData); 
       }) // omit catch for clarity 
     }) // omit catch for clarity 
} 

Ich bin sicher, dass es eine Möglichkeit, diesen Code zu vertrocknen ist, so dass das erste Versprechen in beiden Bedingungen die Benutzer auf das zweite Versprechen geht, aber ich kann nicht herausfinden, wie. Soll ich einen Generator benutzen oder gibt es eine Möglichkeit, dies mit Versprechen zu tun, die ich nicht bekomme?

Answer 1

Da der einzige Teil, der zwischen den beiden Zweigen, anders zu sein scheint, ist, was Sie User.filter() passieren, Sie diesen Wert in eine lokale Variable setzen kann in einer Bedingung und dann eine Version des Codes mit der Variable ausführen.

Sie können auch zu Ihrer gekettet Methoden vereinfachen unnötige Verschachtelung zu entfernen:

var arg; 
if (paramExists) { 
    arg = ...  // some logic 
} else { 
    arg = ...  // some different logic 
} 
// call database with this query 
User.filter(arg).then(function (user) { 
    return Data.filter(...); 
}).then(function (data) { 
    // join data and user 
    res.send(joinedData); 
}); // omit catch for clarity 

Sie auch ein ternäres verwenden:

var arg = paramExists ? someLogic : someOtherLogic; 
// call database with this query 
User.filter(arg).then(function (user) { 
    return Data.filter(...); 
}).then(function (data) { 
    // join data and user 
    res.send(joinedData); 
}); // omit catch for clarity 

---

Wenn Sie sowohl user und data zugreifen müssen zum Senden der Antwort (ein wenig schwer in Ihrem Pseudocode zu sagen), dann könnten Sie Ihre Verschachtelung behalten:

var arg = paramExists ? someLogic : someOtherLogic; 
// call database with this query 
User.filter(arg).then(function (user) { 
    return Data.filter(...).then(function(data) { 
     // join data and user 
     res.send(joinedData); 
    }); 
}); // omit catch for clarity 

Answer 2

Sie können etwas wie folgt tun. Ich nehme an, dass Data.filter ein Versprechen auch zurückgibt.

if (paramExists) { 
    // call database with this query 
    User.filter(/* do something with param */) 
     .then(sameCondition); // omit catch for clarity 
} else { 
    // call database with this query 
    User.filter(/* do something with header */) 
     .then(sameCondition); // omit catch for clarity 
} 

function sameCondition(user) { 
    return Data.filter(/* same in both conditions */) 
    .then(function (data) { 
     // join data and user 
     res.send(joinedData); 
    }); // omit catch for clarity 
} 

Answer 3

etwas tun ...

var task1 = paramExists ? 
    User.filter(/* do something with param */) : 
    User.filter(/* do something with header */) 

var doFilter = task1.then(function() { 
    return Data.filter(/* same in both conditions */) 
}) 

doFilter.then(function (joinedData) { 
    res.send(joinedData); 
});