newアクションのバリデーションエラー後のURLについて

Question

お世話になります。

newアクションで新規登録画面を開いて、必要項目を入力し、[登録する]ボタンを押したとします。
そこでバリデーションエラーが発生し、新規登録画面に戻るのですが、URLはcreateになってしまいます。
URLがcreateのまま[F5]ボタン等で更新した場合、Routing Errorになってしまします。

バリデーションエラーでnew画面に戻ったらURLもnewに戻す方法をご教授いただきたいです。

例
http://localhost:3000/○○○○○/new
↓　画面項目入力後[登録する]ボタン押下
バリデーションエラー発生
↓
http://localhost:3000/○○○○○/create (new画面だけどURLはcreate)

以下設定です。

#controller
  def new
    @schcols = Schcol.new
    @schmaster_schname = params[:sch_name]
    @schmaster_schid = params[:sch_id]
  end
  def create
    @schcols = Schcol.new(schcols_params)
    if @schcols.save
      redirect_to schcols_path, notice: "#{params[:sch_name]}を登録しました。"
    else
      @schmaster_schid = params[:sch_id]
      @schmaster_schname = params[:sch_name]
      render :new
    end
  end

#model
  validates :sch_id, :sch_year, :classification, :item_id, :sch_col1,  presence: true
  validates :sch_year, numericality: true
  validates :sch_id, uniqueness: { scope: [:sch_year, :classification, :item_id, :sch_col1],
             message: I18n.t('errors.messages.key_duplicated_schcol')}

動作環境
ruby 2.5.3p105
Rails 5.2.3

説明がわかりにくいかもしれませんが、宜しくお願い致します。

Comments

[maisumakun]

routingはどのようになっていますでしょうか（resourcesを使った場合、URLに「/create」という文字は入りません）。

[magellan]

maisumakun様
ありがとうございます。
routesの設定は以下になります。
get '/schcols/:sch_id/:sch_name/new', to: 'schcols#new', as: 'schcols_new'
post '/schcols/:sch_id/:sch_name/create', to: 'schcols#create', as: 'schcols_create'

Answer 1

get '/schcols/:sch_id/:sch_name', to: 'schcols#new', as: 'schcols_new'
post '/schcols/:sch_id/:sch_name', to: 'schcols#create', as: 'schcols_create'

ではだめですか？

Comments

[magellan]

winterboum様
返答ありがとうございます。
その方法ですと、他の場所に影響がでてしまい、うまくいきませんでした。

今回、初めてRuby on Railsで開発をしています。
開発を始めて最初の方に作った画面を見直していて修正していたのですが、
おそらくRuby on Railsの本来の開発技法とは違う作り方をしてるのかなと、
開発を進めているうちに思っていました。
今回の場合もsch_idがわかれば、SQLを投げてsch_nameを取得できるのですが、
SQLを何度も投げるのを嫌い、sch_nameを渡すという方法を採用していました。
どちらの方法が「Rails的」なのかわかりませんが、リロードできないというのはおそらく
作りに問題があったのだと思います。
もう一度見直してできる限り綺麗な形で書き直してみます。
winterboum様、並びに返答を頂きましたbamboo-nova様、maisumakun様、貴重なお時間を
さいていただきましてありがとうございました。
また機会がありましたら宜しくお願い致します。

Answer 2

render :new

ではなく、こういうことがしたいということでしょうか？

redirect_back(fallback_location: root_path)

質問の意図を履き違えていたらごめんなさい汗

Comments

[magellan]

bamboo-nova様
ありがとうございます。
ルートに戻りたいわけではなく、バリデーションエラーを出した上でURLがnewになるようにしたいのです。
現状はcreateになってしまうので…。
更新やブラウザの戻るを使用しなければ問題ないのですが…。