GAS 土日祝日を除きメール下書き作成をしたい

Question

１．前提・実現したいこと
土日祝日を除いてメールの下書きが作成されるようにしたい

２．発生している問題・エラーメッセージ
条件分岐のコードを入れたところプログラムが作動しなくなりました。
（条件分岐なしだと作動します）
お力添え頂けますと幸いです。

3.該当ソースコード

function createmail() {

//曜日の取得  
  var date = new Date();
  var day = date.getDay(); 
  
  //今日が土日なら処理を中断
    if (day===0 || day===6) {
       return;
  }

Logger.log(day);
Logger.log(date);

//祝日の取得
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var holidaysheet = ss.getSheetByName("祝日一覧");
  var lastRow = holidaysheet.getLastRow()
  var holiday = holidaysheet.getRange(1,1,lastRow).getValues().slice(1); 　


//祝日一覧の日付とdateの日付が一致した場合処理を中断
for (var i=0 ; i<holiday.length ; ++i){
  if(holiday[i]=date){
    return;
}

Logger.log(holiday);


//FMT詳細
var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getSheetByName("FMT");

//toの取得
  var range = sheet.getRange("C2");
  var to = range.getValue();
  
//ccの取得
  var range = sheet.getRange("C3");
  var cc = range.getValue();

//subjectの取得
  var range = sheet.getRange("C4");
  var subject = range.getValue();



//当日すべてのイベントが格納されてる箱
var values = [];　　

　//イベントの取得
　var calendar = CalendarApp.getCalendarById("カレンダーID");
　var today = new Date();
　var events = calendar.getEventsForDay(today);
　var length = events.length;

　//繰り返し処理
　for (var i=1 ; i<length ; ++i){
　
　  var event = events[i]; 　　　　　//eventsから配列を取り出す
　  var title = event.getTitle();　//eventのタイトルを取得　
　

　  var start = event.getStartTime();　　　　　　　　　　　　　　　　 //eventのスタート時間の取得
　   start = Utilities.formatDate(start,"Asia/Tokyo","HH:mm");　
　  var end = event.getEndTime();　　　　　　　　　　　　　　　　　　//eventのエンド時間の取得
　   end = Utilities.formatDate(end,"Asia/Tokyo","HH:mm");   
　   var time = `${start} - ${end}`  　　　　　　　　　　　　　　　

　　　values.push(`${time} ${title}`)　
　　}



//イベントをスプレッドシートにセット　　　
  var daily = sheet.getRange("C5");
  var titles = daily.setValue(values.join("\n"));　　


//bodyの取得
  var range = sheet.getRange("C6");
  var body = range.getValue(); 


//optionsの指定
options = {

  cc:cc
}

//下書き作成
GmailApp.createDraft(to,subject,body,options);

}
}

Comments

[退会済みユーザー]

「うまく作動しない」とは具体的にどういうことでしょうか？
期待する動作と合わせて、具体的に記載してください。

[otiteruneko]

以下のコードをいれたところ、下書き作成もされなくなってしまいました。
（条件分岐なく、メールの下書きを作成するのみのコードだと作動しました）

やりたいこととしては、土日祝日を除いてメールの下書きが作成されるようにしたいです。

▼コード
＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
//曜日の取得
var date = new Date();
var day = date.getDay();

//今日が土日なら処理を中断
if (day===0 || day===6) {
return;
}

Logger.log(day);
Logger.log(date);

//祝日の取得
var ss = SpreadsheetApp.getActiveSpreadsheet();
var holidaysheet = ss.getSheetByName("祝日一覧");
var lastRow = holidaysheet.getLastRow()
var holiday = holidaysheet.getRange(1,1,lastRow).getValues().slice(1); 　

//祝日一覧の日付とdateの日付が一致した場合処理を中断
for (var i=0 ; i<holiday.length ; ++i){
if(holiday[i]=date){
return;
}

Answer 1

下記のように
var day = date.getDay();
の直後に
Logger.log(day);
Logger.log(date);
を入れて、曜日の数字および日付が意図するものになっているか確認してください。

function createmail() {

//曜日の取得  
  var date = new Date();
  var day = date.getDay(); 

Logger.log(day);
Logger.log(date);  
  //今日が土日なら処理を中断
    if (day===0 || day===6) {
       return;
  }
(以下略）

意図する時刻になっていない場合は、設定が日本時間以外になっている可能性があります。

下の図のように、左側の歯車のマークをクリックして、
タイムゾーンを「(GMT+09:00) GMT+09:00」
に設定してください。

---

追記

    if (holiday[i] = date) {
      return;
    }

これだと「=」が1つで、代入になっているため、常にtrueになってしまいます。

  
    if (holiday[i] == date) {

のように、=を「==」に変えてみてください。

追記２

また、
・祝日のシートデータがわからないので推測ですが、おそらくholidayは時刻が0:00:00となるので、上記比較の際にdateの時刻も0:00:00にそろえる必要がある。

・holidayの数だけ処理をループしているため、同じ下書きが作られてしまう。
といった問題があります。

これらを修正してみたのが下記です。
（全部の情報が公開されておらず、一部推測しているため、うまく動かない可能性はあります。
可能ならスプレッドシートを共有していただいた方が解決の近道かと思います）

function createmail() {

  //曜日の取得  
  var date = new Date();
  var day = date.getDay();

  //今日が土日なら処理を中断
  if (day === 0 || day === 6) {
    return;
  }

  Logger.log(day);
  Logger.log(date);

  //祝日の取得
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var holidaysheet = ss.getSheetByName("祝日一覧");
  var lastRow = holidaysheet.getLastRow()
  var holiday = holidaysheet.getRange(1, 1, lastRow).getValues().slice(1);

  // 時刻を0:00:00にする。
  date.setHours(0, 0, 0, 0);
  //祝日一覧の日付とdateの日付が一致した場合処理を中断
  for (var i = 0; i < holiday.length; ++i) {
    if (holiday[i] == date) {
      return;
    }
  }

  Logger.log(holiday);


  //FMT詳細
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getSheetByName("FMT");

  //toの取得
  var range = sheet.getRange("C2");
  var to = range.getValue();

  //ccの取得
  var range = sheet.getRange("C3");
  var cc = range.getValue();

  //subjectの取得
  var range = sheet.getRange("C4");
  var subject = range.getValue();



  //当日すべてのイベントが格納されてる箱
  var values = [];

  //イベントの取得
  var calendar = CalendarApp.getCalendarById("カレンダーID");
  var today = new Date();
  var events = calendar.getEventsForDay(today);
  var length = events.length;

  //繰り返し処理
  for (var i = 1; i < length; ++i) {

    var event = events[i]; 　　　　　//eventsから配列を取り出す
    var title = event.getTitle();　//eventのタイトルを取得　


    var start = event.getStartTime();　　　　　　　　　　　　　　　　 //eventのスタート時間の取得
    start = Utilities.formatDate(start, "Asia/Tokyo", "HH:mm");
    var end = event.getEndTime();　　　　　　　　　　　　　　　　　　//eventのエンド時間の取得
    end = Utilities.formatDate(end, "Asia/Tokyo", "HH:mm");
    var time = `${start} - ${end}`

    values.push(`${time} ${title}`)
  }



  //イベントをスプレッドシートにセット　　　
  var daily = sheet.getRange("C5");
  var titles = daily.setValue(values.join("\n"));


  //bodyの取得
  var range = sheet.getRange("C6");
  var body = range.getValue();


  //optionsの指定
  options = {

    cc: cc
  }

  //下書き作成
  GmailApp.createDraft(to, subject, body, options);

}

Comments

[otiteruneko]

ありがとうございます＞＜
先程確認したのですが、ログは『5.0』『Fri May 13 07:50:59 GMT+09:00 2022』とでており
タイムゾーンも『GMT+09:00』となっておりました。が下書き作成されせず。。

コードが間違っているのでしょうか・・

[退会済みユーザー]

追記しました。
if (holiday[i] = date) {
を
if (holiday[i] == date) {
に修正してみてください。

[otiteruneko]

ご確認ありがとうございます！
追記していただいたように『holiday[i] == date』に書き換えたのですが、
今度は際限なく下書きが作成されるようになりました。。
（実行停止を押さないと止まらないほどに下書きが。。）

何度もご確認をいただいている中申し訳ございませんが、
もしまだお力添えいただけるようでしたら、どこが間違えてるのか教えていただけますと幸いです..＞＜

[退会済みユーザー]

修正・追記しました。

[otiteruneko]

追記ありがとうございます！
いただいたメソッド追加したところ無事条件分岐で作成することができました！
日付だけでなく、時間も一致しないと上手く動かないのですね..とても勉強になりました。m(_ _)m
お忙しい中、親身になってご回答いただき本当にありがとうございました！！！！

Answer 2

Logger.log(day);
Logger.log(date);

これをif文の前に移動させて、そいつになに出てくるのか見てみては