GAS indexOfメソッドで複数文字列を検索したい

Question

GASで変数に代入した長いテキスト文から、
複数の検索ワードのうち1つでも一致するかどうか調べるものを作りたいと思っています。
しかし、何度かやってみたのですが、以下コードの場合であれば"xxxx"しか検索を実行していなさそうです。

質問ですが以下コードであれば全てのワードを検索実行できていますでしょうか。
できていない場合どのように書き直すべきでしょうか。
お手数お掛け致しまして申し訳ないです。

※利用イメージはGmailの本文を調べて、
特定の文字列を含む場合はアラートを出せるようにしたいのです。

  var test
  var sub = str.indexOf("xxxx"||"yyyy"||"zzzz"||"ffff"||"dddd");  
  if(sub != -1){test = '一致';}

Answer 1

まず、ソースについて軽くコメントを。

  // ;を忘れている。なくても動くが、つけるべき。
  var test
  
  // strってなんですか? おそらくGmailの本文が入った文字列が入っているのだと思いますが。
  var sub = str.indexOf("xxxx"||"yyyy"||"zzzz"||"ffff"||"dddd");
  
  if(sub != -1){test = '一致';}

strというのが別に与えられているとして、
確かにこれでは期待通り動作しません。

こうした場合はindexOfを使うのではなく、例えばmatchを使って

 var test;
 var sub = str.match(/xxxx|yyyy|zzzz|ffff|dddd/);
 // matchの場合、該当なしならばnullを返すため、変更   
 if(sub !== null){test = '一致';}

と書くと、期待通り動くと思います。

追記

禁止文字がたくさんあるとき、上では対応するのが大変になるので、
そういうときにはindexOfを使ったほうがよいかもしれません。例えばこのような関数を書いてみました。

/** 
　mailTextに、forbiddenWords中に指定された文字列が一つでもあればtrue,一つもなければfalseを返す
  @param {string} mailText メール本文の文字列
  @param {Array<string>} forbiddenWords 禁止文字の配列
**/
function isIncludedForbiddenWords(mailText,forbiddenWords){
  return forbiddenWords.some(function(word){return (mailText.indexOf(word) !== -1);});
}

Comments

[退会済みユーザー]

>>TKatayamaさん

ご回答大変ありがとうございます。
お蔭様で無事動きました。
追記頂いた関数は私の勉強不足で大変申し訳ないですが、
構造を理解するまで参考書を片手に読む必要がありそうです。
こちら理解し次第活用させて頂きます。

本当に助かりました！
ありがとうございます。

[退会済みユーザー]

一つ忘れていました。
strも別でvar = strで割当てていたのですが、
記載漏れておりました。申し訳ないです。getplaintextでメール本文を代入しています。