A curious piece of Code from http://blog.davidchelimsky.net/wp-content/uploads/2010/11/duplication.pdf

before_filter_assign_widget.rb

class WidgetsController < ApplicationController
  before_filter :assign_widget, :only => [:show, :new, :edit, :create:, :update, :destroy]

  def show; end
  def new; end
  def edit; end

  def create
    if @widget.save
      redirect_to(@widget, :notice => 'Widget was successfully created.')
    else
      render :action => "new"
    end
  end

  def update
    if @widget.update_attributes(params[:widget])
      redirect_to(@widget, :notice => 'Widget was successfully updated.')
    else
      render :action => "edit"
    end
  end

  def destroy
    @widget.destroy redirect_to(widgets_url)
  end

  private

    def assign_widget
      @widget = params[:id] ? Widget.find(params[:id]) : Widget.new(params[:widget])
    end
end

Is the before_filter bad? And why?