rails_best_practices is a gem to check quality of rails app files according to ihower’s presentation from Kungfu RailsConf in Shanghai China
Pull request Compare This branch is 966 commits behind flyerhzm:master.
Fetching latest commit…
Cannot retrieve the latest commit at this time.
Failed to load latest commit information.



rails_best_practices is a gem to check quality of rails app files according to ihower’s presentation Rails Best Practices from Kungfu RailsConf in Shanghai China.
rails_best_practices is a code static parser tool.


ihower’s presentation


sudo gem install rails_best_practices --source http://gemcutter.org


At the root directory of rails app

rails_best_practices .

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

And default rails_best_practices will do parse codes in vendor, spec, test and stories directories. If you need, see the command options:

$ rails_best_practices -h
Usage: rails_best_practices [options]
    -d, --debug                      Debug mode
        --vendor                     include vendor files
        --spec                       include spec files
        --test                       include test files
        --stories                    include stories files
    -h, --help                       Show this message


If you got NoMethodError or any syntax error, you should use debug mode to detect which file rails_best_practices is parsing and getting the error.

rails_best_practices -d .

Then give me the error stack and the source code of the file that rails_best_practices is parsing error.

Customize Configuration

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

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: { }
UseBeforeFilterCheck: { }
MoveCodeIntoControllerCheck: { }
MoveCodeIntoModelCheck: { }
MoveCodeIntoHelperCheck: { array_count: 3 }
ReplaceInstanceVariableWithLocalVariableCheck: { }


  • Lesson 1. Move code from Controller to Model
    1. Move finder to named_scope
    2. Use model association
    3. Use scope access
    4. Add model virtual attribute
    5. Use model callback
    6. Replace Complex Creation with Factory Method
    7. Move Model Logic into the Model
    8. model.collection_model_ids (many-to-many)
    9. Nested Model Forms (one-to-one)
    10. Nested Model Forms (one-to-many)
  • Lesson 2. RESTful Conventions
    1. Overuse route customizations
    2. Needless deep nesting
    3. Not use default route
  • Lesson 3. Model
    1. Keep Finders on Their Own Model
    2. Love named_scope # same as Move finder to named_scope
    3. the Law of Demeter
    4. DRY: metaprogramming
    5. Extract into Module
    6. Extract to composed class
    7. Use Observer
  • Lesson 4. Migration
    1. Isolating Seed Data
    2. Always add DB index
  • Lesson 6. View
    1. Move code into controller
    2. Move code into model
    3. Move code into helper
    4. Replace instance variable with local variable
    5. Use Form Builder # not implement, use http://github.com/justinfrench/formtastic
    6. Organize Helper files # not implement, it’s rails default behaviour



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