コードの改善

Question

ここに私のコードサンプルがあります。それがさらに改善できるかどうか教えてください。

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 

Answer 1

ここにはいくつか改善があります。

あなたは

.map(&:strip) 

で

.map { |db_name| db_name.strip } 

を置き換えることができ、また、あなたはあなたがそれについて好きではない何

%w{master model sybsystemdb}