初學者重構問題

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，並留下所有的引擎蓋下這骯髒的實施細則。