bragging-detector icon indicating copy to clipboard operation
bragging-detector copied to clipboard

一部の単語がホワイトスペースにより認識できない

Open kai0310 opened this issue 4 years ago • 2 comments

起こっている問題 / The Problem

  • PS5の様な単語であるとAPIの方でPS 5と空白が開いた状態で認識されてしまっている.結果としてマッチしないので自慢を検知できない状況がが発生している.

関係するissues,PR / Related Issues,PR

  • #11

スクリーンショット / Screenshot

スクリーンショット 2021-01-11 22 54 00

期待する見せ方・挙動 / Expected Behavior

  • PS5PS 5でも認識させる様にする

起こっている問題の再現手段 / Steps to Reproduce

  1. 反応する単語にPS5を追加
  2. PS5が含まれる文章を話す

動作環境・ブラウザ / Environment

  • macOS
  • Chrome

kai0310 avatar Jan 11 '21 14:01 kai0310

解決する手段として以下を提示していただきました. ただ,2ではjsonにスペースを入れるという点と事前にサジェストされる候補にあるものにしか有効でなく,ユーザが追加した単語では認識できないため1の方法が望ましいと思われます.

  1. 検知する際に正規表現でスペース取り除く様にする。そうするとこのjsonファイルのみならず、通常の入力ボックスからも検知できるようになります(スペース入ることをそもそも想定していないので、副作用はないはず)。
  2. jsonファイルにスペース入れる形で入れて、どこかに理由をコメントで入れる。

kai0310 avatar Jan 11 '21 14:01 kai0310

1の方法が良さそうですね。日本語を対象としているので半角スペースが除去されても問題ないと思います。 もし、分かち書きを行うように変更することになる場合や、英語に対応する場合に不都合がでる可能性があるかもしれませんが、今は考える必要はないでしょう。

yoidea avatar Jan 12 '21 07:01 yoidea