Rails, controllerのロジックをmodelに移すリファクタリングする方法が知りたいです。

Question

タスク管理のアプリを作っているところですが
fat controllerを作りたくないのでmodelにほとんど移行したいのですが
途中paramsの渡し方やscopeの使い方がなんもわからんです。

th.end_pointer onclick='window.location="#{tasks_path(end_time: @end_time_num)}"' role="link"
      = end_time_sub
    th.create_pointer onclick='window.location="#{tasks_path(created_at: @created_at_num)}"' role="link"
      = created_at_sub

def created_at_sub
  params[:created_at].present? ? '登録日時▼' : '登録日時△'
end

def end_time_sub
  params[:end_time].present? ? '終了期限▼' : '終了期限△'
end

現在の設定

scope :desc_end_time, -> { order("tasks.end_time DESC") }
scope :desc_created_at, -> { order("tasks.created_at DESC") }

def index
  if params[:created_at].present?
    @tasks = Task.order("tasks.created_at DESC")
  elsif params[:end_time].present?
    @tasks = Task.order("tasks.end_time DESC")
  else
    @tasks = Task.order("tasks.created_at DESC")
  end
end

やろうとして失敗している状態

scope :select_desc, ->(params[:created_at], params[:end_time]){
    if create.present?
      order("tasks.created_at DESC")
    elsif end_time.present?
      order("tasks.end_time DESC")
    else
      order("tasks.created_at DESC")
    end
  }

参考にしているサイト等はありすぎてまとまってません????

Answer 1

def index
  @tasks = if params[:created_at].blank? && params[:end_time].present?
             Task.desc_end_time
           else
             Task.desc_created_at
           end
end

これでいいと思いますが、どうしてもscope1つにまとめたいなら

scope :select_desc, lambda do |created_at, end_time|
                      if created_at.blank? && end_time.present?
                        order("tasks.end_time DESC")
                      else
                        order("tasks.created_at DESC")
                      end
                    end

def index
  @tasks.select_desc(params[:created_at], params[:end_time])
end

こうなるんですかね。
scope内で条件分岐するのはどうかなと思いますが。

ちなみに

lambda do |created_at, end_time|
  if created_at.blank? && end_time.present?
    order("tasks.end_time DESC")
  else
    order("tasks.created_at DESC")
  end
end

は

->(created_at, end_time) {
  if created_at.blank? && end_time.present?
    order("tasks.end_time DESC")
  else
    order("tasks.created_at DESC")
  end
}

と同じです。(複数行あるときは->ではなくlambdaを使うのが好ましい)

Comments

[Naruhub]

とてもわかりやすくご回答頂きありがとう御座います！
-> しか使ったことがなかったのですが
lamda とほぼ同じで lamda を選ぶのが好ましい時など知り得ませんでした????
ありがとう御座います！！！