質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

1回答

4261閲覧

Railsでパラメータを加工する処理をDRYに書く方法

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

1クリップ

投稿2016/04/13 00:29

###前提
ユーザに紐づくデータが多数あるモデルを扱っているシステムで、
下記のようなコードが頻出するような状態となっています。

ruby

1def create 2 @item = Item.new(params[:item]) 3 @item.user_id = current_user.id # この部分 4 @item.save

この、@item.user_id = current_user.idをDRYに書く方法について、
ご意見を頂きたいです。

※ 公の質問なので簡素にしていますが、本来はもう少し複雑なパラメータの加工を想定しています。

###考えていること
コントローラの責務、モデルの責務、などを考えると、
どちらかと言えばコントローラなのかもしれませんが、
しっくりこず。。。

参考に見たサイト

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

これは,view側でリクエスト投げる前の処理で:itemformの中でid をhiddenパラメータなどでもたせたりidが漏れるとやばいパラメータの場合はApplicationController(汎用性が必要ない場合はそのコントローラー)で加工処理のメソッドを定義してbefore_actionで前処理してますね。
最適解かどうかはわかりませんが参考になればと思います。

投稿2016/04/13 08:20

編集2016/04/13 09:05
退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2016/04/13 08:24

ご回答ありがとうございます。 この質問は、MVCのどの責務に持たせるのか、 または、他のレイヤーを設けて対応するか、 という趣旨の質問でしたので、意図から外れているように思います。
退会済みユーザー

退会済みユーザー

2016/04/13 09:04 編集

ApplicationControllerクラスでメソッド定義すればDRYに書けるという趣旨で書かせていただいたのですが,意図から外れていましたでしょうか? 私は一つ一つのActionを冗長にするのが嫌なので,initializerに全てパラメータを代入できるようにコントローラ側かviewで整形を行っています。 基本的にparams含むパラメータの整形はコントローラの責務というよりコントローラでやらなければ入力がどんどんたらい回しになり,可読性を下げるのではないかと思います。 ドメイン駆動についてはそこまで詳しくありませんが,paramsのpermitもコントローラでやりますし,そこで整形するのが一般的かと思われます。 重たい処理をコントローラにやらせるべきではないのはその通りで,viewに埋め込むという選択をするということもあるという感じで使い回していますが,そこまで重たい処理が効いてくるレベルの事態であれば,railsの問題ではなくリソースの問題だと思うので,railsで完結しない話になるのではないかと思います。railsで扱うべき層ではないというのが質問者様の他のレイヤーという感じになるのでしょうか?そういった理論があるのなら私も勉強したいです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問