ruby on rails - ¿Qué se entiende por ''Tamaño de condición de rama de asignación demasiado alto'' y cómo solucionarlo?

ruby-on-rails code-metrics

Question

En mi aplicación Rails, uso Rubocop para buscar problemas. Hoy me dio un error como este: El Assignment Branch Condition size for show is too high . Aquí está mi código:

def show
  @category = Category.friendly.find(params[:id])
  @categories = Category.all
  @search = @category.products.approved.order(updated_at: :desc).ransack(params[:q])
  @products = @search.result.page(params[:page]).per(50)
  rate
end

¿Qué significa esto y cómo puedo solucionarlo?

Answer 1

El tamaño de la Condición de rama de asignación (ABC) es una medida del tamaño de un método. Se determina esencialmente contando el número de asignaciones A , ranchos B y declaraciones condicionales C. (mas detalle..)

Para reducir el tamaño de ABC, puede mover algunas de esas asignaciones a llamadas before_action:

before_action :fetch_current_category, only: [:show,:edit,:update] 
before_action :fetch_categories, only: [:show,:edit,:update] 
before_action :fetch_search_results, only: [:show,:edit,:update] #or whatever

def show
  rate
end

private

def fetch_current_category
  @category = Category.friendly.find(params[:id])
end

def fetch_categories
  @categories = Category.all
end

def fetch_search_results
  @search = category.products.approved.order(updated_at: :desc).ransack(params[:q])
  @products = @search.result.page(params[:page]).per(50)
end