2012-01-11 2 views
0

Вот мой пример кода, сообщите мне, можно ли его улучшить?Улучшение кода

excludedb = if File.exist?(arg) 
    IO.read(arg).split(',').map { |db_name| db_name.strip }.delete_if { |db_name| db_name == "" } 
else 
    ["master", "model", "sybsystemdb", "sybsystemprocs", "tempdb", "sybsecurity", "pubs2", "pubs3", "dbccdb", "sybmgmtdb"] 
end 
+0

Что сделаем Вам не нравится об этом? –

+1

Это должно быть на [codereview.stackexchange] (http://codereview.stackexchange.com/). –

ответ

1

Вот пара крошечных улучшений.

Вы можете заменить

.map { |db_name| db_name.strip } 

с

.map(&:strip) 

А также вы можете использовать массив строк буквального

%w{master model sybsystemdb} 
+0

@deceze: спасибо, backticks действительно были излишними :-) –

Смежные вопросы