a code metric tool for rails projects
Ruby Shell
Pull request Compare This branch is 631 commits ahead, 959 commits behind flyerhzm:master.
Fetching latest commit…
Cannot retrieve the latest commit at this time.
Failed to load latest commit information.



Build Status

Coderwall Endorse

Click here to lend your support to: rails best practices and make a donation at www.pledgie.com !

rails_best_practices is a code metric tool to check the quality of rails codes.

it supports following ORM/ODMs:

  • activerecord
  • mongoid
  • mongomapper

following template engines:

  • erb
  • haml
  • slim

rails_best_practices works well only in ruby 1.9.2 and ruby 1.9.3 so far.


At the root directory of rails app

rails_best_practices .

or html output

rails_best_practices -f html .

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

$ rails_best_practices -h
Usage: rails_best_practices [options]
    -d, --debug                      Debug mode
    -f, --format FORMAT              output format
        --without-color              only output plain text without color
        --with-textmate              open file by textmate in html format
        --with-mvim                  open file by mvim in html format
        --with-hg                    display hg commit and username, only support html format
        --with-git                   display git commit and username, only support html format
        --template TEMPLATE          customize erb template
        --output-file OUTPUT_FILE    output html file for the analyzing result
        --silent                     silent mode
        --vendor                     include vendor files
        --spec                       include spec files
        --test                       include test files
        --features                   include features files
    -x, --exclude PATTERNS           Don't analyze files matching a pattern
                                     (comma-separated regexp list)
    -o, --only PATTERNS              analyze files only matching a pattern
                                     (comma-separated regexp list)
    -g, --generate                   Generate configuration yaml
    -v, --version                    Show this version
    -h, --help                       Show this message


Homepage: http://rails-bestpractices.com

Online Service: http://railsbp.com

Github: http://github.com/railsbp/rails_best_practices

RDoc: http://rdoc.rails-bestpractices.com

Team Blog http://rails-bestpractices.com/blog/posts

Google Group: https://groups.google.com/group/rails_best_practices

Wiki: http://github.com/railsbp/rails_best_practices/wiki

Issue Tracker: http://github.com/railsbp/rails_best_practices/issues


rails_best_practices gem is rewritten based on ripper instead of ruby_parser to support ruby 1.9 new syntax.

gem install rails_best_practices

or add in Gemfile

gem "rails_best_practices"


If you install the rails_best_practices with bundler-installed github-sourced gem, please use the following command instead.

bundle exec rails_best_practices .

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

First run

rails_best_practices -g

to generate rails_best_practices.yml file.

Now you can customize this configuration file, the default configuration is as follows:

AddModelVirtualAttributeCheck: { }
AlwaysAddDbIndexCheck: { }
DryBundlerInCapistranoCheck: { }
HashSyntaxCheck: { only_symbol: false, only_string: false }
IsolateSeedDataCheck: { }
KeepFindersOnTheirOwnModelCheck: { }
LawOfDemeterCheck: { }
LongLineCheck: { max_line_length: 80 }
MoveCodeIntoControllerCheck: { }
MoveCodeIntoHelperCheck: { array_count: 3 }
MoveCodeIntoModelCheck: { use_count: 2 }
MoveFinderToNamedScopeCheck: { }
MoveModelLogicIntoModelCheck: { use_count: 4 }
NeedlessDeepNestingCheck: { nested_count: 2 }
NotUseDefaultRouteCheck: {  }
NotUseTimeAgoInWordsCheck: {}
OveruseRouteCustomizationsCheck: { customize_count: 3 }
ProtectMassAssignmentCheck: {}
RemoveEmptyHelpersCheck: {}
RemoveTabCheck: {}
RemoveTrailingWhitespaceCheck: { }
RemoveUnusedMethodsInControllersCheck: { except_methods: [] }
RemoveUnusedMethodsInHelpersCheck: { except_methods: [] }
RemoveUnusedMethodsInModelsCheck: { except_methods: [] }
ReplaceComplexCreationWithFactoryMethodCheck: { attribute_assignment_count: 2 }
ReplaceInstanceVariableWithLocalVariableCheck: { }
RestrictAutoGeneratedRoutesCheck: { }
SimplifyRenderInControllersCheck: {}
SimplifyRenderInViewsCheck: {}
UseBeforeFilterCheck: { customize_count: 2 }
UseModelAssociationCheck: { }
UseMultipartAlternativeAsContentTypeOfEmailCheck: {}
UseObserverCheck: { }
UseParenthesesInMethodDefCheck: {}
UseQueryAttributeCheck: { }
UseSayWithTimeInMigrationsCheck: { }
UseScopeAccessCheck: { }

You can remove or comment one review to disable it, and you can change the options.


Move code from Controller to Model

  1. Move finder to named_scope (rails2 only)
  2. Use model association
  3. Use scope access
  4. Add model virtual attribute
  5. Replace complex creation with factory method
  6. Move model logic into the Model

RESTful Conventions

  1. Overuse route customizations
  2. Needless deep nesting
  3. Not use default route
  4. Restrict auto-generated routes


  1. Keep finders on their own model (rails2 only)
  2. the law of demeter
  3. Use observer
  4. Use query attribute
  5. Remove unused methods in models
  6. Protect mass assignment


  1. Use multipart/alternative as content_type of email


  1. Isolating seed data
  2. Always add db index
  3. Use say with time in migrations


  1. Use before_filter
  2. Simplify render in controllers
  3. Remove unused methods in controllers


  1. Remove empty helpers
  2. Remove unused methods in helpers


  1. Move code into controller
  2. Move code into model
  3. Move code into helper
  4. Replace instance variable with local variable
  5. Simplify render in views
  6. Not use time_ago_in_words


  1. Dry bundler in capistrano


  1. Remove trailing whitespace
  2. Remove tab
  3. Hash Syntax
  4. Use parentheses in method def
  5. Long line

Write Your Own Check List

If you want to write your own check list (some check list only for your rails projects), please read this first, How to write your own check list?


If you want to add your rails best practices into the gem, please post your best practices on http://rails-bestpractices.com

Contact Us

We provide rails consulting services, you can contact us by twitter or email.

Follow us on twitter: http://twitter.com/railsbp

Send us email: team@railsbp.com

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