コードレビュー、というわけではないですが

デバッグツールとしては、pry が断然オススメです！！
http://labs.timedia.co.jp/2011/12/rubyist-should-use-pry.html
ソースコード中に、 binding.pry と書いておいて、いつも通り Rails アプリを起動するだけでブレークポイントを設定出来ます。
色々できるので超便利です。詳しくはまた直接でも紹介します。

あとは、Rails best practices という gem を使うと、コードのお作法を指摘してくれます。
http://blog.scimpr.com/2012/08/27/rails3-2%E3%81%A7%E3%82%B3%E3%83%BC%E3%83%89%E3%81%AE%E3%81%8A%E4%BD%9C%E6%B3%95%E3%82%92%E6%8C%87%E6%91%98%E3%81%97%E3%81%A6%E3%82%82%E3%82%89%E3%81%86%EF%BD%9Erails-best-practices/

よって、ここは以下のようにしておくと良い感じと思います。
```ruby
# console and debug
group :development do
  gem 'pry-rails'
  gem 'pry-debugger'
  gem 'rails_best_practices', :require => false
end
```