JavaScriptで土日祝日を除いた即日発送表示がしたい

Question

実現したいこと

JavaScriptを使って、土日祝日を除いた営業日の13時までは当日発送(今日の日付)、13時以降は1営業日後の日付(平日であれば翌日、土日祝は1営業日)を表示させたい。

前提

時間帯の条件分岐はできるのですが、祝日のカウントがされません。
(4/7を祝日指定しても4/7発送と出てしまう)

該当のソースコード

const holiday = [
  '2023/4/6',
  '2023/4/7'
];

dayWek = new Array("日", "月", "火", "水", "木", "金", "土");

const objDate = new Date();
const objYMD = '(' + ((objDate.getMonth() + 1)) + '/' + objDate.getDate() + ')';
let shipDate = null;
const hour = objDate.getHours();

function shippingExpress() {
  if (!isHoliday(objDate) && hour <= 12) {
    shipDate = objDate
  } else {
    shipDate = objDate.getDate()
    do {
      objDate.setDate(objDate.getDate() + 1);

      if (!isHoliday(objDate)) {
        shipDate = objDate
      }
    } while (shipDate === null)
  }

  if (shipDate !== null) {
    const year = objDate.getFullYear();
    const month = objDate.getMonth() + 1;
    const date = objDate.getDate();
    const weekday = dayWek[objDate.getDay()];
    const outputToday = document.getElementById("js-getToday");
    const outputScheduleDate = document.getElementById("js-shipTiming");
    outputToday.innerHTML = `${objYMD}`;
    outputScheduleDate.innerHTML = `${year}<small>年</small>${month}<small>月</small>${date}<small>日</small><small>(${weekday})</small>`;
  }
}

function isHoliday(date) {
  objDate.getDay(date) == 6 || objDate.getDay(date) == 0 || holiday.includes(formatDate(date, 'YYYY/MM/DD'));
}

function formatDate(date, format) {
  format = format.replace(/YYYY/, date.getFullYear());
  format = format.replace(/MM/, date.getMonth() + 1);
  format = format.replace(/DD/, date.getDate());
  return format;
}

よろしくお願いいたします。

Comments

[YellowGreen]

isHolidayにdateを渡しているのに、objDateの土日判断をしていますね。
getDay()は引数は不要です。

function isHoliday(date) {
return date.getDay() == 6 || date.getDay() == 0 || holiday.includes(formatDate(date, 'YYYY/MM/DD'));
}

と修正してはいかがでしょうか。

[nana7777]

コメントありがとうございます！修正してみたのですが、今日は土曜日で明日が日曜ですが明日発送の日付になっています。土日祝日判断がうまくできていないのでしょうか。。

[YellowGreen]

isHoliayは、こちらでも動作確認できでます。
日付に応じてtrueとfalseが帰ります。

} else {
の次の行の
shipDate = objDate.getDate()
があるので、
do {
...
} while (shipDate === null)
が機能していません。

shipDate = objDate.getDate()
を削除かコメントにしてみるとどうなりますか？

[YellowGreen]

実行する関数は、shippingExpressなのですよね。
せっかく、shipDateを求めているのに
代入する値はobjDateからになっているのが不思議です。
同じ値を参照しているので変わりはないのですが、
shipDateは条件判断だけに使っているということでしょうか。

[YellowGreen]

2023/04/08 02:50 編集の
shipDate = objDate.getDate() を削除かコメントにしてみるとどうなりますか？
をスクリプトに反映（削除またはコメントアウト）してから、

if (shipDate !== null) {
の次の行に
console.log('shipDate: ' + shipDate);
console.log('objDate: ' + objDate);
を挿入してshipDate（objDateも同じものを参照）の値をログで見ることができます。
今日実行すると、
shipDate: Mon Apr 10 2023 HH:mm:ss GMT+0900 (Japan Standard Time)
objDate: Mon Apr 10 2023 HH:mm:ss GMT+0900 (Japan Standard Time)
（HH:mm:ssは実行時刻が入る）と表示されます

[nana7777]

追加のコメントをいただきありがとうございます！！(確認が遅くなりました)

shipDate = objDate.getDate()
を削除すると無事に祝日もカウントして思い通りの表示をしてくれるようになったかと思います。
(今日が日曜日なので、明日を祝日設定にしてみて明日どう表示されるか確認してみます)

今回の即日発送部分以外にも◯日後発送の実装をしていて、先にobjDateを使っていたのとそのコードも参考にしつつ、他のコードも参考にしていたのでごっちゃになっていました。

おっしゃる通り
console.log('shipDate: ' + shipDate);
console.log('objDate: ' + objDate);
を入れると両方同じものが返ってきました。

普段はJavaScriptを自分で書くこと機会がほとんどなく、勉強してもしても毎回悩んだり忘れたりするので、とっても助かりましたm(_ _)m
ご親切にありがとうございました！

Answer 1

function isHoliday(date) {
//  ↓ 少なくともこれが必要
  return objDate.getDay(date) == 6 || objDate.getDay(date) == 0 || holiday.includes(formatDate(date, 'YYYY/MM/DD'));
}

Comments

[nana7777]

回答ありがとうございます！return 抜けていました！