Напишите лучшую функцию переключателя?

Question

Я новичок в Ruby.
Как лучше писать эту функцию? я могу вместо этого использовать хеш-таблицу.

def readable_status(status) 
    if status == "1" 
    return "go" 
    end 
    if status == "2" 
    return "stop" 
    end 
    if status == "3" 
    return "die" 
    end 
end 

Answer 1

Если вы хотите использовать хэш (как на ваш вопрос) вы могли бы сделать :

def readable_status(status) 
    readable = { "1" => "go", "2" => "stop", "3" => "die" } 
    readable[status] || "default value" 
end 

Answer 2

уверен, просто используйте

def readable_status(status) 
    m = {'1' => 'go', '2' => 'stop', '3' => 'die'} 
    m[status] 
end 

вы можете сделать это Oneliner, если вы хотите:

... 
{'1' => 'go', '2' => 'stop', '3' => 'die'}[status] 

Answer 3

Как насчет

def readable_status(status) 
    %w{go stop die}[status.to_i - 1] 
end 

Answer 4

У меня возникли неудачные попытки с помощью Hash-es для этой проблемы из-за субоптимальной спецификации, которая была предоставлена ​​мне (что означает: бизнес изменил спецификацию во время разработки).

Хеши хороши, пока вам не нужно написать что-то более сложное, чем одно значение. Если вам нужно изменить эти одиночные значения на методы, вам нужно переписать все, так как хэши берут значение методов, вызывая их , когда определяется хэш. И если впоследствии значение возвращаемого метода изменится, хеши не будут изменены.

И остается читаемым на английском :-)

def readable_status(status) 
    case status 
    when "1" then "go" end 
    when "2" then "stop" end 
    when "3" then "die" end 
    end 
end 

Answer 5

Если хэш определяется внутри метода, хэш (и каждая строка в нем) воссоздан каждый метод вызывается. Определение константы предотвращает это; использование метода для доступа к значениям сохраняет гибкость, как сказал @Phrogz.

READABLE_STATUS_TABLE = {'1'=>'go', '2'=>'stop', '3'=>'die'} 

def readable_status(status) 
    READABLE_STATUS_TABLE[status] 
end