Обещание с res.send();

Question

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

if (isbot(req.headers['user-agent'])){ 
    var url= req.protocol+'://'+req.get('host')+ req.originalUrl 
    phridge.spawn() 
     .then(function (phantom) { 
     var page = phantom.createPage(); 
      return page.run(url, function (url, resolve, reject) { 
       var page = this; 
       page.open(url, function (status) { 
        // handle page after load 
       }); 
      }) 
      .then(function (contnt) { 
       res.send(contnt); 
      }) 
      .then(phridge.disposeAll()); 
      .catch(function (err) { 
       console.error(err.stack); 
      }) 

     } 
else { 
    next(); 
} 

Вопрос заключается в том - как механик res.send() работает с обещанием? Будет phridge.disposeAll() выполнено?

Answer 1

Вы совершаете многочисленные ошибки. Перед написанием этого кода вы должны убедиться, что вы знакомы с программированием стиля Promise. См. Последний раздел.

---

В этом случае, нет, потому что

 .then(function (contnt) { 
      res.send(contnt); 
     }) 

часть не возвращает Promise.

В этой части, если вы уверены, что res.send не будет поднимать какие-либо исключения, можно было бы написать:

 .then(function (contnt) { 
      res.send(contnt); 
      return new Promise() 
     }) 

И последнюю часть,

 .then(phridge.disposeAll()) 

также проблематично, вы должны изменить его должно быть

 .then(() => phridge.disposeAll()) 

, даже если это конец цепи, и нет необходимости создавать ne w Promise, вы должны написать его так, потому что функция then() принимает функции, а не результат, чтобы быть ее аргументом.

И вам нужно убедиться, что каждая ветка .then() возвращает объект Promise, пока вы их цепляете. (Я не проверял другие, так как я не знаю, что они возвращают.)

---

ОК, есть несколько ошибок, я видел избыточный ; после then() ветви. Я не уверен, есть ли еще проблемы.

Я думаю, что проблема более серьезная: вы не понимаете Promise программирование стиля. Вы должны прочитать ES6 Promise docs или библиотеку Promise (например, bluebird, зависит от того, от какой библиотеки зависит ваша библиотека).

Answer 2

Я бы объединил res.send и disposeAll(). Не нужно слишком усложнять код. res.send является синхронным и возвращает логическое значение.

phridge.spawn() 
    .then(function (phantom) { 
    var page = phantom.createPage(); 
     return page.run(url, function (url, resolve, reject) { 
      var page = this; 
      page.open(url, function (status) { 
       // handle page after load 
      }); 
     }) 
     .then(function (contnt) { 
      res.send(contnt); 
      phridge.disposeAll() 
     }) 
     .catch(function (err) { 
      console.error(err.stack); 
     })