代码改进

时间:2012-01-11 07:20:50

标签: ruby

这是我的代码示例,请告诉我是否可以进一步改进?

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

1 个答案:

答案 0 :(得分:1)

以下是一些微小的改进。

您可以替换

.map { |db_name| db_name.strip }

.map(&:strip)

还可以使用字符串数组文字

%w{master model sybsystemdb}