Напишите метод group_by самостоятельно

Question

Мне нужно написать метод group_by самостоятельно. Это то, что у меня есть до сих пор:

module Enumerable 
    def group_by(&b) 
    solution = {} 
    self.each {|key,val| 
     b.call(var) 
     solution = { key=> (val) } 
    } 
    end 
end 

ary = [1,2,3,4,5,6,7,8,9,10] 

p ary.group_by() { |i| i%3 } 

Я не понимаю. Я надеюсь, что вы можете мне помочь.

Answer 1

Я не совсем уверен, как я могу помочь, кроме публикации решения, но может быть, какое-то описание с тем, что вам трудно найти?

Несколько вопросов я заметил:

• Вы используете мимо двух аргументов в блоке, когда массив имеет только 1, значение
• вы вызываете блок с var, который не существует в текущем scope, может быть, вы имели в виду val?
• вы не проверить, если что-то уже существует в новом словаре решения
• перезаписываются словарь SOLUTION каждый раз, когда вы перебираете новое значение в массиве

Вот что я придумал:

module Enumerable 

    def group_by 
    solution = {} 
    each do |value| 
     key = yield value 
     if solution.key?(key) 
     solution[key] << value 
     else 
     solution[key] = [value] 
     end 
    end 
    solution 
    end 

end 

ary = [1, 2, 3, 4, 5] 
p ary.group_by { |i| i % 3 } 

выход:

{1=>[1, 4], 2=>[2, 5], 0=>[3]} 

вы можете сп eck, если блок был предоставлен, если кто-то пытается использовать эту функцию неправильно, подумайте об использовании инструкции unless block_given?, но, возможно, вы можете попробовать реализовать это самостоятельно.

Answer 2

module Enumerable 
    def group_by &b; inject({}){|h, e| (h[b.call(e)] ||= []).push(e); h} end 
end 

[1,2,3,4,5,6,7,8,9,10].group_by{|i| i % 3} 
# => {1 => [1, 4, 7, 10], 2 => [2, 5, 8], 0 => [3, 6, 9]} 

Answer 3

Другим решение для сравнения:

module Enumerable 
    def group_by 
    {}.tap do |group| 
     each{ |value| (group[ yield(value) ] ||= []) << value } 
    end 
    end 
end 

• использует tap, чтобы избежать неприглядной картины

  thing = {} 
  # do stuff with thing 
  thing # return it 
  

• использует ||= для создания новой коллекции массива уже нет

В качестве альтернативы:

module Enumerable 
    def group_by 
    Hash.new{ |h,k| h[k]=[] }.tap do |group| 
     each{ |value| group[ yield(value) ] << value } 
     group.default = nil # remove the default_proc when done 
    end 
    end 
end 

Answer 4

С моей backports gem:

module Enumerable 
    def group_by 
    return to_enum(:group_by) unless block_given? 
    result = {} 
    each do |o| 
     key = yield(o) 
     ary = result.fetch(key){ result[key] = [] } 
     ary << o 
    end 
    result 
    end 
end 

В отличие от всех решений, представленных до сих пор, она проходит RubySpec.