[ruby-core:49215] Re: [ruby-trunk - Feature #7241] Enumerable#to_h proposal

On 2012/11/11 0:47, jballanc (Joshua Ballanco) wrote:
>
> Issue #7241 has been updated by jballanc (Joshua Ballanco).
>
>
> =begin
> Clojure has a function (({into})) that might fit the bill.

This indeed looks very promising.

> An equivalent Ruby implementation might look something like the following:
>
>      class Hash
>        alias :<<  :merge!
>      end

I might be wrong, but my guess is that constructing lots of 
one-key/value hashes isn't very efficient. Two-element arrays should be 
quite a bit more efficient. So we could define this as follows (in the 
end in C, but here just in Ruby):

class Hash
   def << (other)
     case other.class
     when Array
       store(other[0], other[1])
     when Hash
       merge! other
     end
     self
   end
end

(some additional tweaks may be needed for Array-like and Hash-like objects).


>      module Enumerable
>        def into(coll)
>          coll = coll.dup
>          each do |elem|
>            coll<<  yield(elem)
>          end
>          coll
>        end
>      end
>
>      chars = (97..107).into({}) { |i| { i =>  i.chr } }
>      p chars
>
>      require 'prime'
>      prime_chars = chars.into([]) { |k, v| k.prime? ? v : nil }
>      p prime_chars.compact

It would be great to have a version that avoided "compact". Or maybe 
only that version would be okay? This would use "concat" instead of 
merge! (with Hash#concat an alias for Hash#merge!). Because neither 
Hashes nor Strings can be nested, there would actually not be any 
difference for those, but for Array, the preceeding code could be 
simplified to:

       require 'prime'
       prime_chars = chars.into___([]) { |k, v| k.prime? ? [v] : [] }

I often want a "collect" method where I'm not forced to collect exactly 
one item per item of the original collection. If collect weren't an 
alias to map, I think it would even make a lot of sense to use the word 
"collect" for this (map: one-to-one, collect: one-to-many).

Regards,    Martin.


>      char_string = chars.into("") { |k, v| "#{k}=>#{v}, " }
>      p char_string
> =end
>
> ----------------------------------------
> Feature #7241: Enumerable#to_h proposal
> https://bugs.ruby-lang.org/issues/7241#change-32755
>
> Author: nathan.f77 (Nathan Broadbent)
> Status: Rejected
> Priority: Normal
> Assignee:
> Category: core
> Target version:
>
>
> I often use the `inject` method to build a hash, but I always find it annoying when I need to return the hash at the end of the block.
> This means that I often write code like:
>
>      [1,2,3,4,5].inject({}) {|hash, el| hash[el] = el * 2; hash }
>
> I'm proposing an `Enumerable#to_h` method that would let me write:
>
>      [1,2,3,4,5].to_h {|h, el| h[el] = el * 2 }
>
>
> I saw the proposal at http://bugs.ruby-lang.org/issues/666, but I would not be in favor of his implementation.
> I believe the implementation should be similar to `inject`, so that the hash object and next element are passed to the block. The main difference to the `inject` method is that we would be modifying the hash in place, instead of relying on the block's return value.
>
> As well as providing support for the case above, I have also considered other cases where the `to_h` method would be useful.
> I thought it would be useful if symmetry were provided for the `Hash#to_a` method, such that:
>
>      hash.to_a.to_h == hash  # =>  true
>
> (See example 2)
>
>
> I've allowed developers to provide a symbol instead of a block, so that each element in the collection will be passed to that named method. (See example 3)
>
> Finally, hashes can be given a default value, or a Proc that returns the default value. (See examples 4&  5)
>
>
> Heres an example implementation that I would be happy to rewrite in C if necessary:
>
>
>      module Enumerable
>        def to_h(default_or_sym = nil)
>          if block_given?
>            hash = if Proc === default_or_sym
>              Hash.new(&default_or_sym)
>            else
>              Hash.new(default_or_sym)
>            end
>            self.each do |el|
>              yield hash, el
>            end
>          elsif !default_or_sym.nil?
>            hash = {}
>            self.each do |el|
>              hash[el] = el.send(default_or_sym)
>            end
>          else
>            return Hash[*self.to_a.flatten(1)]
>          end
>          hash
>        end
>      end
>
>
> Examples
> ----------------------------------------------
>
>
> # 1) Build a hash from array elements
>
>      [1,2,3,4,5].to_h {|h, el| h[el] = el * 2 }
>
> =>  {1=>2, 2=>4, 3=>6, 4=>8, 5=>10}
>
>
> # 2) Provides symmetry for Hash.to_a (i.e. you can call hash.to_a.to_h)
>
>      [[1, 2], [3, 4], [5, 6]].to_h
>
> =>  {1=>2, 3=>4, 5=>6}
>
>
> # 3) Build a hash by calling a method on each array element
>
>      ["String", "Another String"].to_h(:size)
>
> =>  {"String"=>6, "Another String"=>14}
>
>
> # 4) Hash with default value
>
>      [4,5,6,5].to_h(0) {|h, el| h[el] += el }
>
> =>  {4=>4, 5=>10, 6=>6}
>
>
> # 5) Hash with default value returned from Proc
>
>      default_proc = ->  hash, key { hash[key] = "go fish: #{key}" }
>      [4,5,6].to_h(default_proc) {|h, el| h[el].upcase! }
>
> =>  {4=>"GO FISH: 4", 5=>"GO FISH: 5", 6=>"GO FISH: 6"}
>
>
>
> Thanks for your time, and please let me know your thoughts!
>
>
> Best,
> Nathan Broadbent
>
>