Возвращение оказать в Рубин на рельсы


Разве это плохо, не оказываете? Я делаю это, чтобы не допустить перенаправления.

def create
    @patient = Patient.new(params[:patient])
    if  !@patient.save
      @patient.user.password = ''
      @patient.user.password_confirmation = ''
      return render is_admin? ? 'new_admin' : 'new'
    else
      #Fire off an e-mail
      PatientMailer.welcome_email(@patient).deliver
    end

    if current_user == @patient
      sign_in @patient
    else
      redirect_to patients_path
    end
  end


4182
10
задан 25 февраля 2011 в 06:02 Источник Поделиться
Комментарии
1 ответ

Пишу это, как это выглядит на визуализации возвращает осмысленное значение, которое возвращается по созданию и используется код, вызывающий создать, что это не так. Так что вместо этого я бы написал:

render is_admin? ? 'new_admin' : 'new'
return

Отсюда ясно, что визуализация используется исключительно для его побочные эффекты и создавать не возвращает значений (кроме нуля).

8
ответ дан 25 февраля 2011 в 06:02 Источник Поделиться