web-dev-qa-db-ja.com

この関数を約束する方法-nodejs

約束を返す必要があるajax呼び出しがあります。機能は以下の通りです

client.tickets.create(ticket,  function(err, req, result) {
  if (err) {    
    logger.error(err);

    return false;
  }

  return JSON.stringify(result);
});

次のアクションを実行する前に、この関数が実行されるのを待つ必要があります。この機能をどのように約束できますか？

次のことを試しましたが、Cannot call method then of undefinedというエラーが表示されました。

return client.tickets.create(ticket).then(function(result){
    return JSON.stringify(result);
},function(err){
    logger.error(err);
    return false;
});
javascriptnode.jspromisezendesk
7
chaithu

create()はPromiseではないため、エラーが発生します。非同期関数の約束は非常に簡単です（nodejsには最近Promiseサポートが組み込まれています）：

function createTicket(ticket) {
    // 1 - Create a new Promise
    return new Promise(function (resolve, reject) {
        // 2 - Copy-paste your code inside this function
        client.tickets.create(ticket, function (err, req, result) {
            // 3 - in your async function's callback
            // replace return by reject (for the errors) and resolve (for the results)
            if (err) {
                reject(err);
            } else {
                resolve(JSON.stringify(result));
            }
        });
    });
}

// 4 - consume your promise with then() (resolved promise) and catch (rejected promise)
createTicket(ticket).then(function (result) {
    // deal with result here
}).catch(function (err) {
    // deal with error here
});
28
Shanoor

非同期コードをpromiseに手動でラップするのではなく、 Bluebird のようなライブラリを使用してそれを行うことをお勧めします。

var Bluebird = require('bluebird');

//either 
client.tickets = Bluebird.promisifyAll(client.tickets);
//or 
client.tickets.createAsync = Bluebird.promisify(client.tickets.create);
...
    return client.tickets.createAsync(ticket)
      .then(JSON.stringify)
      .catch(err => {
        logger.error(error); 
        return false
      });
9
mido