Strong Parametersがうまく使えなくて困っております。

Question

Railsにて勤怠管理アプリでパラメータの中身がnilになってしまう？

※Markdown形式が慣れていないので、うまく書けていなかったらすいません 汗

##環境
AWS cloud9
Rails 5.1.7
ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-linux]

##現状で困っていること
拠点ページというページ内で、拠点情報の新規追加ができない。
なぜか、コンソールからは新規作成が正常にできる。

##解決したいこと
Strong Parametersを使って拠点ページにデータを作成ができるように
実装したい。

#現状のコード以下を記載↓
index.html.erb
bases_controller.rb
_new_form.html.erb
routes.rb
schema.rb

###ここからはコードを記載

<% provide(:title, 'Bases') %>
<% provide(:class_text, 'base') %>
<% provide(:button_text, '作成する') %>
<h1> 拠点一覧 </h1>

<div class="col-md-10 col-md-offset-1">
  <table class="table table-striped table-bordered table-condensed table-hover" id="table-bases">
    <thead>
      <tr>
        <th></th>
        <th></th>
        <th class="center">拠点番号</th>
        <th class="center">拠点名</th>
        <th class="center">勤怠種類</th>
      </tr>
    </thead>
    
    <tbody>
        <% @bases.each do |base| %>
          <tr>
            <td>
              <button type="button" data-toggle="collapse" data-target="#<%= base.base_number %>" class="btn btn-primary">修正</button>
            </td>
            <td>
              <%= link_to "削除", base, method: :delete,
                  data: { confirm: "削除してよろしいですか？" }, class: "btn btn-primary" %>
            </td>
            <td class="center"><%= base.base_number %></td>
            <td class="center"><%= base.base_name %></td>
            <td class="center"><%= base.information %></td>
          </tr>
          <tr>
            <td>
              <div id="<%= base.base_number %>" class="collapse">
                <div class="panel-body">
                  <%= render 'edit_form' %>
                </div>
              </div>
            </td>
          </tr>
        <% end %>
    </tbody>
  </table>
    
  <tr>
    <td>
      <div class="panel panel-default">
        <div class="panel-heading">
          <button type="button" data-toggle="collapse" data-target="#add_base" class="btn btn-lg btn-primary">拠点情報追加</button>
        </div>
        <div id="add_base" class="collapse">
          <div class="panel-body">
            <%= render 'new_form' %>
          </div>
        </div>
      </div>
    </td>
  </tr>
</div>

class BasesController < ApplicationController
  before_action :admin_user, only: [:new, :create, :edit, :update, :index, :show, :destroy]
  before_action :set_base, only: [:edit, :update, :destroy]
  

  def new
    @base = Base.new
  end
  
  def create
    @base = Base.new(base_params)
    if @base.save
      flash[:success] = '拠点情報を作成しました。'  
      redirect_to bases_url
    else
      flash[:danger] = "拠点情報の作成に失敗しました。" + @base.errors.full_messages.join("<br>")
      redirect_to bases_url
    end
  end
  
  def edit
  end

  def update
    if @base.update_attributes(base_params)
      flash[:success] = "拠点情報を更新しました。"
      redirect_to bases_url
    else
      flash[:danger] = "拠点情報の更新に失敗しました。" + @base.errors.full_messages.join("<br>")
      redirect_to bases_url
    end
  end
  
  def index
    @bases = Base.all
  end
  
  def show
  end
  
  def destroy
    if @base.destroy
      flash[:success] = "拠点情報を消去しました。"
      redirect_to bases_url
    else
    end
  end
  
  private
  
    def set_base
      @base = Base.find(params[:id])
    end
    
    def base_params
      params.require(:base).permit(:base_number, :base_name, :information)
    end
end

<%= form_with(model: @base, local: true) do |f| %>
  <!--<!%= render 'shared/error_messages', object: @base %>-->

  <%= f.label :base_number, class: "label-#{yield(:class_text)}" %>
  <%= f.number_field :base_number, class: "form-control" %>

  <%= f.label :base_name, class: "label-#{yield(:class_text)}" %>
  <%= f.text_field :base_name, class: "form-control" %>

  <%= f.label :information, class: "label-#{yield(:class_text)}" %>
  <%= f.text_field :information, :placeholder => "出勤または退勤を入力してください。", class: "form-control" %>

  <%= f.submit yield(:button_text), class: "btn btn-primary btn-block btn-#{yield(:class_text)}" %>
<% end %>

Rails.application.routes.draw do

  root 'static_pages#top'
  get '/signup', to: 'users#new'
  
  # ログイン機能
  get    '/login', to: 'sessions#new'
  post   '/login', to: 'sessions#create'
  delete '/logout', to: 'sessions#destroy'

  resources :bases
  resources :users do
    collection { post :import }
    member do
      get 'edit_basic_info'
      patch 'update_basic_info'
      get 'attendances/edit_one_month'
      patch 'attendances/update_one_month'
    end
    resources :attendances, only: :update
  end
end

ActiveRecord::Schema.define(version: 20210130115200) do

  create_table "attendances", force: :cascade do |t|
    t.date "worked_on"
    t.datetime "started_at"
    t.datetime "finished_at"
    t.string "note"
    t.integer "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["user_id"], name: "index_attendances_on_user_id"
  end

  create_table "bases", force: :cascade do |t|
    t.integer "base_number"
    t.string "base_name"
    t.text "information"
    t.integer "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["user_id"], name: "index_bases_on_user_id"
  end

  create_table "users", force: :cascade do |t|
    t.string "name"
    t.string "email"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.string "password_digest"
    t.string "remember_digest"
    t.boolean "admin", default: false
    t.string "affiliation"
    t.datetime "basic_time", default: "2021-01-29 23:00:00"
    t.datetime "work_time", default: "2021-01-29 22:30:00"
    t.time "designated_work_start_time", default: "2000-01-01 00:00:00"
    t.time "designated_work_end_time", default: "2000-01-01 09:00:00"
    t.string "employee_number"
    t.string "uid"
    t.boolean "superior", default: false
    t.index ["email"], name: "index_users_on_email", unique: true
  end

end

##作成ボタンを押した後の症状について
写真のように作成ボタンを押すと、paramの中が無いか、空だよと返ってきます。
parametersには値が入っていると思うんですが、
なぜかこのようなエラーが起きます。
何が悪さをしているのか？または、私自身のパラメータの渡し方がおかしかったのか？

解決方法が全然検討がつかず、意気消沈しております。
どなたか解決に向けての道しるべを指して頂けると
大変、ありがたいです！！
何卒、よろしくお願いいたします。

Comments

[winterboum]

直接原因は明確です
２つめの画像のParamatersを見ると　"base" => { "base_name" => .... } の様になっていないため、params[:base] がnilになってしまってます。
form_with model: @base ならそうなってくれるはzなのに。なぜ？？
というところがわからない。

[yoshi544]

ご返信ありがとうございます！
言われてみれば、確かに。。。

いきなりbaseの下のネストされた値を(base_name: ....)
更新しようとしているみたいなニュアンスでよろしいでしょうか？
なんでこうなってしまうのでしょうか？
form_withの書き方がおかしいのでしょうか？
それともほかに原因があるのでしょうか？
requireが効いてない？ということなんでしょうか？

[winterboum]

そこが分かってれば、回答に書いてます。。。
わからんのです。不思議。

Answer 1

def create
    @base = Base.new(base_params)
end

@base = Base.create(base_params) にしたらどうなるでしょうか。

ActionController::ParameterMissing
というエラーメッセージはわかりにくいですね。
フォームパラメータが無いと怒られている気がしますが、きっと関数の引数がEmptyだと言う意味だと思います。@baseが無いのに params.require(:base) # @base依存 を呼んだということでは無いかと思います。

追記
Base.newはRubyのクラスインスタンスを作成します。（言語レベル）
Base.createはBaseクラスのシングルトンメソッドです。（フレームワークレベル）
その中でごちょごちょやってから、クラスインスタンスを作って返してくれる関数です。Railsが追加した機能です。

蛇足
Baseのように言語やフレームワークが使いそうなクラス名（データベーステーブル名）は避けて、支店なら BranchOfficeとか、配送拠点なら DistributionSateliteとか被る可能性の少ない名前にしたほうが安全でわかりやすいと思います。

Comments

[yoshi544]

仕事が立て込んでしまい、返信遅くなりました！
値が渡せていないだけで、記述はおかしくないというのが
やはり、皆様のご見解なのかと感じました。
蛇足で記載していただいた様に、クラス名の記載については
自分でもこれからは、気をつけていこうと思います。
まずは、ちょっと思いついたことがあるので、コメントいただいた事を
ヒントに実装してみます。