コントローラー内での配列の要素を取り出し方

Question

カラムで保存されている配列から要素のidを再度、他のテーブルで検索をかけて
whereでまとめたいのですが、うまくできません。

  def show
    @category_item = CategoriesItem.find_by(item_id: @item.id)
    @small_category = Category.where(id: @category_item.small_category_id)
  end

@category_itemの中身

 id: 1,
 big_category_id: 8,
 item_id: 51,
 small_category_id: "[0, 15, 30, 35]",
 created_at: Mon, 25 May 2020 02:34:19 UTC +00:00,
 updated_at: Mon, 25 May 2020 02:34:19 UTC +00:00>

small_category_idの中身の[0,15,30,35]をCategoryテーブルでwhereをかけて
Categoryテーブルから情報を持ってきたい。

現在のやり方だと、変数の中身を見てみると毎回whereの中身が [] になってしまいます。
宜しくお願いします。

  has_many :categories_items
  has_many :categories, through: :categories_items
  accepts_nested_attributes_for :categories_items

	belongs_to :item, optional: true
	belongs_to :category, optional: true

  has_many :categories_items
  has_many :categories, through: :categories_items

Answer 1

正しく、正規化および関連付けを行ってデータを作り直した方がいいです。

どうしても、このまま運用したいというのならば

  def show
    @category_item = CategoriesItem.find_by(item_id: @item.id)
    @small_category = Category.where(id: YAML.parse(@category_item.small_category_id))
  end

---

これの何が駄目かというのを説明すると
複数件のCategoriesItemとそれに関連づいたCategoryを取得しようとした際に
データベースに複数回問い合わせが必要になるのが問題になりえます。

Comments

[Masashige1005]

すみません、一応関連付けはしております。
載せるのを忘れておりました。
現在付け加えました。

Answer 2

Category.where(id: "[0, 15, 30, 35]")

のように文字列で検索をかけてしまっているからではないでしょうか。

Category.where(id: [0, 15, 30, 35])

配列で検索をかけるようにすれば正しく動くようになると思います。

修正の方法はいくつかあると思いますが、
例えば PostgreSQL をお使いであれば以下のような設定で対処できると思います。

class Category
  attribute : small_category_id, :integer, array: true

Comments

[Masashige1005]

integer型で配列って保存できますか？
少し前まではinteger型でsmall_category_idを定義していたのですが、うまくカラムに保存されないのでstring型にしました。ちなみにDBはmysql2を使用しております。
質問ばかりですみませんが、"[0, 15, 30, 35]"　→　[0, 15, 30, 35]の直し方も教えていただけると幸いです。
to_iだと一番前の0しか取得できなかったと思います。

[rhiroe]

asmさんも回答でおっしゃっていますが、そもそも一つのカラムに配列を突っ込むというのはRDBの運用上真っ当ではないです。
RDBのルールとして「1つのレコードの1つのカラムには単一のデータを入れる」というものがあるからです。
配列は構造を持ったデータなのでこのルールから外れています。

> "[0, 15, 30, 35]"　→　[0, 15, 30, 35]の直し方

いくつか方法がありますが、例えば
JSON.parse("[0, 15, 30, 35]")
"[0, 15, 30, 35]".delete('[]').split(', ').map(&:to_i)
など。
eval("[0, 15, 30, 35]")
でも可能ですが絶対にやめてください。

でもどちらにせよこれらはお勧めできません。
可能であればDBの設計を見直した方がいいです。

[Masashige1005]

カラムに配列入れるのはあまり良くないのですね・・・
今のテーブル設計上だと配列の方がまとまってる感じがしたので採用したのですが、配列から取り出すのは運用上良くないのは勉強になりました。
再度、DBの設計を検討し直します。