GASのfor文で繰り返すと1回目で時間フォーマットが合わないとエラーになる

Question

実現したいこと

当日の日付にあった当番をSlackに通知する

詳細

スプレッドシートに記載してある「日付」「名前」「当番内容」「担当時間」から、日付が当日の日付とマッチした場合にSlackに通知するbotをつくっています。

2023/03/28　さとう　そうじ　9:00
2023/03/29　たなか　せんたく　10:00
2023/03/28　いとう　せんたく　11:00

上記のようなデータがスプレッドシートにあり、当日の日付が「2023/03/28」の場合、Slackには2行通知されるとおもいますが、「Utilities.formatDate」の「"JST","HH:mm"」と指定しているとエラーになります。
フォーマットを指定しない場合エラーはでませんが、下記のように2行目だけが正常に通知され1行目はフォーマットされてないデータでSlack通知されてしまいます。

2023/03/28　さとう　そうじ　Sat Dec 30 1899 09:00:00 GMT+0900 (Japan Standard Time)
2023/03/28　いとう　せんたく　11:00

1行目も2行目も正しい時間表記で通知するにはどうしたら良いのでしょうか？

発生している問題・エラーメッセージ

エラーメッセージ　Exception: The parameters (String,String,String) don't match the method signature for Utilities.formatDate.

該当のソースコード

function myFunction() {
  
 var webhook = "https://hooks.slack.com/xxxxxx"; 

 var today = new Date();
 var date =  today.getFullYear() + '年' +(today.getMonth() + 1) + '月' + today.getDate() + '日';

 var spread_sheet = SpreadsheetApp.openByUrl('https://docs.xxxxxxxxxxxxx');
 var target_sheet = spread_sheet.getSheetByName("xxxxxxxxxxxxx
 var data_arrays = target_sheet.getDataRange().getValues(); 
 var message = "";
 var num=0;
 
  data_arrays.forEach(function( data_array ){
   cell_date = new Date(data_array[0]);
    if(cell_date.getFullYear() == today.getFullYear() && cell_date.getMonth() == today.getMonth() && cell_date.getDate() == today.getDate())
   {
     for (var i = 1; i <= 1; i++) {
       start_time = data_array[0*i]; //日付
       start_time2 = data_array[1*i]; //なまえ
       start_time3 = data_array[2*i]; // 掃除内容
       start_time4 = data_array[3*i]; // 時間
       if(start_time != ""){         
         time_text = start_time 
         message += Utilities.formatDate(start_time,"JST","MM/dd") + start_time2 + start_time3 + Utilities.formatDate(start_time4,"JST","HH:mm") + "\n";
       }
     }
     num++;
     return false;
   }
 })

 var jsonData = {"username": "おしらせbot", "text": message};
 var payload = JSON.stringify(jsonData);
 var options =
      {
        "method" : "post",
        "contentType" : "application/json",
        "payload" : payload
      };

  // 1件もない時は通知しない処理をいれる
  if ( num != 0 ) {
    UrlFetchApp.fetch(webhook, options);
  }
}

Answer 1

当初の回答に修正を加えました。

--<当初の回答>--

forEachの中なので、
for(;;)ループはいらないと思います。

そして、ver i も不要なので配列インデックスは、
そのまま0,1,2,3として、

start_time = new Date(data_array[0]);
start_time2 = data_array[1];
start_time3 = data_array[2];
start_time4 = new Date(data_array[3]);

と日付、時刻は一度オブジェクトにしてから代入しましょう。

for (var i = 1; i <= 1; i++) {
の行とfor(;;)ループを閉じる
}
の行を削除します。

(iPhoneからなので、回答が整形できてるか確認できませんが)

オブジェクトにすると、何かしらの日付になるので、

if(start_time != ""){         
  time_text = start_time 
  message += Utilities.formatDate(start_time,"JST","MM/dd") + start_time2 + start_time3 + Utilities.formatDate(start_time4,"JST","HH:mm") + "\n";
}

のところも、if文を外して、
使っていないのなら、
time_text = start_time
の行も削除しましょう。

--<以降は回答の追加です。>--

まず、日付、時刻の全角入力やセミコロンの入力を訂正して日付オブジェクトを生成する補助関数を
myFunctionの前か後に保存します。

//日付、時刻の文字列から日時のオブジェクトを返す
function genDate(_date) {
  if (Object.prototype.toString.call(_date) != "[object Date]") {
    //日付オブジェクトでない→セミコロン、全角数字を修正
    _date = _date.toString().replace(/[：；;]/g, ":");
    _date = _date.replace(/[０-９]/g, s => String.fromCharCode(s.charCodeAt(0) - 0xFEE0));
    //'22:22'→'1989/12/30 22:22:00'
    var hhmm = /^\d+:\d+$/;
    if (hhmm.test(_date)) {
      _date = "1899/12/30 " + _date + ":00";
    }
    return new Date(_date);
  } else {
    //日付オブジェクトならそのまま返す
    return _date;
  }
}

そしてforEach文をこれに合わせて修正します。

  data_arrays.forEach(function (data_array) {
    var cell_date = genDate(data_array[0]);//日付
    if (cell_date.getFullYear() == today.getFullYear() &&
      cell_date.getMonth() == today.getMonth() &&
      cell_date.getDate() == today.getDate()) {
      start_time = cell_date;
      start_time2 = data_array[1];//なまえ
      start_time3 = data_array[2];//掃除内容
      start_time4 = genDate(data_array[3]);//時刻
      message += Utilities.formatDate(start_time, "JST", "MM/dd") +
        start_time2 + start_time3 +
        Utilities.formatDate(start_time4, "JST", "HH:mm") + "\n";
      num++;
      // return false;//←この行は不要
    }
  })

最後に不要な行がもう一つありましたので、削除（コメントに）します。

  // var date = today.getFullYear() + '年' + (today.getMonth() + 1) + '月' + today.getDate() + '日';

以上で、日付、時刻に全角数字など不用意な入力があっても、ある程度は対応できるようになります。

Comments

[manmosu]

フォーマットの形成とループを正しくしたら解決しました。
大変助かりました。ありがとうございます！

[YellowGreen]

あと、日付や時刻を質問者さんだけが入力しているのなら問題ないと思いますが、いろんな方が入力する場合は、どうしても時刻にセミコロンを使ったり、数字を全角で入力したりということが起きます。
そのような場合に対応できるスクリプトの修正例をお示ししたいのですが、しばらく後になると思います。

[YellowGreen]

修正をアップしました。

Answer 2

スプレッドシートに記載している「担当時間」の具体値の型が文字列になっているのではないでしょうか。

Comments

[manmosu]

ご回答ありがとうございます。表示形式を時間にしても、書式なしテキストなどにしてみても、エラーが解消しない状況でした。

[arcxor]

表示形式を時間にした際の実際のセルに表示された内容をご提示ください。

[manmosu]

時：分で表示形式を時間にして、実際のセルには「9:00」「10:00」「11:00」と表示されています。

[manmosu]

yellowgreenさんの回答にて、フォーマットの形成とループを正しくしたら解決しました。ご回答いただきありがとうございました！助かりました。