[リファクタリング] 共通処理をまとめる時にモジュールでまとめる時とそうでない時は？

Question

リファクタリングをしていて、「モジュールで共通化したほうがいい時」と「クラスとして切り出したほうがいい時」があるように感じていますが、言語化はあまりできていません。

Railsの用語を用いますが、私の経験則では、「クラスコンテキストで定義されているcontrollerのコールバックやmodelのassosiationの定義をまとめる時はモジュールを使う」「共通のメソッドを定義したい時は値オブジェクトなどのクラスとして切り出す」という持論があり、クラスとして切り出した方がユニットテストが書きやすい、があると思っています。
(クラスコンテキストの共通化はモジュールを使わざる得ないので選択の余地はないのですが、、、)

こちらの主張を裏付けるような文章など、もしくは反対意見などはありますでしょうか？

Answer 1

こちらの主張を裏付けるような文章など、もしくは反対意見などはありますでしょうか？

Rails標準として用意された機能としてConcernsがあります。これは、複数のクラスに組み込んで使えるモジュールの形をとっています。

また、バリデーションを共通化したい場合はValidatorを書くなど、専用の道具立てがある場合はそれを使うのが適切です。

Comments

[退会済みユーザー]

ありがとうございます！