GASを使ったスプレッドシートからChatworkに自動通知ツール

Question

実現したいこと

実現したいこと
スプレッドシートからChatworkに自動通知させたいです。
毎日9～10時をトリガー自動で実行する仕様にしています。
スプレッドシートE列に入れた実行予定日に基づいて通知を行います。

送りたい日と通知は下記の通りです
① E列に入れた実行予定日と同じ実行日のとき「当日通知」の文章を送る
② E列に入れた実行予定日の前日のとき「前日通知」の文章を送る
※ ただし土日祝日が実行予定日の前日の場合は「前営業日」の文章に変更して送付するイメージです。

また実行後に　実行日時（前日）|| 実行日時（当日）に実行日付を書き込みますが、
書き込まれている部分は再実行をしないようにしたいです。

発生している問題・分からないこと

GASでスプレッドシートからChatworkに自動通知するプログラムを組んでいます。
以下の条件分岐をさせて通知を行いたいのですが、当日通知が送られません。

送りたい日と通知は下記の通りです
① E列に入れた実行予定日と同じ実行日のとき「当日通知」の文章を送る
② E列に入れた実行予定日の前日のとき「前日通知」の文章を送る
※ ただし土日祝日が実行予定日の前日の場合は「前営業日」の文章に変更して送付するイメージです。

スプレッドシートは以下の構成です
実行している日は20230426です

N0. ||ルームID ||通知タイトル ||投稿内容 ||通知トリガー ||実行日時（前日）|| 実行日時（当日）
N0. ||ルームID ||通知タイトル ||投稿内容 ||2024/04/26|| 　　　　　　　　||
N0. ||ルームID ||通知タイトル ||投稿内容 ||2024/04/30||　　　　　　　 　||
N0. ||ルームID ||通知タイトル ||投稿内容 ||2024/05/14|| 　　　　　　　　|| 　　　　　

該当のソースコード

function sendExpenseNotification() {
  // 現在の日付と時刻を取得
  const today = new Date();
  Logger.log("今日: " + today.toLocaleString("ja-JP"));
  const holidays = getHolidays();
  Logger.log("祝日: " + holidays.join(", "));

  // スプレッドシート読み込み
  const ss = SpreadsheetApp.getActiveSpreadsheet();
  const sheet = ss.getSheetByName('自動通知');
  const lastRow = sheet.getLastRow();

  // ChatWorkAPIクライアント作成
  const client = ChatWorkClient.factory({ token: token }); 

  // スプレッドシートに記載がある分だけ繰り返す
  for (var i = 2; i <= lastRow; i++) {
    const [roomId, messageItem, message, triggerDate, prevDayExecuted, todayExecuted, prevDayExecutedDate, todayExecutedDate] = sheet.getRange(i, 2, 1, 9).getValues()[0]; 
    if (!triggerDate) continue; 

    // F列に日付が入っている場合は前日実行済みなのでスキップ
    if (prevDayExecutedDate) continue;

    // G列に日付が入っている場合は当日実行済みなのでスキップ
    if (todayExecutedDate) continue;

    const reservationDate = new Date(triggerDate); 
    Logger.log("トリガー日: " + reservationDate.toLocaleString("ja-JP"));
    
    // 実行日がトリガー日の前営業日より前の日付やトリガー日より後の日付には通知しない
    const prevWeekday = getPreviousWeekday(today);
    Logger.log("前営業日: " + prevWeekday.toLocaleString("ja-JP"));
    if (today < prevWeekday || today >= reservationDate) {
      continue;
    }

    // 通知トリガーが実行日の4日前より未来の場合は通知しない
    const fourDaysBeforeTrigger = new Date(reservationDate);
    fourDaysBeforeTrigger.setDate(reservationDate.getDate() - 4);
    Logger.log("通知トリガーの4日前: " + fourDaysBeforeTrigger.toLocaleString("ja-JP"));
    if (today < fourDaysBeforeTrigger) {
      continue; 
    }

    // 当日通知実行
    if (
      isSameDate(today, reservationDate) &&
      !todayExecuted &&
      !prevDayExecutedDate &&
      !todayExecutedDate
    ) {
      Logger.log("■　今日は実行日です。");
      // 当日通知を送信
      const todayMsg = `※連絡※【本日は${messageItem}です】\n${message}`;
      client.sendMessage({
        room_id: roomId,
        body: todayMsg
      });
      // 当日通知実行日時の記録
      sheet.getRange(i, 7).setValue(today);
    }

    // 前日通知実行
    const prevDay = new Date(reservationDate);
    prevDay.setDate(reservationDate.getDate() - 1);
    Logger.log("前日: " + prevDay.toLocaleString("ja-JP"));

    // 前日が平日かどうかをチェック
    const isPrevDayWeekday = isWeekday(prevDay);
    // 前日が祝日かどうかをチェック
    const isPrevDayHoliday = isHoliday(prevDay);

    if (isPrevDayWeekday && !isPrevDayHoliday && !prevDayExecuted) {
      Logger.log("■　前日は平日のため前日通知日です");
      // 平日の場合、前日通知を送信
      const prevDateMsg = `※連絡※【明日は${messageItem}です】\n${message}`;
      client.sendMessage({
        room_id: roomId,
        body: prevDateMsg
      });
      // 前日通知実行日時の記録
      sheet.getRange(i, 6).setValue(today);
    } else if (!isPrevDayWeekday || isPrevDayHoliday) {
      Logger.log("■　前日は土日祝日のため前営業日通知日です");
      // 平日でない場合または祝日の場合、前営業日通知を送信
      const prevWeekdayMsg = `※連絡※【${messageItem}につきまして】当日は休日のため、前営業日(${today.getMonth() + 1}月${today.getDate()}日)に投稿いたします。\n${message}`;
      client.sendMessage({
        room_id: roomId,
        body: prevWeekdayMsg
      });
      // 前営業日通知実行日時の記録
      sheet.getRange(i, 6).setValue(today); // 前日の前営業日実行
    }
  }
}


//==================================================================//  
// 関数                                                                                                                                                     //  
// =================================================================// 

// 日付が同じかどうかを比較する関数
function isSameDate(date1, date2) {
  return (
    date1.getFullYear() === date2.getFullYear() &&
    date1.getMonth() === date2.getMonth() &&
    date1.getDate() === date2.getDate()
  );
}


// 指定された日付が平日かどうかを判定する関数※1が月曜、5が金曜
function isWeekday(dateToCheck) {
  const day = dateToCheck.getDay();
  return day >= 1 && day <= 5; 
}


// 指定された日付が土曜日または日曜日かどうかを判定する関数※ 0が日曜、6が土曜
function isWeekend(dateToCheck) {
  const day = dateToCheck.getDay();
  return day === 0 || day === 6; 
}


// Googleカレンダーから日本の祝日を取得する
function getJapaneseHolidays() {
  const calendarId = 'ja.japanese#[e-mail]'; 
  const now = new Date(); // 現在の日付を取
  const oneYearFromNow = new Date(now.getFullYear() + 1, 0, 1);
  const events = CalendarApp.getCalendarById(calendarId).getEvents(now, oneYearFromNow); 
  const holidays = events.map(event => event.getDate()); 
  return holidays;
}
function getHolidays() {
  const now = new Date(); 
  const oneYearFromNow = new Date(now.getFullYear() + 1, 0, 1);
  // 日本の祝日カレンダーのIDを定義
  const holidayCalendarId = 'ja.japanese#[e-mail]';
  const calendar = CalendarApp.getCalendarById(holidayCalendarId);
  const holidays = calendar.getEvents(now, oneYearFromNow);

  const holidayDates = [];

  // 各イベント（祝日）について繰り返し処理
  for (const holiday of holidays) {
    // 祝日の日付を取得し、日本のタイムゾーンで"yyyy/MM/dd"形式にフォーマット
    const holidayDate = Utilities.formatDate(holiday.getStartTime(), Session.getScriptTimeZone(), "yyyy/MM/dd");
    // 祝日の日付を配列に追加
    holidayDates.push(holidayDate);
  }

  return holidayDates; // 祝日の日付を含む配列を返す
}

function isHoliday(dateToCheck) {
  const holidays = getHolidays(); // 祝日のリストを取得

  // 指定された日付が祝日のリストに含まれているかどうかを確認
  const formattedDateToCheck = Utilities.formatDate(dateToCheck, Session.getScriptTimeZone(), "yyyy/MM/dd");
  return holidays.includes(formattedDateToCheck);
}

// 指定された日付の前の平日（土日を除く前の日）を取得する関数
function getPreviousWeekday(triggerDate) {
  const previousDate = new Date(triggerDate);

  do {
    previousDate.setDate(previousDate.getDate() - 1);
  } while (!isWeekday(previousDate) || isHoliday(previousDate));

  return previousDate;
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

祝日の条件分岐のコードを追加いたしました。
その際にソースコード変更したため当日実行がされなくなってしまいました。

また以前実行したものを再実行してしまうようになってしまいました。
↓こちらのコードで対応しています
const reservationDate = new Date(triggerDate);
Logger.log("トリガー日: " + reservationDate.toLocaleString("ja-JP"));

// 実行日がトリガー日の前営業日より前の日付やトリガー日より後の日付には通知しない
const prevWeekday = getPreviousWeekday(today);
Logger.log("前営業日: " + prevWeekday.toLocaleString("ja-JP"));
if (today < prevWeekday || today >= reservationDate) {
  continue;
}
// 通知トリガーが実行日の4日前より未来の場合は通知しない
const fourDaysBeforeTrigger = new Date(reservationDate);
fourDaysBeforeTrigger.setDate(reservationDate.getDate() - 4);
Logger.log("通知トリガーの4日前: " + fourDaysBeforeTrigger.toLocaleString("ja-JP"));
if (today < fourDaysBeforeTrigger) {
  continue; 
}

補足

Apps Script

Answer 1

コードを全てチェックしたわけではありませんが、
気になる部分が2箇所あります。

1) 日付の大小比較で当日を含めて比較をしています

    // 実行日がトリガー日の前営業日より前の日付やトリガー日より後の日付には通知しない
    const prevWeekday = getPreviousWeekday(today);
    Logger.log("前営業日: " + prevWeekday.toLocaleString("ja-JP"));
    if (today < prevWeekday || today >= reservationDate) {
      continue;
    }

の部分で、
「トリガー日より後の日付」とコメントに書いてあるのですが、
実際のコードが today >= reservationDate になっているので
同じ日付の場合であっても当日の予定は通知されなくなってしまいます。
ここは > で比較すべきです。

2) 実行日が時刻を含んだままになっています

  // 現在の日付と時刻を取得
  const today = new Date();
  Logger.log("今日: " + today.toLocaleString("ja-JP"));

ここのログを見ると時刻が含まれていることがわかると思います。
シートに入力した実行予定日には時刻は入力されていないと思いますので、
午前零時になっています。
これも

    const reservationDate = new Date(triggerDate);
    Logger.log("トリガー日: " + reservationDate.toLocaleString("ja-JP"));

のログで確認できます。
このことにより、>= や　> 等で日付の大小比較をすると
実行日の方が実行予定日より時刻の分だけ大きいと判断されてしまいます。

実行日と実行予定日を日付のみで比較できるように

  // 現在の日付と時刻を取得
  const today = new Date();
  today.setHours(0, 0, 0, 0); // 時刻を午前零時にする
  Logger.log("今日: " + today.toLocaleString("ja-JP"));

このように、時刻部分を午前零時にしておきます。

ちなみに、時刻を全て午前零時で統一しておくと
等しいかどうかの比較はisSameDate関数を使わなくても

if (today.getTime() === reservationDate.getTime()) {
  ...
}

のように日付オブジェクトのメソッド1つで比較できるようになります。

Comments

[atlanticSalmon]

ご希望どおりの結果になるかどうかはわかりませんが、
少なくとも実行日と実行予定日が同じときに
日付の比較でcontinueされなくなるので、

// 当日通知実行
if (

のif文には到達すると思います。
それ以上は確認していません。

[Rin22209853]

ありがとうございました。
ご回答いただきました箇所を修正したところ正常に動きました。

コードを書いているうちに訳が分からなくなってきてしまっていたので、大変助かりました。
丁寧にご説明いただき誠にありがとうございました。