スプレッドシートの最終行の値をドキュメントでreplaceする

Question

前提・実現したいこと

初心者です。
自社（工場）で故障が起きた際に、作業者がFORMを送信。それをトリガーにして、複数のメールアドレスに自動メールが行くような仕組みをつくろうとしています。
メール本文に故障の場所や設備種類などFORMで回答したいくつかの事項を織り込んだメールにしたいのですが…。また送付先の複数メールアドレスは、スプレッドシートにメアドのリストを対象としています。

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

var factory = rangeValues2[LastRow][4];//工場
という行で、下記のエラーが出ます。

TypeError: Cannot read property '4' of undefined

該当のソースコード

function sendMail() {
  
  //故障報告がされたら、自動でリストにあるメアドへ通知メールを送る

  // Googleドキュメントからメールの件名と本文を取得
  var doc = DocumentApp.openById('XXX');
  var subject = doc.getName();
  var text = doc.getBody().getText();

  // Googleスプレッドシートからデータ取得

  var ss = SpreadsheetApp.openById("ZZZ");
  //var spreadsheet = SpreadsheetApp.getActive();

  var sheet2 = ss.getSheetByName('故障報告');//故障報告シートを定義
  var range2 = sheet2.getDataRange();//該当シートのデータ範囲を取得
  var rangeValues2 = range2.getValues();//取得したデータ範囲の配列を取得
  var LastRow = range2.getLastRow();//データ範囲の最終行を取得

    var factory = rangeValues2[LastRow][4];//工場
    var type = rangeValues2[LastRow][5];//設備種類
    var reporter = rangeValues2[LastRow][3];//報告者メアド
    var datetime = rangeValues2[LastRow][2];//報告日時（タイムスタンプ
    var status = rangeValues2[LastRow][13];//状態
    var demand = rangeValues2[LastRow][16];//要求   

    var replacedText = text.replace('{工場}',factory).replace('{設備種類}',type).replace('{報告者メアド',reporter).replace('{タイムスタンプ}',datetime).replace('{要求}',demand);


  var sheet = ss.getSheetByName('故障通知メアド');
  var range = sheet.getDataRange();
  var rangeValues = range.getValues();

  for(var i = 1; i < rangeValues.length; i++) {
    var name = rangeValues[i][0];
    var position = rangeValues[i][1];
    var email = rangeValues[i][2];

    var replacedText = text.replace('{氏名}', name).replace('{役職}', position);
    
    MailApp.sendEmail(email, subject, replacedText);

試したこと

始めたばかりで、分からないことだらけですが、自分なりにネットでいろいろ探してみたのですが…。
お恥ずかしいような初歩的なことな気がしていますが、職場にも友人にも詳しい人がおらず、独学でやるしかない状況です。どうかよろしくお願いします。

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

Comments

[sawa]

解決された場合は、「解決済み」としましょう。

[sactac]

すみません、いまやりました。
初めてで使い方がわからなくて、教えていただきありがとうございます。

Answer 1

コードの中でセルを（配列で）指定する場合、行の始まり、列の始まりはゼロからスタートします。

たとえば、1行目、1列目を指定する場合は、
rangeValues2[1][1]　ではなく、それぞれ1を引いて
rangeValues2[0][0]
としなければなりません。

同じように、10行目、4列目を指定する場合は、
rangeValues2[9][3]
と書く必要があります。

---

【現状】
「故障報告」シートに、たとえば10行まで入力済みの状態で。sendMailを実行すると、

  var rangeValues2 = range2.getValues();//取得したデータ範囲の配列を取得
  var LastRow = range2.getLastRow();//データ範囲の最終行を取得

の段階で、
・rengaValues2は、10行×16列のデータの塊（配列）になっており
・LastRowには10が代入されています。

次に

    var factory = rangeValues2[LastRow][4];//工場

というように、 rangeValues2[10][4] にアクセスしようとしていますが、
これは実際には11行目　5列目のデータを取得する意味になり、10行までしかないrangeValuesの範囲を超えているため、エラーになってしまいます。

ですから、コード上は、指定する行・列の数字から1を引かなければなりません。

【修正】
アクセスする行・列の数値を1引きます。

(略)
  var sheet2 = ss.getSheetByName('故障報告');//故障報告シートを定義
  var range2 = sheet2.getDataRange();//該当シートのデータ範囲を取得
  var rangeValues2 = range2.getValues();//取得したデータ範囲の配列を取得
+  var LastRow = range2.getLastRow() - 1; //データ範囲の最終行を取得（1を引く）
+    var factory = rangeValues2[LastRow][3];//工場
+    var type = rangeValues2[LastRow][4];//設備種類
+    var reporter = rangeValues2[LastRow][2];//報告者メアド
+    var datetime = rangeValues2[LastRow][1];//報告日時（タイムスタンプ
+    var status = rangeValues2[LastRow][12];//状態
+    var demand = rangeValues2[LastRow][15];//要求  
(略)

Comments

[sactac]

ありがとうございます！！解決しましたーーー(TT
配列はゼロからと教わっていたのに、列番号ばかりに気を取られて、行の方は忘れていました。
本当に助かりました、できて感動です。
また何かございましたら、お助けくださいm(__)m