🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
Ruby on Rails 5

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

Ruby on Rails

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

Active Record

Active Recordは、一つのオブジェクトに対しドメインのロジックとストレージの抽象性を結合するデザインパターンです。

Q&A

1回答

1421閲覧

ActiveRecordのクエリメソッドはどの階層に記述するべきか

widget11

総合スコア221

Ruby on Rails 5

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

Ruby on Rails

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

Active Record

Active Recordは、一つのオブジェクトに対しドメインのロジックとストレージの抽象性を結合するデザインパターンです。

0グッド

0クリップ

投稿2019/11/26 01:57

編集2019/11/26 01:59

概してビジネスロジックはモデルに書き、複数にまたがるようなモデル操作を行う場合はService層に処理を逃がすというのがMVCの定説だと思いますが、いまいち腑に落ちない点があります。


例えば単純にBookテーブルから1件テーブルからレコードを取得してViewに表示したい場合

class BookController < ApplicationController def show @book = Book.find_by(user_id: current_user.id) end end

と書けると思いますが以上のように1行の短いコードの場合でもモデル層に逃がすべきなのでしょうか?

class Book < ApplicationRecord  belongs_to :user   def find_book(id) Book.find_by(user_id: id) end end


また例えばこのshowアクションで、Bookと共に関連のないモデル(例えばですがCountry)をView側で表示したい場合以下のように書くとします

class BookController < ApplicationController def show @book = Book.find_by(user_id: current_user.id)   @country = Country.find_by(user_id: current_user.id) end end

これをモデルクラスにロジックを逃がす場合、BookコントローラーのshowアクションでCountryを表示したいからCountryモデルに対する操作をBookモデルで行うことは悪手なのでしょうか?

class Book < ApplicationRecord    belongs_to :user   def find_book(id) Book.find_by(user_id: id) end def find_country(id) Country.find_by(user_id: id) end end

こうすればコントローラー側でBookモデルのインスタンスを1つ作るだけでいいので便利な気もするんですが。。。

 class BookController < ApplicationController  def show    books = Book.new  @book = books.find_book(current_user.id)    @country = books.find_country(current_user.id)  end  end

正直Railsでどうソフトウェア設計を行っていくべきか皆目見当ついていません。
ネットで初心者向けリファレンス見ると結構コントローラー側でビジネスロジック書いている例も多いですし。。。
宜しくお願い致します。

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

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

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

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

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

guest

回答1

0

以上のように1行の短いコードの場合でもモデル層に逃がすべきなのでしょうか?

この程度であれば、別にメソッドを立てるのは冗長になるだけかと思います。

これをモデルクラスにロジックを逃がす場合

Rails的には、Userモデルにhas_oneをかければ、user.bookuser.countryなどとして自分で書かなくても使えるようになります。わざわざBookに実装するのは屋上屋を架すだけです。

投稿2019/11/26 02:32

maisumakun

総合スコア145963

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

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

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

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

ただいまの回答率
85.36%

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

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

質問する

関連した質問