【Ruby】同じコードを書かないためだけにbefore_filterを使っていませんか？

結論

「同じコードを書かない」のは目的でなく、
品質を上げる１つの手段。

本題

例えば以下のようなプログラムがあるとする。

class TestsController < ApplicationController

  def index
    @tests = Test.all
  end

  def show
    @test = Test.find(params[:id])
  end

  def edit
    @test = Test.find(params[:id])
  end

  # 略
end

「同じコードを２回(以上)書いている！
DRY(Don't Repeat Yourself)に反している！」

と思い、いままで以下の様に書き換えていました。

class TestsController < ApplicationController
  before_filter :set_test, only: [:show, :edit]

  def index
    @tests = Test.all
  end

  def show
  end

  def edit
  end

  # 略

  private
  
  def set_test
    @test = Test.find(params[:id])
  end
end

しかし、こう書くことでバグのリスクが少し高まります。
例えば、新しいactionを追加したとき、
before_filterにactionの追加を忘れる、ということも起きうる。

上記の例ぐらい文量が少なければ問題ないと思います。
が、ソースコードが増える、複数の人がコードを書くようになると、
抜け漏れが起きる可能性も高まります。

だから敢えて品質向上を目的に、
各actionに同じコードを書く。
これによって、ソースを見る人は各actionの中だけを見て
プログラムを書くことができます。

class TestsController < ApplicationController

  def index
    @tests = Test.all
  end

  def show
    @test = Test.find(params[:id])
  end

  def edit
    @test = Test.find(params[:id])
  end

  # 略
end

before_filterを使う方が良い時

actionに関わらず、コントローラーに必要な処理はbefore_filterで書く。
※ ログイン認証など

なぜならbefore_filterを使わない場合、
各actionに記入漏れをする可能性があるからです。

改めて結論

規則は規則ですが、
バグの起きにくいように書きましょう。

関連記事

Railsのフィルターオプション only と except の使い分け
DRY (Don't Repeat Yoursel) の意味を勘違いしてたかも