NodeJS - promisy

Question

Siema mam pytanie odnośnie promises,

Stworzyłem taki przykład:

var createUser = (x) => {
  console.log(x)
};


router.post('/create', function (req, res, next) {
  var newProduct = {
    title: req.body.name,
    price: req.body.price,
    description: req.body.description,
    quantity: req.body.quantity,
    category: req.body.category
  }
  var product = new Product(newProduct);
  var sum = 0;
  product.save()
    .then((products) => {
      res.send(products)
      setTimeout(function () {
        console.log("siema1")
        var sum = 5 + 5
      }, 5000);
      return sum;
    })
    .then((sum) => {
      return createUser('demdemdemmmmm');
      console.log(sum)
    })
    .then(() => {
      return setTimeout(function () {
        console.log("siema2")
      }, 2000);
    })
    .catch((err) => {
      console.log(err)
    })
});

I zmierzam do tego, że powinno wykonać się po kolei gdy w momencie wykona sie wczesniejszy, dopiero wtedy wykona sie następny, .then 1,2,3 a wykonuję się 2,3,1  Dlaczego tak?? Może mi ktoś wytłumaczyć to??

Comments

.then 1,2,3 a wykonuję się 2,3,1

Dlaczego tak miało by się wykonać? W pierwszym i trzecim then() dałeś timery, które wykonują się po (co najmniej) 5 i 2 sekundach. Drugi then() wykona się od razu, przy czym console.log(sum) nie powinno się wykonać, bo jest to wyrażenie języka JavaScript (więc nie podlega hoistingowi), a przed nim jest słówko return. Zatem same metody then() wykonują się w dobrej kolejności, lecz umieszczenie w nich setTimeout() powoduje, że w konsoli widzisz inną kolejność.

Poza tym, czy wiesz że:

    .then(() => {
      return setTimeout(function () {
        console.log("siema2")
      }, 2000);
    })

Zwrócenie setTimeout powoduje zwrócenie identyfikatora tego timera, ponieważ aby anulować setTimeout i setInterval potrzebujesz właśnie ich identyfikatora. Więc, jeśli nie chcesz później anulować timera, to zwracanie go jest troszkę bezcelowe.

Answer 1

Jeśli dobrze rozumiem to co chcesz zrobić...

funkcja w setTimeout wykonuje się po 5s zupełnie niezależnie od promise'ów. Jeśli chcesz żeby setTimeout był częścią "kolejki" .then, to trzeba go owinąć:

// https://stackoverflow.com/a/39538518
function delay(t) {
   return new Promise(function(resolve) { 
       setTimeout(resolve, t)
   });
}

Wtedy można go używać tak jak inne funkcje zwracające promise:

product.save()
  .then((products) => {
    res.send(products);
    return delay(5000);
  }).then(() => {
    console.log("siema1");
    var sum = 5 + 5;
    return sum;
  }).then((sum) => {
    console.log(sum)
    createUser('demdemdemmmmm');
    return delay(2000);
  }).then(() => {
    console.log("siema2");
  }).catch((err) => console.log(err));