初学者重构问题

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

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

--------------解决方案-------------

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

就像是

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

我觉得最革命性的重构将使用XML对象映射库像happymapper并留下所有的引擎盖下的这个肮脏的实施细则。

分类:Ruby on Rails的 时间:2015-03-15 人气:2
本文关键词: 红宝石,Ruby on Rails的
分享到:

相关文章

Copyright (C) 55228885.com, All Rights Reserved.

55228885 版权所有 京ICP备15002868号

processed in 0.281 (s). 10 q(s)