Gmailでの指定日時送信（同時刻に複数送信したい場合）について

Question

Googleフォームで受けつけた内容をもとに、GoogleAppsScirptを使ってGmailで指定日時にメールを送信したいのですが、
同じ日時に複数の人への送信ができません。
コードのどこが間違っているか、またどのようにすればいいか教えていただけないでしょうか。
よろしくお願いいたします。

送信についての条件

• 送信する日時：９時、10時、11時など、毎時０分。下書きメールの件名に記載
• 送信する数：日時によって違う
• 送信先：毎回異なる
• 送信メール：Googleフォームの回答受けつけ時に、その内容をもとに下書きメールを別のscriptであらかじめ作成

スクリプトが行っていること

下書きのメールを取得し……

• 下書きメールの件名に　{西暦年/月/日 時:分}（例:{2019/01/31 13:00})　があれば、その日時を抽出して、指定日時送信のscriptが実行された時間よりも件名の日時が過去の日時であれば送信する。
• 件名に　{西暦年/月/日 時:分}　がない、もしくは件名の日時がscript実行時の日時よりも未来であれば送信しない。

スクリプトのトリガー

最初はトリガーを１分に１回で設定していたが、GASのGmail読み取り制限に引っかかってしまったため、
現在は特定の日時にトリガーを作成するscriptを使用して、指定日時送信メールのscriptを実行させている。

問題点

関数のデバッグは、件名の日時を過ぎてから行っているが、
下書きメールの件名　{西暦年/月/日 時:分}　の日時が同じものが複数あっても、画像の赤枠部分の１件しか送信されていない。
なお、下書き16件分の中にはGmailに設定してある返信定型文や、件名に**{西暦年/月/日 時:分}**がないものも含まれる。

ちなみに、件名に**{西暦年/月/日 時:分}**がある下書きメールを全て削除すると、ログでは全ての下書きメールが出てきます。

function delaySendDraftMail() {
  var drafts = GmailApp.getDraftMessages();
  var len = drafts.length;
  Logger.log('下書きの数(len):' + len);
  //下書きがなければ終了
  if (!len) {
    return false;
  }
  //現在時刻を取得
  var now = (new Date()).getTime();
  //下書きの数だけ繰り返す
  for (var i = 0, l = len; i < l; i++) {
    var mes = drafts[i];   
    Logger.log('型(mes):' + typeof mes);
    
    if ('object' !== typeof mes) {
      continue;
    }
    //件名を取得
    var str = mes.getSubject();    
    Logger.log('件名(str):' + str);
    //件名から日時を抽出
    var match = str.match(/^(\{(\d{4}[/\-]\d{1,2}[/\-]\d{1,2} \d{1,2}:\d{1,2})\}) ?(.*)?/);
    Logger.log('件名から抽出した日時(match):' + match);
    //日時が抽出できないならスキップ
    if (!match || !match[1]) {
      continue;
    }
    //時間を取得
    var time = (new Date(match[2].replace(/\-/g,'/')+' +09:00')).getTime(); 
    Logger.log('時間(time):' + time);
    //時間を取得できない、または未来の時間なら無視
    if(!time || (time && time>now)){
      continue;
    }
    //各情報をセット
    var to = mes.getTo();
    var subject = match[3] || '';
    var body = mes.getPlainBody();
    var options = {}, val;
    //必要な情報がなければスキップ
    if (!to || !body) {
      continue;
    }
    // From を変更した場合でも対応
    var from = mes.getFrom();
    var aliases = GmailApp.getAliases();   
    Logger.log('別名アドレス(aliases):' + aliases);
    
    for (var i = 0, l = aliases.length; i < l; i++) {
      var val = aliases[i];
      // From エイリアス一覧にマッチすれば From として使用
      if (-1 !== from.indexOf(val)) {
        options['from'] = val;
        break;
      }
    }
    if (val = mes.getCc()) {//Cc
      options['cc'] = val;
    }
    if (val = mes.getBcc()) {//Bcc
      options['bcc'] = val;
    }
    if (val = mes.getBody()) {//HTML本文
      //bodyにdivタグがあればHTMLとみなす
      if ( val.indexOf('<div')!==-1 ) {
        options['htmlBody'] = val;
      }
    }
    //添付ファイル
    if (val = mes.getAttachments()) {
      options['attachments'] = val;
    }
    // 送信！
    var status = GmailApp.sendEmail(to, subject, body, options);
    Logger.log('送信(status):' + status);
    
    //送信したら下書きをゴミ箱へ
    if (status) {
      mes.moveToTrash();
    }
  }
}

Comments

[t_obara]

問題点を見つけるために、コードを最小限にしてどのように振る舞うかを確認するのが効果的なことがあります。今回もたとえば、何もしないようにして問題なくメールリストアップできるか、ゴミ箱にだけ移動させたら実施できるかなどちょっとずつ処理を追加して何をしたことで想定外の動作をするか確認してみてはいかがでしょうか。
また、sendEmailの戻りはGmailAppオブジェクトなので、それを参照して送信できたかを確認するのは間違った使い方だと思います。

Answer 1

// From を変更した場合でも対応
    var from = mes.getFrom();

（省略）

//添付ファイル
    if (val = mes.getAttachments()) {
      options['attachments'] = val;
    }

この部分が不要だったので削除し、送信元メールアドレスと名前は毎回同じでいいので、その部分を以下のように書きかえました。

// var options = {}, val;　←これはやめた
    var options = {
    　from: '●●●●●●@●●●.●●●.jp', 
      name: 'あああああああああ'
    };

これで同じ日時に複数送信できました。
なんだか恥ずかしい結果ですが、解決して良かったです。

Answer 2

※全面書きかえしました。t_obara様のご指摘に感謝します

原因は、draftsをループしているカウンタ変数iおよび上限変数lと
別名(alias)をループしているカウンタ変数iおよび上限変数lが同じであることではないでしょうか。

下記概念コードで示しますが、hereのところにあるlは最初のループから「1」が出力され、ループは1回で終端します。

function q175842() {
  var limit = 5;
  for(var i = 0, l = limit; i < l; i++) {
    var alius = 1;
    for(var i = 0, l = alius; i < l; i++) {
      Logger.log("inner");
    }
    Logger.log(l); //here
    Logger.log("outer");
  }
}

従って提示コードが上手く動くのは、下書きとaliasが上手く強調している場合のみで
例えば下書きが3通あるときは、上から順にaliasの指定が3件、2件、1件となっている特殊状況です。

Comments

[t_obara]

ドキュメントによると、GmailDraftではなくGmailMessageのようなのですよね。で、GmailMessageにはmovoToTrashメソッドがあります。
https://developers.google.com/apps-script/reference/gmail/gmail-app#getdraftmessages
https://developers.google.com/apps-script/reference/gmail/gmail-message#movetotrash

[papinianus]

ご指摘ありがとうございます。getDraftsとgetDraftMessagesを混同いたしておりました。
ご指摘および当初より回答で懸念を表明しておりました停止するならエラーが発生するはず、の2点を考慮し、全コードを読み直して、回答を書き換えました。

[papinianus]

質問者様へ
質疑で指摘されているstatusの件ですが、私の技量ではtry/catchくらいしか思いつきませんでした。
```
var sentSuccess = false;
try {
GmailApp.sendEmail();
sentSuccess = true;
} catch (e) {
// sentSuccess = false;
}　finally　{
if(sentSuccess) { mes.moveToTrash(); }　
}
```
みたいな

[mozuq]

お二方ともに、ご回答ありがとうございます。

■t_obata様
ちょっとずつ処理をしていく、確かにそのとおりですね。
ログを確認することに夢中で、思いつきませんでした。
sendEmailの件につきまして、ご指摘いただきありがとうございます。

■papinianus様
原因を教えていただきありがとうございます。
今回、私が質問として書いたコードですが、ネットにあるものを参考にしました。
原因とコードを照らし合わせ、コードの中に自分には不要な箇所があることがわかり、書きかえると同じ日時に複数のメールを送信することができました。
なお、try～文の提示につきましてもありがとうございます。
こちらにつきましては、もう少し自分で勉強してコードを変えていきたいと思います。