Ruby on Rails: Usuń wiele kluczy mieszających

Często piszę to:

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

Ścieżka usuwania nie wydaje się odpowiednia, podobnie jak:

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

Czy jest coś prostszego i czystszego?

Domyślam się, że nie jesteś świadomy Hash # z wyjątkiem metody ActiveSupport, która dodaje do Hash.

Pozwoliłoby to na uproszczenie kodu do:

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

Nie musiałbyś też małpować łatek, ponieważ zespół Rails zrobił to za Ciebie!

Podczas używania Hash#exceptrozwiązuje problem, pamiętaj, że wprowadza potencjalne problemy z bezpieczeństwem . Dobrą praktyczną zasadą postępowania z danymi od odwiedzających jest użycie metody białej listy. W tym przypadku użycie Hash#slicezamiast.

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

Byłbym całkowicie zadowolony z kodu, który zamieściłeś w swoim pytaniu.

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

Innym sposobem wyrażenia odpowiedzi dmathieu może być

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

Odpalić małpę?

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

Nie wiem, co Twoim zdaniem jest niewłaściwe w proponowanym rozwiązaniu. Przypuszczam, że chcesz delete_allmetodę na Hash czy coś? Jeśli tak, odpowiedź Tadmana dostarcza rozwiązania. Ale szczerze, jednorazowo, myślę, że twoje rozwiązanie jest niezwykle łatwe do zastosowania. Jeśli używasz tego często, możesz chcieć zawrzeć to w metodzie pomocniczej.