redirect_to and return と書いて良いのか?

redirect_to root_url unless logged_in?

みたいなコードを初めのうちはつい書いてしまいハマる。
正しくは、以下のように return しなければならない。

unless logged_in?
  redirect_to root_url
  return 
end

しかし1行で書けていたものが4行になってしまうのはいかにも不格好である。

あちこちでコードを読んでいるとたまにこんなのを見かける。

redirect_to root_url and return unless logged_in?

これは見た目もかっこいいし意味も判り易くて cool だ。

ところで、 and return ってなんだ?

これは何の事はない、 redirect_to の戻り値がたまたま true なのを利用して、 and で受けて次の文を書いているだけである。
redirect_to が万が一 false や nil を返してきたら return は実行されない。
ちなみに redirect_to の実装を見ると、少なくとも現時点では false や nil にならないことは確実だが、将来に渡って保証されているわけではないので気持ち悪い。

and の代わりに true/false に関わらず確実に実行する接続詞として ; があるので、以下のように書いてみる。

redirect_to root_url; return unless logged_in?

残念ながらこれはうまくいかない。 unless より ; のほうが優先順位が高いので、

redirect_to root_url
return unless logged_in?

という意味になってしまう。

仕方なくカッコでくくる。

(redirect_to root_url; return) unless logged_in?

これは正しく動くが、かっこわるいよねえ。カッコ付けててかっこ悪いってやつ?

さらに他のコードを読み漁ると、こんな書き方もある。

return redirect_to(root_url) unless logged_in?

and return 記法に比べればcoolさは劣るが、比較的スマートな書き方だ。何の意味も無い値を return することについては一瞬不安になるが、 action の戻り値を気にしなければならなくなるような仕様変更は将来的にも有り得ないと思われるので、問題ないだろう。

というわけで、結論としては return redirect_to 記法をオススメ、というところで筆を置くつもりだったのだが、何やら公式っぽいドキュメントに、「and return」って書くといいよ! と記述されてるので、いいのかもしれんね。
(だったら redirect_to の実装の末尾に true の一行を足しておきたいなあ)

Comment are closed.