redirect_to! = Return

Question

Solo quiero dar una confirmación sobre el comportamiento de redirect_to.

que tienen código que se parece a:

if some_condition 
    redirect_to(path_one) 
end 

redirect_to(path_two) 

Si el some_condition = true me sale error:

Render and/or redirect were called multiple times in this action. Please note that you may only call render OR redirect, and at most once per action.

Parece que el método sigue ejecutando después de la llamada redirect_to. ¿Necesito crear un código como este:

if some_condition 
    redirect_to(path_one) 
    return 
end 

redirect_to(path_two) 

?

Answer 1

Sí, tiene que volver de método cuando se hace redirigir. En realidad, solo agrega encabezados apropiados para el objeto de respuesta.

Puede escribir más forma rubyish:

if some_condition 
    return redirect_to(path_one) 
end 

redirect_to(path_two) 

u otra manera:

return redirect_to(some_condition ? path_one : path_two) 

o de otra manera:

redirect_path = path_one 

if some_condition 
    redirect_path = path_two 
end 

redirect_to redirect_path 

Answer 2

De http://api.rubyonrails.org/classes/ActionController/Base.html:

If you need to redirect on the condition of something, then be sure to add “and return” to halt execution.

def do_something 
    redirect_to(:action => "elsewhere") and return if monkeys.nil? 
    render :action => "overthere" # won't be called if monkeys is nil 
end 

Answer 3

También se puede hacer

redirect_to path_one and return 

que se lee bien.

Answer 4

Vale la pena señalar que hay es sin necesidad de return a menos que tenga cualquier código después de redirect_to, como en este ejemplo:

def show 
    if can?(:show, :poll) 
    redirect_to registrar_poll_url and return 
    elsif can?(:show, Invoice) 
    redirect_to registrar_invoices_url and return 
    end 
end 

Answer 5

para consolidar el "camino rubyish" ejemplo en Eimantas' answer a dos líneas de código :

return redirect_to(path_one) if some_condition 

redirect_to(path_two)