GASの配列を使用して文字入力する速度を速めたい

Question

前提・実現したいこと

GASの配列を使用して文字入力する速度を速めたい

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

以下のコードで上記に記載の内容を実現したいのですが、
処理速度が遅く、現場で使用できるレベルではありません。

該当のソースコード

function　勤怠() {
  var sheet1 = SpreadsheetApp.openById('*********');
  var date = new Date(); //現在日時のDateオブジェクトを作る
  var day = date.getDate();//現在の「日」を取得
  date.setDate(day+1);//前日日付にしたいので-1する
  var yesterday = Utilities.formatDate(date, 'JST', 'MM');//日付の表示形式を整形する
  var sh1 = sheet1.getSheetByName("勤怠");
  var sh5 = sheet1.getSheetByName(yesterday + '月1部');//【ト
  var col = sh5.getRange(7,3).getValues();
  var i = sh5.getRange(sh5.getMaxRows(), 5).getNextDataCell(SpreadsheetApp.Direction.UP).getRow()
  var ary1 = sh5.getRange(10,col,i,1).getValues();
  var ary2 = sh5.getRange(10,6,i,1).getValues();
  var ary3 = sh1.getRange("A6:A32")
  for(let j = 0; j < ary2.length ;j++){
  var KEY=ary2[j][0]
  if (ary3.createTextFinder(KEY).findNext()) {
  Logger.log("存在します！");
  ary1[j][0]="欠勤"
  sh5.getRange(10+j,col).setValue(ary1);  
  }
  }
}

試したこと

ここに問題に対して試したことを記載してください。

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

Comments

[norapomu]

コードの挿入でコードを挿入しないと回答してくれる方が見にくいため、修正してください。
また、質問が「Javaの質問」になっていますが、エクセルのVBAではないでしょうか？

[Nihsi2407]

コードの挿入でコードを挿入しないと回答してくれる方が見にくいため、修正してください。
⇒初心者の為、ご容赦下さい。また、こちらの意味あいが不明なのですが、ご教授いただけないでしょか？

Answer 1

• データ規模と今どれだけかかっているかの計測値とどれくらいになったら 現場で使用できるレベル になるのかの目標値がなければ、有効な高速化は難しいです。

• 質問がかわったら、別の質問を立ててくださらないと辛いです(質問が変わって何やってるかを編集から追う必要があるので)。つらいっていうのは回答しづらいってことです。

• 色々と推測でやっているので、もしかしたら意図した動作でないかもしれないです。サンプルデータがほしいです。

const q314116 = () => {
    const book = SpreadsheetApp.openById('1tIjXivR9bVB0_7edrBhtv9z7lKI5zuz7bEPa_kawP_w');// 【つくば2】入出庫＿シフト
    const now = new Date();
    const yesterday = new Date(now.getFullYear(),now.getMonth(),now.getDate()-1);
    const attendance = book.getSheetByName("勤怠").getRange("A6:A32").getValues().flatMap(e=>e);
    const monthlySheet = book.getSheetByName((yesterday.getMonth() + 1).padStart(2, '0') + '月1部');//【つくば2】入出庫＿10月シフト
    const monthlyData = monthlySheet.getDataRange().getValues();
    const absentCol = monthlySheet[6][2];
    const nameCol = 5;
    const listLength = monthlyData.length;
    for(let i = 9; i < listLength; i++) {
        const name = monthlyData[i][nameCol];
        if(!attendance.includes(name)) continue;
        monthlyData[i][absentCol] = "欠勤";
    }
    monthlySheet.getRange(1,1,listLength,monthlyData[0].length).setValues(monthlyData);
}

Comments

[Nihsi2407]

コードサンプルを作成頂き、ありがとうございました！
サンプルコードにより、無事目的の動作とスピードとなることができました！

Answer 2

var ary4 = sh1.getRange("A3").getValues();

for(let j = 0; j < ary1.length ;j++){
  ary3[0][0]=ary2[j][0]
  sh1.getRange("A1").setValues(ary3)
  if(ary4="TRUE"){
  ary1[j] ="欠勤"
  sh5.getRange(10+j,col).setValue(ary1);
}

ary4は単一セル（A3）なのですべての行が同じ結果になるのは当然です。
また、ary4は2次元配列ですしif(ary4="TRUE")ここは条件判定式になってないし、プログラムにも欠陥があります。

TRUEの判定が下った時のみ、該当のセル１つに欠勤の文字を入力したい

TRUEの判定箇所が1箇所なので、何がしたいのか全く理解できませんので、これ以上の回答はつけようがありません。

Comments

[Nihsi2407]

返信ありがとうございます。
ary4に関してはary4[0][0]と記載
if(ary4="TRUE")に関してはif(ary4=="TRUE")
このように修正させて頂ければと良いでしょうか？

また、セル上の式に関して補足説明もさせていただきました。
ご確認の程よろしくお願い致します。

[macaron_xxx]

ary4にはgetValuesしたタイミングの値しか入っていないので、いくらforの中でA1の値を書き換えてもary4は変わりません。
forの中でary4を毎回取得することでも対応できますが、とても無駄で時間がかかる処理なので、A1,A2,A3の役割をすべてプログラム上で実現した方がいいでしょう。

[Nihsi2407]

ご指摘いただいた内容を修正いたしましたところ、動作はできる様になりました。
ありがとうございます。
ただ、おっしゃっているように処理速度に問題があるようなのですが、こちらを高速かする手段はないのでしょうか？
ご確認の程よろしくお願い致します。

[macaron_xxx]

細かく言うと色々あるんですが、どこに時間がかかってるのか、Logから探すのがいいでしょうね。

sh5.getRange(10+j,col).setValue(ary1);
をforの外に出して、sh5.getRange(10,col,i,1).setValues(ary1);
だけでも早くなるのでは。