GASで行を取得して、別シートの最終行に転記する

Question

当方プログミラング系統は完全に素人で、お見苦しい点も多々あるかと思いますが、ご容赦いただけますと幸いです。

前提・実現したいこと

Google Spreadsheetにて、
GASで計算シートに入力した値(A~C列)をもう一つのログシート最終行に転記するようなマクロを作成したいのですが、
うまく挙動しない(実行されるものの、変化がない)ため質問させていただきました。
解決法をご存知の方おられましたらご教示いただきたいです。
よろしくお願いします。

function cpaste() {
//計算シートとログシートを設定
var calcsheet = SpreadsheetApp.getActive().getSheetByName("計算");
var logsheet = SpreadsheetApp.getActive().getSheetByName("ログ");
 
//ログシートの最終行を取得
var dblastrow=logsheet.getLastRow()+1
 
//値が入力されている間は繰り返し
let i = 2;
  while(calcsheet.getRange(i,3).getValue()){
    i++;
  }
  
//ログシートの最終行に各データを貼り付け
logsheet.getRange(dblastrow,1).setValue(calcsheet.getRange(i,1).getValue())
logsheet.getRange(dblastrow,2).setValue(calcsheet.getRange(i,2).getValue())
logsheet.getRange(dblastrow,3).setValue(calcsheet.getRange(i,3).getValue())

}

Answer 1

while で繰り返し処理を期待しているなら、以下を手直ししてみてください。

・var dblastrow=logsheet.getLastRow()+1　は while の中へ （最終行が変わっていくから）

・i++; }　の前に「最終行に各データを貼り付け」の処理を入れる　（While 内に入れて、処理が終わってから iを増やす）

Comments

[as.y4nd4]

試したところ期待通りに実行することができました。
本当にありがとうございます。

[sawa]

慣れてきたら、まとめて取得、まとめて貼付け（getValues、setValues）のやり方も挑戦してみてください。
今のやり方だと一つ一つ処理してるので、量が多いと結構時間かかります。

[as.y4nd4]

おっしゃる通り、3行程度なら気になりませんが10行を超えると少し億劫なくらいの処理時間でしたので、
これを機会にまとめて処理するコードにも挑戦してみたいと思います。