[Rails6]フォームを送れるようにしたい

Question

######※何か情報が足りないなどあれば申し付けください
##実現したいこと
現在railsでQAサイトを作っています
質問投稿画面でタイトルと内容を空にして投稿すると以下のようなエラーになります。

空で登録するを押すと

こうなります。
これは@tagsがnilということでしょうか？
ちなみにテーブル構造はquestionモデルとtagモデルが多対多の構造になっています。

この問題は質問投稿のコントローラのquestionモデルの方でvalidationを設定すると起きるようになります。
逆に言えばvalidationを外せば投稿できます。

##関連しそうなソースコード
####app/models/question.rb

class Question < ApplicationRecord
  belongs_to :user
  has_many :answers, dependent: :destroy
  has_many :question_tags, dependent: :destroy
  has_many :tags , through: :question_tags
  accepts_nested_attributes_for :question_tags, :allow_destroy => true
  validates :title, presence: true
  validates :body, presence: true
end

####app/models/question_tag.rb(中間テーブル)

class QuestionTag < ApplicationRecord
  belongs_to :tag 
  belongs_to :question
end

####app/models/tag.rb

class Tag < ApplicationRecord
  has_many :question_tags, dependent: :destroy
  has_many :questions , through: :question_tags
end

#####app/views/questions/new.html.erb

<h1>質問入力</h1>
<% if question.errors.any? %>
  <div id="error_explanation">
    <h2><%= pluralize(question.errors.count, "error") %> prohibited this question from being saved:</h2>

    <ul>
      <% question.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
    </ul>
  </div>
 <% end %>
 
<%= form_with(model: question, local: true) do |form| %>

  <div class="field" id="title">
    <h6>タイトル</h6>
    <%= form.text_field :title %>
  </div>

  <div class="field" id="body">
    <h6>質問内容</h6>
    <%= form.text_field :body %>
  </div>
  
  
  <div class="field" id="checkbox">
    <% @tags.each do |t| %>　<-- ここでエラーになります。
      <%= form.label t.name %>
      <%= check_box_tag "question[tag_ids][]", t.id, @question.tags.include?(t) %>
      <br/>
    <% end %>
  </div>
  
  <div class="actions" id="submit">
    <%= form.submit %>
  </div>
<% end %>

####app/controllers/questions_controller.rb

class QuestionsController < ApplicationController
  before_action :set_question, only: %i[ show edit update destroy ]
 
  
  # GET /questions or /questions.json
  def index
    #@questions = current_user.questions.all
    @q = Question.ransack(params[:q])
    @questions = @q.result(distinct: true)
  end

  # GET /questions/1 or /questions/1.json
  def show
  end

  # GET /questions/new
  def new
    @tags = Tag.all <--全てのタグを取ってくる
    @question = Question.new
  end

# GET /questions/1/edit
  def edit
    @tags = Tag.all
  end

  # POST /questions or /questions.json
  def create
    @question = current_user.questions.build(question_params)
    respond_to do |format|
      if @question.save
        format.html { redirect_to @question, notice: "質問を投稿しました" }
        format.json { render :show, status: :created, location: @question }
      else
        format.html { render :new, status: :unprocessable_entity }
        format.json { render json: @question.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /questions/1 or /questions/1.json
  def update
    respond_to do |format|
      if @question.update(question_params)
        format.html { redirect_to @question, notice: "更新しました" }
        format.json { render :show, status: :ok, location: @question }
      else
        format.html { render :edit, status: :unprocessable_entity }
        format.json { render json: @question.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /questions/1 or /questions/1.json
  def destroy
    @question.destroy
    respond_to do |format|
      format.html { redirect_to questions_url, notice: "質問を削除しました" }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_question
      @question = Question.find(params[:id])
    end

    # Only allow a list of trusted parameters through.
    def question_params
      params.require(:question).permit(:user_id, :title, :body, :best_answer_id, {:tag_ids => []})
    end
end

#routes.rb

Rails.application.routes.draw do

  devise_for :admin_users, ActiveAdmin::Devise.config
  ActiveAdmin.routes(self)
  resources :reactions
  get 'answers' => 'answers#index'
  resources :questions, shallow: true do
    resources :answers, shallow: true do
      resources :reactions
    end
  end
  devise_for :users, controllers: { omniauth_callbacks: 'omniauth_callbacks' ,
    registrations: 'users/registrations',
    sessions: 'users/sessions' 
  }
  get 'users/show', to: 'users#show'
  resources :users 
  resource :user, except: [:new, :create, :destroy]
  root 'questions#index'
  get 'pages/index'
  get 'pages/show'
  get 'questions', to:'questions#index'
  get 'static_pages/home'
end

##試してみたこと
questionモデルとタグモデルが多対多なのでそれが関係しているのかなと思い探してはみましたが見つかりませんでした。

###追記
routes.rbとquestionモデルを修正しました。
つけるとエラーが起こるvalidationは:titleと:bodyです

Comments

[yoshi_10_11]

見直していただけるとおわかりかと思いますが、「```」にうまくコードが書かれていなかったりするので、一度編集していただいたほうが良いかと思います。

また、回答を考える上でいくつかご質問がございます。

・question_controller.rbについて、createがありませんが書いていただくことはできますでしょうか？もしくは、実装されていませんか？

・validationを消した場合にエラーが発生する、とのことですが、具体的にどのvalidationコードを消したり追加したりしたのか教えていただけますか？

・念の為、routes.rbも見せていただきたいです

[jackson]

修正依頼ありがとうございます！
question_controller.rbのcreateは今回関係ないかもと思って消してました。申し訳ございません。
またvalidatonを消した場合ではなく、validationを付けた場合にエラーが発生します。
つけるとエラーが起こるvalidationは:titleと:bodyです

Answer 1

@tagsがnilだ、ってことです。
validationをかけると create で else に行きview new に行きます
その時に @tags未定義なので。
elseで @tags = Tag.allしてください

Comments

[jackson]

ありがとうございます！createの方でも@tags = Tag.allを記述したらエラーが起こらなくなりました！ちなみにあと2つ質問がございます、1点目はnewアクションの方だけでこのエラーが起こらないように制御することは可能でしょうか？
2点目はwinterboumさんがおっしゃるelseで@tags = Tag.allをするというのはcreateでTag.allをするという認識で間違いないでしょうか?

[winterboum]

1点目の質問の意味がわかりません
２点めもわからん。create　の else　に書け　と言ってるのですから createですが、、、、

[jackson]

ありがとうございます。意味がわかりました。少し構造を勘違いしていたようです

[winterboum]

実は質問の元になってる疑問は分かってました。
非常によく有る誤解です。
インスタンス変数は「そのインスタンス」でのみ使えます。
controllerも同じく、で、勘違いするのは、「actionが終わるとそのインスタンスは消える」「action毎に新しいインスタンスが作られる」ということを 忘れる・知らない ということです