我试图仅根据传递的不同信息显示表中的某些记录,如果没有满足任何要求,则重定向到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
答案 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