for構文の回数が一回多くなる

Question

お世話になります。
こちらのサイトを参考に、スプレッドシートに登録されている人にメールを一括送信するgasを組んでいます。
https://liapoc.com/bulk-transmission.html

実際に実行すると、メールは送信されますが、for 構文が最後の空の列を取得して終わります。
動作が一回分多いのですが、これをどう変更すればよろしいでしょうか？

よろしくお願いいたします。

function myFunction() {
 

var mySheet = SpreadsheetApp.getActive().getSheetByName('連絡先');
var rowSheet　=　mySheet.getDataRange().getLastRow(); 
 

var docTest=DocumentApp.openById("/*documentID*/"); 
var strDoc=docTest.getBody().getText();  
 
for(var i=2;i<=rowSheet;i++){
//for(var i=2;i<=rowSheet;i--){修正のご指摘をいただきました。
 
var strEmail=mySheet.getRange(i,1).getValue();　
var strSei=mySheet.getRange(i,2).getValue();
var strMei=mySheet.getRange(i,3).getValue();
 
var strBody=strDoc.replace(/{姓}/g,strSei).replace(/{名}/g,strMei);
 
 
/* メール表題、fromアドレス、差出人名を準備 */
var strSubject = strSei + " " + strMei+"様 ●●●●●●●●"; //姓 名 様 メールタイトル
var strFrom="/*Mailadress*/"; //From
var strSender="/*Name*/"; //差出人

 
   /* メールを送信 */
    MailApp.sendEmail(
      strEmail, //toアドレス
      strSubject,  //メールタイトル
      strBody,　//本文
      {
        from: strFrom,　//fromアドレス
        name: strSender　//差出人
      }
    ); 
    
  }
}

Comments

[ikedas]

掲載しておられるのは、実際に実行したコードでしょうか? 相違がないかよく確認して、違っていれば訂正してくださいませんか。

[TD0]

/* を除いて、全て実行した通りのコードをお載せしています。

[ikedas]

forの行は本当にこうなっているんですか……?

[TD0]

すみません、++で実行しています^^;

[ikedas]

質問本文を訂正してください。この追記・修正の依頼は隠れてしまうので、他の人が気づきません。

[TD0]

ご指摘ありがとうございました。

Answer 1

• var rowSheet　=　mySheet.getDataRange().getLastRow();

ikedas様が回答で引用された仕様のとおりなので、冗長です。var rowShet = myShet.getLastRow();とすべきです。

• ロジックにより、ループが1回多くなる理由(今回はあてはまらないように見えるので参考程度に)

手続型プログラミングでありがちなのは、forの終了条件の判定で、不等号でイコールを含むかどうかを誤っているパターン。これに対処するには、そもそもforをしない、という選択を考えるべきです。具体的にはArray.prototype.forEachを利用するなどをご検討ください。

• 今回の1回多い理由と対処

ロジック上の誤りには見えないので、データ起因です。つまり回答は自分で探すしかないです。そこで探しかた。
どうやってループが1回多いと思ったかが疑問です。MailAppは、宛先が不正だとエラーになるので、仮にA列が空白だったら、メールは送信されないため(その後の実行も中断する)ので気付けないはず(例えばAからC列は5行しかデータがないのに、F6にタイプミスなどで文字があるときには、A6が空白なので、メール送信エラーで結局5通しか送信できない。あり得ないが送信できたとして、空白の宛先なので誰も知りようがない)。
仮に、メールのあてさきや姓、名が同じものが2通来たことを根拠に判定しているとしても、それは行が非表示になっているなどして見えないだけで、ループが1回多いかどうか分からないはず。
さらに、この2つの組み合わせとして、質問者様は3行と思っているのに、getLastRowは6を取得し、5行目のA列がないため、1回しか多くないようにみえてるパターンもあり得ます。
などと考えると、gasが取得しているrowSheetやそれぞれの都度のstrEmailなどをLogger.logして見ていくほか原因も対処法も不明です
(一般論として、strEmailなどが""だったらbreak/continueするというのは対処としてなくはないですが、メールが2通来ているパターンは対処できません)

• A列の最終行を知りたい場合

A,B,Cはペアなのでこれだけでは上手くいきませんが、A列にあるデータの個数(スプレッドシート関数で言うCOUNTA(A:A))は、下記のように取得できます。ただし、こちらはCOUNTA相当なので、途中にA列にデータがない行があると、A列のデータがある最終行を取得できない(足りなくなる)ので、ご注意ください。

function getLastRowOfA(sheet) {
  return sheet.getRange("A:A").getValues().filter(String).length; //"A:C"とやっても動かないのでご注意ください
}

Comments

[TD0]

他の条件式についても順次勉強に励みます…

ループが多いと気づいたきっかけは実行トランスクリプトからです。記載されているメールアドレス全てに確実にメールは送信されているため、確認をしたところ最後のメール送信の後、もう一度アドレス・姓名が空の状態で送信を試みていました。メールは正しく、一行一件送信されています。
ご指摘の通りループが止まらない状態なのかもしれません。エラーが発生して強制的に終了しているだけなのかもしれないです…

元データには空白等はありません。A列の最終行取得のアイデアを拝借して再構築してみます。

[papinianus]

フォームの回答だと考えにくいですが、空白がある、のではなく使用していない列（Z列とか）に余計なデータがあるのでは、というのが推測です。
確認方法としては
Logger.log(mySheet.getDataRange().getValues())とすると取得した全データが見れるので思ったデータになっているか見てください

[TD0]

ご指摘の通り、行末にデータが含まれていました。
(共同編集者がいたため、作成過程のメモを下欄に残していたようです。)
大変丁寧にご指導いただいて、ありがとうございました。
今後とも何卒よろしくおねがいいたします。

Answer 2

SheetのGetDataRange()の説明によると

This is functionally equivalent to creating a Range bounded by A1 and (Range.getLastColumn(), Range.getLastRow()).

説明がちょっとわかりにくいですが、mySheet.GetDataRange()は、シートの左上から、データが入っている一番右・一番下までの範囲を返すようです。ですから最終行より下 (左から3つめまでのセルとは限らない) にデータや関数が入っていたりすれば、mySheet.GetDataRange().getLastRow()はその行番号を返します。つまりmySheet.getLastRow()と同じになるようです。

特定の列だけを調べてデータが入っている最後の行を返すようなメソッドはないようです (あるのかもしれませんが私は見つけられませんでした)。不要なデータのあるセルを消すか、またはセルが空であれば繰り返しをやめるようにしてはどうでしょうか。

Comments

[TD0]

繰り返しをやめるという選択肢、検討してみます。
次のステップとしてonFormsubmitのトリガー設定と、
アドオンとしての公開を実践してみようとしていたため、
mySheet.GetDataRange().getLastRow()を設定していました。
（mySheet.getLast.Row()はご指摘の通り結果は同じだと把握しています。）

特定列に絞れないんですね…セルに関しては一定規則のもとフォームから転送されるデータですので空は存在しないものとして構築しています。