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

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

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

Ruby on Rails 6は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

受付中

railsで中身がほぼ同じメソッドが二つあるので、リファクタリングしたい

kokosi1062
kokosi1062

総合スコア39

Ruby on Rails 6

Ruby on Rails 6は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

1回答

0評価

0クリップ

123閲覧

投稿2022/01/21 08:33

編集2022/01/24 14:13

devise.rbファイルの中身ですが、リファクタリングしたいです。
login_userlogin_user_at_nilをそれぞれ使いたいのですが、
ほぼ中身は同じなので、リファクタリング出来ないかと考えています。
違いは、changed_at: nilがあるかないかです。
アドバイスいただけますと幸いです。どうぞよろしくお願いいたします。

ruby

module ControllerMacros def login_user(user = nil) if user.blank? main = create(:main) create(:status, main: main) create(:ip, main: main) user = create(:user, main: main) end allow(controller).to receive(:authenticate_user!).and_return(true) @request.env['devise.mapping'] = Devise.mappings[:user] @user = user sign_in @user end def login_user_at_nil(user = nil) if user.blank? main = create(:main) create(:status, main: main) create(:ip, main: main) user = create(:user, main: main, changed_at: nil) end allow(controller).to receive(:authenticate_user!).and_return(true) @request.env['devise.mapping'] = Devise.mappings[:user] @user = user sign_in @user end

リファクタリング試した結果

ruby

def login_user_at_nil(user = nil) login_user_sub(user, changed_at: nil) end def login_user(user = nil) login_user_sub(user, {}) end def login_user_sub(university_user, option) if user.blank? main= create(:main) create(:status, main: main) create(:ip, main: main) user = create(:user, main: main, option) end allow(controller).to receive(:authenticate_user!).and_return(true) @request.env['devise.mapping'] = Devise.mappings[:user] @user = user sign_in @user end

良い質問の評価を上げる

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

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

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

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

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

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

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

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

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

まだ回答がついていません

会員登録して回答してみよう

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

ただいまの回答率
87.20%

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

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

質問する

関連した質問

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

Ruby on Rails 6

Ruby on Rails 6は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。