この閉鎖は正しいですか、私の推論に間違いがありますか？

Question

Google Scriptsでスクリプトコードを高速化するためにクロージャを使用しようとしています。概念は私には新しかったが、クロージャが以下のコードで正しく適用されるかどうかは疑問だ。 （コードが動作する）

背景：このコードは、1か月にどれだけ割合が進んでいるかを計算することです。

関数が呼び出されるたびにnow変数とmonth変数が更新されることは望ましくありませんが、ドキュメントが開かれている場合は一度だけ更新されます。私が本質的に望むのは、すべての変数が一度しか設定されておらず、関数が複数回呼び出された場合にその関数が答えを返すということです。

これはクロージャを使用する正当な理由ですか、正しく使用していますか？

アドバイスと忍耐ありがとうございます。

// calculate progress with second precision 
var progressCalc = (function() { 

    const stMonth = SpreadsheetApp.getActive() 
    .getRangeByName("pStartdate") 
    .getValue(); 

    const eoMonth = SpreadsheetApp.getActive() 
    .getRangeByName("pEndOfMonthDate") 
    .getValue(); 

    const now = new Date(); 

    var unixProgressEoMonth = unixTime(eoMonth)-unixTime(stMonth) 
    var unixProgressNow = unixTime(now)-unixTime(stMonth) 

    return function() { return unixProgressNow/unixProgressEoMonth;}; 

})(); 

function progress() { 
    Logger.log(progressCalc()); 
    return progressCalc(); 
} 

Answer 1

私はGoogleのスクリプトで私のスクリプトコードをスピードアップするために、クロージャと協力しようとしています。

次に、progressCalcが呼び出されるたびに計算を評価しないでください。

これはクロージャを使用するのに適していますか？

いいえ実際には、計算結果が一定であるため、関数に入れて何度も実行する必要はありません。ファンクションを保存する代わりに、すぐに結果値を保存してください。

var progressValue = (function() { 

    const stMonth = SpreadsheetApp.getActive() 
    .getRangeByName("pStartdate") 
    .getValue(); 

    const eoMonth = SpreadsheetApp.getActive() 
    .getRangeByName("pEndOfMonthDate") 
    .getValue(); 

    const now = new Date(); 

    var unixProgressEoMonth = unixTime(eoMonth)-unixTime(stMonth) 
    var unixProgressNow = unixTime(now)-unixTime(stMonth) 

    return unixProgressNow/unixProgressEoMonth; 
})(); 

function progress() { 
    Logger.log(progressValue); 
}