Expressでtwitterの検索結果を表示できない

Question

Twitter API を使って取得した結果を、express を使って出力したいと考えています。

参考

を見ながら、クエリの結果を取得すること自体はできたのですが、それをexpressに組み込むことがうまくできません。
どうすれば、http://localhost:3000/twitterにアクセスしたとき、tweet取得結果をjsonで返してくることができるでしょうか?

お知恵を拝借願えないでしょうか。必要な情報ありましたら、ご連絡ください。
コンソールに出力できていることから、認証系は問題ないと判断しています。

twitter.js

var express = require('express');
var router = express.Router();
var Twitter = require('twitter');
var client = new Twitter(中略);

var data = client.get('search/tweets', {q: 'ダナン'}, function(error, tweets, response) {
    console.log(tweets);
 });

 router.get('/',(req,res,next)=>{
    res.json(data);
 });

module.exports = router;

api、というものを使うのが初めてなので、見様見真似なのですが、どうぞよろしくお願いいたします。

Answer 1

リファレンスのexpress.Routerの部分をよく読んでください。

Express でのルーティング

Routerは、他のファイルから呼び出して利用可能なルーティングモジュールを作成するための機能です。
これ単体では動きません。

module.exports = router;

で行っているのは、ここで作成したモジュールを他のファイルから呼び出せるよう登録しているだけです。
動かすには、他のファイルからこのモジュールを呼び出し、起動する必要があります。

ここで質問ですが、あなたが作ろうとしているのはファイルをモジュールに分割する必要があるほど行数の多いものですか？
そうでないなら、Routerを使う必要はないです。素直にexpressの標準機能であるappを使いましょう。
以下のようにしたら単体で動くようになります。

const express = require('express');
const app = express();
const Twitter = require('twitter');
const client = new Twitter(中略);
const PORT = 3000;

app.get('/twitter', async (req,res)=>{
    const data = await client.get('search/tweets', {q: 'ダナン'});
    console.log(data);
    res.json(data);
});

app.listen(PORT);
console.log(`Server running at ${PORT}`);

ここで、client.get... の場所を移動したのにも理由があります。
app.get の引数に与えた関数は、http://localhost:3000/twitterへのリクエストがあるたびに呼ばれます。
一方で、その外にある処理は、プログラムの起動時に1回だけ呼ばれます。
app.get の外にclient.getがあると、http://localhost:3000/twitterに何度リクエストを送っても、プログラムを再起動しない限りは常に同じレスポンスが返されます。
あなたが作ろうとしているのは、きっとリクエストのたびに検索し直してくれるものですよね？

以上、長くなりましたがお役に立てれば幸いです。

Comments

[masaking]

本当にご丁寧にありがとうございます。
routerがあるのは、express-generetorを使うとはやい、というのを見て試したからで、おっしゃる通り、ページがあまりないので、appのほうがシンプルになると思いました。
また、getの親子関係というか、expressにgetするのと、twitterにgetするののどちらがどちらに依存するのか、あるいは互いに独立なのか、という点について、しっかり整理して書いていこうと思います。

[takeaship]

お役に立てたようで良かったです。
また、私から解説はせずさらっと流しましたが、以下の点も重要なので調べておくことをおすすめします。

* varではなくconstを使うべき理由
* client.getの前のawaitの意味