max_by возвращение неверного значения

Question

у меня есть хэш, который выглядит следующим образом

@hash = { 1=>[], 2=>[], 3=>[], 4=>[],5=>[], 6=>[], 7=>[ 
[{"value"=>1.58, "title"=>"sun", "quantity" => 2}], 
[{"value"=>1.99, "title"=>"sophia", "quantity" => 5}], 
[{"value"=>6.30, "title"=>"roam", "quantity" => 15}], 
[{"value"=>3.981, "title"=>"jia", "quantity" => "4"}]], 8 => [], 9 => [], 10 => [] } 

, и я пытаюсь найти максимальное значение, основанное на стоимость тега для каждого конкретного значения индекса, такого как

из пут должны выглядеть следующим образом

@hash = { 1=>nil, 2=>nil, 3=> nil, 4=>nil,5=>nil, 6=>nil, 7=>[ 
[{"value"=>6.30, "title"=>"roam", "quantity" => 15}], 8 => nil, 9 => nil, 10 => nil } 

Я надеюсь, что ожидаемый выход ясно

Так что я пытаюсь получить максимум от следующего кода

for t in 0..10 
    if !@hash[t].nil?                  
     @hash[t] = @hash[t].max_by{|i| i[0]['value'].to_i} 
     puts "----------" 
     puts @hash.to_s 
    end 
    end 

, но я получаю только первое значение формы хэширования и нуль для других индексов вместо придания максимального значения для данного индекса. В путах в этом цикле дает мне это что неправильно

[{"title":"sun","value":1.58, "quantity => 2"}], null, null, null, null.. 

не знает, что это неправильно

Answer 1

здесь:

@hash = { 1=>[], 2=>[], 3=>[], 4=>[],5=>[], 6=>[], 7=>[ 
[{"value"=>1.58, "title"=>"sun", "quantity" => 2}], 
[{"value"=>1.99, "title"=>"sophia", "quantity" => 5}], 
[{"value"=>6.30, "title"=>"roam", "quantity" => 15}], 
[{"value"=>3.981, "title"=>"jia", "quantity" => "4"}]], 8 => [], 9 => [], 10 => [] } 

@hash.each do |k, v| 
    if v.empty? 
    @hash[k] = nil 
    else 
    @hash[k] = [v.flatten.sort{ |v1, v2| v2['value'] <=> v1['value'] }.first] 
    end 
end 

puts @hash.inspect 
#=> {1=>nil, 2=>nil, 3=>nil, 4=>nil, 5=>nil, 6=>nil, 7=>[{"value"=>6.3, "title"=>"roam", "quantity"=>15}], 8=>nil, 9=>nil, 10=>nil} 

Answer 2

Я думаю, что это делает то, что вы хотите:

@hash.map do |key, things| 
    [ 
    key, 
    things.max_by { |thing| thing.first["value"].to_i } 
    ] 
end.to_h 

Простите бесполезные имена переменных, я не совсем уверен, что представляет ваша структура данных Я не могу выбрать лучшие имена.

Отметьте, что это не редактирует @hash, но возвращает новый хеш.

Answer 3

Использование #max_by по запросу (Не знаю, почему вы, как и многие другие ответы, хотят уничтожить исходный хэш. Все ниже методы не являются деструктивными и возвратит запрошенные результаты.)

@hash.map.with_object({}) do |(k,v),obj| 
    obj[k] = v.empty? ? nil : [v.flatten.max_by{ |h| h["value"]}] 
end 
#=>{1=>nil, 2=>nil, 3=>nil, 4=>nil, 5=>nil, 6=>nil, 7=>[{"value"=>6.3, "title"=>"roam", "quantity"=>15}], 8=>nil, 9=>nil, 10=>nil} 

Это может быть более чистым, если вы не хотите, чтобы 7-й элемент представлял собой массив из 1, который в любом случае не имеет для меня никакого смысла, например

@hash.map.with_object({}) do |(k,v),obj| 
    obj[k] = v.flatten.max_by{ |h| h["value"]} 
end 
#=> {1=>nil, 2=>nil, 3=>nil, 4=>nil, 5=>nil, 6=>nil, 7=>{"value"=>6.3, "title"=>"roam", "quantity"=>15}, 8=>nil, 9=>nil, 10=>nil} 

Почему вам нужен массив одного элемента, когда все это будет делать, это добавить сложность? Если вы не добавите этот элемент позже?

Или другой способ

@hash.map.with_object({}) do |(k,v),obj| 
    obj[k] = v.max_by{|a| a.is_a?(Array) ? a[0]['value'] : a.to_f} 
end 

Answer 4

@hash = { 1=>[], 2=>[], 3=>[], 4=>[],5=>[], 6=>[], 7=>[ 
[{"value"=>1.58, "title"=>"sun", "quantity" => 2}], 
[{"value"=>1.99, "title"=>"sophia", "quantity" => 5}], 
[{"value"=>6.30, "title"=>"roam", "quantity" => 15}], 
[{"value"=>3.981, "title"=>"jia", "quantity" => "4"}]], 8 => [], 9 => [], 10 => [] } 

@hash.each do |key, values| 
    if values.empty? 
    @hash[key] = nil 
    else 
    @hash[key] = [values.flatten.max_by{|h| h["value"]}] 
    end 
end 

p @hash 

Answer 5

Вот код, который даст нужный результат:

output_hash = {} 

@hash.keys.each do |k| 
    output_hash[k] = @hash[k].flatten.max{ |a, b| a["value"] <=> b["value"] } 
end 

puts output_hash 

Answer 6

Это легко может быть сделано с помощью одного лайнера, вы можете заменить sort_by с max_by если вам просто нужно одно значение

Сортировка значений descendin г

@hash.values.reject(&:empty?).flatten.sort_by{|x| x['value']} 

Максимальное значение хэш

@hash.values.reject(&:empty?).flatten.max_by{|x| x['value']}