初学者重构问题

Question

我想知道重构这个的最佳或最正确的方法。

series = Hash.new
series[:series_id] = doc.xpath("Data/Series/id").inner_text if not doc.xpath("Data/Series/id").nil?

Answer 1

series = { :series_id => doc.xpath("Data/Series/id").try(:inner_text) }

Answer 2

像

这样的东西

series = Hash.new
id_element = doc.xpath("Data/Series/id")
series[:series_id] =  id_element.inner_text unless id_element.nil?

Answer 3

我认为最具革命性的重构是使用xml来对象映射库，如happymapper，并将所有这些脏实现细节留在幕后。