質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.58%

Railsで掲示板を作成中。複数カテゴリを付与して、form_withを使い保存したい。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 508

score 7

前提・実現したいこと

初学者です。Ruby on Rails5.2.4.2を使用し、こちらを参考にして掲示板を作成しております。
スレッドのトピックスと、カテゴリを紐づけるために、TopicCategoryRelationという中間テーブルを作成しました。
複数カテゴリが付与可能な状態で、form_withを使用し、コントローラでparamsの制限も記述したのですが、下記エラーで前に進めない状態です。
初歩的な問題かもしれませんが、ご教示いただけると嬉しいです。

発生している問題・エラーメッセージ

ターミナル上でのエラー文

Started POST "/topics/create" for ::1 at 2020-03-29 16:06:13 +0900
Processing by TopicsController#create as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"S1AKhFNNuBbJAUzoxfNLDNYKZ1nR7JSaqU5nCQShD04bvwg+pSivdrl+J5U3Q58Ajp6QP1EqsJv0yksicJvgAg==", "topic"=>{"title"=>"テスト1", "category_ids"=>["", "0", "1"]}, "commit"=>"トピックス作成"}
Completed 400 Bad Request in 1ms (ActiveRecord: 0.0ms)

ActionController::ParameterMissing (param is missing or the value is empty: topics):
app/controllers/topics_controller.rb:18:in `topic_params'
app/controllers/topics_controller.rb:12:in `create'

localhost:3000 ブラウザのエラー文

ActionController::ParameterMissing in TopicsController#create
param is missing or the value is empty: topicswhere line #17 raised:

Extracted source (around line #18):
16          <%= c.label do %>
17    def topic_params
18      params.require(:topics).permit(:title, {:category_ids => []})
19    end
20  end

該当のソースコード

routes.rb

Rails.application.routes.draw do
  root 'topics#index'
  get 'topics/thread/:id' => 'topics#thread', as: :topics_show
  post 'topics/create' => 'topics#create'
end

コントローラ

class TopicsController < ApplicationController
  def index
    @topicsAll = Topic.all
    @topics = Topic.new
    @topics.topic_category_relations.build
  end

  def thread
  end

  def create
    @topics = Topic.create(topic_params)
    @topics.save
    redirect_to ("/topics/thread/{@topics.id}")
  end

  def topic_params
    params.require(:topics).permit(:title, {:category_ids => []})
  end
end

index.html.erb(トップページ)

<h1>トピック一覧</h1>
<ul>
  <% @topicsAll.each do |topic| %>
    <li>
      <%= link_to topic.title, topics_thread_path(topic.id) %>
        <% topic.topic_category_relations.each do |tcr| %>
          <%= tcr.category.category_name %>
        <% end%>
    </li>
  <% end %>
</ul>

<h1>トピック新規登録</h1>
<%= form_with model: @topics, url: {controller: 'topics', action: 'create' }, local: true do |f| %>
  <%= f.label :title, 'トピックスタイトル' %>
  <%= f.text_field :title %>
  <p>
  <%= f.label 'カテゴリ' %>
    <%# 存在するカテゴリの数だけチェックボックスを作成 %>
    <%= f.collection_check_boxes(:category_ids, Category.all, :id, :category_name) do |c| %>
          <%= c.check_box + c.text%>
    <% end %>
  </p>
  <%= f.submit 'トピックス作成'%>
<% end %>

topicモデル

class Topic < ApplicationRecord
    has_many :topic_category_relations
    has_many :categories, through: :topic_category_relations
end

categoryモデル

class Category < ApplicationRecord
    has_many :topic_category_relations
    has_many :topics, through: :topic_category_relations
end

中間テーブルモデル

class TopicCategoryRelation < ApplicationRecord
    belongs_to :topic
    belongs_to :category
end

schema.rb

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

  create_table "topic_category_relations", force: :cascade do |t|
    t.integer "topic_id"
    t.integer "category_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

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

試したこと

・topicテーブルに値を1つ作成
・categoryテーブルに値を3つ作成
・ローカルサーバー再起動
・index.html.erbの下記部分の、{:category_ids => []}をcategory_ids: []という記述に変えましたが、変化なし。

params.require(:topics).permit(:title, {:category_ids => []})

モデルの定義に問題がありそうですが、なかなか見当がつかないので、ご教示いただけると嬉しいです。

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

Ruby: 2.5.0
Rails: 5.2.4.2
OS: MacOS
IDE: VSCode

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

ストロングパラメータのキーを複数形から単数形にすれば解決しませんか。

params.require(:topics)

params.require(:topic)

フォームから送られているキーが topic なのに対し、許可しているキーが topics なので topics キーに値はないよと怒られてるように見えます。

{"topic"=>{"title"=>"テスト1", "category_ids"=>["", "0", "1"]}, "commit"=>"トピックス作成"}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/03/29 19:06 編集

    なんと、、正常に動きました!有難うございます!

    キャンセル

  • 2020/03/29 19:10

    エラーメッセージやログが必要なので別で質問を投稿していただけると助かります〜。

    キャンセル

  • 2020/03/29 19:11

    すみません、自己解決しました!
    ご親切に有難うございます!^^

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.58%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る