GASの配列格納後のセル指定。for以下に記述すると[列]部が「undefined」となる

前提・実現したいこと

gasにてガントチャートに類似したタスク進捗管理表を作成中。
高速化のため配列処理への書き換えを試みていますが、
下記問題でつまづいています。
やりたいこととしては、作業開始日/完了日の入力有無により未着手、仕掛中、完了を表示したいです。
下記問題はつまづいているところまで記載しています。
（ifの作成はこれから）

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

配列格納後、セルを指定しようとしたところ、for以下に記述すると[列]部が「undefined」となる。forが無い場合、同じ記述内容でも指定セルの値は正しく取得できる。

■for以下に記述　※「undefined」になる
function hairetu_test() {

var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getActiveSheet();
var lastRow = sheet.getLastRow();

//二次元配列として取得
var values = sheet.getRange(1,1,lastRow,39).getValues();
//for用の変数
var count = values.length;

var c1 = 11;
var c2 = 12;

for(var r_test = 4 ; r_test <= count; r_test++){
var array_jissekistart = values[r_test][c1]//ここの「c1」11が取得できず「undefined」になる
var array_jissekiend = values[r_test][c2]

Logger.log(array_jissekistart);
}
}

■forが無い場合　※正しく取得
function hairetu_test() {

var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getActiveSheet();
var lastRow = sheet.getLastRow();

//二次元配列として取得
var values = sheet.getRange(1,1,lastRow,39).getValues();
//for用の変数
var count = values.length;

var c1 = 11;
var c2 = 12;
var r_test = 4
var array_jissekistart = values[r_test][c1]
var array_jissekiend = values[r_test][c2]

Logger.log(array_jissekistart);
}

試したこと

変数では無く数値で入力した場合も同様に「undefined」とfor無しは正しく取得できる。
Logger.logで確認済み。Webで調べても記述内容の誤りが見つけきれず。

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

ここにより詳細な情報を記載してください。

行動規範の内容に同意します

回答2件

ベストアンサー

典型的コードミスであります。

for(var r_test = 4 ; r_test <= values.length; r_test++){
  var array_jissekistart = values[r_test][c1]
  var array_jissekiend = values[r_test][c2]

  Logger.log(array_jissekistart);
}

原因は配列の仕様に対する勘違いかと思います。
配列は「0」から数えますが、lengthメソッドは要素数を1から数えます。
そのため、要素が1つ多いと判断され、参照エラーを起こしていると思われます。

for(var r_test = 4 ; r_test < values.length; r_test++){
  var array_jissekistart = values[r_test][c1]
  var array_jissekiend = values[r_test][c2]

  Logger.log(array_jissekistart);
}

また、要素数を1つずつ行うのでしたら、for~of構文またはforeachメソッドを用いるのが現在のトレンドであります。前者は対応しておりませんでしたが、現在では実装されたので問題なく使うことができます。

投稿2020/03/23 07:47

zigutabi

総合スコア57

rsmt

2020/03/23 09:33

ありがとうございます！！！ forでも正しく取得できました。初歩的なミスすいません！！for～of試してみます。

行動規範の内容に同意します

理由は簡単です。

forループで回した最後のcountの場所に値が入っていないからではないかと。r_test<=countではなくr_test < countにすれば。forは0からカウントしますので、ラストが10行目ならr_test < 10とします（が10は11行目になります）。なので、countを使うならばcount - 1としたら最後まで取れるのではないかと。。

javascript
1function hairetu_test() {
2  var ss = SpreadsheetApp.getActiveSpreadsheet();
3  var sheet = ss.getActiveSheet();
4  var lastRow = sheet.getLastRow();
5  
6  //二次元配列として取得
7  var values = sheet.getRange(1,1,lastRow,11).getValues(); 
8  //for用の変数
9  var count = values.length;
10  var c1 = 11;
11
12  for(var r_test = 4 ; r_test < count; r_test++){
13    var array = values[r_test][c1];
14    
15    Logger.log(r_test);
16  }
17}