javascriptでtwitterキーワードを取得しgoogleスプレッドシートで比較・抽出する

Question

前提・実現したいこと

javascriptでtwitterキーワードを取得しgoogleスプレッドシートで比較・抽出する
プログラムを書いている際につまづきました。
解決法を教えていただける方、よろしくお願いします。

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

TwitterキーワードJSONを取得しスプレッドシートに書き込んだあとの「文字列比較」がうまくいきません。
A列とC列を比較し、A列のキーワードがC列の文章に含まれる場合、C列をそのまま返すようにしたいです。
いまのプログラムでは完全一致になってしまい、キーワード同士の一致したものしか取り出すことができません。（文章の中にキーワードがある場合は抽出できていない）

該当のソースコード

function isContain() {
   
    var sheet = SpreadsheetApp.getActive().getSheetByName("twitter1");
    var keywordA = sheet.getRange(1, 1, sheet.getLastRow(), 1).getValues();
    var titleC = sheet.getRange(1, 3, sheet.getLastRow(), 1).getValues();
    
  var xxx = keywordA.filter(function(e){return titleC.filter(function(f){return e.toString() == f.toString()}).length > 0 });

Logger.log(xxx) 

}

試したこと

Google検索して解決しようと思いましたが、力及ばずでした。

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

ここにより詳細な情報を記載してください。

Answer 1

var xxx = keywordA.filter(function(e){return titleC.filter(function(f){return e.toString() == f.toString()}).length > 0 });

無名関数が入れ子になっている箇所ですが、自力で追えないなら
ワンライナーで一気に書かずに↓こんな感じに分解して確実に処理するべきと思います。

ログも追加したのでどこの判定が質問者さんの意図と違うのか分かると思います。

  var xxx = keywordA.filter(function(keyA){
    return titleC.filter(function(keyC){
      // BUG: ここが完全一致判定になっている。部分一致を調べるにはどうしたらいい？
      let result = keyA.toString() == keyC.toString(); 

      Logger.log(`${result ? 'o' : 'x'}: keyA=[${keyA.toString()}], keyC=[${keyC.toString()}]`);
      return result;
    }).length > 0
  });

ヒント追記

let hoge = 'アイエェー、ニンジャナンデニンジャ';
let fuga = 'ニンジャ';
let piyo = 'にんじゃ';

Logger.log(hoge.indexOf(fuga)); // 6 と出力
Logger.log(hoge.indexOf(piyo)); // -1 と出力

頑張ってください。

Comments

[spa]

回答いただきありがとうございます。おかげさまで間違っている箇所がわかりましたが、まだレベルが低いのでどう修正すればよいかわかりませんでした。
let result = keyA.toString() == keyC.toString(); で同じ文字列のみを比較してしまっているのはわかりましたが、部分一致を表す不等号は存在しない？ので、String.prototype.includes()を使って判定をしないとダメなんでしょうか。

[退会済みユーザー]

発想を変えましょう。AとCを直接比較する必要はないです。

これがStringで使える機能ですが、この中のAPI1つに元の文字列に対して、指定した文字列が含まれる場合に何文字目にあると知ることができます。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/String

それから、Webを有効活用しましょう。
めっちゃ出てきますよ？
https://www.google.com/search?q=javascript+%E9%83%A8%E5%88%86%E4%B8%80%E8%87%B4+%E6%96%87%E5%AD%97%E5%88%97&oq=javascript+%E9%83%A8%E5%88%86%E4%B8%80%E8%87%B4+%E6%96%87%E5%AD%97%E5%88%97&aqs=chrome..69i57.11936j0j7&sourceid=chrome&ie=UTF-8

[spa]

ありがとうございます。両方のページとも自分で検索して見つけてはいたのですが、なかなかうまく行かなくて困っていました。
もうちょっと発想を変えてみます。

[退会済みユーザー]

いやいや、全然読んでないでしょｗ
1つ目のURL開いたらスクロールさせずにAPI名と機能概要が書いてあるので見つけられます。
API名をクリックすると次のページでは動作するサンプルが見られます。

リファレンスは正直うんざりするのも分かるんですが、もう少し頑張りましょう。
APIが分かったら、その結果が【-1じゃない時】は【部分一致している】という判断をすればOKです。

[spa]

すみません。読んではいたのですがどうも頭が堅くて応用が効かないのです・・・
「文字列の中の部分文字列の存在や位置をチェックする indexOf() メソッド」を使って-1以外が返ってくれば「キーワードが入ってるぞ」という判断をする、ということですよね？
違ったらすみません。

[退会済みユーザー]

その認識で合っていますが、【先にコードを手元で書いて、自分の目で動きを見て】
それで想定通りならそれで良し。そうでなかったらそれを踏まえて再質問をする。
その方がご自身の学習の為にも良いと思います。
頑張ってください。

[spa]

let result = keyC.toString().indexOf(keyA.toString()) !== -1 && keyA.toString().length >1 && keyC.toString().length >1;

ヒントをいただいたので完成させることができました。ご指導いただきありがとうございます。
空白同士は一致させない処理もなんとか文字数のカウントで対処できました。
感謝です(^^)

[退会済みユーザー]

解決できたようで何よりです。
ただ、コードの1行の長さが気になりますね…。
目安として3項を超えるようなら分割して整理した方がプログラムがスッキリし、拡張がしやすくなります。

> let result = keyC.toString().indexOf(keyA.toString()) !== -1 && keyA.toString().length >1 && keyC.toString().length >1;
【keyCを文字列にしてKeyAを文字列にしたものを部分検索して見つからず、かつkeyAを文字列にした時の長さが2以上で、かつkeyCを文字列にした時の長さが2以上になった時は真】
このプログラムを日本語にした文章だって意味を把握するのって大変だと思います。
変数の名前の付け方にも気を回せると格段に読みやすくなります。

---
// 値を文字列に変換する
let strKeyA = keyA.toString();
let strKeyC = keyC.toString();

// キーワードを含んでいるかどうか
let isExist = strKeyC.indexOf(strKeyA) !== -1;

// strKeyAとstrKeyCの両方が2文字以上になっているかどうか
let isExceed = (strKeyA.length > 1) && (strKeyC.length > 1);

// 結果
let result = (isExist && isExceed);
---

> 空白同士は一致させない処理もなんとか文字数のカウントで対処できました。
多分、空白2文字以上とかそういう時に対応できなくなりそうです。
Stringクラスのtrim()を使うと、文字列の最初と最後の空白文字等が除去した文字列を取得できます。
その上でlength=0の時は除外する処理をした方が良さそうですね。

すでに長文コメントになっているので、これで終わります。失礼しました。