質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.34%

  • JavaScript

    17536questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Node.js

    2006questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • LINE Messaging API

    171questions

LINE Messaging APIとlivedoor天気の連携

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 413

d0nchaaan

score 2

 前提・実現したいこと

LINEで地名を送信したら、その地名の天気を返してくれるbotを作っています。
天気情報はlivedoor天気のWeather Hacksから持ってきます。

実行環境はnode.jsです。

 発生している問題・エラーメッセージ

undefined:1
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
^

SyntaxError: Unexpected token < in JSON at position 0
    at JSON.parse (<anonymous>)
    at Request._callback (C:\Users\user_name\Desktop\GoogleGit\wetherbot\LINE_Weather.js:197:15)
    at Request.self.callback (C:\Users\user_name\Desktop\GoogleGit\wetherbot\node_modules\request\request.js:185:22)
    at emitTwo (events.js:126:13)
    at Request.emit (events.js:214:7)
    at Request.<anonymous> (C:\Users\user_name\Desktop\GoogleGit\wetherbot\node_modules\request\request.js:1157:10)
    at emitOne (events.js:116:13)
    at Request.emit (events.js:211:7)
    at IncomingMessage.<anonymous> (C:\Users\user_name\Desktop\GoogleGit\wetherbot\node_modules\request\request.js:1079:12)
    at Object.onceWrapper (events.js:313:30)

 該当のソースコード

const express = require('express');
const line = require('@line/bot-sdk');
const PORT = process.env.PORT || 3000;
var json = null; /** jsonを入れる変数 */

/* LINEBotを使うためのAPI設定 */
'use strict';
const config = { // アクセストークンとシークレットをconfigに格納
  channelAccessToken: '',
  channelSecret:''
};

const app = express();

app.post('/webhook', line.middleware(config), (req, res) => {
  console.log(req.body.events);
  Promise
    .all(req.body.events.map(handleEvent))
    .then((result) => res.json(result));
});

const client = new line.Client(config);
var cityCode;

/* LINEからPOSTをキャッチして処理するAPI */
function handleEvent(event) {
  if (event.type !== 'message' || event.message.type !== 'text') {
    return Promise.resolve(null);
  }
  /**受け取ったメッセージが連想配列にあるかを判定 */
  if (typeof city.prop !== event.message.text) {
    cityCode = city[event.message.text]; // あった場合はcityCodeに都市ID(東京なら130010)を入れる
  }
  return client.replyMessage(event.replyToken, {
    type: 'text',
    text: strBody //実際に返信の言葉を入れる箇所
  });
}

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

/**連想配列で都市コードを定義 */
var city = {
  '稚内': '011000',
  '旭川': '012010',
  '留萌': '012020',
  '網走': '013010',
  '北見': '013020',
  '紋別': '013030',
  '根室': '014010',
   ~~~~~~~~~~~~~~~
   ~~~~~~~~~~~~~~~
};
/* livedoor天気から天気情報を取得するAPI */
var URL = 'http://weather.livedoor.com/forecast/webservice/json/v1?city=' + cityCode; // 都市コードを足して取得する地域のURLを決める

/** livedoor天気から天気情報を取得 */
let request = require('request');
request.get(URL, function (err, res, body) {
  if (err) {
    console.log('Error: ' + err.message);
    return;
  }
  json = JSON.parse(body);

 試したこと

handleEvent()関数内ではcityCodeに値が入っているのは確認済みです。
この関数外でcityCodeを使えれば上手くいきそうと睨んでいます。
handleEvent内に既にreturn文があるためreturnで変数を返すという方法もできませんでした。

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

プログラミングを初めて3カ月ほどですので、わかりにくいところや、ほかに必要な情報などあれば仰ってください

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+2

エラーの原因はlivedoor天気のAPIを呼ぶときに、cityCodeが正しく渡せておらずエラー画面のhtmlが返ってきているがjsonとして処理しようとして失敗しているように見えます。
requsetに渡しているURLの中身を確認してください。

呼び出し方法に関しては下記を参考にしてもらい、handler内でrequestの実行が完了した後にreplyMessageを呼び出すように修正すると、とりあえず動くようになると思います。

let request = require('request');
request.get(URL, function (err, res, body) {
  if (err) {
    console.log('Error: ' + err.message);
    return;
  }
  json = JSON.parse(body);

  // 天気情報取り出してstrBodyに格納する

  client.replyMessage(event.replyToken, {
    type: 'text',
    text: strBody //実際に返信の言葉を入れる箇所
  });
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/05 22:35

    変数の参照範囲が原因だったんですね!解決しましたありがとうございます!

    キャンセル

  • 2018/07/06 01:02

    よかったです!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.34%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • JavaScript

    17536questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Node.js

    2006questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • LINE Messaging API

    171questions