APIを関数で呼び出したいが「undefined」が表示される

Question

Node.jsにてWebAPIのレスポンスデータを読み出す際の処理を関数としてまとめたいのですが、関数として実行すると戻り値がundefinedとなります。
関数として記述しなければ、処理を実行できますが、
同様の処理を繰り返し実行させたいため、関数として記述したいです。

よろしくお願いします。

const request = require('request');

//定期実行(2秒ごと)
setInterval(function() {
  //ticker情報を取得
  var cb_ticker = get_ticker('https://api.crypto-bridge.org/api/v1/ticker');

  console.log('レスポンス:' + cb_ticker);
}, 2000);

//ticker情報を取得
function get_ticker(ticker){
  var URL = ticker;
  var res = 'res-test';
  request.get({
    uri: URL,
    headers: {'Content-type': 'application/json'},
    json: true
  }, function(err, req, data){
    res = data[1].id;
    return res;
  });
}

Answer 1

requestモジュールの処理は非同期のため、戻り値はcallback関数で受け取る必要があります。
つsync-request

Comments

[zentsu]

callback関数を使う必要があるとわかり、参考になりました。ありがとうございます！

Answer 2

戻り値がundefinedとなります

get_tickerは何もreturnしていないので当然undefinedになります。そしてrequestは非同期に取得するのでその結果をget_tickerが戻り値として返すことは不可能です。

get_tickerがcallbackを受け取るようにするか、

const request = require('request')

setInterval(function () {
  get_ticker('https://api.crypto-bridge.org/api/v1/ticker', (err, result) => {
    if (result) console.log(result)
  })
}, 5000)

//ticker情報を取得
const get_ticker = (ticker, cb) => {
  cb = cb || (() => {})
  request.get({
    uri: ticker,
    json: true,
  }, function (err, req, data) {
    if (err) {
      cb(err)
      return
    }
    cb(null, data[1].id)
  })
}

Promiseを返すようにすればできます。

const request = require('request')

setInterval(function () {
  get_ticker('https://api.crypto-bridge.org/api/v1/ticker').then(result => {
    console.log(result)
  })
}, 5000)

const get_ticker = (ticker) => new Promise((resolve, reject) => {
  request.get({
    uri: ticker,
    json: true,
  }, function (err, req, data) {
    if (err) {
      reject(err)
      return
    }
    resolve(data[1].id)
  })
})

もっとも、requestのPromiseバージョンであるrequest-promiseなどを使えばもっとすっきり書けます。

const rp = require('request-promise')

setInterval(function () {
  get_ticker('https://api.crypto-bridge.org/api/v1/ticker').then(result => {
    console.log(result)
  })
}, 5000)

const get_ticker = (ticker) => rp.get({
  uri: ticker,
  json: true,
}).then(data => data[1].id)

async/awaitでもできます。

const rp = require('request-promise')

setInterval(async function () {
  const result = await get_ticker('https://api.crypto-bridge.org/api/v1/ticker')
  console.log(result)
}, 5000)

const get_ticker = (ticker) => rp.get({
  uri: ticker,
  json: true,
}).then(data => data[1].id)

Comments

[zentsu]

複数の手法で実現する方法をご教授いただきありがとうござます。
試して見たところ、自分の環境でもやりたいことを実現できました。

何を知るべきかが明確になり、大変勉強になりました。
すぐに実現できるソースコードまで書いていただき、ありがとうござました！！