私の正規表現一致が見つからないときにオブジェクト

Question

私はトラブル文を返すが生じています

を通過するためのループを防ぐ他の場合：私はフレーズを入力すると

function ai(message) { 
    if (username.length < 3) { 
     username = message; 
     send_message("Oh, well hello " + username + ", My name is Donald J Trump and i'm a big boy"); 
    } else { 
     for (i = 0; i <= botChat.length; i++) { 
      var re = new RegExp(botChat[i][0], "i"); 
      if (re.test(message)) { 
       var length = botChat[i].length - 1; 
       var index = Math.ceil(length * Math.random()); 
       var reply = botChat[i][index]; 
       send_message(reply); 
      } 
     } 
    } 
} 

それが正しくごとのように配列の最初の行を一致させることができますforループ。私が抱えている問題は、elseステートメントを追加しようとすると、配列を適切にループしなくなるということです。

私が試してみました：

else if (re.test(message) === false) { 
    send_message("i can't be expected to know everything"); 
} 

をしかし、それは動作しません、それはbotChat[0][0]過ぎてループからループを防止します。

私も試してみた：

if (send_message().val() == "") { 
    // ... 
} 

しかし、私のコードは、もはや働くたびに。私の配列に「上記の一致が見つからない場合はこれを選択してください」という何かを追加する方法がありますか？

if/elseが動作するように、または私のコードを作業するだけの方法はありますか？

は、codepenです。

Answer 1

私はあなたのcodepenをチェックしsend_message()戻りundefinedとundefinedは.val()メソッドを持っていないため、これらの行が例外を投げているので、ライン190および194 console.log(send_message().val());は、ループを破壊しているようです。

正規表現が正常に動作しています。新しいRegExpオブジェクトを反復ごとに作成するのではなく、配列if (botChat[i][0].test(message))で定義されているものを使用することをお勧めします。オーバーヘッドは少なくなります。