javascriptの繰り返し処理を正常化したい

Question

実現したいこと

00秒の時にメールを送信するプログラムを記述したい。

前提

gas でJavaScript系のプログラムを記述しています。

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

繰り返し処理が終了しない。
処理が終了しないので、エラーメッセージは表示されない。

該当のソースコード

const today = new Date();
const body = String(today.getMonth() + 1) + ’/‘ + today.getDate() + ‘-‘ + today.getHours() + ‘:’ + today.getMinutes() + ‘.’ + today.getSeconds() ;
console.log(body);


const sec = today.getSeconds();
console.log(sec);
for (let i=0;i<999999999;i++){
  if (sec <= 0){
  console.log(sec);
  GmailApp.sendEmail(address,subject,body);
  break
  }
}

試したこと

i<9999999999のところの数字をいじってみましたが、うまくいきません。

補足情報（FW/ツールのバージョンなど）

secをconstで宣言していることがおかしいのか、
breakではforを終了できないのどちらかだと考えています。

Answer 1

ビジーループで00秒を待つ、なんてことはしてはいけません。無駄にCPUコストを消費することになります。

GAS でしたら Utilities.sleep(msec) で待つとか、トリガーで時間指定しましょう。

Comments

[takahashi]

ごめんなさい。トリガーでやってもいいのですが、プログラムコードで指定してみたいです。
(secの宣言をforの中に入れると、)構文的でエラーはでないのに、うまく動かない理由を知りたいんです。

[int32_t]

現在試しているコードが提示されていないのではっきりした事は言えませんが、質問文のコードの sec の宣言を移動しただけではダメです。today の値も更新しないと today.getSeconds() はいつも同じ値を返します。

[takahashi]

todayを定義しているところのコードを追加しました。

[int32_t]

today = new Date(); と sec の宣言をループの中に入れて、for (...) を while (true) にすれば意図どおりのコードにはなると思います。999999999 回程度のループだったら1秒もかからずに終わるので。

[takahashi]

解決しました。
ありがとうございました

Answer 2

secは最初の行で代入されて、それっきりどこからも変更されてる様子はありません。

なので、それ以降何回ループさせようと、secの値は変わりませんよ

Comments

[takahashi]

secの宣言をforの中に移動してみましたが、それでもうまくいきませんでした。

[y_waiwai]

それはまた別の問題。
エラーメッセージよく読もう

[takahashi]

forが終了しないので、エラーメッセージが表示されません。

Answer 3

最終的にこのようになりました。

while(true){
 const today = new Date();
 const body = String(today.getMonth() + 1) + ’/‘ + today.getDate() + ‘-‘ + today.getHours() + ‘:’ + today.getMinutes() + ‘.’ + today.getSeconds() ;
 const sec = today.getSeconds();
  if (sec <= 0){
  console.log(body);
  GmailApp.sendEmail(address,subject,body);
  break;
  }
}