質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

Q&A

解決済

1回答

1005閲覧

【GAS】gmailの本文の合致キーワードを複数転記する際のエラーについて

takamine07

総合スコア9

Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

0グッド

0クリップ

投稿2019/03/11 02:20

以前にこちらで質問させていただいた件の追加質問です。
以前の質問内容:
GASを利用してgmailの特定ラベルに振り分けられたメールのうち、
特定のキーワードに合致する部分をスプレッドシートに転記する
その際に複数合致箇所がある場合はそのすべてを転記したい

例:本文内容 抜粋
対象No.:123456789
対象No.:123412341
対象No.:123456788,123456787
(項目数は決まっていない、1~6個程度)

スプレッド出力イメージ(特定キーワードは"1234とする"、上記の3番目はカンマ形式のままでも可)
123456789 | 123412341 | 123456788 | 123456787

https://teratail.com/questions/144880


こちらの回答で作成していただいたスクリプトを利用することで合致キーワード転記の実現はしましたが、
本文内に合致キーワードが存在し無い場合、スクリプトエラーとなってしまいます。
その場合は合致キーワード以外の部分(差出人・日時など)だけを取り出してスプレッドシートに転記したいのですが、可能でしょうか。

エラー箇所:
if (result.length > maxLength) {
→TypeError: null からプロパティ「length」を読み取れません。
というエラーが出ます。

何度もすみません…よろしくお願い致します。


参考コード:
function searchContactMail() {
/* Gmailから特定条件のスレッドを検索しメールを取り出す */
var mail_query = ('ラベル条件');
var myThreads = GmailApp.search(mail_query, 0, 50); //条件にマッチしたスレッドを取得
var myMsgs = GmailApp.getMessagesForThreads(myThreads); //スレッドからメールを取得する →二次元配列で格納

var valMsgs = [];

/* 各メールから日時、送信元、件名、内容を取り出す*/
var maxLength = 0;
for(var i = 0;i < myMsgs.length;i++){
valMsgs[i] = [];
valMsgs[i][0] = myMsgs[i][0].getDate();
valMsgs[i][1] = myMsgs[i][0].getFrom();
valMsgs[i][2] = myMsgs[i][0].getTo();
valMsgs[i][3] = myMsgs[i][0].getSubject();
valMsgs[i][4] = myThreads[i].getPermalink();
var result = myMsgs[i][0].getPlainBody().match(/1234[^\n]*/g); // 正規表現で本文中の文字列を取り出す。
Array.prototype.push.apply(valMsgs[i], result); // 元の配列に取り出した配列を追加する。
if (result.length > maxLength) {
maxLength = result.length; // 取り出した配列の最大の長さを取得しておく
}
}

maxLength += 5; // ここで固定項目数分加算(項目数=カラム数を固定する)

/* カラム数に足りない行にカラムを追加する /
valMsgs.forEach(function(x) {
if (x.length < maxLength) {
/
足りないカラム数分だけ空の配列を用意して追加する */
var s = Array.apply(null , Array(maxLength - x.length)).map(function() {
return "";
});
Array.prototype.push.apply(x, s);
}
});

/* スプレッドシートに出力 */
if(myMsgs.length>0){
SpreadsheetApp.getActiveSheet().getRange(2, 1, i, maxLength).setValues(valMsgs); //シートに貼り付け
}
}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

javascript

1if (result !== null && result.length > maxLength) {

あらためてコードを読み直しましたが、↓こちらのほうがいいかもしれない。

javascript

1var result = myMsgs[i][0].getPlainBody().match(/1234[^\n]*/g); // 正規表現で本文中の文字列を取り出す。 2//ここに↓を入れたほうがいいかもしれない 3if(result === null) { result = [];} 4Array.prototype.push.apply(valMsgs[i], result); // 元の配列に取り出した配列を追加する。

投稿2019/03/11 02:23

編集2019/03/11 02:30
papinianus

総合スコア12705

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

takamine07

2019/03/11 02:50

お早い回答ありがとうございます!! 2番目に頂いたコードで、今試してみて考えられるパターンですべて動くようになりました。 本当にありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問