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

มีอะไรที่ง่ายและสะอาดกว่านี้ไหม?

ฉันคาดเดาว่าคุณไม่ทราบแฮช # ยกเว้นวิธีที่ ActiveSupport เพิ่มลงในแฮช

มันจะช่วยให้รหัสของคุณง่ายต่อการ:

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วิธีการในแฮชหรืออะไร? ถ้าเป็นเช่นนั้นคำตอบของ tadmanจะให้คำตอบ แต่ตรงไปตรงมาฉันคิดว่าทางออกของคุณนั้นง่ายมาก หากคุณใช้งานบ่อยครั้งคุณอาจจำเป็นต้องปิดมันด้วยวิธีการช่วยเหลือ