Railsの太ったモデルをダイエットさせる方法について - メドピア開発者ブログ

こんにちは。メドピアのRuby(Rails)化をお手伝いしている@willnetです。最近はよくリファクタリングをしています。 今回は、最近僕がリファクタリングしている内容についてまとめようと思います。 メドピアではFat Model/Controllerを避けるために、rubocopの設定を利用しクラスの行数が300行以下になるよう制限しています*1。最近300行を超えるモデルが出てきたので、一部の処理を別のクラスに切り出し始めました。 このとき、Railsが提供している機能であるconcernsを利用すると楽に行数を減らすことができますが、それだとrubocopの指摘を回避できるという意味しかないので、なるべく委譲(composition)を利用して処理を別クラスに移していっています*2。 複数モデルにまたがる処理を切り出す Railsアプリケーションを書いていると、複数のモデルを一度

[okbm]

強い気持ちでserviceクラス

[tzt]

ActiveRecordは結局トランザクションスクリプトに行き着くってことよな。

[t-cyrill]

まだまだ先は長そう。頑張って欲しい。

[zentarou]

モデル名と同じディレクトリ作ってそこにクラス置いちゃう

[donkeyhole]

1メソッドが長いのはダメだと思うけど、1クラス(モデル)がでかいのは、外に切り出しても、そもそもの複雑度は変わらないと思う。サービス/業務レベルで刈り込む必要があるのでは。

[atico]

create.postsとrelationをひっぱってきている時点でPOROになりきっていない。型のないRubyで別クラスにインスタンスを渡すと、むしろ影響範囲がよみきれなくなる恐れが。

[uva]

フラットに置かれた大量のファイルよりディレクトリ作って仕分けた方がわかりやすいはず、っていうのと似てる。その構造と命名を適切にできてさえいれば・・・

[ota42y]

委譲大事……Fat Modelだとmodelを調べれば良かったのに、POROだと適切なクラスを探し出す必要が生まれるので、クラス名の名前付けが大変そう……

[morygonzalez]

こういうのやるととてもコードの見通しが良くなるのでよい。ただどの範囲で切り出してクラス化するかが初心者には難しいと思う。あとファィルが増えてファイル移動が忙しくなる。

[tikkss]

“それだけではなく、メール送信の対象となるユーザを外部から注入できるようにしたことで、テストをしやすくなっています。”

[iga_k]

今日の現場で説明用に使わせていただきましたありがとうございます！

[halleruya]

“多くのwebサービスではユーザが起点となる処理が多いので、結果としてUserモデルにメソッドが集まってしまいUserモデルがどんどんFatになります。数百行のレベルで済めば良い方で、千行を超えるUserモデルを持つプロジェ

[syossan]

まだRailsで消耗してるの？

[quanon]

大好きなアプローチ。ただ PORO のインターフェイスをある程度しっかり設計しておかないと事故る。あと、複数のモデルにまたがっていなくてもビジネスロジックよりの処理は積極的に外に出したい。

[sinsoku]

非同期処理で通知送れば普通のpost.saveとActiveJobで事足りる気がするけど、transactionにする必要あったのかな。

[crema]

プログラミングの話ですw

[h5y1m141]

POROなやつを使うことで、色々整理できて、他の人が途中で加わってもわかりやすい構造になるんじゃいかなと考えられるようにちょっとづつなってきた。最初のUserモデルみたいなやつは、 結構過去何度も見てる。。

[el-condor]

良い。Railsでも、もっとPOROが使われるべき/GoFでいうとFacadeパターンかな。

[fukumura]

メドピアもRailsの開発が活発になってきているのでリファクタリングが重要です！