hazisarashi/stream_xml_to_hash — pull request #6 by iori (2013-03-09 16:42:41+00:00)
Title: Dev/refactoring

テストは通ってます。

87 if !@now_tag.empty? and during_capture?
93 @capture_data[:text] = text if during_capture? and !@capture_data.has_key? :text

この辺りのif式の条件が何を示しているか解らないので

protected

# 省略

def hoge? # 適切な名前
@now_tag.empty? and during_capture?
end

みたいにすると可読性上がると思うので、いい名前つけて関数に分けると良いと思いました。
(僕はドメインを理解していないので、名前わからんちん)