带重定向的应用程序控制器级别验证

Question

我不确定确保每个用户都具有某些必要属性的最佳方法，并且如果他们不是我想将其重定向到“新”页面，例如

class ApplicationController < ActionController::Base 
    protect_from_forgery 
    before_filter :authenticate_user!, :valid_location 

    def valid_location 
    if (current_user.location.nil? || current_user.location.city.nil?) 
     redirect_to new_user_locations_path(current_user.id) 
    else 
     true 
    end 
end 

上面的例子有缺陷，因为它创建了一个重定向循环。我可以明确地使用一些关于创建这种验证的建议。谢谢

Answer 1

创建重定向循环的原因是因为valid_location方法也在负责new_user_locations_path的控制器上调用。为了防止这种情况，您需要确保控制器不会使用skip_before_filter（在Rails 4中为skip_before_action）运行该过滤器。类似的问题是answered here。

class LocationsController < ApplicationController 
    skip_before_filter :valid_location, only: [:new, :create] 
    #... 
end 

因为valid_location返回真/假布尔，我建议重命名方法valid_location?或invalid_location?和重构逻辑一点：

class ApplicationController < ActionController::Base 
    protect_from_forgery 
    before_filter :authenticate_user!, :redirect_invalid_locations 

    private 
    def redirect_invalid_locations 
    redirect_to(new_user_locations_path(current_user)) if invalid_location? 
    end 

    def invalid_location? 
    current_user.try(:location).try(:city).nil? 
    end 
end 


class LocationsController < ApplicationController 
    skip_before_filter :redirect_invalid_locations, only: [:new, :create] 
end  

在位置控制器