こんな事がありました

他の人が書いたGoogleAppsScriptのコードを見ていて(と書くことによって「私じゃないよ！」を強調したい下心が見えますね)、Dateオブジェクトを渡すとその1週間後(7日後)が日本の祝日かどうかを判定する関数 isHolidayNextWeek() がありました。

こんな感じで使われていました。

function main() {
  const date = new Date("2023-10-02") // 翌週の 2023-10-09 は スポーツの日です 
  if(isHolidayNextWeek(date)){
    console.log(`${Utilities.formatDate(date,"JST","yyyy-MM-dd")} の1週間後は祝日です`)
  }
}

function isHolidayNextWeek(date) {
  const nextWeekMonday = new Date(date.setDate(date.getDate() + 7));
  const calendar = CalendarApp.getCalendarById('ja.japanese#holiday@group.v.calendar.google.com');
  const count = calendar.getEventsForDay(nextWeekMonday).length;
  if (count) return true
  return false
}

↓ main() の実行結果がこちら

2023-10-09 の1週間後は祝日です

…え。こちらの想定としては

2023-10-02 の1週間後は祝日です

でした。

これが参照渡しってやつか！！

JavaScript における「値渡し」「参照渡し」の説明は別のところで調べてもらうとして

プリミティブは値渡し
オブジェクトは参照渡し

のようです。

(2023-05-03追記: 正確には「JavaScriptには参照渡しは存在しない」ようです。私は「参照値という値渡し」という表現がピンと来ました。参照先)

今回の date はオブジェクト(Dateクラスのインスタンス？) なので main() の中の date と isHolidayNextWeek() の中のdate は「同じものを参照している」ということ、、、なんだろうな。

function isHolidayNextWeek(date) {
  const nextWeekMonday = new Date(date.setDate(date.getDate() + 7)); // dateにsetしちゃダメ！
...

ここが date.setDate() なので、dateにsetしちゃってるのよね。だから main() の中のdateも7日後になっちゃってる、と。

これ、頭ではわかっててもうっかりやっちゃいそう。。。

こうすればいいよね

const nextWeekMonday = new Date(date.getFullYear(), date.getMonth(), date.getDate() + 7);

dateオブジェクトは変更しないで、dateから取得した値によって new Date() する、と。

これ、気をつけないと「A関数からB関数にオブジェクトを渡して、B関数の中でオブジェクトを変更しちゃうと、その変更にA関数からは気づけない」みたいなことになりますよね。