重构一个选项散列是如何从另一个构建的

Question

此代码是为ActiveAdmin编写的，但实际上并不重要。 我如何优化它？谢谢。

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

我认为“优化”你的意思是重构。您可以

• 使用Hash#slice（Rails的方法）来构建menu_options
• 消除双重否定用if和any?
• 使用||测试options[:permitted_params]

一起：

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] || []))) 
    ...