Discordのbot(server.js)で計算クイズを作りたい

Question

前提・実現したいこと

DiscordのbotをJavaScript(server.js)で作ろうとしています。

具体的には、
botがランダムな計算問題を出題する
↓
回答者が正解すれば再度ランダムな計算問題を出題する
の動きをするように設計したいです。

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

if文の中で計算問題の答え(var ans_convertと定義)を出しておき、回答者のdiscord上の回答と一致したら次の問題に行くようにしたいのですが、下記のようなエラーが発生し、Discord上ではどんな入力でも正解となってしまいます。
（sendMsg(ans_convert)しようとしたときに、ans_convertが未定義だよというエラー）

DiscordAPIError: Cannot send an empty message
    at /rbd/pnpm-volume/52872b23-6ab3-4b15-be7a-8668464d4907/node_modules/.registry.npmjs.org/discord.js/11.6.4/node_modules/discord.js/src/client/rest/RequestHandlers/Sequential.js:85:15
    at /rbd/pnpm-volume/52872b23-6ab3-4b15-be7a-8668464d4907/node_modules/.registry.npmjs.org/snekfetch/3.6.4/node_modules/snekfetch/src/index.js:215:21
    at processTicksAndRejections (internal/process/task_queues.js:88:5) {
  name: 'DiscordAPIError',
  message: 'Cannot send an empty message',
  path: '/api/v7/channels/769246390768369697/messages',
  code: 50006,
  method: 'POST'
}

該当のソースコード

汚いコードで申し訳ありませんが…

//discordBotのseerver.jsの必要部分のみ抜粋
var nowQuiz = 0;
var ans = 0;
var ans_convert = 0;
if (message.content.match(/クイズ/)){
    nowQuiz++;
    sendMsg(message.channel.id, 'クイズをするよ');
    sendMsg(message.channel.id, '参加者は ' + message.author.username);
    sendMsg(message.channel.id, "割り算は整数部分だけ答えればOK!");
    
    var num1 = Math.floor( Math.random() * 100 );
    var num2 = Math.floor( Math.random() * 100 );
    var operator = ['+', '-', '×', '÷']
    var chooseOperator = Math.floor( Math.random() * 4 );
    
    sendMsg(message.channel.id, nowQuiz + "問目: " + num1 + operand[chooseOperator] + num2 + ' = ?');

    if(chooseOperator == 0){//足し算
      var ans = num1 + num2;
    }else if(chooseOperator == 1){//引き算
      var ans = num1 - num2;
    }else if(chooseOperator == 2){//掛け算
      var ans = num1 * num2;
    }else if(chooseOperator == 3){//割り算
      var ans = Math.floor(num1 / num2); //整数部分のみ
    }
    var ans_convert = RegExp(ans); //正規表現で使用できるように変換
    return;
  }
  
  //このif文でans_convertが引き継がれない
  //どんな回答でもif文を通り、OK!と表示されてしまう
  if (nowQuiz==1 && message.content.match(ans_convert)){
    sendMsg(message.channel.id, "OK!");
    nowQuiz++; //2問目
    sendMsg(message.channel.id, ans_convert); //ここで上記のエラー発生

    //以下、同様の処理

    return;
  }

試したこと

nowQuizの値はif文を超えて引継ぎが出来ているようです。
（if (nowQuiz==1 && message.content.match(ans_convert))文内でsendMsg(nowQuiz)を実行するとちゃんと正しい値が返ってきます）

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

package.jsonの中身を記載します

{
"//1": "describes your app and its dependencies",
"//2": "https://docs.npmjs.com/files/package.json",
"//3": "updating this file will download and update your packages",
"name": "hello-express",
"version": "0.0.1",
"description": "A simple Node app built on Express, instantly up and running.",
"main": "server.js",
"scripts": {
"start": "node server.js"
},
"dependencies": {
"express": "^4.17.1",
"discord.js": "^11.6.4",
"querystring": "^0.2.0"
},
"engines": {
"node": "12.x"
},
"repository": {
"url": "https://glitch.com/edit/#!/hello-express"
},
"license": "MIT",
"keywords": [
"node",
"glitch",
"express"
]
}

Answer 1

一刻も早く var をやめて let と const に置き換えろ。

あと変数のスコープは if ブロックの中で閉じています。if ブロックの中で宣言した変数は if ブロックの中でしか通用しません。

追記：
「変数のスコープはブロックの中で閉じている」というよりは「変数宣言はブロックの中で閉じている」と言ったほうが適切でしたね。なので nowQuiz についての説明は以上です。次に正解の記録ですが、「if の外に変数があればよい」ので

...
    let ans = 0
    if(chooseOperator == 0){//足し算
      ans = num1 + num2
    }else if(chooseOperator == 1){//引き算
      ans = num1 - num2
    }else if(chooseOperator == 2){//掛け算
      ans = num1 * num2
    }else if(chooseOperator == 3){//割り算
      ans = Math.floor(num1 / num2) //整数部分のみ
    }
...

とすれば問題なく正解をキャッチできます。宣言は外にありますから let ans に代入することが出来ます。

var は変数のスコープを破壊する厄介者です。一刻も早く使用をやめて let と const に乗り換えてください。

Comments

[kei0005]

回答ありがとうございます。2つほど質問があり恐縮ですが、
①変数のスコープは if ブロックの中で閉じているのであれば、どのようにして正解の値を記録し、Discord上の回答者の回答と一致しているかどうかを判定すればよいでしょうか。
②変数nowQuizのスコープがifブロックの中で閉じずに他のifブロックへ越境しているのはなぜなのか、見当は付きますでしょうか

[kei0005]

問題が無事解消されました
この度はありがとうございます

[kei0005]

var, let, constの使い分けもちゃんとできるようになる必要がありそうですね

[A_kirisaki]

var は使わないでください！心からのお願いです。