初心者のリファクタリングの質問

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

私が最も革新的なリファクタリングがhappymapperのようなマッピングライブラリ、オブジェクト、フードの下にあるすべてのこの汚い実装の詳細を残すために、XMLを使用することだろうと思う

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

Answer 3

ような何か。