【GAS】表付きのメールを配信したい

Question

表付きのメールを配信したく下記コードを書いたのですが、
表が上手くつかず困っています。
添削をしていただきたいです。

送りたい表（内容は変更・増減あり）

担当	業務	内容	リンク
中村	配信	配信業務	/spredsheet/
中村	分析	分析業務	/spredsheet/
西野	マーケ	広報業務	/spredsheet/

現状の表
担||undefined||
|:--|:--:|--:|

当||undefined||
|:--|:--:|--:|

コードは下記です。

function sendMail() {
  const SS = SpreadsheetApp.getActiveSpreadsheet();
  const SS1 = SS.getSheetByName('業務一覧');
  
  const subject = "業務配信"
  const style = '<font face=Sans Serif">';
  
  const to = '@gmail';
  
  const column = SS1.getRange('A:A').getValues();
  const LastRow = column.filter(String).length+0;
  const value1 = SS1.getRange("A1:D" + LastRow).getValue();

//デザイン
  var style1 = "style='border-collapse: collapse; border:1px solid #000000;text-align:center;'><font face='Meiryo UI' color ='#FFFFFF'>"
  var style2 = " style='border-collapse: collapse; border:1px solid #000000;'><font face='Meiryo UI' color ='#000000'>";
  var style3 = " style='border-collapse: collapse; border:1px solid #000000;'>"

//メール本文
  const item = []
  niboshi.push(style+"<br>業務棚卸の日です。<br>下記の表のご確認を。<br>");
               
  item.push("<table style='border-collapse: collapse;' cellpadding='3'>");
  item.push("<tr" + style3);
  
  for(var j=0;j<2;j++)
 {
  item.push("<th bgcolor='#000000' nowrap"+ style1 + value1[0][j] +"</font></th>");
 }
  item.push("</tr>");
  for(var i = 1;i<value1.length;i++){
  item.push("<tr" + style3);
  for(var k=0;k<2;k++)
 {
  item.push("<td" + style2 + value1[i][k] +"</font></td>");
 }
  item.push("</tr>");
 }
  item.push("</table><br>");
  item.push(style+"<br>業務の変更は下記に<br> /spreadsheets/")
               
  const options = {};
  options.from = "@gmail"
  options.name = "業務配信"
  options.htmlBody = item.join("<br>");
  MailApp.sendEmail(to,subject,"",options);
  
}

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

Answer 1

2020/02/05 から V8 エンジンが利用可能になり、テンプレートリテラルが使えるので、こう。

/*
準備
 表示メニューから「マニフェスト ファイルを表示」にチェックをつけます。
 appsscript.jsonに `"runtimeVersion":"V8"` を追加します。
//```json: 例
{
  "timeZone": "Asia/Tokyo",
  "dependencies": {
  },
  "exceptionLogging": "STACKDRIVER",
  "runtimeVersion":"V8"
}
//```
* */

const sendMail = () => {
    const JobListSheet = SpreadsheetApp.getActive().getSheetByName('業務一覧');
    const subject = '業務配信';
    const to = '@gmail';

    const vals = JobListSheet.getDataRange().getValues().filter(e=>e[0] !== '');//A列が埋まているものを取り出す

    const heading = baseStyle('業務棚卸の日です。<br>下記の表のご確認を。');

    const header = rowStyle(vals[0].slice(0,2).map(headerStyle));
    const data = vals.slice(1).map(e=>rowStyle(e.slice(0,2).map(cellStyle).join(''))).join('');
    const table = tableStyle(`${header}${data}`);

    const footer = baseStyle('業務の変更は下記に<br> /spreadsheets/');

    const body = `${heading}${table}${footer}`;

    const options = {
        from:'@gmail',
        name:'業務配信',
        htmlBody:body
    };
    GmailApp.sendEmail(to,subject,'',options);

}
const nl = `<br>`;
const baseStyle = baseContent => `<font face="Sans Serif">${nl}${baseContent}${nl}</font>`;
const headerStyle = headerContent => `<th bgcolor='#000000' nowrap style='border-collapse: collapse; border:1px solid #000000;text-align:center;'><font face='Meiryo UI' color ='#FFFFFF'> ${headerContent}</font></th>`;
const cellStyle = cellContent => `<td style='border-collapse: collapse; border:1px solid #000000;'><font face='Meiryo UI' color ='#000000'>${cellContent}</font></td>`;
const rowStyle = rowContnt => `<tr style='border-collapse: collapse; border:1px solid #000000;'>${rowContnt}</tr>`;
const tableStyle = tblContent => `${nl}<table style='border-collapse: collapse;' cellpadding='3'>${tblContent}</table>${nl}`;

Comments

[slime_Hoimi]

いつもありがとうございます！
同じ機能のコードがこんなにシンプルにかけてしまうのですね。。。
いつも大変勉強になります。ありがとうございます。

Answer 2

机上デバッグしただけですが。

//value1は配列を期待してそうなので
//const value1 = SS1.getRange("A1:D" + LastRow).getValue(); //誤 getValue
const value1 = SS1.getRange("A1:D" + LastRow).getValues();  //正 getValues


//niboshi.push(style+"<br>業務棚卸の日です。<br>下記の表のご確認を。<br>"); //誤 niboshi
item.push(style+"<br>業務棚卸の日です。<br>下記の表のご確認を。<br>");      //正 item

あと、constってGASで使えましたっけ。ローカルで開発して、claspでpushしてるんでしょうか。

これぐらいそれぞれの変数の中身をログ出力でもすればすぐに分かるはずです。
まずは自分でデバッグしてみましょう。

Comments

[slime_Hoimi]

ありがとうございます！助かりました！

コードを書くのに必死になっててデバックのことが完全に頭から抜けてました。。。
constはGASしかかじっていないので、どこでも使えるものだと思ってました。。。
実際使えるみたいなので基本はこちらを使うようにしています。

niboshiはすみません、猫好きなため変数をいつも猫関連にしてまして。。。
お恥ずかしい、ありがとうございます。