【GAS】Gmailとスプレッドシートの連携/自動転記について

Question

前提・実現したいこと

Gmailで受信している問合せメールの内容の一部を、スプレッドシート/GASを使って下記の流れで自動的に転記できるようにしたいと考えています。

　・特定のラベル付き+未読状態のメールを取得する
・メール内から該当箇所のみをスプレッドシートに転記する
・取得が終わったメールは既読状態に変更する

■元のメールデータ/例文
下記のようなメールから「問合せNo、問合せ日時、氏名、フリガナ、電話番号」の部分のみを取り出してスプレッドシートに転記したい。
(上記の項目名部分で文字列検索をする？)

XX様

この度はお問い合わせありがとうございます。
確認し、こちらからご連絡させて頂きます。

------問い合わせ日時------
問合せNo　　　：000000
問合せ日時　　 ：2019/12/31 00:00:00

--------お客様情報--------
氏名　　　　　：山田太郎 様
フリガナ　　　：ヤマダタロウ 様
電話番号　　　：000-0000-0000

////////////////////////////
株式会社〇〇
住所
電話
メール
///////////////////////////

発生している問題・エラーメッセージ

・該当箇所のみをスプレッドシートに転記する
部分をどう記述すればいいかわからない状態です。

当方非プログラマーのため、メソッドの動作などを検索しながらサンプルコードを組み合わせて作成しておりましたが、
文字列のいい検索方法がわかりませんでした。

該当のソースコード

function getMail(){
  var objSpreadsheet = SpreadsheetApp.getActiveSpreadsheet(); 　　
  var objSheet = objSpreadsheet.getSheetByName("シート1"); //出力先のシートを指定
  var sheet = SpreadsheetApp.setActiveSheet(objSheet);
  
  //メールチェック用の変数
  var start = 0;
  var max = 500;

 //配列"threads"にラベルが問合せ、状態が未読のスレッドを検索、代入。
  var threads = GmailApp.search('label:問合せ is:unread',start,max);

 //変数"row"にスプレッドシートの最終行の次行を代入
  var row = sheet.getLastRow() + 1;
 
 //オブジェクト"threads"のプロパティ数ループする
 for(var n in threads){
    var thd = threads[n];
    var msgs = thd.getMessages(); //未読スレッドに含まれるメッセージを配列として取得

    for(var m in msgs){
      var msg = msgs[m];
      var body = msg.getBody(); //メール本文を1データの塊として取得する
      var res  = body.split("\n")　//取得したメール本文を各改行列ごとに配列に代入      
      
      //indexOfで取得したい文字列が入っている配列のインデックス番号を取得→配列で書き込み？search？
      //検索したい文字列に空白が含まれているのは正規表現の\sで対応？

              var number = res.indexOf(/問合せNo\s\s\s：/);
      		var date = res.indexOf("問合せ日時　　：");
      		var name = res.indexOf("氏名　　　　　：");
      		var name2 = res.indexOf("フリガナ　　　：");
      		var tel = res.indexOf("電話番号　　　：");

      
      		//セルへの書き込みを行う
      		sheet.getRange(row,1).setValue(number);
      		sheet.getRange(row,2).setValue(res[date]);
              sheet.getRange(row,3).setValue(res[name]);
     		 sheet.getRange(row,4).setValue(res[name2]);
      		sheet.getRange(row,7).setValue(res[tel]);
            row++
        }
 }
  //スレッドを既読に変更する
  thd.markRead();  
  Utilities.sleep(1000);
  }

試したこと

・「var res = body.split("\n")」の段階で出力→改行部分でデータ分割はできている。
・セルへの書き込みで「number」そのものを出力すると「-1」が出力される。
→検索ができていない状態でしょうか？

補足情報（FW/ツールのバージョンなど）

Answer 1

papinianusさんの回答スクリプトが素晴らしく整理されてきれいに書かれているのでご活用されるのが良いと思います。
質問者さんの問題は（indexOfで指定しているワードが対象と一致していることが前提ですが）おそらくvar res = body.split("\n")が配列になっていないからと思われます。（indexOfは一致するものがないときに-1を返しますので）
var resarray = [];
resarray.push(res);
を足せばうまく動きませんか？

Comments

[papinianus]

配列になっていると思いますし、配列であるか否が原因ではないと思いますが、おっしゃるように配列になってない(つまり本文全体がデカい一つの文字列である)ときに、resarrayにつっこんでも、-1じゃないでしょうか？
Array.prototype.indexOfは`===`での比較しかしない(はず、gasがjavascriptの流儀に乗っていれば)ので、質問者様のように部分文字列や正規表現リテラルを与えても一致させてくれません。
もし仮に部分文字列で一致できたとしても、(resarrayに入れたところで)resをなんとか配列にでも分けてあげないとメール本文全部しか取れないので、部分ごとに分けてシートに書き込みたい目的を達成できない印象を持ちます。

[hiroshi0240]

今、確認しました。おっしゃる通りsplitした時点で配列に格納できてますね。
ということはdate以降のデータは値指定しているので、一致してれば取得できているはずですね。質問者さんのコードのエラーはnumberのindexOf指定に正規表現を使われている部分ですね。
ということは、質問者さんのコードのindexOfをmatchに置き換えればそのまま使えるのかな。

Answer 2

function getMail(){
  var objSpreadsheet = SpreadsheetApp.getActiveSpreadsheet(); 　　
  var objSheet = objSpreadsheet.getSheetByName("シート1"); //出力先のシートを指定
  var sheet = SpreadsheetApp.setActiveSheet(objSheet);

  //メールチェック用の変数
  var start = 0;
  var max = 500;

 //配列"threads"にラベルが問合せ、状態が未読のスレッドを検索、代入。
  var threads = GmailApp.search('label:問合せ is:unread',start,max);

 //オブジェクト"threads"のプロパティ数ループする
 for(var n in threads){
    var thd = threads[n];
    var msgs = thd.getMessages(); //未読スレッドに含まれるメッセージを配列として取得

    for(var m in msgs){
      var msg = msgs[m];
      var body = msg.getBody(); //メール本文を1データの塊として取得する
      var no = body.match(/^問合せNo\s*?：(.+)$/m)[1];
      var dateandtime = body.match(/^問合せ日時\s*?：(.+)$/m)[1];
      var name = body.match(/^氏名\s*?：(.+)$/m)[1];
      var kana = body.match(/^フリガナ\s*?：(.+)$/m)[1];
      var tel = body.match(/^電話番号\s*?：(.+)$/m)[1];
      sheet.appendRow([no, dateandtime, name, kana, "", "", tel]);
    }
  }
  //スレッドを既読に変更する
  thd.markRead();  
  Utilities.sleep(1000);
}

Comments

[macaron_xxx]

これ、たぶん正常に動かないです。
GASではconstでのスコープがグローバルっぽい動き（バグだと思うんですけど）するので、おそらくforの中のconstも再代入されずにずっと1行目のものが追加され続けると思われます。
GASでconstは本当の意味で再代入しない定数以外は使わないほうがいいです。

[macaron_xxx]

function constTest() {
var msgs = ["問合せNo　　　：000000","問合せNo　　　：000001","問合せNo　　　：000002"];

for(var m in msgs){
var msg = msgs[m];
Logger.log(msg);
const no = msg.match(/^問合せNo\s*?：(.+)$/m)[1];
Logger.log(no);
}
}

[papinianus]

ありがとうございます。
信用しているので確認を怠って回答を直しました。
盲目的にconstにしていますが、ブロックスコープが働かないのだとすると(javaの影響でしょうかね？)、constにもあまり意味がないですね。

説明変数を持たずに、直接appendRowに変数を渡したほうがいいのかもしれないです。

[ful_a_o]

コメントが遅くなってしまいすみません！回答頂いたコードで動作致しました。
付け焼刃のツギハギな知識でしたので、これを機に改めて基礎から学習していこうと思います。
papinianus様、hiroshi0240様、macaron_xxx様、ありがとうございました！