Ruby on Rails: Eliminar varias claves hash

Question

a menudo me encuentro escribiendo esto:

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

La estela de eliminaciones no se siente bien y tampoco lo hace:

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

¿Hay algo más simple y ¿limpiador?

Answer 1

Supongo que eres consciente del método Hash#except ActiveSupport añade en Hash.

Esto permitiría a su código para simplificar a:

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

Además, no tendría que parche mono, ya que el equipo de rieles lo hizo por ti!

Answer 2

¿Arrancar un parche de mono?

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 

Answer 3

No sé lo que piensa que está mal con su solución propuesta. Supongo que quiere un método delete_all en Hash o algo así? Si es así, tadman's answer proporciona la solución. Pero francamente, por una sola vez, creo que su solución es extremadamente fácil de seguir. Si está utilizando esto con frecuencia, es posible que desee envolverlo en un método de ayuda.

Answer 4

Otra manera de expresar la respuesta de dmathieu podría ser

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

Answer 5

estaría completamente satisfecho con el código que envió originalmente en su pregunta.

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

Answer 6

Mientras usa Hash#except maneja su problema, tenga en cuenta que introduce potential security issues. Una buena regla empírica para manejar cualquier información de los visitantes es utilizar un enfoque de lista blanca. En este caso, usando Hash#slice en su lugar.

params.slice!(:param_to_remove_1, :param_to_remove_2) 
redirect_to my_path(params)