GASで最後の1行の処理ができない

Question

実現したいこと

GASを用いて、スプレッドシート上で値が入力されている最後の行まで処理を行いたい。

前提

https://teratail.com/questions/tdhhsye5sh2cca
先日上記の質問をした者です。

いただいた回答で納得したつもりだったのですが、やっぱり思っていた挙動と違ったので、再質問させてください。

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

function QRSendMail(){

  var sheetname = 'customer_list';
  const spreadsheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(sheetname); 
  const lastRow = spreadsheet.getLastRow();
  var rangeValues = spreadsheet.getDataRange().getValues(); 

  for(var i=3 ; i<=lastRow ; i++){
    var sending = rangeValues[i][0];

最初、上記のように書いていて、いただいた回答に従い、下記のように修正しました。（ifの条件内の＝の削除）

function QRSendMail(){

  var sheetname = 'customer_list';
  const spreadsheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(sheetname); 
  const lastRow = spreadsheet.getLastRow();
  var rangeValues = spreadsheet.getDataRange().getValues(); 

  for(var i=3 ; i<lastRow ; i++){
    var sending = rangeValues[i][0];

当初の「i<=lastRow」だと、前回の質問のようなエラーが出てしまうのですが、
「i<lastRow」だと、エラーが出ない代わりに最後の行の1つ上の行までしか処理されません。

4～21行に値が入っている時、4～20行が処理されます。

配列の考え方がなかなか理解できず情けないのですが、なるべく分かりやすい説明と解決策をいただけると幸いです。
宜しくお願いいたします。

Comments

[pecmm]

質問文のソースが途中で途切れていますが、forブロック内の記載してない範囲でも i を用いた処理を何か記述されていますか？

もしもそこでrangeValues配列にアクセスしたり、シート(spreadsheet変数)のメソッド呼び出しで i を使っているならば、インデックスがずれているせいで結果がおかしく見えているのかもしれません。

配列の添え字が0開始(処理範囲は3-20)、シート関数の行番号指定は1開始(処理範囲は4-21)となるので
全ての処理でそれらの整合性を取る必要があります。

[退会済みユーザー]

var dept2 = rangeValues[i][6];
のように、別の複数の列の値を取ったあとに、
if(dept2 != ''){
GmailApp.sendEmail(mail, title, text1, options);
spreadsheet.getRange(i+1,2).setValue(today);
}
として、メールを送り、メールを送った日付を2列目の4行目以降のセルに入力するように、書いたつもりです！

[pecmm]

申し訳ないですが、断片的なソースコードでは断片的な回答しかできません。
全文を質問文に追加すればよりよい回答が得られるかもしれません。

エラーが出ないのに部分的な処理しかできなかったのであれば、もしかすると自動メール送信の上限にひっかかっているかもしれません。(私は上限まで送ったことないので、エラーがでるのか単に処理が打ち切られるのか、どうなるのか把握していませんが……)

一時的に実際の処理(シート書き込みとかメール送信とか)をコメントアウトし代わりにconsole.log などにデバッグ出力して、どのデータに対してどんな処理がされる予定であったか確認してみれば分かりやすいかもしれません。

[退会済みユーザー]

iを使っているのは、rangeValuesが[i][0]～[i][11]で複数列に並んでいるものと、
追記したspreadsheet.getRange(i+1,2).setValue(today)のみです。
何回か試して同じように最後の1行のみ処理されないため、メール送信数の上限は関係ないです。

[YellowGreen]

G21のセルの値をご確認ください。
値が入力されていますか？

Answer 1

19：50修正
質問へのコメントを見ての情報も追加して少し分かりやすくなるようにしてみました。
ループ内でB列にデータを書き込む処理を追加してあります。

提示されているコードを実行した結果です。
コードは実行できるよう}を追加しています。

function QRSendMail(){

  var sheetname = 'customer_list';
  const spreadsheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(sheetname); 
  const lastRow = spreadsheet.getLastRow();
  var rangeValues = spreadsheet.getDataRange().getValues(); 

  Logger.log(`lastRow = ${lastRow}`);
  Logger.log(rangeValues);

  for(var i=3 ; i<lastRow ; i++){
    var sending = rangeValues[i][0];
    Logger.log(`i = ${i} : sending = ${sending}`);
    spreadsheet.getRange(i+1,2).setValue(`today${sending}`);
  }
}

実行前のスプレッドシート

出力されたログ

19:42:05 お知らせ 実行開始
19:42:05 情報 lastRow = 21
19:42:05 情報 [[1.0], [2.0], [3.0], [4.0], [5.0], [6.0], [7.0], [8.0], [9.0], [10.0], [11.0], [12.0], [13.0], [14.0], [15.0], [16.0], [17.0], [18.0], [19.0], [20.0], [21.0]]
19:42:05 情報 i = 3 : sending = 4
19:42:05 情報 i = 4 : sending = 5
19:42:05 情報 i = 5 : sending = 6
19:42:05 情報 i = 6 : sending = 7
19:42:05 情報 i = 7 : sending = 8
19:42:05 情報 i = 8 : sending = 9
19:42:05 情報 i = 9 : sending = 10
19:42:05 情報 i = 10 : sending = 11
19:42:05 情報 i = 11 : sending = 12
19:42:05 情報 i = 12 : sending = 13
19:42:05 情報 i = 13 : sending = 14
19:42:05 情報 i = 14 : sending = 15
19:42:05 情報 i = 15 : sending = 16
19:42:05 情報 i = 16 : sending = 17
19:42:05 情報 i = 17 : sending = 18
19:42:05 情報 i = 18 : sending = 19
19:42:05 情報 i = 19 : sending = 20
19:42:05 情報 i = 20 : sending = 21
19:42:07 お知らせ 実行完了

実行した結果のスプレッドシート

ログを見てもらえば、配列の添え字と実際の行の関係が分かるのではないかと思います。

Comments

[退会済みユーザー]

質問へのコメントで追記しているのですが、値を取得したのちにメールを送り、送った日付を記入する処理をしています。
そこの整合性が取れていないのかなと思われますが、どこが間違っているのか分からず困っています。

[YAmaGNZ]

コメントにあったループ内の処理であろう
spreadsheet.getRange(i+1,2).setValue(today);
を追加して、結果のスプレッドシートを追記しました。

[退会済みユーザー]

ご回答ありがとうございましたｍｍ

Answer 2

(既にYAmaGNZさんも似たようなことを提案されていますが……)

function test_QRSendMail(){
  var sheetname = 'customer_list';
  const spreadsheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(sheetname); 
  const lastRow = spreadsheet.getLastRow();
  var rangeValues = spreadsheet.getDataRange().getValues(); 
  
  for(var i=3 ; i<lastRow ; i++){
    var sending = rangeValues[i][0];
    
    // 省略されている処理
    
    console.log('dept2', dept2, dept2 != ''); // ※ここ
    if(dept2 != '') {
      //GmailApp.sendEmail(mail, title, text1, options);
      console.log('GmailApp.sendEmail', mail, title, text1, options); // ※ここ
      //spreadsheet.getRange(i+1,2).setValue(today);
      console.log(`spreadsheet.getRange(${i+1},2), setValue ${today}`); // ※ここ
    }
  }
}

一時的に実際の処理(シート書き込みとかメール送信とか)をコメントアウトし代わりにconsole.log などにデバッグ出力して、どのデータに対してどんな処理がされる予定であったか確認してみれば分かりやすいかもしれません。

少なくとも提示されている範囲では、dept2 の判定辺りが怪しいといえば怪しいです。

Comments

[退会済みユーザー]

ご回答いただきありがとうございましたｍｍ

Answer 3

配列が原因ではなかったと明言したらいいのではないですか？
ここまで引っ張っておきながら、プライバシーですか？
それを伏せて原因を説明すると思いますが、それをしないでおしまいにするのは、後味が悪いですね。
これまで考えて提案してしきた方のことをどのように考えているのでしょうか？

Comments

[YellowGreen]

あなたが配列の処理がおかしいとタイトルに書いて、前回の回答が間違っているかのような質問をあげたので、皆さんあなたに配列を具体例を挙げて説明してきたのですよ。

[退会済みユーザー]

煮詰まって書いていた部分を全て消して書き直したらうまくいきました。
正直なところ、質問に載せておらず消した部分のどこが悪かったのか、一番の原因は分からないままでしたが、
質問した部分は間違っていませんでしたと自己解決のところに記載しています。

考えて回答していただいた方々には感謝しております。
デバッグやログの見方を提案していただいたのは初心者の私にとって勉強にはなりましたが、
直接的な解決方法ではなかったためベストアンサーとしては選べませんでした。

配列が間違っていると思う、という私の質問であるのは確かですが、
載せている部分だけでは間違っているとは断定できないよ、というのが皆さんの総意であると受け取りました。

不快にさせてしまい申し訳ございません。
口論をしたいわけではないので、これ以上のコメントは控えさせていただきます。

[YellowGreen]

当方もこれ以上コメントしたくありません。

[YellowGreen]

上のコメントをこれ以上編集しないでください。

Answer 4

自己解決できました、、

質問に載せていたiの設定は間違っておらず、
載せていなかった部分のコードを一部書き直し、一晩頭を冷やしたら解決できました。
質問しているのにプライバシーに関わりうると判断し載せられなかった部分なので、私が悪いです。
ちょっと煮詰まってしまっていたんだと思います；；

ご迷惑をおかけしました。回答やコメントをしていただいてありがとうございました。

Comments

[YellowGreen]

こんな解決は悲しいです。