スプレッドシートの他のシートから行を転記すると、空の列が追加されてしまう

Question

実現したいこと、問題

スプレッドシートの他のシートから行を転記したいです。
動作自体は問題なくするのですが、全ての動作の後に空白の行が追加されます。
if文ではチェックがついていないなら転記しないように設定しているのでチェックがないものはどこかから来たのではなくて新しく生成されたのだと思いますが、なぜ生成されてしまうのかわかりません。
何行追加されるかは時によります。

該当のソースコード

function repaired() {
  var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();

  var ss_copyFrom = spreadsheet.getSheetByName('修理中');
  var ss_copyTo = spreadsheet.getSheetByName('SHOP');
  var ss_copyTo2 = spreadsheet.getSheetByName('発送中');
  var Rows = ss_copyFrom.getLastRow();
 

for(let i = 2; i < Rows; i++){
  var checkCellRow = i;//値が入る行番号
  var checkCellCol = 4;//値が入る列番号

  var checkCellValue = ss_copyFrom.getRange(checkCellRow,checkCellCol).getValue();
  var receive = ss_copyFrom.getRange(i,13).getValue();

  if(checkCellValue == true && receive === 'ご来店'){
    var copyValue = ss_copyFrom.getRange(checkCellRow,1,1,39).getValues();//checkCellRow(=10)行目の39列目までの値をコピー
    var writeCellRow = ss_copyTo.getLastRow();//コピー先のシートの値が入っている最終行番号をゲット    ss_copyTo.getRange(writeCellRow+1,1,1,39).setValues(copyValue);//コピー先のシートの値が入っている最終行の次の行にcopyValueを書き込み

 ss_copyFrom.deleteRow(checkCellRow);
  }else if(checkCellValue == true && receive !== 'ご来店'){
   var copyValue = ss_copyFrom.getRange(checkCellRow,1,1,39).getValues();//checkCellRow(=10)行目の39列目までの値をコピー
    var writeCellRow = ss_copyTo2.getLastRow();//コピー先のシートの値が入っている最終行番号をゲット

    ss_copyTo2.getRange(writeCellRow+1,1,1,39).setValues(copyValue);//コピー先のシートの値が入っている最終行の次の行にcopyValueを書き込み

 ss_copyFrom.deleteRow(checkCellRow);
  }//if end

 ss_copyTo.getRange(i,19).clearContent();
 ss_copyTo2.getRange(i,19).clearContent();
}//for end

}

試したこと

よろしくお願いします。

Answer 1

時間の経過した質問ですが、GASの未回答を減らしたいため回答いたします。

ざっとしか確認できていませんが、

var Rows = ss_copyFrom.getLastRow();

ここで取得している最終行は、この時点のものであるたるため、

ss_copyFrom.deleteRow(checkCellRow);

シートの行を直接削除しても、rowの値はかわりません。

２行目からRow行目まで回した場合、途中で行を削除するため
最後の方の行は空白行になってしまいます。

例）row が２０（行）だったとする場合、途中で２行削除すると18行目が最終行になりますがrowは２０のため
１９行目と２０行目は空白行になっています。

  } else if (checkCellValue == true && receive !== 'ご来店') {

ここに関しても、checkCellValue は '' のため、trueで通りますし、receive !== 'ご来店'も通るため、
空白行が追加されます。

これを回避するためには、
forの順番を昇順ではなく降順で（最終行から）実行していく必要があります。

for (var i = Rows; i > 2; i--) {
　　//処理
}

※getValueやsetValueを多用するとGASから警告（シートへのアクセスが多すぎると）が出ますので、
先に全ての値を取得し、最後に貼り付ける方がおすすめです。

Comments

[Sou23]

長く困っておりました。ありがとうございます！大変助かりました。

[Tatsunosuke]

お役に立てて良かったです！