Railsのモデルの扱いについて

ただいま、Railsでwebアプリケーションを作成しているのですが、モデルの構成と、そのより良い取り出し方がわからず困っております。

model1
id
name

model2
id
name

model3
id
name

windows
model1_id, model2_id, model3_id, title, sentence

＊ model_id, model2_id, model3_idのうちどれか一つにidが入っており、他の2つはNULLが入っています。

といった4つのテーブルがあるのですが、これをif ~~~.present?を使って振り分けようと思います。

Ruby
1if(windows.model1_id.present?)
2  model = Model1.find(windows.model1_id)
3elsif(windows.model2_id.present?)
4  model = Model2.find(windows.model2_id)
5elsif(windows.model3_id.present?)
6  model = Model3.find(windows.model3_id)
7end

この処理をどこかに記述しようと思ったのですが、どこに記述すればいいかで迷っております。
また、このようなモデルの構造で良いのかも迷っております。

より良い方法を知っている方がいらっしゃいましたら、教えていただけますと幸いです。
よろしくお願いいたします。

行動規範の内容に同意します

回答2件

ご回答いただきありがとうございます。
たしかに、今後もmodel4, 5が増える可能性がありましたので、ご教授頂いた通りのカラム構成を行った方がいいですね！

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

投稿2018/01/03 07:38

junya451054

総合スコア69

ベストアンサー

windowsモデルでいいと思います。

使い方は

ruby
1windows = Windows.find(id)
2windows.model
3# => model_* のインスタンス

という感じでしょうか？

余談ですが、
Windowsのスキーマは

model1_id
model2_id
model3_id
title
sentence

ではなく

model_name
model_id
title
sentence

のほうがいいと思います。（モデルが今後増えたときとか大変でしょう？）

↑のようにすることで

# model_nameに入っている文字列の定数が存在しなければエラー
return エラー処理 unless const_defined?(model_name)
klass = const_get(model_name)
# model_nameに入っている文字列の定数がクラスでなければエラー
return エラー処理 unless klass.is_a?(Class)

klass.find(model_id)

のように書くことができます

投稿2017/12/31 05:27