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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

Q&A

解決済

1回答

1748閲覧

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

angelica-k

総合スコア14

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

0グッド

1クリップ

投稿2017/09/16 11:29

編集2017/09/18 14:28

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

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

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

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

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

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

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

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

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

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

・100行を占めるscope(全て特定条件下の勤怠計算を行うもの)

おっしゃるように一般的な解決方はご指摘のサイトの通りクエリビルダーを作ることで、ファットモデルに対して有効かと思います。
下記の記事は参考になるかと思います。
https://qiita.com/furaji/items/12cef3ec4d092865af88

・出勤、半休など無数のステータス毎に計算するために、ビジネスロジックでswitch

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

投稿2017/09/24 15:19

akichim21

総合スコア93

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

angelica-k

2017/09/25 07:50

ありがとうございます。参考になりました。 クエリビルダーによる分離と悩んだのですが、scopeのメリットが失われることを避けるため、 Concernによる分離を行いました。 http://qiita.com/QUANON/items/d57dc7a19870b88aa162 なんとなくで行っていましたが、コマンドパターンという名前があったのですね。 こちらも学んで、正しく使えるようにしたいと思います。ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問