18

ユーザーとイベントを含むアプリがあります。各ユーザーにはいくつかのイベントがあります。ユーザーが特定のイベントを見たい場合は、次のアクションを実行します。

def show
  begin
    @userEvents = current_user.event
    @event = @userEvents.find(params[:id])
  rescue ActiveRecord::RecordNotFound  
    redirect_to :controller => "main", :action => "index"
  end

  respond_to do |format|
    format.html # show.html.erb
    format.json { render json: @event }
  end
end

ユーザーのイベントが見つからない場合は、ユーザーがURLで遊んだことを意味し、取得しようとしているイベントはユーザーのものではありません。彼をメインページにリダイレクトするか、イベントが見つからないというエラーのあるページを表示したいと思います。上記のコードを実行しようとすると、このエラーが発生します。

AbstractController::DoubleRenderError in EventsController#show

これを修正する最良の方法は何ですか？

4

3 に答える 3

24

リダイレクト後にリターンを置く

begin
 @userEvents = current_user.event
 @event = @userEvents.find(params[:id])
rescue ActiveRecord::RecordNotFound  
 redirect_to :controller => "main", :action => "index"
 return
end
于 2012-10-01T21:46:36.537 に答える
17

アクションメソッドから呼び出しredirect_toが戻らないため、respond_toブロックに移動すると。が発生しDoubleRenderErrorます。これを修正する1つの方法は、次のとおりです。

redirect_to :controller => "main", :action => "index" and return

ただし、より適切な解決策は、この例外を宣言的に救済するか、単にクライアントに伝播させることです。前者は次のようになります。

class YourController < ActionController::Base

  rescue_from ActiveRecord::RecordNotFound, with: :dude_wheres_my_record

  def show
    # your original code without the begin and rescue
  end

  def dude_where_my_record
    # special handling here
  end
end

例外をそのままにしておくと、ユーザーにはpublic/404.html本番モードのページが表示されます。

于 2012-10-01T21:47:39.313 に答える
5

アプリケーションコントローラに、次のように記述してください。

    rescue_from (ActiveRecord::RecordNotFound) { |exception| handle_exception(exception, 404) }

   protected

    def handle_exception(ex, status)
        render_error(ex, status)
        logger.error ex   
    end

    def render_error(ex, status)
        @status_code = status
        respond_to do |format|
          format.html { render :template => "error", :status => status }
          format.all { render :nothing => true, :status => status }
       end
    end

ページerror.html.erbを作成します

<div class="page-header">
  <h1>
    <%= t "errors.#{@status_code}.heading" %>
    <small><%= t "errors.#{@status_code}.subheading" %></small>
  </h1>
</div>
<p><%= t "errors.#{@status_code}.description" %></p>
<% if defined? root_path %>
  <%= link_to t(:return_to_home), root_path %>
<% end %>

およびen.yml

en:
  errors:
    "404":
      description: "The page you are looking for does not exist!"
      heading: "Record not found"
      subheading: ""
于 2015-02-19T08:26:53.680 に答える