Рефакторинг метода для поиска записей по статусу

Question

В настоящее время я использую .find_by_status (params [: status]) в своих @tasks, чтобы найти задачи, которые не закрыты или липкие. (или 4,5).

def self.find_by_status(status) 
    status = status.to_i 
    if status == 0 then 
    status = 1 
    else 
    status = status 
    end 
    if status == 1 || !status then 
    Task.where(["STATUS NOT IN (4,5)"]) 
    else 
    Task.where(:status => status) 
    end 
end 

Я также скопировал это для своей модели билетов, чтобы найти только открытые билеты на главной странице.

Это также сопровождается этим

<% status_active = 1 %> 
<% Task.new.statuses.each do |status| %> 
    <li class="<%= if (params[:status].to_i || status_active) == status[0] then "active" end %>"> 
    <%= link_to status[1], :controller => params[:controller], :action => params[:action], :params => { :status => status[0] } %> 
    </li> 

Я новичок в рельсах, и я действительно изо всех сил на рефакторинга это. Я бы предпочел сделать эти ссылки в ниспадающем фильтре, но с этим я тоже борюсь.

Любая помощь приветствуется!

Answer 1

Этот фрагмент кода:

def self.find_by_status(status) 
    if status.to_i.zero? 
    Task.where(["STATUS NOT IN (4,5)"]) 
    else 
    Task.where(:status => status.to_i) 
    end 
end 

идентична вашей выше коде (Там нет никакого способа, что статус может быть ложным после того, как вы бросили его to_i).

Вы можете очистить код вида вниз:

<% Task.new.statuses.each do |status| %> 
    <li class="<%= 'active' if (params[:status].to_i || Task::STATUS_ACTIVE) == status[0] %>"> 
    <%= link_to status[1], :controller => params[:controller], :action => params[:action], :params => { :status => status[0] } %> 
</li> 
<% end %> 

Я предлагаю добавить STATUS_ACTIVE как константа в модели вместо кодирования его в поле зрения.

Кроме того, факт, что вы указываете контроллеры и действия с помощью параметра, является странным, но, не зная больше о вашем прецеденте, я не могу устранить его.