node.jsの出力について

Question

前提・実現したいこと

Line Massaging APIとfirebaseを連携させ、お天気の情報のweb API(livedoor お天気サービス)から読み取りそれをユーザのメッセージに対して送信する。

該当のソースコード

async function handleEvent(event) {
    if (event.type !== 'message' || event.message.type !== 'text') {
        return Promise.resolve(null);
    }
    usersRef.set(event.message.text);

    var x = event.message.text;
    var obj = { type: 'text', text: x + 'が実行されました' };

    //LINEに対応するコマンドはここへ追加
    switch (x) {

        case 'おはよう': //お天気情報取得
            const RSS = "http://weather.livedoor.com/forecast/rss/area/130010.xml"; // RSSのURLを指定 --- (*1)
            const cli = require('cheerio-httpcli'); // cheerio-httpcli モジュールを取り込む
            // ダウンロードして解析
            cli.fetch(RSS, {}, (err, $, res) => {
                // エラーがあればメッセージを表示
                if (err) { console.log("失敗", err); return; }
                // ダウンロードしたデータから必要な部分を取り出す --- (*2)
                $("item > title").each((index, elem) => {
                    var e = $(elem);
                    console.log(e.text()); // テキスト部分を表示
                    obj = {
                        type: 'text',
                        text: "おはようございます。今週の天気です" > +e.text
                    };
                });
            });

            break;

        case 'lon':
            obj;
            break;

        case 'loff':
            obj;　
            break;

            //例外処理
        default:
            obj = { type: 'text', text: '不可能です。今後に期待して！！' };
            break;
    }
    //返信機能
    return client.replyMessage(event.replyToken, obj);
}
exports.app = functions.https.onRequest(app);

試したこと

objにお天気サービスから読み取った情報(e.text)を代入し、出力しようとしたのすがうまくいきません。
firebaseのログには、読み取ったお天気情報が表示されているので、読み取り自体に問題はないはずです。

補足情報（FW/ツールのバージョンなど）

現在、プログラミング初心者の学生でして、ふとfirebase、LINE、Raspberry Piを用いてスマートホーム を作ろうと
思い立ち、学習しています。ご不明な点ございましたら、すぐに対応させていただきますので、何卒よろしくお願い致します。

Answer 1

コードが古いので軽く整形

// requireはファイルの先頭で行う
const httpcli = require('cheerio-httpcli'); // cheerio-httpcli モジュールを取り込む

// 普通の関数宣言は巻き上げがあるので、こちらの方が推奨
const handleEvent = async event => {
  // ガード節は1行でOK
  if (event.type !== 'message' || event.message.type !== 'text') return;
  usersRef.set(event.message.text);

  // objってなんやねん、おもちゃ箱に「箱」ってラベル貼って管理するの？
  // resultもあまり気の利いた名前じゃないけどまだマシかな
  let result = null;

  const command = event.message.text;
  switch (command) {
    case 'おはよう': //お天気情報取得
      try {
        const url = "http://weather.livedoor.com/forecast/rss/area/130010.xml";
        // await構文使わないとasync関数の意味がないからね！
        const res = await httpcli.fetch(url, {});
        const $ = res.$;
        $("item > title").each((index, elem) => {
          var e = $(elem);
          console.log(e.text()); // テキスト部分を表示
          result = {
            type: 'text',
            text: "おはようございます。今週の天気です" > + e.text
          };
        });
      } catch (err) {
        console.error("失敗", err);
        return;
      }
      break;
    default:
      result = {
        type: 'text',
        text: `${command}は不可能です。今後に期待して！！`
      };
  }

  // resultに何か代入されたら発火の方が良さそう
  if (result) return client.replyMessage(event.replyToken, result);
}

exports.app = functions.https.onRequest(app);

こんな感じだね。
問題は大きく分けて2つ

text: "おはようございます。今週の天気です" > + e.text

e.text()という風に実行してないから値がとれないんじゃない？
これ自体はよくあるうっかりミスだね、次！

$("item > title").each((index, elem) => {

この.eachの意味を理解して使ってる？
eachというのは日本語訳すると各、それぞれ、おのおのといった単語になるんだ
今回だとitem要素の子であるtitle要素を探して、
それぞれ全てを関数にぶちこんで実行という使い方になる。

それ自体はまぁ良いんだよ。
問題はこの後。

result = {
  type: 'text',
  text: "おはようございます。今週の天気です" > + e.text()
};

それぞれのタイトルがresult変数へ代入しちゃってるよね。
そうなると上書き上書きで後勝ちになっちゃうんだよ。
つまり最後の1件の情報しか取得できない。

もしこれが1個だけならこのコードでも動作はするだろうけど、
後から見たエンジニアがかなり困るから、1個想定の場合、eachは絶対に削っておくべきだね。

もし複数タイトルに対応させたいのであれば、
外でresultへ入れる用の変数を作って文字列を足し込む等して対処する必要があるね。

// eachを使う場合
let titles = [];
const $ = res.$;
$("item > title").each((index, elem) => {
  titles.push("おはようございます。今週の天気です > " + $(elem).text());
});
result = {
  type: "text",
  text: titles.join(',')
}

// 配列にするならmapを使った方がスマートだね
const $ = res.$;
const titles = $("item > title").map((_, elem) =>
  "おはようございます。今週の天気です > " + $(elem).text()
).get();
result = {
  type: "text",
  text: titles.join(',')
}

Comments

[Okkawa]

丁寧なご回答ありがとうございます。とても勉強になります。
miyabi様から教えていただいたコードを参考に書き直してみたのですが、
「ReferenceError: $ is not defined」が出てしまいます。

---java script
res.$("item > title")
---
に問題があるようです。
原因を調べたのですが、改善することが出来ません。
何かわかりますでしょうか。よろしければお願い致します。

[miyabi-sun]

うっかりしてた！ごめんね！！

https://www.npmjs.com/package/cheerio-httpcli
これのPromise版にresponse.$があって
結果を自動的に$で包んでくれているからこれ利用すれば$不要じゃんって思ってたけど
2箇所以上で使ってるから宣言した方が綺麗になると思い返したから回答を修正したよ！

[Okkawa]

ご返答ありがとうございます！
出来ました！！これを元にもっと勉強に精進したいと思います！ありがとうございます。
またの機会ありましたらよろしくお願い致します。