【Ruby on Rails】No route matchesエラー

Question

Ruby on Rails初心者です。

editページの登録ボタンクリックでupdateアクションを呼びたいのですが、
No route matchesのエラーが出ます。

単純なミスだと思うのですが、どうしても分からずにいます。。。

分かる方よろしくお願いします。

#エラーメッセージ

#routes.rb

  get '/emp_groups', to: 'emp_groups#index'
  get '/emp_groups/edit', to: 'emp_groups#edit'
  patch '/emp_groups', to: 'emp_groups#update'

#edit.html.erb

				<% if @employee.present? %>
					<%= form_for @employee, url: emp_groups_edit_path(employee_id: @employee.id) do |f| %>
						<% if @has_error %>
							<p class="alert alert-danger">入力内容に不備があります</p>
						<% end %>
						<div class="table-responsive">
							<table id="empgroup_input" class="table">
								<thead>
									<tr>
										<th>部</th>
										<th>課</th>
										<th>グループ</th>
										<th>開始年月日</th>
										<th>行削除</th>
									</tr>
								</thead>
								<tbody>
									<%= f.fields_for :emp_groups do |eg_f| %>
											<tr>
												<td>
													<%= eg_f.collection_select :department_id, Department.all, :id, :dep_name, {include_blank: true }, { class: 'department_select' } %>
												</td>
												<td>
													<%= eg_f.collection_select :section_id, Section.where(department_id: eg_f.object.department_id).all, :id, :sec_name, {include_blank: true }, { class: 'section_select' } %>
												</td>
												<td>
													<%= eg_f.collection_select :group_id, Group.where(section_id: eg_f.object.section_id).all, :id, :grp_name, {include_blank: true }, { class: 'group_select' } %>
												</td>
												<td>
													<%= eg_f.text_field :start_date, class: 'date-picker' %>
												</td>
												<td>
													<button class="btn btn-danger empgroup_destroy">削除</button>
													<%= eg_f.hidden_field :_destroy %>
												</td>
											</tr>
									<% end %>
								</tbody>
							</table>
							<button id="empgroup_add_row" class="btn btn-primary">行追加</button>
						</div>
						<p><%= f.submit "更新", class: "btn btn-primary" %></p>
					<% end %>
				<% end %>
			</div>
		</div>
	</div>

#Controller

class EmpGroupsController < ApplicationController


  def edit
   set_employee
   @employee.emp_groups.build
  end

  def update
  
    if @employee.update(employee_params)
      redirect_to emp_groups_edit_path(employee_id: @employee.id)
    else
      @employee.emp_groups.build
      @has_error = true
      render :edit
    end
  end

  private

    def set_employee
      @employee = Employee.find(params[:id])
    end

    def employee_params
      params.require(:employee).permit(emp_groups_attributes: [:id, :department_id, :section_id, :group_id, :start_date, :_destroy])
    end
end

Answer 1

form_forの引数で、url: emp_groups_edit_pathと固定してあるため、この/emp_groups/editに対して投げられることとなるので、うまく行きません。

メソッドなり直書きなりで、patch '/emp_groups', to: 'emp_groups#update'の方に飛ぶように指定しましょう（メソッドはきちんとPATCHとなっています）。

Comments

[pecchan]

form_forの引数に渡すパスの指定を理解していなかったようです。
有難う御座いました。

[pecchan]

<%= form_for @employee, url: emp_groups_edit_path(employee_id: @employee.id) do |f| %>
を
<%= form_for @employee, url: emp_groups_path(id: @employee.id) do |f| %>
に変更しました。

有難う御座いました。

今からじっくり勉強します。

Answer 2

こんにちは。

実際に動かして確認していませんが、ぱっと見では、

 <%= form_for @employee, url: emp_groups_edit_path(employee_id: @employee.id) do |f| %>

のところが、単に

 <%= form_for @employee do |f| %>

でよいのではないかと思われます。

参考までに、Railsガイド「Action View フォームヘルパー」
https://railsguides.jp/form_helpers.html
から、以下、引用です。

RESTfulなリソースを扱っている場合、レコード識別(record identification)を使用するとform_forの呼び出しがはるかに簡単になります。これは、モデルのインスタンスを渡すだけで、後はRailsがそこからモデル名など必要なものを取り出して処理してくれるというものです。　
## 新しい記事の作成
# 長いバージョン
form_for(@article, url: articles_path)
# 短いバージョン(レコード識別を利用)
form_for(@article)
　 
## 既存の記事の修正
# 長いバージョン
form_for(@article, url: article_path(@article), html: {method: "patch"})
# 短いバージョン
form_for(@article)

あるいは、rails generate scafflod で、何らかのモデルのサンプルアプリを
作っておき、それをかたわらに参照しながら作っていくとよいかもしれません。

参考になれば幸いです。

Comments

[pecchan]

有難う御座います。
仰る通り、
<%= form_for @employee do |f| %>
だけで動きました。

しかしながら何故ちゃんと動作（emp_groupsがないのにちゃんとemp_groupsのコントローラに対して動作する）するのか理解出来ていません・・・orz

リンク先も読みましたが理解出来ませんでした。

頑張ります、有難う御座いました。

[jun68ykt]

私の回答は、pecchanさんの理解をかえって混乱させてしまう回答だったかもしれないと思っています。
なので、BAもらってよかったのかな？と思っていたりします。

まず、私の回答に挙げた Rails ガイドからの引用は適切ではないと思えてきたので、
「emp_groupsがないのにちゃんとemp_groupsのコントローラに対して動作する」ような
formタグが生成される理由の根拠としては、申し訳ありませんが、
その引用は取り下げたいと思います。

では、なぜ、<%= form_for @employee do |f| %> でうまくいくのでは? と思ったかというと、
端的にいうと、カンです。
もう少し詳しくいえば

　　Railsがよしなに解釈して適切な<form>を作ってくれのではないか？

という、いってみれば、「やってみて結果OKならラッキー」的な、「世界のRailさんお願い！」的な
かなりいいかげんなものです。
ですが、パラメータを省いて何かをやったときにRailsがよしなにやってくれることの
すべてを正確に頭に入れるのは無理なので、現場の経験では、こういうことはままあったりします。
(もちろんこのような場当たり的なのは、本当はよくありません。）

[pecchan]

<%= form_for @employee do |f| %>
ですと、所属の登録は出来てましたが、社員一覧へ遷移しました。
意図する動きは所属一覧でした。

そこで申し訳ないですが、BAを変更させていただきました。すいません。

とんでもないです。
回答以外のそういう経験則が一番参考になります。
有難う御座います。

遷移先は、社員一覧なのに所属データが登録出来てて最初は混乱しましたが、
登録出来たのはやはりRailsがアソシエーションを辿ってよろしくやってくれたんでしょうね。

お気遣い有難う御座います。

参考先ページを今夜はゆっくり読み解きます。