teratail header banner
teratail header banner
質問するログイン新規登録
Google スプレッドシート

Google スプレッドシートは、フリーで利用できる表計算ソフト。Webアプリのためインターネットに接続することで利用できます。チャートやグラフの作成のほか、シートを他のユーザーと共有したり、同時に作業を進めることも可能です。

Google Apps Script

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

コールバック

コールバックは他のコードに引数として渡されるコードのことを指します。

Q&A

0回答

899閲覧

GASでコールバック関数findIndexが一部上手く機能しなくなりました

mmyk

総合スコア2

Google スプレッドシート

Google スプレッドシートは、フリーで利用できる表計算ソフト。Webアプリのためインターネットに接続することで利用できます。チャートやグラフの作成のほか、シートを他のユーザーと共有したり、同時に作業を進めることも可能です。

Google Apps Script

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

コールバック

コールバックは他のコードに引数として渡されるコードのことを指します。

0グッド

0クリップ

投稿2022/05/28 07:28

編集2022/05/30 00:35

0

0

スプレッドシートにメールからデータをGASで取り込んでいます。
今まで、メールから抽出した数列が存在するスプレッドシート上のデータが存在する該当行を、
findIndex関数で検索しチェックしていました。
これがうまく機能しなくなってしまい困っています。
行数が返ってくる数列と、「-1」が返ってしまう数列があります。

スプレッドシート
イメージ説明

Google

1// 書き込むシートを取得 2var ss = SpreadsheetApp.getActiveSpreadsheet(); 3var sheet = ss.getSheetByName('データ一覧'); 4 5//データ配列取得 6var lastrow = sheet.getLastRow() 7var data = sheet.getRange(2, 1, lastrow, 22).getValues(); 8 9function SearchMail() { 10 11 // 検索条件に該当するスレッド一覧を取得 12 var threads = GmailApp.search('newer_than:1m',0,50); 13 threads.forEach(function(thread) {  // スレッドを一つずつ取り出す 14  var messages = thread.getMessages(); // スレッド内のメール一覧を取得 15 messages.forEach(function(message) { // メールを一つずつ取り出す 16 17 //subject 18 var Subject = message.getSubject(); 19 if (Subject.match(/注文キャンセル/)){  //Cancelメール 20 getCancel(message,Subject); 21 } 22 }); 23 }); 24} 25 26function getCancel(message,Subject){ 27  28 //ID(refid) 29 var refid = Subject.substring(Subject.indexOf(":")+1,Subject.indexOf(")")); 30 31 if(refIndex(refid) !== -1){ 32 //該当#idの行にXつける 33 var i = refIndex(refid); 34 if(data[i][12] == ""){ 35 sheet.getRange(i + 2,13).setValue("X"); 36 } 37 } 38} 39 40//該当のRefIDのインデックスを検索するコールバック関数 41function refIndex(refid){ 42 var refIndex = data.findIndex(data => data[1] == refid); 43 return refIndex; 44}

イメージ説明

上手くいかなかった数字が下記になります
1585396473
989456078
213816337
128789415
158986985
2031366299
754110115
1079534760
945710144

当方、文系のプログラミング初心者で、ネットで検索しながらGASを業務で利用しております。
どうかご教示頂ければ幸いです。

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

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

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

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

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

退会済みユーザー

退会済みユーザー

2022/05/28 08:09

refIndex関数のコードも記載してください。
mmyk

2022/05/30 00:37

ご指摘ありがとうございます。 すみません。1番大切なところがありませんでした。 どうぞ宜しくお願いします。
退会済みユーザー

退会済みユーザー

2022/06/01 00:45 編集

「うまく行かなかった数字」というのは、「データ一覧シートのB列に存在しているにもかかわらず、見つからない(-1)とされてしまった数字」ということですよね? コードを見る限りでは特に問題点は見当たらないのですが、 たとえば、refidとして抜き出した文字の両隣に空白が入っていて、見つからない扱いになっている可能性もあります。 var refid = Subject.substring(Subject.indexOf(":")+1,Subject.indexOf(")")); の直後に console.log(`refid=[${refid}] -> index=${refIndex(refid)}`); といれて、indexが-1となるrefidに何か変なところがないか確認してみてはいかがでしょうか。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだ回答がついていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.30%

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

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

質問する

関連した質問