Rubyonrails 分配分支条件大小太高是什么意思以及如何解决

发布于06月20日

在我的Rails应用程序中，我使用Rubocop来判断问题.今天它给了我一个这样的错误:Assignment Branch Condition size for show is too high.这是我的代码:

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

这意味着什么？我如何修复它？

推荐答案

分配分支条件(ABC)大小是对方法大小的度量.它基本上是通过计算A个选区、B个牧场和C个条件语句的数量来确定的.(more detail..)

为了降低ABC分数，你可以将其中一些作业(job)转移到"行动前要求"中:

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