TypeError: Cannot read property 'parameters' of undefined

Question

前提・実現したいこと

プログラム初心者です。

Dailogflowに質問してWebhookを使いCloudFunctionで回答分岐させDialogflow上に回答を表示させたいと思っています。下記のページと同じ手順で作業していたのですがTypeError: Cannot read property 'parameters' of undefinedと出て困っています。
お手数ですが教えていただけると大変助かります。
参考にしたページ：
https://www.apps-gcp.com/dialogflow-cloud-function-chatbot/

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

TypeError: Cannot read property 'parameters' of undefined	

該当のソースコード

//index.js
exports.chatbottest = function chatbottest(req, res) {

 let mes;
 if(!req.body.result.parameters.boardgame){
  mes = "パンデミック、アグリコラ、カタンについてお答えできます。";
 }
 
 else if(req.body.result.parameters.boardgame == "パンデミック"){
  mes = "病原体から世界を救うボードゲームです";
 }
 
 else if(req.body.result.parameters.boardgame == "アグリコラ"){
  mes = "荒れ果てた土地で豊かな農場を作るゲームです";
 }
 
 else if(req.body.result.parameters.boardgame == "カタン"){
  mes = "未開拓のカタン島で部族を栄えさせるゲームです";
 }
 
 else{
  mes = "すいません、知らないボードゲームです";
 }
 
 res.setHeader('Content-Type', 'application/json');
 res.send(JSON.stringify({"speech": mes, "displayText": mes}));
};

{
//package.json

 "name": "sample-http",
 "version": "0.0.1"
}

Answer 1

req.body.result.parameters.boardgame

req.body.result が undefined であるためエラーが発生しています。
エラーが発生した場合は、固有名詞を取り除いた状態「TypeError: Cannot read property of undefined」などで検索をかけると原因が詳細に書かれたサイトが見つかると思います。

---追記---
req.body.result は Dialogflow から渡ってくる引数になります。
Dialogflow から渡ってきている引数が間違っているのか、Google Cloud functions 側の設定が間違っているのか切り分けが必要になります。

まずは、引数が正しくわたってきているのか、req の中身を確認してください。
中身が入っている場合、Dialogflowの戻り値の形式が変わっているのだと思います。

---再追記---
Dialogflow が返す値の形式に変更が入っているようです。

Dialogflow Webhook Format
v1 ： https://developers.google.com/actions/reference/v1/dialogflow-webhook
v2（現行） ： https://developers.google.com/actions/build/json/dialogflow-webhook-json

v1の時は 「req.body.result」で取れたものが、v2では「req.body.queryResult」となっているようです。
req.body.result.parameters.boardgame の部分を
req.body.queryResult.parameters.boardgame と変更すれば、値がとれるのではないかと思います。

Comments

[yanasan]

回答ありがとうございます。

この場合ですとreq.body.resultをしっかりと定義してあげないといけないということでしょうか？
もし認識違いがあればお手数ですが教えていただけると大変助かります。
無学で申し訳ありません。
よろしくお願いいたします。

[yanasan]

事細かく教えていただきありがとうございます。
やっぱり形式に変更があったのですね。
教えていただいたとおりにやってみようと思います。

[yanasan]

できましたー
ありがとうございます！

Answer 2

データが不定というならともかく自分でオブジェクトを設計しているのであれば
あまりくどいチェックは不要なはずですがちゃんとチェックするならこう

if(typeof req!=="undefined" &&
   req.body &&
   req.body.result &&
   req.body.result.parameters){
  if(req.body.result.parameters.boardgame == "パンデミック"){
    mes = "パンデミック、アグリコラ、カタンについてお答えできます。";
  }else if(){
    ・・・;
  }
}

つまりreqが定義されていればreq.bodyを参照してもエラーにはなりませんが
req.bodyが定義されていなければreq.body.resultはエラーになります

全部すっとばしてtry-catchする手もありますが
問題点の所在をわからなくしてしまう可能性はありますので
おすすめはしません

try{
  if(req.body.result.parameters.boardgame == "パンデミック"){
    mes = "パンデミック、アグリコラ、カタンについてお答えできます。";
  }else if(){
    ・・・;
  }
}catch(e){
  mes = "病原体から世界を救うボードゲームです";
}