Railsでモデルのデータになんらかの処理をしてからビューに渡したいです

Question

閲覧ありがとうございます。

今回Ruby on Railsで、モデルから引っ張ってきたデータに、何らかの処理をしてビューに渡したいと考えております。

例えば、データの作成日から１日以上経過したら、flagをfalseにするなどです。

この場合この処理はどこに書けば良いのでしょうか？

現在
@data_origin = Data.all
１日経過したら@data_origin.flag = false
@data = @data_origin

として、ビューに@dataを渡しているのですが、これは正しい書き方なのでしょうか？
それともモデルファイルに書く処理なのでしょうか？

一応処理は上手くいってるのですが、正しい書き方を知りたいと思い質問させて頂きました。

とても素人質問ですが、お答えを貰えると助かります。

Answer 1

どういう設計になっているのかが分からないので断定できませんが…
本来は、モデル側でデータの処理を完了してからビューに渡すのがセオリーだと思います。

Comments

[affluenceyou]

申し訳ありません。かなり説明不足でした。
私はコントローラ内でデータを処理して、ビューに渡しました。

日時によってデータを処理したいのですが・・・

質問なのですが、モデル内で処理を書くのと、コントローラ内で処理を書くのは、どのような基準でわけられるのでしょうか？
そもそも、モデル内に書く処理がどのようなものかも分かっていないのですが・・・

Answer 2

「1日以上経過している」の必要性によって違ってくると思います。
モデルの関連要素などの状態を変更させたいなど、永続的な変化をモデルに持たせ対なのであれば、バッチ処理などで定期的にモデルクラスのデータを書き換えてあげる方が良いでしょう。
しかし、「1日以上経過している」状態を見せたい、もしくはそれによって見せ方を変えるのであれば、ビュー側で処理してしまう方が良いように思います。

ソース見たところ、フラグを更新した後、保存などは行っていないので、@data.flagはビューの切り分けなどで使っているのではないかとお見受けします。
だとしたら、属性にflagなどつけずに、ビュー側で日付の比較した方が良いのでは?

Comments

[affluenceyou]

なるほど。確かにわざわざ属性を付ける必要はなかったです・・・。　保存する必要があるものだけ属性を付ければいいのですね！　ありがとうございました！！