huaoguo/permission-manager — pull request #1 by Kilen (2014-03-29 03:53:37+00:00)
Title: code_review_at_0329

以下的建议有两种：
1. 一种是代码风格的建议（我下面提到的风格改进是基于现在社区普遍认可的风格）
2. 另外一种是代码的改进建议。

还有一些全局性的问题：
1. model 在一开始的时候就加上 validation 是个好习惯
2. 尽量去掉没用的代码（尤其是 rails 自动生成的一些代码）
3. 我比较倾向一种代码哲学：轻controller，重model，不过由于现在业务逻辑比较简单，所以也体现不了太多东西。