form_withでmodelにうまく値を渡すことができない

Question

前提・実現したいこと

railsでwebサービスを作っております。

ユーザー編集画面で、form_withにおいてmodelに2つの値を渡したいのですが、エラーが出てしまいうまくいきません。

エラーコード、ソースコードは下述いたします。
どこが間違っているか教えていただきたいです。

程度の低い質問でしたら申し訳ございません。
また、質問に必要な情報等に不足があれば、お手数ですがご指摘いただければ、追記いたします。

よろしくお願いいたします。

該当のソースコード

[edit.html.erb]
<% provide(:title, 'edit') %>

<div class="edit-wrapper">
  <h1 class="edit">Edit</h1>
  
  <div class="row signup-forms">
    <div class="col-md-6 col-md-offset-3">
      <%= form_with(model: [@user, @detail], local: true) do |f| %> //該当箇所ここです
        <%= render 'shared/error_messages'%>
      
        <%= f.label :name, "Nickname" %>
        <%= f.text_field :name, class: 'form-control', placeholder: "#{@user.name}" %>
  
        <%= f.label :image %>
        <%= f.file_field :image, class: "form-control image-form" %>
      
        <%= f.label :mother_tongue, "Mother Tongue | 母語" %>
        <%= f.select :mother_tongue, [["English", "English"], ["Japanese", "Japanese"], 
        ["Other","other"]], include_blank: "Please select.", class:"form-control" %>
        
        <%= f.label :japanese_level, "Japanese Level | 日本語のレベル" %>
        <%= f.select :japanese_level, [["Beginner | 初心者", "Beginner | 初心者"], ["Capable of quite basic communication | 簡単な会話ならできる", "Capable of quite basic communication | 簡単な会話ならできる"], 
        ["Capable of daily conversation | 日常会話ができる","Capable of daily conversation | 日常会話ができる"], ["Capable of advanced conversation | 難しい内容の会話ができる","Capable of advanced conversation | 難しい内容の会話ができる"], ["Almost native level | ネイティブとほとんど同レベル","Almost native level | ネイティブとほとんど同レベル"],
        ["Native | ネイティブ","Native | ネイティブ"]], include_blank: "Please select.", class:"form-control" %>
        
        <%= f.label :english_level, "English Level | 英語のレベル" %>
        <%= f.select :english_level, [["Beginner | 初心者", "Beginner | 初心者"], ["Capable of quite basic communication | 簡単な会話ならできる", "Capable of quite basic communication | 簡単な会話ならできる"], 
        ["Capable of daily conversation | 日常会話ができる","Capable of daily conversation | 日常会話ができる"], ["Capable of advanced conversation | 難しい内容の会話ができる","Capable of advanced conversation | 難しい内容の会話ができる"], ["Almost native level | ネイティブとほとんど同レベル","Almost native level | ネイティブとほとんど同レベル"],
        ["Native | ネイティブ","Native | ネイティブ"]], include_blank: "Please select.", class:"form-control" %>
        
        <%= f.label :gender, "Gender(optional) | ジェンダー（任意）" %>
        <%= f.select :gender, [["he/him | 男性", "he/him | 男性"], ["she/her | 女性", "she/her | 女性"], 
        ["they/them | その他","they/them | その他"], ["I don't want to tell | 言いたくない","I don't want to tell | 言いたくない"]], include_blank: "Please select.", class:"form-control" %>
        
        <%= f.label :region, "Region  | 地域" %>
        <%= f.select :region, [["Asia | アジア", "Asia | アジア"], ["Europe ｜ヨーロッパ", "Europe | ヨーロッパ"], 
        ["Africa | アフリカ","Africa | アフリカ"], ["North/Central America | 北/中央アメリカ","North/Central America | 北/中央アメリカ"], ["South America | 南アメリカ","South America | 南アメリカ"],
        ["Oceania | オセアニア","Oceania | オセアニア"], ["Other | その他","Other | その他"]], include_blank: "Please select.", class:"form-control" %>
        
        <%= f.label :purpose, "Purpose of Learning | 学習目的" %>
        <%= f.select :purpose, [["Business | 仕事", "Business | 仕事"], ["School Classes | 学校の授業", "School Classes | 学校の授業"], 
        ["Entrance Exam | 受験","Entrance Exam | 受験"], ["Trip | 旅行","Trip | 旅行"], ["Want to talk with Foreingers | 外国の人と話したい","Want to talk with Foreingers | 外国の人と話したい"], 
        ["Qualifying Exam | 資格試験","Qualifying Exam | 資格試験"], ["Hobby | 趣味","Hobby | 趣味"], ["Other | その他", "Other | その他"]], include_blank: "Please select.", class:"form-control" %>
        
        <%= f.label :self_introduction, "Self Introduction | 自己紹介" %>
        <%= f.text_area :self_introduction, class: 'form-control self-intro', placeholder: "It is recommended that you write in simple English. 英語で書くことをお勧めします。簡単な文でも問題ありません。" %>
        
        <%= f.label :sns_1, "URL of Your SNS 1 (optional) | SNSリンク1（任意）" %>
        <%= f.text_field :sns_1, class: 'form-control' %>
        
        <%= f.label :sns_2, "URL of Your SNS 2 (optional) | SNSリンク2（任意）" %>
        <%= f.text_field :sns_2, class: 'form-control' %>
        
        <%= f.label :sns_3, "URL of Your SNS 3 (optional) | SNSリンク3（任意）" %>
        <%= f.text_field :sns_3, class: 'form-control' %>
        
        
  
        <%= f.submit "Update Your Information", class: "btn btn-primary create" %>
      <% end %>
      
      <%= link_to "Change Email", "/users/#{@user.id}/edit_email", class: "btn btn-light btn-lg" %>
      <%= link_to "Change Password", "/users/#{@user.id}/edit_email", class: "btn btn-secondary btn-lg" %>
    </div>
  </div>
</div>

[users_controller.rb]
class UsersController < ApplicationController
  
  def show
    @user = User.find(params[:id])
  end
  
  def new
    @user = User.new
  end
  
  def signup_jp
    @user = User.new
  end
  
  def create
    @user = User.new(user_params)
    if @user.save
      log_in @user
      flash[:success] = "Welcome to INTE! Your account has been creaeted."
      redirect_to details_path
    else
      flash[:danger] = "Invalid user information. Confirm that you filled all boxes except optional."
      render 'new'
    end
  end
  
  def details
  end
  
  def details_create
    @detail = Detail.new(detail_params)
    if @detail.save
      redirect_to "/users/#{current_user.id}"
    else
      render 'details'
    end
  end
  
  def edit
    @user = User.find(params[:id])
    @detail = Detail.find_by(user_id: @user.id )
  end
  
  def update //該当箇所ここです
    @user = User.find(params[:id])
    @detail = Detail.find_by(user_id: @user.id)
    if @user.update(user_params) || @detail.update(detail_params)
      redirect_to @user
    else
      render 'edit'
    end
  end
    
  
  def edit_email
    @user = User.find(params[:id])
  end

  
  def edit_password
    @user = User.find(params[:id])
  end
  

  
  
  
  private
  
    def user_params
      params.require(:user).permit(:name, :email, :image, :password, :password_confirmation)
    end
    
    def detail_params
      params.permit(:authenticity_token, :user_id, :mother_tongue, :japanese_level, :english_level,
                                     :gender, :region, :purpose, :self_introduction, :sns_1, :sns_2, :sns_3)
    end
    
end

[routes.rb]
Rails.application.routes.draw do
  
  get 'sessions/new'
  root 'static_pages#language'
  get '/home_jp', to:'static_pages#home_jp'
  get '/home_en', to:'static_pages#home_en'
  get '/signup', to:'users#new'
  get '/signup_jp', to:'users#signup_jp'
  post '/signup_jp', to:'users#create'
  get '/details', to:'users#details'
  post '/details', to:'users#details_create'
  get '/login', to:'sessions#new'
  post '/login', to:'sessions#create'
  delete '/logout', to:'sessions#destroy'
  get '/users/:id/edit_email', to:'users#edit_email'
  patch '/users/:id/edit_email', to:'users#update'
  get '/users/:id/edit_password', to:'users#edit_password'
  patch '/users/:id/edit_password', to:'users#update'
  # For details on the DSL available within this file, see https://guides.rubyonrails.org/routing.html
  resources :users
end

###エラーコード

NoMethodError in Users#edit
Showing /home/ubuntu/environment/INTE/app/views/users/edit.html.erb where line #8 raised:

undefined method `user_detail_path' for #<#<Class:0x00007f31b28afa80>:0x00007f31b225baf8>
Did you mean?  user_path

Extracted source (around line #8):
              
  <div class="row signup-forms">
    <div class="col-md-6 col-md-offset-3">
      <%= form_with(model: [@user, @detail], local: true) do |f| %> //ここが赤く表示されています
        <%= render 'shared/error_messages'%>
      
        <%= f.label :name, "Nickname" %>

補足情報（FW/ツールのバージョンなど）

Rails 6.0.3
Ruby 2.6.3
AWS Cloud9を使用

Comments

[winterboum]

modelのcodeも。
特に関連の定義部分

Answer 1

Rails wayに従っていないと、Rails wayの恩恵は得られません。

routes.rbがリソースベースのルーティングでない事が問題です。
そのため、form_with model: [@user, @detail]にてどのURLに送りつければいいのかRailsは判断できません。

• urlを明示する
• routes.rbに細工をする

どちらかが必要になります。

urlを明示する場合

<%= form_with model: [@user, @detail], url: '/details', local: true do |f| %>

とすることになります。

Comments

[takuma1229]

回答ありがとうございます。

<%= form_with model: [@user, @detail], url: '/details', local: true do |f| %>
やってみたのですが、No route matches [PATCH] "/details"のエラーが出てしまいました。

その後[routes.rbにて]
patch '/details', to:'users#detail' とする
=>detail作成画面に飛ばされてしまう
patch '/details', to:'users#update' とする
=>Couldn't find User without an ID(@user = User.find(params[:id])の部分が赤く表示)

といった感じでした...

[asm]

なるほど。
とりあえず動かすだけならば
patch '/details', to:'users#details_create'
です。

用語の問題としてcreate(post)は新規作成で、更新はupdate(patch)を使うのが約束になっています。

[takuma1229]

patch '/details', to:'users#details_create'
にしても、detail作成画面に飛ばされてしまい、うまく動かないんですよね。。。
一応editで入力した情報は送られているようなのですが、saveまではされていないのでデータベースは更新されず,,,

お手数おかけしてしまい申し訳無いです。

[asm]

大改修が必要ですね。。。
そもそも、form_withの送信先がusers#details_createなのかusers#updateなのかが不明
detail_paramsにrequire(:detail)がないのはなぜなのかが不明
です。

[takuma1229]

form_withの送信先はusers#updateにしたいと思っています！

detail_paramsにrequire(:detail)をつけてpatchもto: 'users#update'にしてみましたが、Couldn't find User without an IDのエラーがやはり出てしまいました。。。

[asm]

> @user = User.find(params[:id])
params[:id]というのはユーザー側が弄ったパラメータを送る事が可能なので認証が必要なものには向いていません。
現在ログインしているユーザーを編集する場合はcurrent_user的なものを使う必要があります。