LaravelのModelの扱いに関して

Question

LaravelのModelの肥大化について質問です。
Modelが行数が増えてしまう対策として以下のようなやり方があり実行しました。

• FormRequestを利用
• Service層にロジックを切り分ける

上記の対応をしてもどうしてもModelが大きくなり、
共同開発でコンフリクトが多くなる場合があります。

この場合の対策として1ファイルの量を減らすためにModelを継承したり、Traitを利用して、関連性がある処理ごとにModelを分割する方法を考えています。

解決策として悪くないように思えますが、なにか副作用はあるでしょうか。

Comments

[hentaiman]

ファイル分割よりも、どういう処理の分け方を基準にファイル分割しているのかを把握してそれを解消する方を考える、または考えた結果を質問に書いてみるなど

Answer 1

考え方のアドバイスになって申し訳ないですが、「大きくなってしまったものをどうするか」と小手先でなんとかしようとしているようにも見受けられます。

Howは置いといてWhy
「なぜ肥大化するか」を突き詰める必要があると思います。
そして「どうあるべきか」ルールを決めて
「メンバーで徹底されているか」適切で的確なレビューを行う

適切に設計時点でモデリングできていればそうそうファットモデルになることはないと思います。
一度現在のコードや構成は捨てるくらいの気概で見直さないと枝先の修復にしかならず、気づいたら元に戻ってやり直しになるように思います。

Comments

[makibi5577]

ご回答ありがとうございます。アドバイスでも参考になるので助かります。

Answer 2

継承は絶対に使ってはいけない。

Laravel内部を見れば分かるけど「ただのコードの分割」にはConcernsを使っている。
app/Models/Concerns/内にトレイトを作って置く。
ConcernsはRailsから持ってきた概念だろうけどただの概念なので気にしなくていい。
機能としてはPHPのトレイトを使ってるだけ。

LaravelのModelはリレーションなど細々とした設定が増えるので「Laravelが用意してる機能」以外のことには使わない。
確実にアプリ全体で使って共通化した方がいいメソッド以外はModelに書くことはない。
ModelはDBに関する定義として扱う。

Model内に色々書くのは他のフレームワークの使い方でLaravelの流儀ではない。
FormRequestなんて出てくるのはCakePHP使ってた人がModelにバリデーション関連を書いてるんだろうけどLaravelでは元々そんな使い方はしない。
とはいえこの辺はプロジェクトごとに方針を決めること。

Comments

[makibi5577]

ご回答ありがとうございます。Concernsに関して調べたいと思います。