GASスクルプトで実行するとなぜかメールが多く配信される。

Question

以下のスクリプトを実行すると何故か、最後に一致する人にだけ2個メールを送ってしまいます。
何かこの書き方に間違えがあるのでしょうか。
自分ではどこに問題があるかわかりません。
宜しくお願いします。

function sendAllMail() {
  //　このスクリプトの実行日の取得
 var today = Utilities.formatDate(new Date(), 'Asia/Tokyo','yyyy/MM/dd');; //　今日の日付をyyyy/mm/dd形式に直して取得
 
  // spreadSheetオブジェクトを取得
 let spreadSheet = SpreadsheetApp.getActive()
 
 // 件名と本文が書かれたsheetオブジェクトを取得
 let templateSheet = spreadSheet.getSheetByName("データ項目")
 
 // 件名はD2セルに入ってるのでその値を取得
 let title = templateSheet.getRange("D2").getValue()
 
 // 送信対象のメールアドレス一覧が記載されてるシートを取得
 let sheet = spreadSheet.getSheetByName("回答")

 var col = "A"; 
 var last_row = sheet.getLastRow(); 
 var range = sheet.getRange(col + "1:" + col + last_row); 
 var value = range.getValues(); 
 var array = []; 
 var isDate = []; 

  for(var i = 0; i < value.length; i++){ 
    array.push(value[i][0]); 
    isDate.push(Object.prototype.toString.call(value[i][0])) 
     if (value[i] !== undefined && isDate[i] === "[object Date]"){ 
      array[i] = Utilities.formatDate(array[i], 'Asia/Tokyo','yyyy/MM/dd'); 
     if (array[i] == today){ 
       let targets = sheet.getRange(i+1, 3, i, 5).getValues()
         
       // 本文の「%企業名%」を企業名に置き換えてメールを送信
         try {  
           for (target of targets) {
             var fileurls = target[4].split('/');
             var fileid = fileurls[5];
             var file = DriveApp.getFileById(fileid);//添付ファイル取得
             // 本文はB2セルに入ってるのでその値を取得
             let body = templateSheet.getRange("E2").getValue()
             body = body.replace(/%名前%/, target[0])

             // スプレッドシートのメアド、件名・本文でメールを送信
             GmailApp.sendEmail(target[1], title, body, {attachments: [file]});
             }
          }
           catch (e) {
             continue;
          }     
        }
      }
    }  
  }   

回答シート以下
|タイムスタンプ|受付番号|名前|メールアドレス|性別|年代||
|:--:|:--:|:--:|:--:|:--:|:--:|
|2020/10/18 19:37:14|1|a|[e-mail]|男|5059歳|https://drive.google.com/file/d/xxxx/view?usp=drivesdk|
|||||||
|2020/10/15 10:09:00|2|c|[e-mail]|女|3039歳| https://drive.google.com/file/d/xxx/view?usp=drivesdk|
|||||||
|||||||
|||||||
|2020/10/19 20:34:44|3|d|[e-mail]|男|4049歳|https://drive.google.com/file/d/xxx/view?usp=drivesdk|
|2020/10/17 10:09:07|7|e|[e-mail]|男|5059歳| https://drive.google.com/file/d/xxx/view?usp=drivesdk|
|2020/10/19 20:04:04|46|f|[e-mail]|男|40~49歳| https://drive.google.com/file/d/xxx/view?usp=drivesdk|

Comments

[Daregada]

「回答」シートのサンプルを質問文に載せてください。

[退会済みユーザー]

失礼しました。
回答シート追加しました。

Answer 1

提示されたコードでは、A列の日付のリストをvar value = range.getValues();で取得し、それに基づくforループの内部で、日付が今日と一致した場合にだけlet targets = sheet.getRange(i + 1, 3, i, 5).getValues();でターゲット情報を取得しています。

getRangeがこの形式で4つの引数を取る場合、先頭から順番に、

• 範囲先頭の行番号(1オリジン)
• 範囲先頭の列番号(1オリジン)
• 範囲の行数
• 範囲の列数

を指定します。

あなたのコードでは、(ターゲット情報を1行だけ取得すればいいのにもかかわらず)、iの値の行数だけの複数のターゲット情報を取得し、それらを順番に処理していますね。そうすると、先頭の日付が今日でなくても、今日の日付の下にあるターゲット情報が使われてしまいます。

forによるループは、A列の日付のリストに関する処理だけにして、今日の日付と一致した場合だけ、1行だけのターゲット情報をその都度取得すればいいでしょう。

なお、今日の日付に一致するターゲットが多数ある場合には、日付リストと同様にgetValuesでターゲット情報のリストをあらかじめ作成しておき、それを使った方が処理が軽くなります。

function myFunction() {
  // 今日の日付文字列(yyyy/MM/dd)を取得
  var today = Utilities.formatDate(new Date(), 'Asia/Tokyo','yyyy/MM/dd');
  Logger.log(`toray: ${today}`);
  
  // テンプレートと宛先リストの各シートを取得
  let spreadSheet = SpreadsheetApp.getActive()
  let templateSheet = spreadSheet.getSheetByName("データ項目")
  let sheet = spreadSheet.getSheetByName("回答")

  // テンプレートシートから共通タイトルと本文テンプレートを取得
  let title = templateSheet.getRange("D2").getValue()
  Logger.log(`title: '${title}'`);
  let body_template = templateSheet.getRange("E2").getValue();
  Logger.log(`body_template: '${body_template}'`);
  
  // 宛先リストから日付(A列)だけのリストを取得
  let a1note = `A2:A${sheet.getLastRow()}`;
  Logger.log(`a1note: ${a1note}`);
  let date_values = sheet.getRange(a1note).getValues();
  Logger.log(`date_values: ${date_values}`);
  
  for (let i = 0; i < date_values.length; i++) {
    Logger.log(`loop: ${i}`);
    
    if (date_values[i] === undefined || Object.prototype.toString.call(date_values[i][0]) !== "[object Date]") {
      Logger.log("skipped");
      continue;
    }
    
    // 日付リストから日付文字列(yyyy/MM/dd)を取得
    let target_date = Utilities.formatDate(date_values[i][0], 'Asia/Tokyo','yyyy/MM/dd');
    Logger.log(`target_date: ${target_date}`);

    // 今日の日付文字列と一致するものだけを処理
    if (target_date === today) {
      Logger.log("matched");

      // 行番号は i + 2, 列番号は3(C列)から、「1行・5列分のデータ」(名前、メールアドレスなど)を取得する
      // getValuesの返り値は2次元配列になるので、先頭の要素(1行分の配列)だけを取り出す
      let target_values = sheet.getRange(i + 2, 3, 1, 5).getValues()[0];
      
      // URLを分割してfileidを取得し、ファイルグロブ情報を得る
      let fileid = target_values[4].split('/')[5];
      let attached_file = DriveApp.getFileById(fileid);

      let target_name = target_values[0];
      let target_address = target_values[1];
      
      // 本文のテンプレートから実際の本文を生成
      let body = body_template.replace(/%名前%/, target_name);
      
      Logger.log(`To: ${target_address}, Subject: ${title}, Body: ${body}, Attachments: ${fileid}`);

      GmailApp.sendEmail(target_address, title, body, {attachments: [attached_file]});
    }
  }
}

Comments

[退会済みユーザー]

ご丁寧な説明ありがとうございました。
理解できまし。