Railsでbefore_validationコールバックを使う時の落とし穴

TL;DR: before_validationに登録されているコードがfalseを返すと、obj.saveはfalseを返しobj.save!は例外を投げます。DBには何も登録されません。

Railsのコールバック

便利ですよね！使ってますか？

Rails Guidesの一節を引用してみます。

Callbacks are methods that get called at certain moments of an object's life cycle. With callbacks it is possible to write code that will run whenever an Active Record object is created, saved, updated, deleted, validated, or loaded from the database.

すなわち

コールバックは（ActiveRecordの）オブジェクトのライフサイクルの中の特定の時点で呼び出されるメソッドです。コールバックを利用することで、ARオブジェクトが作られるとき・保存されるとき・更新されるとき・削除されるとき・validationが実行されるとき・DBから読み込まれるとき などに必ず実行されるコードを登録することができます。

controllerになんでもかんでも任せてしまうのを防ぎ、modelの責務をmodelに記述することができるようなるのでとても有用です。

※上記guidesやreferencesと合わせて、昨年末のRoR Advent Calendarに投稿されたこの記事を読むと勉強になります ↓
Ruby - てめえらのRailsはオブジェクト指向じゃねえ！まずはCallbackクラス、Validatorクラスを活用しろ！ - Qiita [キータ]

before_validation

ここから本題です。

例題として、RailsでSNSを構築することを考えます。ユーザーはメールアドレスに紐付けてアカウントを取得し、登録するとメールアドレス確認メールが送られてきます。メールアドレスの確認が完了しないと、SNS上で活動し始めることができません。利用開始後にユーザーがメールアドレスを変更した場合も、確認のメールが送られてきます。

# db/migrate/20140104120000_create_users.rb

class CreateUsers < ActiveRecord::Migration
  def change
    create_table :users do |t|
      t.string :name
      t.string :email
      # some more columns...

      t.string :email_verification_code # メールアドレス確認用乱数
      t.boolean :email_verificated # メールアドレス確認済み権限フラグ
    end
  end
end

このようにテーブルを定義し、以下のようなコードを書きます。

# app/models/users.rb

class User < ActiveRecord::Base
  # validation前に、emailが変更されていればコールバック関数を呼び出す
  before_validation :set_email_verification_code, if: 'self.email_changed?'

  # 保存後、メールアドレス確認メールを送信する
  after_save :send_verification_mail, if: 'self.email_changed?'

  private
  def set_email_verification_code
    # 新しい確認用コードを生成
    self.email_verification_code = SecureRandom.hex(10)
    # 状態を「未確認」にする
    self.email_verificated = false
  end

  def send_verification_mail
    # 略
  end
end

一見、とても自然ですよね？しかしこれでは、いくらすばらしいSNSを作ってもユーザーは集まりません。なんたって、登録できないのですから！

Reference中の一文を引用します。

If the returning value of a before_validation callback can be evaluated to false, the process will be aborted and Base#save will return false. If ActiveRecord::Validations#save! is called it will raise a ActiveRecord::RecordInvalid exception. Nothing will be appended to the errors object.

意訳しますと

before_validationコールバックのコードがfalse（またはfalseと評価されるもの）を返すと、そこで処理が中止されます（validationは実行されません）。user.saveの途中でこれが起こると、user.saveは単にfalseを返します（DBには保存されません）。user.save!の途中でこれが起こると、ActiveRecord::RecordInvalidが起こります。いずれにしても、user.errorsにはいかなるvalidation errorも登録されません。

なんということでしょう！見かけ上はvalidationに失敗しているのとまったく変わらないのに、実はvalidationそのものがスキップされているのです。

上で作ったUserモデルの例では、set_email_verification_codeの最後の式でfalseを代入しているために、メソッドそのものの戻り値がfalseになります。（Rubyでは代入式の戻り値は代入された値ですね。）

これを避けるには単にこのメソッド中の式の順番を入れ替えれば良いだけです。

  def set_email_verification_code
    # 状態を「未確認」にする
    self.email_verificated = false
    # 新しい確認用コードを生成
    self.email_verification_code = SecureRandom.hex(10) # => 戻り値が乱数文字列になる
  end

Railsチームがどのような意図でこの挙動を設計したのかわかりませんが、陥りやすい落とし穴でなおかつ一度ハマると気づいて抜け出すのが困難です（数十分悩みました）。お気をつけください。

半年ほど放置しておりましたがあけましておめでとうございます（笑
今年もマイペースに更新してまいりますので何卒よろしくおねがいします

Vim の mercurial レポジトリの HEAD が ver7.4a になってた

およそ2ヶ月ほど更新が途切れておりましたが生きています。お久しぶりです。

Vimを最強のPython開発環境にする2 - Λlisue's blog この記事を読んで、一念発起、Vimの環境を徹底的に整備しました（Λlisueさんありがとうございます！）。その中で、NeoComplcacheに代わる新しい補完PluginとしてNeoCompleteが紹介されていました。Vim 7.3.885以上かつLuaインタプリタ装備の環境下で使用でき、NeoComplcacheよりも高速に動作するということで、入れてみようとしたのですが、UbuntuのパッケージになっているVimは7.3.5xxだったのでVimをコンパイルする必要がありました。

続きを読む

Ubuntu 13.04 で音声のデジタル出力

夜ふかししてます。こんばんは。

先日、Ubuntu 13.04をインストールしたところ、音声のデジタル出力(S/PDIF)ができなくなったということを記事に書きましたが、簡単な設定変更で治りました！

ので、同じ道を通る方のために書き残しておきます。

続きを読む

Ubuntu 13.04をインストールしてみた

4/25にリリースされたUbuntu 13.04。連休を利用してインストールしてみました。Ubuntuはアップデートの時に不具合が出るというのをよく聞くので、クリーンインストールしています。

ま、Ubuntuのリリースサイクルである半年に一回くらい、ゼロベースでやり直した方が中身がゴチャゴチャしなくて好ましいという考え方もできますね。

続きを読む

Ubuntu with Intel Graphics

Ubuntu導入当初からでしたが、起動して数分内にこの画面が出ます。
f:id:iTakeshi:20130503222841j:plain
「システムプログラムの問題が見つかりました」で、何が悪いの?というと、毎度毎度plymouthd（グラフィカルブートができるようにするやつ）がクラッシュなさっているそうな。

特段の実害がなかったので放置してありましたが、最近どうもChromeでサーフィンしているとカクつくなぁ…ということで、やっぱりグラフィックス周りになんか病巣があるんじゃないかと疑い、いろいろ調べてみたところ…

Intel HD Graphicsドライバ、入ってないんじゃね?

という結論に至りました。えらいこっちゃ。

追記
Ubuntu 13.04でこの問題は修正されておりますので、この記事の適用は12.10よりも古いバージョンに限定されます。

続きを読む

Ubuntu 12.10 をインストールしたら low graphics mode になった話

お久しぶりの更新デス。
前置きを書く文才がないのでいきなり本題へ。

先週まではWindows7のVirtualBox上でUbuntuを動かしていたのですが、最近はUbuntuを使うためにWindowsを立ち上げるという本末転倒な事態に陥っていたのでした。
ということで、SSDを増設してデュアルブートにしてみました！

…と、ここまではよかったのですが、インストールして、初期設定とかカスタマイズとかゴソゴソして、rebootすると、"The system is running in low-graphics mode."というエラー画面が出て、ターミナルエミュレータからしかログインできなくなってしまいます。
Ubuntu serverにSSHするのと（見た目は）まるっきり同じ状態です。

続きを読む