Rails Best Practices

Rails Best Practices

<br />#Before<br /><% form_for @user do |f| %><br /> <%= f.text_field :email %><br /> <% for role in Role.all %><br /> <%= check_box_tag 'role_id[]', role.id, @user.roles.include?(role) %><br /> <%= role.name %><br /> <% end %><br /><% end %><br /><br />class User < ApplicationController<br /> def update<br /> @user = User.find(params[:id])<br /> if @user.update_attributes(params[:user])<br /> @user.roles.delete_all<br /> (params[:role_id] || []).each { |i| @user.roles << Role.find(i) }<br /> end<br /> end<br />end<br /><br /><br />#After<br /><% form_for @user do |f| %><br /> <% for role in Role.all %><br /> <%= check_box_tag 'user[role_ids][]', role.id, @user.roles.include?(role)<br /> <%= role.name %><br /> <% end %><br /> <%= hidden_field_tag 'user[role_ids][]', '' %><br /><% end %><br /><br />class User < ApplicationController<br /> def update<br /> @user = User.find(params[:id])<br /> @user.update_attributes(params[:user])<br /> # @user.role_ids = params[:user][:role_ids]<br /> end<br />end<br /><br /><br />#==========================================<br /><br />#Nested Model Forms (one-to-one) Before<br /><br />class Product < ActiveRecord::Base<br /> has_one :detail<br />end<br />class Detail < ActiveRecord::Base<br /> belongs_to :product<br />end<br /><br /><% form_for :product do |f| %><br /> <%= f.text_field :title %><br /> <% fields_for :detail do |detail| %><br /> <%= detail.text_field :manufacturer %><br /> <% end %><br /><% end %><br /><br /><br />class Product < ApplicationController<br /> def create<br /> @product = Product.new(params[:product])<br /> @details = Detail.new(params[:detail])<br /><br /> Product.transaction do<br /> @product.save!<br /> @details.product = @product<br /> @details.save!<br /> end<br /> end<br />end<br /><br /><br /># ~~~~~~~~~~~~~~~~~~~~~~~~~<br /># Nested Model Forms (one-to-one) After Rails 2.3 new feature<br /><br />class Product < ActiveRecord::Base<br /> has_one :detail<br /> accepts_nested_attributes_for :detail<br />end<br /><br /><% form_for :product do |f| %><br /> <%= f.text_field :title %><br /> <% f.fields_for :detail do |detail| %><br /> <%= detail.text_field :manufacturer %><br /> <% end %><br /><% end<br /><br /><br />class Product < ApplicationController<br /> def create<br /> @product = Product.new(params[:product])<br /> @product.save<br /> end<br />end<br /><br />#Nested Model Forms (one-to-many)<br /><br />class Project < ActiveRecord::Base<br /> has_many :tasks<br /> accepts_nested_attributes_for :tasks<br />end<br /><br />class Task < ActiveRecord::Base<br /> belongs_to :project<br />end<br /><br /><% form_for @project do |f| %><br /> <%= f.text_field :name %><br /> <% f.fields_for :tasks do |tasks_form| %><br /> <%= tasks_form.text_field :name %><br /> <% end %><br /><% end %><br /><br /><br />#===========================================<br />#Extract into Module Before<br /><br />class User < ActiveRecord::Base<br /> validates_presence_of :cellphone<br /> before_save :parse_cellphone<br /><br /> def parse_cellphone<br /> # do something<br /> end<br />end<br /><br />#<br />/lib/has_cellphone.rb<br />module HasCellphone<br /><br /> def self.included(base)<br /> base.validates_presence_of :cellphone<br /> base.before_save :parse_cellphone<br /> base.send(:include,InstanceMethods)<br /> base.send(:extend, ClassMethods)<br /> end<br /><br /> module InstanceMethods<br /> def parse_cellphone<br /> # do something<br /> end<br /> end<br /><br /> module ClassMethods<br /> end<br />end<br /><br /><br /><br />class User < ActiveRecord::Base<br /> include HasCellphone<br />end<br /><br /><br /><br />#=======================================<br /># Extract to composed class Before<br /><br /># == Schema Information<br /># address_city :string(255)<br /># address_street :string(255)<br />class Customer < ActiveRecord::Base<br /> def adddress_close_to?(other_customer)<br /> address_city == other_customer.address_city<br /> end<br /><br /> def address_equal(other_customer)<br /> address_street == other_customer.address_street &&<br /> address_city == other_customer.address_city<br /> end<br />end<br /><br /><br />#Extract to composed class (value object) After<br />class Customer < ActiveRecord::Base<br /> composed_of :address, :mapping => [ %w(address_street street),<br /> %w(address_city city) ]<br />end<br /><br />class Address<br /> attr_reader :street, :city<br /> def initialize(street, city)<br /> @street, @city = street, city<br /> end<br /><br /> def close_to?(other_address)<br /> city == other_address.city<br /> end<br /><br /> def ==(other_address)<br /> city == other_address.city && street == other_address.street<br /> end<br />end<br /><br /><br /><br /># Isolating Seed Data<br /># Before<br /><br />class CreateRoles < ActiveRecord::Migration<br /> def self.up<br /> create_table "roles", :force => true do |t|<br /> t.string :name<br /> end<br /><br /> #Move this to db/seeds.rb<br /> ["admin", "author", "editor","account"].each do |name|<br /> Role.create!(:name => name)<br /> end<br /><br /> end<br /><br /> def self.down<br /> drop_table "roles"<br /> end<br />end<br /><br /># After<br /># /db/seeds.rb (Rails 2.3.4)<br />["admin", "author", "editor","account"].each do |name|<br /> Role.create!(:name => name)<br />end<br />rake db:seed<br /><br />