セルが空なら列を削除、としているのに空でなくても消える

Question

前提・実現したいこと

該当セルが空の場合のみ、その列を削除したいです。

発生している問題

該当セルが空でない場合にも削除されます。

該当のソースコード

var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getSheetByName("ご予約")
var lastRow = sheet.getLastRow()+1;
var lastcolumn = sheet.getLastColumn();
var range = sheet.getRange(1,2,lastRow,lastcolumn);


for(var i = 1; i <= lastcolumn; i++){
if(sheet.getRange(i,6).isBlank){
  Logger.log(sheet.getRange(i,2).getValue());
  Logger.log(sheet.getRange(i,6).getValue());
  sheet.deleteRow(i);
}

試したこと

ログをとって(i,6)が空でないことは確認しました。
よろしくお願いします。

Comments

[itagagaki]

if 内に書いた Logger.log(sheet.getRange(i,6).getValue()); で空でないということは、if の条件式に問題があるということですよね？

Answer 1

var lastRow = sheet.getLastRow()+1;
var lastcolumn = sheet.getLastColumn();

for(var i = 1; i <= lastcolumn; i++){　//←最終列が入ってます
if(sheet.getRange(i,6).isBlank（）){　// ６列目のi行目が空なら・・・

  sheet.deleteRow(i);　//←行の削除
}

列と行がごっちゃになってますのでそこを修正いただければ。
Those rows are out of bounds.　<= 範囲外エラーです。

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

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

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

for (var i = lastRow; i > 1; i--) {
　　//処理
}

Answer 2

if(sheet.getRange(i,6).isBlank){
は
if(sheet.getRange(i,6).isBlank()){
の誤りではないでしょうか。

Comments

[Sou23]

ありがとうございます。その点を修正したのですが、
Those rows are out of bounds.
と出てしまいました。
もしまたお分かりでしたらよろしくお願い申し上げます。

[itagagaki]

getRangeの引数は(row,column)ですが、逆に指定しているように見えますね。