From: mame@... Date: 2019-05-21T04:02:46+00:00 Subject: [ruby-core:92737] [Ruby trunk Feature#15863] Add `Hash#slice!` and `ENV.slice!` Issue #15863 has been updated by mame (Yusuke Endoh). > It should behave in the way Hash#slice does, except one thing Hash#slice! modifies the object. (See, for instance, how Array#slice and Array#slice! work, they return the same value) I agree that it is ideal. However, in practical, it would be difficult to add to Ruby a feature whose behavior is completely different from the existing feature in Rails. If you really want to do this, you need first to persuade the developers of ActiveSupport to deprecate (and then remove) the current `Hash#slice!`, wait for a few years, and finally we can add it to Ruby with a "correct" behavior. ---------------------------------------- Feature #15863: Add `Hash#slice!` and `ENV.slice!` https://bugs.ruby-lang.org/issues/15863#change-78099 * Author: bogdanvlviv (Bogdan Denkovych) * Status: Open * Priority: Normal * Assignee: * Target version: ---------------------------------------- ## Add `Hash#slice!` In https://bugs.ruby-lang.org/issues/8499 we added `Hash#slice`. `Hash#slice!` removes and returns the key/value pairs matching the given keys: ```ruby h = {a: 100, b: 200, c: 300} h.slice!(:a) # => {:a=>100} h # => {:b=>200, :c=>300} h.slice!(:b, :c, :d) # => {:b=>200, :c=>300} h # => {} ``` Note that, this method reflects the behavior of Active Support's `Hash#extract!` method that was added in 2009, see https://github.com/rails/rails/commit/8dcf91ca113579646e95b0fd7a864dfb6512a53b ```ruby h = {a: 100, b: 200, c: 300} h.extract!(:a) # => {:a=>100} h # => {:b=>200, :c=>300} h.extract!(:b, :c, :d) # => {:b=>200, :c=>300} h # => {} ``` (There is a proposal to add `Hash#extract` to Ruby - https://bugs.ruby-lang.org/issues/15831, but it has another method signature) Active Support also has `Hash#slice!`, see https://api.rubyonrails.org/v5.2/classes/Hash.html#method-i-slice-21. It is quite different what this patch proposes, see how it works: ```ruby h = {a: 100, b: 200, c: 300} h.slice!(:a) # => {:b=>200, :c=>300} # AS Hash#slice! h # => {:a=>100} h.slice!(:b, :c, :d) # => {:a=>100} # AS Hash#slice! h # => {} ``` I think `Hash#slice!` in Ruby should work in the same way as `Hash#extract!` from Active Support, there is one argument: - https://bugs.ruby-lang.org/issues/8499#note-31 It should behave in the way `Hash#slice` does, except one thing `Hash#slice!` modifies the object. (See, for instance, how `Array#slice` and `Array#slice!` work, they return the same value) But I would like to discuss it more to choose the right behavior for the proposed method. (Maybe there are good arguments why we should add `Hash#slice!` with behavior as it is in Active Support) ## Add `ENV.slice!` The method removes and returns the key/value pairs matching the given keys. ```ruby ENV.slice!("PORT", "RAILS_ENV") # => {"PORT"=>"3000", "RAILS_ENV"=>"development"} ``` Pull Request: https://github.com/ruby/ruby/pull/2195 Patch: https://patch-diff.githubusercontent.com/raw/ruby/ruby/pull/2195.patch -- https://bugs.ruby-lang.org/ Unsubscribe: