リファクタリングと私は、次のコードを持つ複雑

Question

軽減する方法：

@model = "ford" 
@attribute = "name" 

def grouped 
    grouped = {} 

    @array.each do |r| 
    field = r.car.send(@model) 

    if field.is_a? ActiveRecord::Associations::CollectionProxy 
     field.each do |obj| 
      key = obj.send(@attribute) 
      grouped[key] = [] unless grouped.has_key?(key) 
      grouped[key].push(r) 
     end 
    else 
     key = field.send(@attribute) 
     grouped[key] = [] unless grouped.has_key?(key) 
     grouped[key].push(r) 
    end 
    end 

    grouped 
end 

結果はを：

{ford: [a, b, c]} 

codeclimateは、それが認知の複雑さを持っていることを言います。

このメソッドをどのようにリファクタリングしてクリーナーにすることができますか？

Answer 1

def grouped 
    @array.each_with_object(Hash.new { |h, k| h[k] = [] }) do |r, grouped| 
    case field = r.car.send(@model) 
    when ActiveRecord::Associations::CollectionProxy 
     field.each do |obj| 
      grouped[obj.send(@attribute)] << r 
     end 
    else 
     grouped[field.send(@attribute)] << r 
    end 
    end 
end