Bir seçenek karmasının başka bir

Question

dan nasıl oluşturulduğu Refactor Bu kod ActiveAdmin için yazılmıştır, ancak aslında önemli değildir. Nasıl optimize edebilirim? Teşekkür ederim.

module ActiveAdminSimpleMenu 
    # in options can take: 
    # menu_priority:integer 
    # menu_parent:string 
    # permitted_params:array for some additions to main_fields permitted params 

    def simple_menu_for(klass, options = {}) 
    ActiveAdmin.register klass do 
     menu_options = {} 
     menu_options[:priority] = options[:menu_priority] if options[:menu_priority] 
     menu_options[:parent] = options[:menu_parent] if options[:menu_parent] 
     menu menu_options unless menu_options.empty? 
     permit_params(*(klass.main_fields + (options[:permitted_params] ? options[:permitted_params] : []))) 
     ... 

Answer 1

"En iyi duruma getirme" deyimiyle, refakatörü kastediyorum. Sen menu_options

inşa etmek

• kullanım Hash#slice (Rails yöntemi) Hep birlikte options[:permitted_params]

test etmek if ve any?

kullanımını || kullanarak çift negatif ortadan kaldırabilir:

def simple_menu_for(klass, options = {}) 
    ActiveAdmin.register klass do 
    menu_options = options.slice(:menu_priority, :menu_parent) 
    menu menu_options if menu_options.any? 
    permit_params(*(klass.main_fields + (options[:permitted_params] || []))) 
    ...