GAS 土日祝のみメール送信を停止したい

Question

土日祝（Googleカレンダーの暦通り）のみメール配信を停止したいです。
色々調べ下記コードを書いてみたのですがうまくいきません。。。
トリガーセットなどの問題でしょうか？

初心の質問で恐れ入りますが、ご教示のほどよろしくお願いいたします。

function myfunction(){
    const book = SpreadsheetApp.getActive();
    const bookname = book.getName();

    const to = ['@gmail'];
    const subject = bookname + "更新のお知らせ";
    var content = "各位\n \n";
    content += "いつもお世話になっております。\n";
    content += "ご確認のほどよろしくお願いいたします。";
    MailApp.sendEmail(to.join(","),subject,content);
  
  // 祝日・休日を除く処理
  var currentDate = new Date();
  var weekday = currentDate.getDay();
  if (weekday == 0 || weekday == 6) {
    return;
  }
  var calendar = CalendarApp.getCalendarById('ja.japanese#[e-mail]');
  if (calendar.getEventsForDay(currentDate, {max: 1}).length > 0) {
    return;
  }
}

Answer 1

// 祝日・休日を除く処理
というのが送信のコードより後ろに記述されています。

これをその関数の最初にもってきてみてはどうでしょう

Comments

[slime_Hoimi]

回答ありがとうございます！
単純なことが理解できていなかったようでお恥ずかしい・・・
解決しました、ありがとうございます！

Answer 2

うまくいかない ＝ 土日にメールが送られる、だと理解してます。

10行目でメールを送信した後に曜日を判定しても意味がないです。
曜日を判定して、土日なら中断する return の後にメールを送りましょう。

Comments

[papinianus]

https://teratail.com/questions/128300#reply-195110
> コードの正否は、動作確認をすればわかると思いますので、割愛します。
> 処理の順序としては、
> １．取得したい日付を確定する。
> ２．取得したい日付が、祝日・休日である場合は、処理を抜ける。
> ３．（処理を抜けずにここに来たなら平日）スケジュールデータを取得する。
>とした方が、不要な処理を行わずに済むかと思います。

丁寧な解説があるので活用しましょう。参考になったときは +1 しましょう。

[slime_Hoimi]

いつもありがとうございます！
解説リンクもいただいてありがとうございます。
大変助かりました。