hazisarashi/stream_xml_to_hash — pull request #1 by iori (2013-03-05 08:07:54+00:00)
Title: Update stream_xml_to_hash.rb

注意：Forkして動作確認すべきなんだろうけど、テストない&動作確認用のデータがないのでGitHub上で動作確認しないで書いています。
ごめんなさい。

・複数の引数がある場合は()あった方が可読性が上がるので、つけた方が良いと思います。
・debug_lineという関数にまとめた方が、条件分岐が統一出来るので良いと思います。
・テストが無いので判断出来ないのですが、全てがPublicなメソッドである必要はありますか？
・children_to_hashで最後に明示的にreturnを書いていますが、書かなくて良い。
・injectを使うことによって、new_hashを外に書かなくてもよい.
・一行で部分は、ifを後置したほうがソースコードが短くなるので良い（ケースバイケースだけど）

他にも色々直せそうだけど、時間がないのでここまで(>_<)