Ruby on Rails Code Quality Checklist :
- Each controller action only calls one model method other than an initial find or new.
(Make custom .new or .update methods in the model with all necessary). - Only one or two instance variables are shared between each controller and view.
- All model and variable names are both immediately obvious (to a new developer) and as short as possible without using abbreviations.
- All custom “finds” accessed from more than one place in the code use named_scope instead of a custom method.
- A .find or .find_by_ is never called in a view or view helper.
There are 15 in all, with explanation for each one, so go read and apply.