GAS。関数による間違い検出が働かないのです。

Question

GASで、メールを送信するプログラムを作っています。
spreadsheet上に空欄があれば、popupAttached（関数）により処理が止まるようにしたいのですが、どうも機能していません。一度、個々のプログラムで動作確認してから関数として入れています。（途中で元のプログラムから変更していますが）

送信先のメールアドレスを一か所、空欄にしているのですが、動いておらず

Exception: メールを送信できませんでした: 受信者が指定されていません

と表示されます。タイトルも空欄にして試したのですが、エラーとなりませんでした。

function mail(){

         /*下準備　　spreadsheet、document,driveの指定を行う*/
  const sheet = SpreadsheetApp.getActiveSheet();　　//spreadsheetをアクティブ化
  const lastRow = sheet.getLastRow();  //最終行の取得
  const document_id = 'xxxxxxx'; //dcumentのid
  const doc = DocumentApp.openById(document_id);
  const docText = doc.getBody().getText();
  const drive_id ='xxxxxxxx'　　//ファルダ―ID
  const folder = DriveApp.getFolderById(drive_id);   //driveのフォルダー指定
  
  
  　　　　　/*spreadsheetData*/
  const todayDate = sheet.getRange('H1').getValue();//NOW()関数で自動入力
  const senderName = sheet.getRange('H2').getValue();//差出⼈名
  const subject = sheet.getRange('H3').getValue();//タイトルが空であればエラーを吐く。（関数で判断）
  const CC = sheet.getRange('H4').getValue();//CCの指定
  const BCC = sheet.getRange('H5').getValue();//BCCの指定
  const mailReplayIsok = sheet.getRange('H6').getValue();//replayの有無
  const mailReplayTo = sheet.getRange('H7').getValue();//replay先の変更
  const extention = sheet.getRange('H8').getValue();//ファイルの拡張子指定
  const arrayData = ['B','C','D','F'];
  
  try{
  /*メール送信前にspreadsheetに抜けがないかの確認。もし、抜けていたら。処理を終了させる。*/
    for(let j = 0; j < 4; j++){
      let contentArr = arrayData[j] + ':' + arrayData[j]
      let Length = sheet.getRange(contentArr).getValues().filter(String).length;
      let zero =Istrue(Length,lastRow)
      if(zero ==0);
      else{
        for(;Length < lastRow; Length++){
          let ans = sheet.getRange(j + 2, Length).isBlank();
          if(ans =='true'){
            popupAttached(arrayData[j] + Length,'を入力してください。');
          }
        }     
      }
    }
  
  /*添付ファイルと拡張子の両方が入力されているか？*/
    XOR(recognize(sheet.getRange(3, 6).getValue()),recognize(extention));
  
  /*メール送信。　forLoopでdocumentの置換を行い、個々のdocument作成。その後送信。*/
    for(let i = 2; i <= lastRow; i++){
   　　　　/*spreadsheetからデータを取得*/
      const lastName = sheet.getRange(i, 2).getValue();  //姓
      const firstName = sheet.getRange(i, 3).getValue(); //名
      const score　= sheet.getRange(i, 4).getValue();    //通知する点数
      const recipientName = sheet.getRange(i, 5).getValue();//宛先
      const attached = sheet.getRange(i, 6).getValue();  //添付ファイル
      const body = docText         //置換置換処理
        .replace('{送信日}',todayDate)  
        .replace('{姓}',lastName)
        .replace('{名}',firstName)
        .replace('{点数}',score);
      const attachedFile = attached + '.' + extention
//    const data = folder.getFilesByName('attachedFile').next().getBlob();
    
//    const options = {name: senderName, cc: CC,bcc: BCC,noReply:　mailReplayIsok,replyTo:　mailReplayTo};//差出人、CC、BCC、replayの有無、replay先の変更を設定
   　 GmailApp.sendEmail(recipientName,subject,body);//メール送信 ,options
    }
  }catch(e){
    console.error("エラー：", e.message);
  }
}


function recognize(value){  //空であるか確認。
  if(value == null){
       return 1;    
   }else{
       return 0;
   }
}
 
  
function XOR(x, y){
  const  ans = x*1-y*1;
  if(ans == 0){
    return ;
  }else{
    if(x == 0){
      popupAttached('添付ファイル名','を入力してから実行ボタンを押してください。n\添付ファイルを送信するには、ファイルの拡張子','添付ファイル名の両方の入力が必要です。');
    }else{
      popupAttached('ファイルの拡張子','を入力してから実行ボタンを押してください。n\添付ファイルを送信するには、ファイルの拡張子','添付ファイル名の両方の入力が必要です。');
    }
  }
}
 
    //ポップアップ警告処理。タイトル
function popupTitle(){
   const rezult = Browser.msgBox("タイトルが入力されていません。\nタイトルなしで送信されますか？", Browser.Buttons.YES_NO);
   if(rezult == "yes"){
     const flag = Browser.msgBox("本当によろしいでしょうか？", Browser.Buttons.YES_NO);
     if(flag == "no"){
       Browser.msgBox("タイトルを再入力してから実行してください。", Browser.Buttons);
       throw new Error('終了します。タイトルのミス');
     }else;//実行される。
   }else{
     Browser.msgBox("タイトルを再入力してから実行してください。", Browser.Buttons);
     throw new Error('終了します。タイトルのミス');
   }
}
  
  //ポップアップで終了のお知らせ。
function popupAttached(message1, message2){
   Browser.msgBox( message1 + 'がemptyです。n\' + message1 + message2, Browser.Buttons);
   throw new Error('終了します' + message1 +'のミス');
}
 

function Istrue(sheet,lastraw){
  if(sheet == lastraw ) {
    return 0;
  }else{
    return 1;
  }
}

Comments

[draq]

スプレッドシートからスクリプトエディタを開いて作成しているのでしょうか？

[aass12]

ご指摘ありがとうございます。
スプレッドシート上で開いていませんでした。
しかし、スプレッドシート上で実行し直してもうまくポップアップでの表示が表れてくれません。

Answer 1

こんにちは、少しコードを見ただけですが以下が怪しいと思われます。
修正前

if (ans == "true") {
    popupAttached(arrayData[j] + Length, "を入力してください。");
}

修正後

if (ans) {
    popupAttached(arrayData[j] + Length, "を入力してください。");
}

にしたらどうでしょうか？