Ruby: 保存されない場合

Question

前提・実現したいこと

特定のユーザーが作成したチャットルームに１：１でチャットが出来るようにしたい

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

発言内容が空の場合、データベースに保存されないようバリデーションをかけると以下のエラーになります。

NoMethodError in Messages#create
Showing （省略）views/messages/_side_bar.html.erb where line #14 raised:
undefined method `exists?' for nil:NilClass

<div class="room">
    <ul class="room-name">
      <% if @rooms.exists? %>
      <% @rooms.each do |room| %>
      <li class="list">
        <%= link_to room_messages_path(room) do %>

Request
Parameters:

{"authenticity_token"=>"（省略）==", "message"=>{"content"=>""}, "commit"=>"送信", "room_id"=>"1"}

該当のソースコード

class Message < ApplicationRecord
  belongs_to :room
  belongs_to :user

  validates :content, presence: true
end

class MessagesController < ApplicationController
  def index
    @message = Message.new
    @room = Room.find(params[:room_id])
    @rooms = Room.all.includes(:user)
    @messages = @room.messages.includes(:user)
  end

  def create
    @room = Room.find(params[:room_id])
    @message = @room.messages.new(message_params)
    if @message.valid?
       @message.save
       redirect_to room_messages_path(@room)
    else
      @messages = @room.messages.includes(:user)
      render :index
    end
  end

  private

  def message_params
    params.require(:message).permit(:content).merge(user_id: current_user.id)
  end
end

<div class="wrapper">
  <div class="side-bar">
    <%= render "side_bar" %>
  </div>
  <div class="chat">
    <%= render "main_chat" %>
  </div>
</div>

<div class="message">
  <div class="upper-message">
    <div class="message-user">
      <%= message.user.nickname %>
    </div>
    <div class="message-date">
      <%= l message.created_at %>
    </div>
  </div>
  <div class="lower-message">
    <div class="message-content">
      <%= message.content %>
    </div>
  </div>
</div>

<div class="side-bar-header">
  <div class="side-header-title">
    <a href="#">Idea Trade</a>
  </div>
  <div class="create-room">
    <%= link_to "チャットを作成する", new_room_path %>
  </div>
</div>


<div class="rooms">
  <div class="room">
    <ul class="room-name">
      <% if @rooms.exists? %>
      <% @rooms.each do |room| %>
      <li class="list">
        <%= link_to room_messages_path(room) do %>
        <%= room.name %>
        <% end %>
      </li>
      <% end %>
      <% end %>
    </ul>
  </div>
</div>

<div class="rooms-f">
  <div class="user-name">
    <%= link_to current_user.nickname, edit_user_path(current_user) %>
  </div>
</div>

（省略）

<div class="messages">
  <%= render partial: 'message', collection: @messages %>
</div>

<%= form_with model: [@room, @message], class: 'form', local: true do |f| %>
  <div class="form-input">
    <%= f.text_field :content, class: 'form-message', placeholder: 'type a message' %>
    <label class="form-image">
      <span class="image-file">画像</span>
      <%= f.file_field :image, class: 'hidden' %>
    </label>
  </div>
  <%= f.submit '送信', class: 'form-submit' %>
<% end %>

試したこと

仮説として、エラー文で指摘されているexists?メソッドが空で定義されてないとありますが、そこよりもコントローラーの保存されない場合の条件式が合ってないのではと考えてます。
関連する箇所を思いつく限り弄ってみましたが変化なしです。
indexアクションの

@rooms = Room.all.includes(:user)
@messages = @room.messages.includes(:user)

の、includesでダブってるからまずいのかなと考えましたが、@roomsの箇所を消すとサイドバーのルーム名が表示されずエラーとなります。

Answer 1

create内で@roomsへの代入がないため、createで失敗して再表示を行う際には@roomsがnilとなってしまいます。

Comments

[sasa_nagare]

解決しました！ありがとうございます！！