Arbeit mit Rettung in Schienen

Question

Ich arbeite mit dem folgenden Stück;

def index 
    @user = User.find(params[:id]) 
    rescue 
    flash[:notice] = "ERROR" 
    redirect_to(:action => 'index') 
    else 
    flash[:notice] = "OK" 
    redirect_to(:action => 'index') 
end 

Nun ich jedem Fall, ob ich eine korrekte ID haben oder nicht, ich bin immer auf „OK“ meiner Meinung nach bekommen, was mache ich falsch?

Ich brauche das, wenn ich keine ID in der DB habe, um "ERROR" anzuzeigen. Ich habe auch versucht, rescue ActiveRecord::RecordNotFound zu verwenden, aber das gleiche passiert.

Alle Hilfe ist willkommen.

Answer 1

Der gesamte Code nach dem Ende des Rettungsblocks wird nur interpretiert, wenn im Rettungsblock keine Rückkehr erfolgt. Sie können also am Ende Ihres Rettungsblocks zurückrufen.

def index 
    begin 
    @user = User.find(params[:id]) 
    rescue 
    flash[:notice] = "ERROR" 
    redirect_to(:action => 'index') 
    return 
    end 
    flash[:notice] = "OK" 
    redirect_to(:action => 'index') 
end 

oder

def index 
    @user = User.find(params[:id]) 
    # after is interpret only if no exception before 
    flash[:notice] = "OK" 
    redirect_to(:action => 'index') 
rescue 
    flash[:notice] = "ERROR" 
    redirect_to(:action => 'index') 
end 

Aber in Ihrem Fall ist die bessere rescue_from oder rescue_in_public

wie

class UserController < ApplicationController 
    def rescue_in_public(exception) 
    flash[:notice] = "ERROR" 
    redirect_to(:action => 'index') 
    end 

    def index 
    @user = User.find(params[:id]) 
    flash[:notice] = "OK" 
    redirect_to(:action => 'index') 
    end 
end 

Aber die Verwendung von rescue_in_public zu verwenden, ist nicht wirklich ein guter Rat

Answer 2

Wenn kein user mit diesem id vorhanden ist, wird User.findnil zurückgeben. Die Rückgabe nil ist kein Fehler und löst keinen rescue aus.

Answer 3

Nur eine Gesamt Rails Rettungs Antwort:

Ich fand das sehr cool sein:

@user = User.find(params[:id]) rescue ""