基本的に1文字の変数は何も説明してないから後でわかりにくくなるので避けたほうがいいね。
`Hoge.map {|h| h.name }` とか `rescue => e` みたいにすごく短い範囲でしか使わない場合には使うこともあるけど、今回のようにある程度長い範囲で使う変数名で 1 文字は避けたほうがいいと思います。