是否有更清洁的方法来实现这一目标?

时间:2012-07-19 12:05:18

标签: ruby-on-rails

我试图仅根据传递的不同信息显示表中的某些记录,如果没有满足任何要求,则重定向到home。代码全部正常运行,只是想看看其他人如何处理这个

if current_user.admin?
  @schedules = Schedule.all
elsif current_user.team_id?
  @schedules = Schedule.find_all_by_team_id(current_user[:team_id])
else
  redirect_to root_path, :status => 301, :alert => "Please contact your club administrator to be assigned to a team."
  return
end

2 个答案:

答案 0 :(得分:6)

您应始终将复杂的逻辑从控制器移开。

class Schedule
  def self.for(user)
    case user.role #you should define the role method in User
      when User::ADMIN
        scoped
      when User::TEAM
        where(team_id: user[:team_id])
    end
  end
end

在您的控制器中:

@schedules = Schedule.for(current_user)

redirect_to root_path, :status => 301, :alert => "Please contact your club administrator to be assigned to a team." unless @schedules

答案 1 :(得分:0)

解决问题的方法。

@schedules = Schedule.all if current_user.admin?
@schedules = Schedule.find_all_by_team_id(current_user[:team_id]) if current_user.team_id?
if @schedules.nil?
  redirect_to root_path, :status => 301, :alert => "Please contact your club administrator 
    to be assigned to a team."
else
  #Your normal redirect
end