before_filter pour définir la variable d'instance commune (Ruby On Rails)

Question

Est-ce une pratique courante de faire un filtre comme ceci:

before_filter :get_clients, :only => [:new, :edit, :create, :update] 
... 
def get_clients 
     @clients = Client.accessible_by(current_ability) 
end 

Ma forme a besoin d'accéder à @clients, donc la configuration que je les @clients variables avec avant le filtre au lieu de le faire dans toutes les méthodes. Est-ce que ça a du sens?

Answer 1

Je recommande la construction d'une aide memoized à la place:

def current_clients 
    @current_clients ||= Client.accessible_by(current_ability) 
end 
helper :current_clients 

Cela fonctionne aussi bien comme avant-filtre, mais ne sera pas exécuté à moins que vous devez charger réellement la liste des clients pour une forme/vue. Vous obtenez un chargement paresseux de cette ressource. La responsabilité de savoir comment charger la liste des clients incombe toujours au contrôleur.