Rails：辅助重构

Question

我有一个我觉得很可笑的助手，但我一直想不出改进的方法。这是有问题的帮助程序：

# Shows Admin Menu Button
def admin_toggle_button
  if user_signed_in? && ( current_user.has_role?(:admin) || ( @collection && can?(:curate,@collection) ) )
    if session[:admin_menu] == :on
      link_to( 'Admin Tools', edit_shared_path(:admin_menu => :off), :remote=>true, :class => 'selected', :id => 'admin_toggle_button', :title => 'Hide Admin Menu' )
    else
      link_to( 'Admin Tools', edit_shared_path(:admin_menu => :on), :remote=>true, :id => 'admin_toggle_button', :title => 'Show Admin Menu' )
    end
  end
end

在我的应用程序的菜单栏中，我调用 admin_toggle_button ，该帮助程序确定该按钮是否应该存在及其状态。

要显示管理菜单按钮，需要有登录用户，并且该用户需要是管理员，或者该用户需要查看允许他管理（编辑）的集合。

我的问题是：像这样的辅助方法是正常的吗——即你是否发现你时不时地需要如此复杂的方法——或者我是否遗漏了一些东西？您能建议一种改进此方法的方法吗？

Answer 1

您可以为第一个条件创建一个方法，以便可以在其他地方重用它。

def can_view_admin_stuff?
  user_signed_in? && ( current_user.has_role?(:admin) || ( @collection && can?(:curate,@collection) ) )
end

def admin_toggle_button
  return '' unless can_view_admin_stuff?

  if session[:admin_menu] == :on
    link_to( 'Admin Tools', edit_shared_path(:admin_menu => :off), :remote=>true, :class => 'selected', :id => 'admin_toggle_button', :title => 'Hide Admin Menu' )
  else
    link_to( 'Admin Tools', edit_shared_path(:admin_menu => :on), :remote=>true, :id => 'admin_toggle_button', :title => 'Show Admin Menu' )
  end
end

但是，是的，我想找到这样的方法是正常的，特别是如果您只使用链接的条件。如果您将它用于代码的其他部分，那么最好有一个助手。

Answer 2

该方法并不是非常复杂。最复杂的部分是访问检查，即使这样也不算太糟糕。需要帮助者的原因之一是让类似的事情远离你的视野，这样你就不会做任何错事。

两个 link_to 变体之间的重复可能需要注意。您可以对其进行一些重构，但是：

def admin_toggle_button
  return '' if !user_signed_in? || !( current_user.has_role?(:admin) || ( @collection && can?(:curate,@collection) ) )
  opts = {
    :id     => 'admin_toggle_button',
    :remote => true,
    :title  => 'Show Admin Menu'
  }
  admin_menu = :on
  if session[:admin_menu] == :on
    opts[:title] = 'Hide Admin Menu'
    opts[:class] = 'selected'
    admin_menu   = :off
  end
  link_to('Admin Tools', edit_shared_path(:admin_menu => admin_menu), opts)
end

这种方法突出显示了两个可能的 link_to 调用之间的差异。如果 session[:admin_menu] != :on 更常见，那么您可能想要反转逻辑以启动 opts 和 admin_menu “隐藏管理菜单”设置，然后根据需要将其调整为 != :on 大小写。

def admin_toggle_button
  return '' if !user_signed_in? || !( current_user.has_role?(:admin) || ( @collection && can?(:curate,@collection) ) )
  opts = {
    :class  => 'selected',
    :id     => 'admin_toggle_button',
    :remote => true,
    :title  => 'Hide Admin Menu'
  }
  admin_menu = :off
  if session[:admin_menu] != :on
    opts[:title] = 'Show Admin Menu'
    opts.delete(:class)
    admin_menu   = :on
  end
  link_to('Admin Tools', edit_shared_path(:admin_menu => admin_menu), opts)
end