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

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

ただいまの
回答率

87.77%

FATモデルからscopeを分離するためのリファクタリングについて

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 1,230

score 14

前提・実現したいこと

【前提】
勤怠管理のシステムを改修しております。私はruby,railsはそこまで詳しくありません。

スピード重視で開発してきたようで、勤務時間計算を担当しているモデルが
700行を超えるFATモデルになっております。
今後の機能改修のために、今のうちにリファクタリングしたいと考えています。
(リファクタリングは一気に完了させずに、段階に分けてしようと思っています)

このモデルがFATモデルになっている原因は、素人ながら見たところ下記だと思っています。
・100行を占めるscope(全て特定条件下の勤怠計算を行うもの)
・出勤、半休など無数のステータス毎に計算するために、ビジネスロジックでswitch
を使って分岐させていること

【実現したいこと】
勤務時間計算を行うモデルですので、責務はそれほどかけ離れていないと思っています。
そのため、まずはモデルの持つ機能をそのままに、scopeを分離したいと考えています。

発生している問題

リファクタリングの方針をどのようにするか、悩んでおります。
下記についてアドバイスをいただけないでしょうか。
・scopeを分離する方法
・またはそれ以外のリファクタリング方法

試したこと

scopeの分離は下記をヒントにしています。
肥大化したActiveRecordモデルをリファクタリングする7つの方法
しかし、上記サイトのコードでは毎回.allを実行するため、数万件のデータを扱うには
パフォーマンスが悪すぎると感じました。

次に、下記サイトをヒントにしました。
Rubyでメソッドチェイン可能なクエリオブジェクトを書く

しかし、上記のサイトのコードでは下記の問題がありました。
・リファクタリング前よりも、総コード量は増加する。(問題かはわかりませんが。。。)

補足情報(言語/FW/ツール等のバージョンなど)

Rails 4.2.0

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

>>・100行を占めるscope(全て特定条件下の勤怠計算を行うもの)
おっしゃるように一般的な解決方はご指摘のサイトの通りクエリビルダーを作ることで、ファットモデルに対して有効かと思います。
下記の記事は参考になるかと思います。
https://qiita.com/furaji/items/12cef3ec4d092865af88

>>・出勤、半休など無数のステータス毎に計算するために、ビジネスロジックでswitch
を使って分岐させていること
上記はstatusごとに共通の親クラスを持った子クラスを作り、時間計算ロジックやその他を実装していけば良いのではないでしょうか。
コマンドパターンに近いかと思います。
http://morizyun.github.io/ruby/design-pattern-command.html

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/25 16:50

    ありがとうございます。参考になりました。

    クエリビルダーによる分離と悩んだのですが、scopeのメリットが失われることを避けるため、
    Concernによる分離を行いました。
    http://qiita.com/QUANON/items/d57dc7a19870b88aa162

    なんとなくで行っていましたが、コマンドパターンという名前があったのですね。
    こちらも学んで、正しく使えるようにしたいと思います。ありがとうございました。

    キャンセル

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

  • ただいまの回答率 87.77%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る