如何清理此Ruby代码？

Question

的问题

我想分析的文本框的线条，以产生从中项目。

我有它的工作，但它看起来很丑陋。

如何清理“if/else”混乱？

代码

class LineParser 
    def self.parse(line) 
    line_matches = line.chomp.match(/(?<name>[[:print:]]+) \$(?<price>\d+\.*\d*)(+?(?<description_text>[^\+#]([[:print:]][^\+#])*))?(+?(\+(?<quantity>\d+)))?(+?#(?<cat1>[[:print:]][^#]*))?$/) 
    return matches_to_hash(line_matches) 
    end 
    def self.matches_to_hash(matches) 
    hash = {} 
    keys = [:name, :price, :description_text, :quantity, :cat1] 
    keys.each do |key| 
     if key == :price 
     hash[key] = matches[key].to_f 
     elsif key == :quantity 
     if matches[key].nil? 
      hash[key] = 1 
     else 
      hash[key] = matches[key].to_i 
     end 
     else 
     hash[key] = matches[key] 
     end 
    end 
    hash 
    end 
end 

Answer 1

从解析中删除显式返回。

class LineParser 
    def self.parse(line) 
    line_matches = line.chomp.match(/(?<name>[[:print:]]+) \$(?<price>\d+\.*\d*)(+?(?<description_text>[^\+#]([[:print:]][^\+#])*))?(+?(\+(?<quantity>\d+)))?(+?#(?<cat1>[[:print:]][^#]*))?$/) 
    matches_to_hash(line_matches) 
    end 
    def self.matches_to_hash(matches) 
    { 
     :price   => matches[:price].to_f, 
     :quantity   => (matches[:quantity] || 1).to_i, 
     :name    => matches[:name], 
     :description_text => matches[:description_text], 
     :cat1    => matches[:cat1] 
    } 
    end 
end 

Answer 2

这

if matches[key].nil? 
    hash[key] = 1 
else 
    hash[key] = matches[key].to_i 
end 

相当于

hash[key] = (matches[key] || 1).to_i 

而对于if/else语句链有超过一对夫妇的分支，也许一个case语句更合适。

Answer 3

class LineParser 
    def self.parse(line) 
    line_matches = line.chomp.match(/(?<name>[[:print:]]+) \$(?<price>\d+\.*\d*)(+?(?<description_text>[^\+#]([[:print:]][^\+#])*))?(+?(\+(?<quantity>\d+)))?(+?#(?<cat1>[[:print:]][^#]*))?$/) 
    return matches_to_hash(line_matches) 
    end 
    def self.matches_to_hash(matches) 
    hash = {} 
    [:name, :price, :description_text, :quantity, :cat1].each do |key| 
     case key 
     when :price 
     hash[key] = matches[key].to_f 
     when :quantity 
      hash[key] = 1 if matches[key].nil? 
      hash[key] = matches[key].to_i unless matches[key].nil? 
     else 
     hash[key] = matches[key] 
    end 
    hash 
    end 
end 

Answer 4

整理matches_to_hash我会做：

def self.matches_to_hash(matches) 
    hash = {} 
    hash[:name] = matches[:name] 
    hash[:price] = matches[:price].to_f 
    hash[:description_text] = matches[:description_text] 
    hash[:quantity] = matches[:quantity].nil? ? 1 : matches[:quantity].to_i 
    hash[:cat1] = matches[:cat1] 
    hash 
end 

不过，我想我会只是改变解析到：

def self.parse(line) 
    line_matches = line.chomp.match(/([[:print:]]+) \$(\d+\.*\d*)(+?([^\+#]([[:print:]][^\+#])*))?(+?(\+(\d+)))?(+?#([[:print:]][^#]*))?$/) 
    hash = {} 
    hash[:name] = $1 
    hash[:price] = $2.to_f 
    hash[:description_text] = $3 
    hash[:quantity] = $4.nil? ? 1 : $4.to_i 
    hash[:cat1] = $5 
    hash 
end 

Answer 5

我不能完全肯定，为什么你循环通过按键时，每个键做不同的事情。为什么不一一处理呢？

class LineParser 
    def self.parse(line) 
    line_matches = line.chomp.match(/(?<name>[[:print:]]+) \$(?<price>\d+\.*\d*)(+?(?<description_text>[^\+#]([[:print:]][^\+#])*))?(+?(\+(?<quantity>\d+)))?(+?#(?<cat1>[[:print:]][^#]*))?$/) 
    return matches_to_hash(line_matches) 
    end 
    def self.matches_to_hash(matches) 
    hash = {} 
    hash[:price] = matches[:price].to_f 
    hash[:quantity] = (matches[:quantity] || 1).to_i 
    hash[:name] = matches[:name] 
    hash[:description_text] = matches[:description_text] 
    hash[:cat1] = matches[:cat1] 
    hash 
    end 
end 

注意：我也偷了Thilo发布的关于数量的聪明点。