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

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

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

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

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

オブジェクト指向

オブジェクト指向プログラミング(Object-oriented programming;OOP)は「オブジェクト」を使用するプログラミングの概念です。オブジェクト指向プログラムは、カプセル化(情報隠蔽)とポリモーフィズム(多態性)で構成されています。

Q&A

解決済

2回答

825閲覧

(ruby on rails)モデルにどこまでの責務を持たせればいいかが分からない

widget11

総合スコア221

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

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

オブジェクト指向

オブジェクト指向プログラミング(Object-oriented programming;OOP)は「オブジェクト」を使用するプログラミングの概念です。オブジェクト指向プログラムは、カプセル化(情報隠蔽)とポリモーフィズム(多態性)で構成されています。

0グッド

1クリップ

投稿2019/03/17 18:31

自身のスキルアップの為に最近はソフトウェア設計、オブジェクト指向(OOP)に挑戦してます。ネットで調べつつ、自身の制作物に適用させています。

オブジェクト指向には単一責任の原則なる言葉があると思います。
単一責任の原則は参考記事によると

変更する理由が同じものは集める、変更する理由が違うものは分ける。

クラス(オブジェクト)が担う責任は1つに限定すべきである(かつその責務は完全にカプセル化されるべきである)。

また参考記事2によると

1つのクラスは1つの仕事しかしない

だそうです。
基本的にクラスというものは属性と振る舞い(メソッド)を持ち得るクラスで、且つモデルクラスはビジネスロジック(概してそのモデルが参照するテーブルの属性に対するCRUDを行う?)を書くべきクラスだと考えています。(ActiveRecordは属性を書かなくてもいいので便利ですね)
勿論例えばActiveRecordモデルクラスでwebAPIに対してリクエストを投げる処理を書かない等分かりやすいものは分かるのですが、例えば

class TApplicant < ApplicationRecord belongs_to :t_work belongs_to :t_user_account def apply_works(user_id, work_id) TApplicant.create(t_work_id:work_id, t_user_account_id:user_id) end def self.countapplicant(work_id) self.where(t_work_id: work).count end end

以上のように、とある仕事に応募する(TApplicantテーブルにユーザーを登録する)メソッドと登録者数を集計するメソッドがあるとします。
どちらもTApplicantモデルに関わり何らかの処理を行うメソッドですが、片方はCRUDのCを行い、もう片方はマッチする条件をカウントしているだけのメソッドです。
単一責任の原則に照らすと、要素カウントするメソッドを書くのはアウトなのかもしれませんが、正直コード数が少ない上にTApplicantクラスに対して無縁のメソッドでは無いと思います。

このような場合でもクラス分けして責務を分離させるべきなのでしょうか?
このようなオブジェクト指向の話となると、比較的プレーンなrubyのクラスの話が多いので、railsではどうするべきなのか迷っています。
よろしくお願い致します

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

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

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

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

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

guest

回答2

0

ベストアンサー

規模の小さいプログラムでは、CRUD を一つのクラスにまとめておくというのも良くやると思います。

クラス設計にはこれが正解というものが存在せず、結構センスが問われるので難しいところだと思いますが、あまり冗長になりすぎないようにするのも大切です。

提示していただいたコードも、これだけを見ればそのままでも悪くないと思います。

ただし、規模の大きなプログラムになると、全体の設計を考慮する必要があります。
今は小さなクラスでも将来的に拡張される可能性もありますので。

投稿2019/03/17 22:29

nskydiving

総合スコア6500

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

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

0

workとuseをつなぐためだけのクラスなので、あまり気にしないと思います。
それよりもworkやuserのクラスの方が重要な気がします。
idddなんかを読むと、少し世界が開くと思います。

投稿2019/03/18 17:43

urbainleverrier

総合スコア200

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問