Rubin kürzer Array Iteration

Question

Ich versuche, herauszufinden, ob es einen kürzeren Weg ist, den Code unten zu tun:

$my_array.each do |eh| 
    row = Array.new 
    eh.each do |k,v| 
    if k == 'state' 
     if eh[k] == "stopped" 
     row << eh[k].red 
     elsif eh[k] == "running" 
     row << eh[k].green 
     else 
     row << eh[k].yellow 
     end 
    else 
    row << eh[k] 
    end 
    end 
    data << row 
end 

Die oben ist für die Verwendung von terminal-table und colorize

Answer 1

Wie wäre es damit :

color_map = { 'stopped' => :red, 'running' => :green } 
data = $my_array.map do |eh| 
    eh.map do |k,v| 
    if k == 'state' 
     v.send color_map.fetch(v, :yellow) 
    else 
     v 
    end 
    end 
end 

Es verwendet map statt each manuell Aufbau der Arrays zu vermeiden. Es verwendet auch einen Hash, um den Status der Farbe zuzuordnen, anstatt if-Anweisungen zu verwenden.

Sie könnten es weiter und entfernen Sie die innere if Anweisung, wenn Sie möchten, aber das ist zumindest eine Verbesserung.