Railsで処理を別クラスに切り出す方法について - メドピア開発者ブログ

こんにちは。メドピアのRuby(Rails)化をお手伝いしている@willnetです。最近はエンジニアが増えた影響か、Railsの質問に答えていることが多いです。 以前、Railsの太ったモデルをダイエットさせる方法についてというタイトルでPOROを使っていこうという話を書きました。その際にコード例などもなるべく多く載せるようにしたのですが、このエントリだけを読んだ状態では、いざ「POROを使ってみよう！」としたときにまだ悩む余地がありそうです。 POROはその名の通り普通のRubyオブジェクトなので、いろんな書き方ができてしまいます。それなりに経験がある人でないと、どのように書いたらいいんだろう…と悩んで時間を使ってしまいそうですね。さらに、複数人で開発しているチームだと書き方のバラツキも気になるところです。きっと、POROを書くときのお作法が決まっている方が開発しやすいはず。 そこで、

[willnet]

春の新作です(\( ⁰⊖⁰)/)

[tera_chan3700]

よか記事

[el-condor]

一つのクラスに纏めるほど関連性の強い処理が複数というのは経験上ほぼないのでpublicメソッド一つという流儀には基本賛成しつつcallではなく振る舞いを記述するようなメソッド名をつける派

[ene0kcal]

内容にもよるが、多くはインスタンスメソッドだと思う。業務とオブジェクト指向理解していれば自明で分かるはずなのだが。

[als_uz]

わかる。RubyだとPOROになるのかそうか

[naari_3]

サービス層のイメージはこれ

[h5y1m141]

“クラスメソッドで同様のことをやろうとするとクラスインスタンス変数を更新するコードになり、結果としてスレッドセーフではないコードになってしまいます” なるほど！ こういう感じで説明すれば良いのか

[youko03]

“一つのクラスには一つの公開インターフェース”

[d4-1977]

リファクタリングの話としてもいいなあ。

[volpe0104]

インスタンスメソッドを使う理由とかがわかりやすい。重複コードを共通化して分岐を増やすよりも多少重複しても分岐を減らしてメンテ性を上げるのか。クラスはサービスとかに置くのかな。

[tikkss]

すごく良いなぁ。PORO に切り出した後のコードはどのディレクトリに配置するんだろう

[seiya-orz]

このスタイル好きなんだけどチームメンバーのスタイルと合わなくてやらなくなってしまった

[whitecrane89200]

わかる。丁寧でわかりやすい

[sinsinchang]

「POROに切り出したとき〜統一する」統一させるのは大事ですね。すべてが同じカタチで作られていれば読んだ人は迷わない。

[quanon]

みんな同じこと考えるんだな、やっぱり。

[kwhrtsk]

mastodonのサービス層が割とそんな雰囲気ですね。 https://github.com/tootsuite/mastodon/blob/master/app/services/notify_service.rb

[zentarou]

クロージャ付きの関数（群）でいいんだよなぁ

[nomnel]

ほぼ同じやり方してた。 self.call(args) で new(args).call するの呼び出しが短くなっていいかも

[another931244]

ムズそうだけど、後で読んでみるか...。

[fukumura]

負債を大きくしない話。