Ruby on Rails：複数のハッシュキーを削除する

私はよくこれを書いています。

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

削除の軌跡は正しくなく、どちらも正しくありません。

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

よりシンプルでクリーンなものはありますか？

私はあなたがハッシュを知らないことを推測している#exceptActiveSupportがHashに追加するHashメソッドに。

これにより、コードを次のように簡略化できます。

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

また、Railsチームがパッチを作成したので、モンキーパッチを適用する必要はありません。

を使用Hash#exceptすると問題が処理されますが、潜在的なセキュリティ問題が発生することに注意してください。訪問者からのデータを処理するための良い目安は、ホワイトリストアプローチを使用することです。この場合、Hash#slice代わりに使用します。

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

質問で最初に投稿したコードに完全に満足します。

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

dmathieuの答えを表現する別の方法は、

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

サルのパッチを起動しますか？

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

提案されたソリューションのどこが悪いと思うかわかりません。あなたdelete_allはハッシュか何かのメソッドが欲しいと思いますか？もしそうなら、タッドマンの答えは解決策を提供します。しかし、率直に言って、1回限りの場合、あなたのソリューションは非常に簡単に理解できると思います。これを頻繁に使用する場合は、ヘルパーメソッドでラップすることをお勧めします。