配列中のキーワードを持つ場合のみ、特定のタグ付けを行う方法

Question

###前提・実現したいこと

スクレイピングを活用した動画サイトを作っています。

キーワードの配列を作り、その配列に入ったキーワードを含むタイトルだった場合、その動画に対して特定のタグ付けを行うといった機能を実装したいと思っています。

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

配列に入ったキーワードを含むタイトルかどうか、という条件式の書き方がわかりません。

###ソースコード

# encoding: utf-8

#タグ付けするためのキーワード配列を用意

     kawaii[]="かわいい,カワイイ,可愛い,子犬,息子,赤ちゃん,仔犬,ゴールデンレトリバー,柴犬,少女,フレンチブルドッグ,ポメラニアン,秋田犬,かわいすぎる,孫,puppy"
      omoshiro[]="歌う,バカ,いびき,爆笑,芸,面白い,面白,おもしろ,funny"
      howto[]="しつけ,躾,躾け,トレーニング,無駄吠え,対処法,方法,手法,解説,how"
      kando[]="涙,助け,救う,感動"


#以下で配列に含まれるキーワードを含んだタイトルにタグ付けしたいです。イメージでは以下のように書かれるのかと想像していましたが、これではタグ付けされませんでした。

      if dog.title.include?(kawaii)
      dog.description="かわいい"
      elsif dog.title.include?(omoshiro)
      dog.description="おもしろ"
      end
      
      dog.save
      else
      end
    end

###補足情報(言語/FW/ツール等のバージョンなど)

Ruby 2.2.2
Rais 4

Answer 1

こんな感じですかね？

@keys = {
  感動: "涙,助け,救う,感動",
  かわいい: "かわいい,カワイイ,可愛い,子犬,息子,赤ちゃん,仔犬,ゴールデンレトリバー,柴犬,少女,フレンチブルドッグ,ポメラニアン,秋田犬,かわいすぎる,孫,puppy",
  おもしろ: "歌う,バカ,いびき,爆笑,芸,面白い,面白,おもしろ,funny",
  HowTo: "しつけ,躾,躾け,トレーニング,無駄吠え,対処法,方法,手法,解説,how"
}

def searchTag(s)
  @keys.each do |k, v|
    words = v.split(",")
    words.detect do |word|
      return k if s.index(word)
    end
  end
end

puts searchTag "かわいすぎる面白いポメラニアン"   # かわいい
puts searchTag "少女に挑みかかる爆笑無駄吠え狼"   # かわいい
puts searchTag "涙が出るほど面白い赤ちゃん"       # 感動

（見ての通りkeysの順番を考えないと変なタグが付きます。）

Comments

[mannnakakunn]

ご指導ありがとうございます。
重複してキーワードが入ってるってことも想定しないといけないんですよね笑
keysの順番が大事なんですね。

@keysの部分（インスタンス変数？）をこのように記述できることさえ知りませんでした。
このような方法ならスマートにキーワードを変更することもできますね。
感動しました！ベストアンサーにさせていただきます。

以下、素人の戯言です。

このままsearchTagメソッドを使った時に、キーワードを持つタイトルの、それぞれのタグ付けには成功しました。
ですが、キーワードを持たない場合に何故かnilが出力されず、以下のような　＠keysの中身そのままが、出力されてしまいました。（たぶん私のコードの書き方が悪いでしょう）

{:感動系動画？=>"心揺さぶる,涙,助け,救う,感動", :かわいい系動画?=>"かわいい,カワイイ,可愛い,子犬,息子,赤ちゃん,仔犬,ゴールデンレトリバー,柴犬,少女,フレンチブルドッグ,ポメラニアン,秋田犬,かわいすぎる,孫,puppy", :おもしろ系動画?=>"歌う,バカ,いびき,爆笑,芸,面白い,面白,おもしろ,funny", :HowTo系動画?=>"しつけ,躾,躾け,トレーニング,無駄吠え,対処法,方法,手法,解説,how"}

そこで、

@keys = {
感動系動画？: "心揺さぶる,涙,助け,救う,感動",
かわいい系動画?: "かわいい,カワイイ,可愛い,子犬,息子,赤ちゃん,仔犬,ゴールデンレトリバー,柴犬,少女,フレンチブルドッグ,ポメラニアン,秋田犬,かわいすぎる,孫,puppy",
おもしろ系動画?: "歌う,バカ,いびき,爆笑,芸,面白い,面白,おもしろ,funny",
HowTo系動画?: "しつけ,躾,躾け,トレーニング,無駄吠え,対処法,方法,手法,解説,how",
普通系動画?: " "
}

のような形（ちょっと強引な形？）で、普通系動画？:" "のように定義してあげると、なんでもないない動画に普通系動画というタグをつけることができました。
これでいいのかよくわかりませんが（苦笑）

とても勉強になりました。ありがとうございます。

Answer 2

下記のように指定すれば一番最初に見つかったものをかえします。

kawaii = %w(かわいい カワイイ)
dog.title = 'カワイイいぬ'
kawaii.detect{ |str| dog.title.index(str)} => # カワイイ
dog.title = '普通のいぬ'
kawaii.detect{ |str| dog.title.index(str)} => # nil

Comments

[mannnakakunn]

ご指導ありがとうございます。こちらを参考にして、コードを書きましたが、期待通りの動きをしてくれました。

中身としては。。。

１：%w(...)で、キーワードを配列に入れる。
２：配列中のキーワードをdetectメソッドで探す。探す方法は３へ。
３：ブロック引数strに、キーワードそれぞれを入れ、ブロック繰り返す。indexできた場合、そのキーワードが返ってくる。

というような解釈でいいのでしょうかね・・・。ともあれとても勉強になりました。ありがとうございます。