Home

Please go to https://rails-bestpractices.com

How to write your own check list?

Default Check list:

• 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
  9. Nested Model Forms

• RESTful Conventions
  1. Overuse route customizations
  2. Needless deep nesting
  3. Not use default route

• 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
  8. Use Attribute Query

• Mailer
  1. Use multipart/alternative as content_type of email

• Migration
  1. Isolating Seed Data
  2. Always add DB index
  3. Use Say with Time in Migrations

• Controller
  1. Use before_filter
  2. DRY Controller # not implement, use https://github.com/josevalim/inherited_resources

• 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 https://github.com/justinfrench/formtastic
  6. Organize Helper files # not implement, it’s rails default behaviour

• Deploy
  1. Dry bundler in capistrano

• Other
  1. Remove Trailing Whitespace