実録！Railsのはまりポイント10選

実録！
Railsのはまりポイント10選
2012/03/26
第2回渋谷Edge Rails勉強会
藤田武雄

Copyright © Drecom Co., Ltd.

自己紹介
• 藤田武雄
• 株式会社ドリコム
• ソーシャルゲーム事業本部
• アプリケーションエンジニア
• 最近は主に社内ライブラリの開発を
担当


今日の内容
• Railsを使用したソーシャルゲームの開
発時にはまったポイントをご紹介

• 実際に社内であった事例です
• ほとんどRails3採用後の話


Case1
and/or


はまり例
a = b or c
• こういう意味にしたかった
a = (b or c)
• 実際の動きは
(a = b) or c
• bの値をaに代入し、それが偽であればc
を評価する


演算子の優先順位
• and/orは優先順位が一番低い
• 対策1: 括弧をつける
a = (b or c)

• 対策2: &&/¦¦を使う
a = b || c


ちなみにRails本体は
• Contributing to Ruby on Rails
• 5.3 Follow the Coding Conventionsに
は

• Prefer &&/¦¦ over and/or
• と書かれている
• http://guides.rubyonrails.org/
contributing_to_ruby_on_rails.html


Case2
scope


はまり例
scope :started, where(‘start_at <= ?’, Time.now)

• どういうことが起こりうるか
• 開始時刻になってもキャンペーンがス
タートしない

• 設定した時刻にバナーが切り替わら
ない


なぜ?
• クラスがロードされた時点でwhereの
中にあるTime.nowの評価が行われる

• 結果、アプリケーションサーバを起動
した時間で固定されてしまう


lambdaを渡す
scope :started, lambda { where('start_at
<= ?', Time.now) }
• lambdaの中身は都度評価される
• edgeではwhereを直接書くやり方は
deprecatedとなった

• https://github.com/rails/rails/commit/
0a12a5f8169685915cbb7bf4d0a7bb48
2f7f2fd2


Case3
クラスのインスタンス変数


インスタンス変数の遅延
初期化
@foo ||= bar
• 初期化の手間を省くためによく使われ
る

• barの実行結果をキャッシュする効果も
• クラスのインスタンス変数では注意が
必要


よくない例
class Baz
def self.foo
@foo ||= bar
end
end
• barがDBから取得した値だとDBを更新
しても返り値が変化しない！


起こったこと
• データを追加したのに反映されない
• よくわからんからunicornを再起動
• うまくいった

• そんなこともありました


なぜ?
• 通常はオブジェクトのインスタンスの
寿命は1リクエストの間だけ

• 一方、クラス自体はリクエストを返し
た後も生き続ける

• クラスのインスタンス変数に入れる
とずっと保持される


対策
• むやみにクラスのインスタンス変数に
入れない

• 意味をわかっていてやるのであればOK


Case4
textあふれ


データ破損
• textカラムにJSONを格納していた
• ある日突然データが壊れた！


原因
• mysqlのtextは65535バイトまで
• それ以上のものを保存すると後ろが切
れるが保存自体エラーにはならない

• 読み込むとJSONとしては壊れているの
でパースエラー


対策
• その1: validates_length_ofを設定
• その2: 文字数を減らす
• その3: mediumtext(16MBまで)などに
変更する

• どれぐらい増えるか見積もりしておけば
はみ出ないように設計見直しできるはず

• AR::Store(YAML)にも気をつけよう

Case5
index名長さ制限


index名の長さ
• 複数カラムにindexを張った時に長さ制
限に引っかかった

• mysqlは64バイトまでしか使えない
index_テーブル名_on_カラム名_and_カラ
ム名_…


index名を短くする
• デフォルトを変更するモンキーパッチ
をあてた

• テーブルごとにuniqueになればよいの
でテーブル名不要
カラム名_and_カラム名_...


パッチ
module ActiveRecord
module ConnectionAdapters
module SchemaStatements
def index_name(table_name, options)
if Hash === options
if options[:column]
"#{Array.wrap(options[:column]) * '_and_'}"
elsif options[:name]
options[:name]
else
raise ArgumentError, "You must specify the index name"
end
else
index_name(table_name, :column => options)
end
end
end
end
end

Case6
tinyint


tinyintの扱いの罠
• アプリリリース前にDBAの協力の下、
テーブル構成をチューニングした

• int → tinyint
• するとアプリの動作がおかしくなった
• DBに入っている数値は変わってないの
に取ってきた値が全然違う！


原因と対策
• ActiveRecordはtinyintのカラムを
booleanとして扱う

• 対策: boolean以外ではtinyintを使わな
い(smallintにするなど)


Case7
参照分散時のミス


data_fabric
• 参照分散のためのplugin
• 利用者数が増え、masterだけでさばききれな
くなったため導入

• action単位でaround_ﬁlterをかけて振り分け
• slaveの情報を元にmaster側を更新しない
ようにaction単位にした

• アクセスの多いマイページなどはslaveへ逃が
す


事故発生！
• あるときレプリケーション遅延が起
こってslaveの古い情報を元にmasterに
更新がかかってしまった

• データ不整合発生！
• そんな実装はしていない認識だった
• 調査するとマイページのviewで呼び出
しているhelperからupdateが…


対策
• helperからupdateしない！
• 参照振り分け用around_ﬁlterの外側に
before_ﬁlterを設定し、update処理部
分を移した


Case8
ランキング集計


ランキング集計
• イベントランキングを毎時集計
• 一人ずつselectして順位をupdateする実装
になっていた

• 参加ユーザ数が増えると集計時間がどんど
ん長くなる

• 集計時間が30分を超えた時点で状況を検知
• 前月までは大丈夫だったのに…

対応前のサーバ負荷
• DBサーバのCPU使用率


対策
• 方針としてはtransaction数を減らす
• 裏で新規テーブルを作成して、1000件
ずつbulk insert

• 全部入ったらrenameで入れ替え
• bulk insertにはactiverecord-import
を使用


コード例
CREATE TABLE IF NOT EXISTS
tmp_scores LIKE scores;
TRUNCATE TABLE tmp_scores;
class TmpScore < ActiveRecord::Base; end
TmpScore.import([:user_id, :value, :rank],
scores)
RENAME TABLE scores TO scores_old,
tmp_scores TO scores;
DROP TABLE scores_old;

対応後のサーバ負荷


別の方法
• Redisのsorted setを使う
• リアルタイムにランキングを更新でき
る

• 数万人規模でも大丈夫
• そのままでは同率順位を表現できない


Case9
double submit
protection


double submit protection

• 二重送信を検知してくれるplugin
• 二重送信を防ぎたいフォーム(orリンク)
のあるページでsessionにtokenを保存
し、パラメータにも同じ物をつける

• 次のページでは両者を付きあわせて、
session内のtokenを無効にする


sessionの罠
• Railsのsessionの扱いではまった
• レスポンスを返すまでsessionの実体が
更新されない！

• 処理に時間がかかるとすり抜けてしま
う


処理の流れ
user app session memcache
request load

write

ココが問題
response write


対策
• レスポンス返すまで待っていられない
のでsessionを使うのをやめる

• pluginに手を入れてtokenをRedisに保
存するように変更


Case10
daemon-spawn +
resque


daemon-spawn
• rubyプログラムをデーモン化するため
のライブラリ

• resqueのworkerをデーモン化した時に
問題が起こった


何が起こったか
• rails runnerで起動していた
• redisへのコネクションを張るpluginが
あった

• forkするときにコネクションがコピー
されて全workerで共有されてしまい
redisへの接続が不安定に


対策
• rails runnerを使わなくした
• 自前で必要なものをrequireするスクリ
プトに書き換え


もうひとつの問題
• stopした時に処理中のプロセスが落と
される

• 処理中ステータスの変なデータが残っ
てしまった


原因
• daemon-spawnのstop時のシグナルが
デフォルトではTERM

• rescueはTERMを受け取ると処理中で
も即終了してしまう


対策
• daemon-spawnの起動スクリプトで明
示的にQUITを指定した

• ドキュメントに載ってない
• 実装を読んでみて指定できることがわ
かった


まとめ
• 社内ではまった例をご紹介しました
• pluginではまったときは実装を追いか
けることも必要


ご清聴ありがとうございました


実録！Railsのはまりポイント10選

Recommended

Recommended

More Related Content

What's hot

What's hot (20)

Viewers also liked

Viewers also liked (20)

Similar to 実録！Railsのはまりポイント10選

Similar to 実録！Railsのはまりポイント10選 (20)

More from Drecom Co., Ltd.

More from Drecom Co., Ltd. (17)

実録！Railsのはまりポイント10選

Editor's Notes