[GAS]スプレッドシートにfor文で値を入力

Question

前提・実現したいこと

今アクティブなシートではない
他のスプレッドシートを取得して
3行目の列の中からif文で該当する文字列と同じ
列を探し出し、その列の
42行目に×を入力させる。

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

for文以降の処理が行われない。

該当のソースコード

function myfunction(){
    var sh1 = SpreadsheetApp.openById("スプレッドシートのID")
    Logger.log(sh1.getName());
    var sh2 = sh1.getSheetByName('シートの名前');
    Logger.log(sh2.getName());
    var lastrow = sh2.getLastRow();//最終行取得
    Logger.log(lastrow);
    var getName1 = sh2.getRange(2, 3, lastrow-1, 1);
    Logger.log(getName1);
    var getName2 = getName1.getValues();
    Logger.log(getName2);
    var getDate1 = sh2.getRange(2, 5, lastrow-1, 1);
    var getDate2 = getDate1.getValues();
    Logger.log(getDate2);
    var kakunin = sh2.getRange(2, 42, lastrow-1, 1);
    var kakuninV = kakunin.getValue();

    for(var i=0;i<lastrow.length;i++){
     if(getName2[i][3]== "文字列"){
     　sh2.getRange(i,42).setValue("×");
     }
    }
}

試したこと

ログでvar kakuninVまでは取得できていることが分かっているので
for文以降の記述が誤っている？

Comments

[papinianus]

ABCの方向は列ですが、かりに3行目がC列の意味ならこれではさがせません。配列は0から始まるのでCは2だからです。
また、getRangeで最終行マイナス1していますが、これだと最終行が取れないので、最終行に目的の値があるとき、何も起こらない、という事象になります。

Answer 1

for(var i=0;i<lastrow.length;i++)
の部分ですが、lastrowが最終行までの行数なのでi<=lastrowだけにすればよいかと。