http://peepcode.com/products/draft-rails-code-review-pdf
有料のPDFですが、なかなかいいのでまとめ。Railsベカラズ集。
セッションをDB保存
間違い
デフォルトのまま。
正解
Memcachedが速いと言われるがdbとappが一緒のサーバーであればActiveRecordで十分。3つぐらいのサーバーまではActiveRecordで十分。
カスタム設定ファイルを使え
間違い
別サーバーにつなぐときにAPIキーなどをコードに書くな。
environment.rbに書いてもまだ甘い。
正解
config/config.ymlに書く
http://kpumuk.info/ruby-on-rails/flexible-application-configuration-in-ruby-on-rails/
http://blog.bleything.net/2006/06/27/dry-out-your-database-yml
繰り返す文字列には定数を使え
間違い
emailとかよく使う文字列はtypoする。この間違いを探すのは難しい。
ドメインの設定もproductionとdevelopmentでは違う。
正解
environments/development.rb、environments/production.rbに
APP_DOMAIN = ‘******’と違う設定をしておいて、
default_url_options[:host] = APP_DOMAIN
FROM_EMAIL,、DO_NOT_REPLY_EMAIL、 CUSTOMER_SUPPORT_EMAILとか
http://agilewebdevelopment.com/plugins/app_config
時間の記録はUTCで
間違い
ローカルゾーンで記録
正解
UTCで保存。ユーザーの時間に合わしやすいから。
environment.rb => config.active_record.default_timezone = :utc
MySQL => CONVERT_TZ
http://dev.mysql.com/doc/refman/5.0/en/time-zone-support.html
http://redhanded.hobix.com/inspect/showingPerfectTime.html
ActiveRecordをループの中に入れない
間違い
ループのなかにfindを入れない。
正解
:includeを使って結果をループで回す。:joinとか:selectも
http://railscasts.com/episodes/22
http://blog.jcoglan.com/includebydefault/
バイナリフィールドに気をつけろ
間違い
class CreatePhotos < ActiveRecord::Migration
def self.up
create_table :photos do |t|
t.column :caption, :string
t.column :data, :binary
t.column :updated_at, :datetime
end
end
こんなときに
def index
@photos = Photo.find :all
end
こんな事をしては駄目
正解
def self.find_all_for_listing
find(:all, :select => “id, caption, updated_at”)
end
もしくはwith_ scopeを使う。
http://agilewebdevelopment.com/plugins/scope_out
http://www.dcmanges.com/blog/21.html