ストロングパラメーターの中身に指定したものを全て取得したい

Question

解決したいこと

ストロングパラメーターの中身に指定したものを全て取得したい

##現状
目標管理アプリを製作中です。(利用環境は、「rails 5.2.3」「ruby 2.5.1」)
まず、新規作成ページから作成しました。このページで入力した値をコントローラーで取得しようとストロングパラメーターを設定したのですが、作成を実行すると、以下のエラーが発生しcreateアクションが正しく実行しませんでした。
「undefined local variable or method `target' for #TargetsController:0x00007f94aa1117c8 Did you mean? target_url」

ストロングパラメーター定義部分がエラー該当箇所なのですがどこかおかしな点はありますか？
ちなみにform_withを使っています。
データベース(モデル)はTargetモデル(目標の情報を入っている)、Categoryモデル(カテゴリの情報が入っている)、Priorityモデル(優先度の情報が入っている)の３つがあります。
※関係あるかわかりませんが、Bootstrapも使用してます。

エラー全文

NameError in TargetsController#create
undefined local variable or method `target' for #<TargetsController:0x00007f94aa1117c8> Did you mean? target_url
Extracted source (around line #42):
40
41
42
43
44
45
              
  private
  def target_params
    params.require(:target).permit(:title, :price, :date, :memo, :created_at).merge(category_id: target.category.id,priority: target.priority.id)
  end

  def move_to_index

Rails.root: /Users/ren0826nosuke/Desktop/projects/target-app

Application Trace | Framework Trace | Full Trace
app/controllers/targets_controller.rb:42:in `target_params'
app/controllers/targets_controller.rb:20:in `create'
Request
Parameters:

{"utf8"=>"✓",
 "authenticity_token"=>"F5MW8QuNuGxrebg66IJL2TkYLX233VnhbnpqO1cvrFH05j6rkv8bN+GmxtrEJBWAKtIzHbtyzSVkACcDi6NBng==",
 "target"=>
  {"title"=>"aaaaa",
   "category_id"=>"1",
   "price"=>"100",
   "date(1i)"=>"2020",
   "date(2i)"=>"4",
   "date(3i)"=>"14",
   "memo"=>"",
   "priority"=>"1",
   "created_at(1i)"=>"2020",
   "created_at(2i)"=>"4",
   "created_at(3i)"=>"14"},
 "commit"=>"作成"}
Toggle session dump
Toggle env dump
Response
Headers:

None

新たに出たエラー全文

NoMethodError in Targets#create
Showing /Users/ren0826nosuke/Desktop/projects/target-app/app/views/targets/new.html.erb where line #10 raised:

undefined method `map' for nil:NilClass
Extracted source (around line #10):
8
9
10
11
12
13
              
    <div class="form-group">
      <%= form.label :category_id, "カテゴリ" %><br>
      <%= form.select :category_id, @category.map{|category|[category.name,category.id]} %>
　　　　　#上記selectの部分がエラー該当箇所です。
    </div>

    <div class="form-group">

Rails.root: /Users/ren0826nosuke/Desktop/projects/target-app

Application Trace | Framework Trace | Full Trace
app/views/targets/new.html.erb:10:in `block in _app_views_targets_new_html_erb__2269560302185157887_70138228332500'
app/views/targets/new.html.erb:2:in `_app_views_targets_new_html_erb__2269560302185157887_70138228332500'
app/controllers/targets_controller.rb:24:in `create'
Request
Parameters:

{"utf8"=>"✓",
 "authenticity_token"=>"F5MW8QuNuGxrebg66IJL2TkYLX233VnhbnpqO1cvrFH05j6rkv8bN+GmxtrEJBWAKtIzHbtyzSVkACcDi6NBng==",
 "target"=>
  {"title"=>"aaaaa",
   "category_id"=>"1",
   "price"=>"100",
   "date(1i)"=>"2020",
   "date(2i)"=>"4",
   "date(3i)"=>"14",
   "memo"=>"",
   "priority"=>"1",
   "created_at(1i)"=>"2020",
   "created_at(2i)"=>"4",
   "created_at(3i)"=>"14"},
 "commit"=>"作成"}
Toggle session dump
Toggle env dump
Response
Headers:

None

試してみた事

createアクションにbinding.pryでparamsの中身を見てみると以下の通りでした。
:category_id, :priorityが許可されていないと出ていますがなぜなのかはわかっていません。。

[1] pry(#<TargetsController>)> params
=> <ActionController::Parameters {"utf8"=>"✓", "authenticity_token"=>"F5MW8QuNuGxrebg66IJL2TkYLX233VnhbnpqO1cvrFH05j6rkv8bN+GmxtrEJBWAKtIzHbtyzSVkACcDi6NBng==", "target"=>{"title"=>"aaaaa", "category_id"=>"1", "price"=>"100", "date(1i)"=>"2020", "date(2i)"=>"4", "date(3i)"=>"14", "memo"=>"", "priority"=>"1", "created_at(1i)"=>"2020", "created_at(2i)"=>"4", "created_at(3i)"=>"14"}, "commit"=>"作成", "controller"=>"targets", "action"=>"create"} permitted: false>
[2] pry(#<TargetsController>)> target_params
Unpermitted parameters: :category_id, :priority
NameError: undefined local variable or method `target' for #<TargetsController:0x00007f94aa1117c8>
Did you mean?  target_url
from /Users/ren0826nosuke/Desktop/projects/target-app/app/controllers/targets_controller.rb:42:in `target_params'

記述コード

コントローラーの詳細は以下の通りです。

targets_controller.rb

class TargetsController < ApplicationController
  # before_action :move_to_index, only: [:new]

  def index

  end

  def new
    @target=Target.new
    @category=Category.all
    @priority=Priority.all
  end

  def show

  end

  def create
    binding.pry
    @target = Target.create(target_params)
    if @target.save
      redirect_to "/"
    else
      render :new
    end
  end

  def edit

  end

  def update

  end

  def delete

  end

  private
  def target_params
    params.require(:target).permit(:title, :price, :date, :memo, :created_at).merge(category_id: target.category.id,priority: target.priority.id)
  # 上記がエラー該当箇所です
  end

  def move_to_index
    redirect_to "/"
  end
end

新規作成ページの詳細は以下の通りです。

new.html.erb

<div class="container">
  <%= form_with model: @target, local:true do |form| %>
    <div class="form-group">
      <%= form.label :title , "タイトル" %>
      <%=form.text_field :title ,class:"form-control",placeholder:"タイトル" %>
    </div>

    <div class="form-group">
      <%= form.label :category_id, "カテゴリ" %><br>
      <%= form.select :category_id, @category.map{|category|[category.name,category.id]} %>
    </div>

    <div class="form-group">
      <%= form.label :price , "予算・価格" %>
      <%=form.text_field :price ,class:"form-control", placeholder:"¥" %>
    </div>

    <div class="form-group">
      <%= form.label :date , "期日" %><br>
      <%=form.date_select(
                  :date,
                  use_month_numbers: true,
                  start_year:        1998,
                  end_year:          (Time.now.year + 100),
                  class:"form-control"
      )%>
    </div>

    <div class="form-group">
      <%= form.label :memo , "メモ" %>
      <%=form.text_area :memo ,class:"form-control",placeholder:"メモ" %>
    </div>

    <div class="form-group">
      <%= form.label :priority, "優先度" %><br>
      <%= form.select :priority, @priority.map{|priority|[priority.name,priority.id]} %>
    </div>

    <div class="form-group">
      <%= form.label :created_at , "作成日" %><br>
      <%=form.date_select(
                  :created_at,
                  use_month_numbers: true,
                  start_year:        1998,
                  end_year:          (Time.now.year + 100),
                  class:"form-control"
      )%>
    </div>

    <%=form.submit "作成" ,class:"btn btn-primary btn-lg active"%>

  <% end %>
</div>

モデルの詳細は以下の通りです。

category.rb

class Category < ApplicationRecord
  has_many :targets
end

priority.rb

class Priority < ApplicationRecord
  has_many :targets
end

targets.rb

class Target < ApplicationRecord
  belongs_to :category
  belongs_to :priority

  validates :title, presence: true
  validates :price, numericality: { only_integer: true },presence: true
end

一応schema.rbも載せておきます。

schema.rb

# This file is auto-generated from the current state of the database. Instead
# of editing this file, please use the migrations feature of Active Record to
# incrementally modify your database, and then regenerate this schema definition.
#
# Note that this schema.rb definition is the authoritative source for your
# database schema. If you need to create the application database on another
# system, you should be using db:schema:load, not running all the migrations
# from scratch. The latter is a flawed and unsustainable approach (the more migrations
# you'll amass, the slower it'll run and the greater likelihood for issues).
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema.define(version: 2020_04_14_040458) do

  create_table "categories", force: :cascade do |t|
    t.string "name"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  create_table "priorities", force: :cascade do |t|
    t.string "name"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  create_table "targets", force: :cascade do |t|
    t.string "title"
    t.text "memo"
    t.date "date"
    t.integer "category_id"
    t.integer "priority_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer "price"
  end

end

Comments

[winterboum]

エラーは抜粋ではなく、全体を載せてください

[ren0826nosuke]

すみません。
エラー文追記しました！

Answer 1

キツキツで書いてある部分を半角スペースで開けてみると解決しました。
それだけのために何時間も悩んでいるのがバカみたいで下がいい勉強になりました！

Answer 2

params.require(:target).permit(:title, :price, :date, :memo, :created_at).merge(category_id: target.category.id,priority: target.priority.id)
に使われている target　が未定義なためです。
category_id も priority_id も params で渡ってきますからmergeする必要はないです。

params.require(:target).permit(:title, :price, :date, :memo, :category_id,priority_id) とすればOK

ただし、viewにある select :priority は :priority_id にする必要があります

Comments

[ren0826nosuke]

おかげさまでストロングパラメーターの定義部分のエラーは解決しました！ありがとうございます！
ですが、new.html.erbで新たにエラーが出てしまったのでエラー文を追記しました。確認のほどよろしくお願いします。