NoMethodError が解決出来ません。

Question

実現したいこと

宿泊管理アプリを作成しており、ユーザーごとに管理している施設の登録をしたいです。

発生している問題・分からないこと

deviseを使いユーザー認証を行い、そのユーザー情報に紐づけて施設の登録をするのですが、そこでタイトルのエラーが発生しており困っています。

エラーメッセージ

undefined method `rooms' for nil:NilClass
Extracted source (around line #8):
            
  def index
    @rooms = @current_user.rooms.all
  end

該当のソースコード

class User < ApplicationRecord
  has_many :rooms

  has_one_attached :photo
  has_many :reservations
  has_many :guest_reviews, class_name: "GuestReview", foreign_key: "guest_id"
has_many :host_reviews, class_name: "HostReview", foreign_key: "host_id"


  validates :full_name, :email, presence: true, length: {maximum: 50}, allow_blank: true


  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
end

class Room < ApplicationRecord
  belongs_to :user

  has_one_attached :photos
  has_many :reservations
  has_many :guest_reviews
  has_many :reviews

  validates :r_name, :r_self, :r_price, :r_create, :r_address, presence: true, allow_blank: true
  validates :r_price, numericality: {only_integer: true, greater_than_or_equal_to: 1}

  def average_rating
    guest_reviews.count == 0 ? 0 : guest_reviews.average(:stars).round(2).to_i
  end

  def self.looks(search,keyword)
    if serach == "pefect_match"
      @room = Room.where("r_name LIKE?","#{keyword}")
    elsif search == "forward_match"
      @room = Room.where("r_name LIKE?","#{keyword}")
    elsif search == "backworkd_match"
      @room = Room.where("r_name LIKE?","#{keyword}")
    elsif search == "partial_match"
      @room = Room.where("r_name LIKE?","#{keyword}")
    else
      @room = Room.all
    end
  end

end

class RoomsController < ApplicationController
  before_action :set_room, except: [:index, :new, :create]
  before_action :authenticate_user!, except: [:show]
  protect_from_forgery except: [:upload_photo]
  before_action :is_authorised, only: [:listing, :pricing, :description, :photo_upload, :amenities, :location, :update]
  

  def index
    @rooms = current_user.rooms.all
  end

  def new
    @room = current_user.rooms.build
  end

  def create
    @room = current_user.rooms.build(room_params)
    if @room.save
      redirect_to listing_room_path(@room), notice: "保存完了"
    else
      flash[:alert] = "問題"
      render :new
    end
  end

<h1>Rooms#new</h1>
<p>Find me in app/views/rooms/new.html.erb</p>

<div class="container">
    <div class="card mt-4">
        <div class="card-body">
            <h4 class="mt-4 mb-4"><b>部屋の新規登録</b></h4>
            <%= form_for @room, multipart: true do |f| %>
                <div class="row mb-4">            
                    <div class="col-md-4">
                        <div class="mb-2">
                            <label>施設名</label>
                            <%= f.text_field :r_name,
                                id: "r_name", class: "form-control" %>
                        </div>
                    </div>
                    <div class="col-md-4">
                        <div class="mb-2">
                            <label>写真</label>
                            <%= image_tag room_url(@room), :size => '150x100', class: "img-fluid img-thumbnail rounded-pill" %>
                            <h4 style="margin-left: 5.5rem;"></h4>
        
        
                            <button class="btn btn-info text-light w-100" type="button" data-bs-toggle="collapse" data-bs-target="#collapse1" aria-expanded="false" aria-controls="collapse1">
                                <i class="fa-solid fa-cloud-arrow-up"></i>写真アップロード
                            </button>
                            <div class="collapse" id="collapse1">
                            <div class="card card-body">
                            <%= form_for :rooms, url: rooms_url(current_user), action: :update, method: :post do |form|  %>
                                <%= form.file_field :photos, class: "input-group-text", onchange: "this.form.submit();" %>
                                <%= form.submit %>
                                
                                    
                               <% end %>
                            </div>

                        </div>
                    </div>
                </div>
                <div class="row mb-4">          
                    <div class="col-md-4">
                        <div class="mb-2">
                            <label>紹介</label>
                            <%= f.text_area :r_self,
                                id: "r_self", class: "form-control" %>
                        </div>
                    </div>
                    <div class="col-md-4">
                        <div class="mb-2">
                            <label>料金</label>
                            <%= f.text_field :r_price,
                                id: "r_price", class: "form-control" %>
                        </div>
                    </div>
                    <div class="col-md-4">
                     <div class="mb-2">
                            <label>作成日</label>
                            <%= f.date_field :r_create,
                                id: "r_create", class: "form-control" %>
                        </div>
                     </div>
                     <div class="col-md-4">
                      <div class="mb-2">
                            <label>住所</label>
                            <%= f.text_field :r_address,
                                id: "r_address", class: "form-control" %>
                      </div>
                     </div>
                     <div class="col-md-4">
                     <div class="mb-2">
                           <label>宿泊可能人数</label>
                           <%= f.text_field :people,
                               id: "people", class: "form-control" %>
                     </div>
                    </div>
                </div>
                <%= f.submit "登録する", class: "btn btn-danger w-100" %>        
            <% end %>
        </div>
    </div>    
</div>

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

作成元のサイトも確認しましたが記載したコードで作動しており、モデル間の関係も入力してるのでnillになる理由が分かりません。

補足

下記サイトを参考に作成しています。
https://mrradiology.hatenablog.jp/VacationRental

Answer 1

user.rbの修正

User モデルに has_many :rooms と belongs_to :rooms が両方定義されていましたが、これらは矛盾していると思われます。
User は複数の Room を持つので、has_many :rooms のみを使用し、belongs_to :rooms を削除した方が良いと思います。

room.rbの修正

こちらもbelongs_to :rooms という不必要な関連が含まれているため、削除した方が良いと思います。

rooms_controller.rbの修正

deviseを使用してますでしょうか？
@current_user ではなく current_userメソッドを使用するので、current_userかと思います。
Rails deviseで使えるようになるヘルパーメソッド一覧
https://qiita.com/tobita0000/items/866de191635e6d74e392

  def index
    @rooms = @current_user.rooms.all
  end

Comments

[G23]

ご指摘有難うございます。
pryを使ってデバックし確認した所下記のようになりました。
[1] pry(#<RoomsController>)> current_user
=> nil
[2] pry(#<RoomsController>)> rooms
NameError: undefined local variable or method `rooms' for #<RoomsController:0x0000000000a708>
Did you mean? rooms_url
from (pry):2:in `create'
[3] pry(#<RoomsController>)> build
NameError: undefined local variable or method `build' for #<RoomsController:0x0000000000a708>
from (pry):3:in `create'
[4] pry(#<RoomsController>)>
roomsはroomsテーブルとの関連付けの為のものという認識でしたが違うのでしょうか？
またログイン状態にも関わらずcurrent_userがnilになる原因はなんでしょうか？
初歩以前の問題ですが、お願い致します。

Answer 2

@current_user が nil ということですね。
さて、この変数名は正しいでしょうか？
公式のドキュメントは英語ですが、deviseのヘルパーについてはこちらを参照してみてください。
https://github.com/heartcombo/devise?tab=readme-ov-file#controller-filters-and-helpers

Comments

[winterboum]

＞ この変数名は正しいでしょうか？
それクサイですね

[G23]

@current_userの部分をcurrent_usrに変更した所下記の部分がnill classになりました。
undefined method `build' for nil:NilClass
この場合、buildの前にあるroomsがnillということでしょうか？

[mather]

そうだと思います。

[G23]

pryを使ってデバックし確認した所下記のようになりました。
[1] pry(#<RoomsController>)> current_user
=> nil
[2] pry(#<RoomsController>)> rooms
NameError: undefined local variable or method `rooms' for #<RoomsController:0x0000000000a708>
Did you mean? rooms_url
from (pry):2:in `create'
[3] pry(#<RoomsController>)> build
NameError: undefined local variable or method `build' for #<RoomsController:0x0000000000a708>
from (pry):3:in `create'
[4] pry(#<RoomsController>)>
roomsはroomsテーブルとの関連付けの為のものという認識でしたが違うのでしょうか？
またログイン状態にも関わらずcurrent_userがnilになる原因はなんでしょうか？
初歩以前の問題ですが、お願い致します。