h1. rails_best_practices

rails_best_practices is a gem to check quality of rails app files according to ihower's presentation Rails Best Practices.
rails_best_practices is a code static parser tool based on ruby_parser.

*************************************************

h2. Resources

"ihower's presentation":http://www.slideshare.net/ihower/rails-best-practices
"Wiki":http://wiki.github.com/flyerhzm/rails_best_practices

*************************************************

h2. Install

<pre><code>
sudo gem install rails_best_practices --source http://gemcutter.org
</code></pre>

*************************************************

h2. Usage

at the root directory of rails app

<pre><code>
rails_best_practices .
</code></pre>

notice the period at the end, it can be the relative or absolute path of your rails app.

*************************************************

h2. Customize Configuration

Copy <code>rails_best_practices.yml</code> in the root directory of rails_best_practices gem to <code>config</code> directory
Now you can customize the configuration, the default configuration is as follows:

<pre><code>
MoveFinderToNamedScopeCheck: { }
UseModelAssociationCheck: { }
UseScopeAccessCheck: { }
AddModelVirtualAttributeCheck: { }
# UseModelCallbackCheck: { }
ReplaceComplexCreationWithFactoryMethodCheck: { attribute_assignment_count: 2 }
MoveModelLogicIntoModelCheck: { called_count: 4 }
# ManyToManyCollectionCheck: { }
# NestedModelFormsCheck: { }
OveruseRouteCustomizationsCheck: { customize_count: 3 }
NeedlessDeepNestingCheck: { nested_count: 2 }
NotUseDefaultRouteCheck: {  }
KeepFindersOnTheirOwnModelCheck: { }
LawOfDemeterCheck: { }
UseObserverCheck: { }
IsolateSeedDataCheck: { }
AlwaysAddDbIndexCheck: { }
UseFilterCheck: { }
MoveCodeIntoControllerCheck: { }
MoveCodeIntoHelperCheck: { }
ReplaceInstanceVariableWithLocalVariableCheck: { }
</code></pre>

*************************************************

h2. Progress

* Lesson 1. Move code from Controller to Model
## [-Move finder to named_scope-]
## [-Use model association-]
## [-Use scope access-]
## [-Add model virtual attribute-]
## Use model callback
## [-Replace Complex Creation with Factory Method-]
## [-Move Model Logic into the Model-]
## model.collection_model_ids (many-to-many)
## Nested Model Forms (one-to-one)
## Nested Model Forms (one-to-many)

* Lesson 2. RESTful Conventions
## [-Overuse route customizations-]
## [-Needless deep nesting-]
## [-Not use default route-]

* Lesson 3. Model
## [-Keep Finders on Their Own Model-]
## [-Love named_scope-] # same as Move finder to named_scope
## [-the Law of Demeter-]
## DRY: metaprogramming
## Extract into Module
## Extract to composed class
## [-Use Observer-]

* Lesson 4. Migration
## [-Isolating Seed Data-]
## [-Always add DB index-]

* Lesson 5. Controller
## [-Use before_filter-]
## DRY Controller

* Lesson 6. View
## [-Move code into controller-]
## Move code into model
## [-Move code into helper-]
## [-Replace instance variable with local variable-]
## Use Form Builder
## Organize Helper files

*************************************************

Copyright © 2009 Richard Huang (flyerhzm@gmail.com), released under the MIT license