Сухие обещания в javascript

Question

У меня есть следующий код в узле v6.3.0, запущенный на api, который запускает два отдельных обещания в зависимости от условности того, существует ли параметр в запросе POST.

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 
} 

Я уверен, что есть способ DRY этот код так, что первое обещание в обоих случаях передает пользователю второе обещание, но я не могу понять, каким образом. Должен ли я использовать генератор, или есть способ сделать это с обещаниями, которые я не получаю?

Answer 1

Поскольку единственная часть, которая, кажется, отличается между двумя ветвями, что вы передаете User.filter(), вы можете поместить это значение в локальную переменную в условном выражении, а затем запустить одну версию кода с использованием переменной.

Вы также можете упростить прикованные методы слишком, чтобы удалить ненужные вложенности:

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 

Вы можете также использовать трехкомпонентный:

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 

---

Если вам необходимо получить доступ как user и data для отправки ответа (немного сложно сказать в вашем псевдокоде), вы можете сохранить свою вложенность:

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

Вы можете сделать что-то следующее. Я предполагаю, что Data.filter также дает обещание.

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

ли что-то вроде этого ...

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); 
});