ruby-on-rails-3.1 - このコードを改良する方法 メンタルブロック Rails

Question

これら 2 つの方法を 1 つにまとめたいと考えています。しかし、私の人生では、方法がわかりません。NOOBビッグタイム。助けていただければ幸いです。

  def is_active_no_category
      'active' if params[:category].blank?
  end


  def is_active(category)
      'active' if  params[:category] == category.name.parameterize
  end

Answer 1

以下の関数では、引数として a を渡すことcategoryはオプションです。引数を渡さない場合は、メソッドのように機能しますis_active_no_category。引数を渡す場合。それはあなたのis_active方法のように機能します。

def is_active(category = nil)
  'active' if (category.present? && params[:category] == category.name.parameterize) || (category.nil? && params[:category].blank?)
end

ステートメントをよりコンパクトにすることができるかもしれませんがif、関数の正確な要件を述べていないため、最も完全なソリューションを提供しました.