Ruby on Rails : Supprimer plusieurs clés de hachage

Question

Je me retrouve souvent à écrire cela :

params.delete(:controller)  
params.delete(:action)  
params.delete(:other_key)  
redirect_to my_path(params)  

La traînée de suppressions ne semble pas correcte, et elle ne l'est pas non plus :

[:controller, :action, :other_key].each do |k|
  params.delete(k)
end

Y a-t-il quelque chose de plus simple et de plus propre ?

Answer 1

Je suppose que vous n'êtes pas au courant de la Hash#except ActiveSupport ajoute une méthode à Hash.

Cela permettrait de simplifier votre code à :

redirect_to my_path(params.except(:controller, :action, :other_key))

De plus, vous n'auriez pas à faire de patch singulier, puisque l'équipe Rails l'a fait pour vous !

Answer 2

En utilisant Hash#except gère votre problème, sachez qu'il introduit les problèmes de sécurité potentiels . Une bonne règle de base pour traiter les données des visiteurs est d'utiliser une liste blanche. Dans ce cas, l'utilisation de Hash#slice à la place.

params.slice!(:param_to_keep_1, :param_to_keep_2)
redirect_to my_path(params)

Answer 3

Je serais tout à fait satisfait du code que vous avez posté dans votre question.

[:controller, :action, :other_key].each { |k| params.delete(k) }

Answer 4

Une autre façon de formuler la réponse de dmathieu pourrait être

params.delete_if { |k,v| [:controller, :action, :other_key].include? k }

Answer 5

Allumer un patch de singe ?

class Hash
  def delete_keys!(*keys)
    keys.flatten.each do |k|
      delete(k)
    end

    self
  end

  def delete_keys(*keys)
    _dup = dup
    keys.flatten.each do |k|
      _dup.delete(k)
    end

    _dup
  end
end