【GAS】google calendarで、特定の予定があるときは動作しないようにしたい

Question

【GAS】google calendarで、特定の予定があるときは動作しないようにしたい

営業日だけ動かしたいプログラムがあり、google calendarに「休業日」という予定が入っている場合は動かさないようにしたいです。

どちらのコードが正しいのか確認していただきたいです

ほぼ同じプログラムを個人ブログで公開されている方のコードを拝見しましたが、納得がいかず伺っています。

以下における**「休業日かどうか（indexOf==~）」という所が、コードの下に記載の①②のどちらが正しいでしょうか**？

【候補】
①indexOf==-1
②indexOf>-1

function BusinessDay(date){
/*以下、追記*/
  if (date.getDay() == 0 || date.getDay() == 6) {
    return false;//①平日かどうか。0が日曜日で、6が土曜日の時はfalse
  }
  var calJa = CalendarApp.getCalendarById('ja.japanese#[e-mail]');
  if(calJa.getEventsForDay(date).length > 0){
    return false;//②祝日かどうか。日本の祝日カレンダーの今日の予定数が0より多い時はfalse
  }
/*以上、追記*/
  var myCalendar = CalendarApp.getCalendarById('[e-mail]');
  var todayEvents = myCals.getEventsForDay(date);
  var eventTitles =[];　/* イベントの数だけ繰り返し */
  for(var i=0;i<todayEvents.length;i++){
    eventTitles.push(todayEvents[i].getTitle()); //イベントのタイトルを配列に追加
  }
/* 休業日かどうか */
  if(eventTitles.indexOf('休業日') == -1){
    return false;
  }
  return true;
}

①indexOf==-1

if(eventTitles.indexOf('休業日') == -1){
    return false;
  }
  return true;

②indexOf>-1

if(eventTitles.indexOf('休業日') > -1){
    return false;
  }
  return true;

私は②だと思うのですが、①でされている方がいるのでどうなのかなと思っております。
①だと、「休業日」という予定が見つからない、つまり営業日に「false」が返されてしまい、作動しないと思うので。

ご意見いただければ幸いです！

Comments

[Zuishin]

多分その二つは関数名が違うと思います。営業日かどうか求める関数と、休業日かどうか求める関数ではありませんか？

[saralira]

ご指摘ありがとうございます！
簡潔にしようと省略していたのですが、実は、休業日か判定する前に、①平日かどうか、②祝日かどうかを判断するスクリプトがあるのです。
それを踏まえるといかがでしょう…？
お手すきの際に確認いただけると幸いです。

[Zuishin]

判定する前のことは無関係で、営業日かどうか求める関数であるなら、営業日に true を返すべきだし、休業日かどうか求める関数なら、営業日に false を返すべきだと思います。
これは関数の名前だけの問題です。

[saralira]

なるほど、ありがとうございます。
それでは、営業日にtrueを返したい場合は、以下で返せているという認識で大丈夫でしょうか？

if(eventTitles.indexOf('休業日') > -1){ return false; }
return true;

※何度もお手数お掛けしすみません！！

[Zuishin]

eventTitles に「休業日」が無ければ true が返ります。
それが本当に休業日かどうかまではわかりませんが、休業日には必ずあるのならそれで正しいはずです。

[saralira]

わかりました。
お答えいただきありがとうございます！
本当に感謝してます…！

Answer 1

Zuishinさんとやり取りさせていただいて解決しました。

営業日にtrueを返したい場合は、以下の通り。eventTitles に「休業日」が無ければ true が返る。

if(eventTitles.indexOf('休業日') > -1){ return false; } 
return true;