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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails 6

Ruby on Rails 6は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

695閲覧

undefined method `user_id' for nil:NilClassについて

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails 6

Ruby on Rails 6は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2022/10/05 01:15

困っている箇所

rooms_controllerのindex箇所の掲載者のuserを入れる@anotherEntriesの箇所でエラーが起こってしまいました。
undefined method `user_id' for nil:NilClass

エラー

NoMethodError at /rooms undefined method `user_id' for nil:NilClass Hint: Something is `nil` when it probably shouldn't be.

rooms_controller

class RoomsController < ApplicationController before_action :authenticate_user! def index @currentEntries = current_user.entries myRoomIds = [] @currentEntries.each do |entry| myRoomIds << entry.room.id end @anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?',@plan.user_id) end def create @plan = Plan.find(params[:entry][:plan_id]) @room = Room.create(room_name: @plan.title) @entry1 = Entry.create(room_id: @room.id, user_id: current_user.id) @entry2 = Entry.create(params.require(:entry).permit(:user_id, :room_id).merge(room_id: @room.id)) redirect_to "/rooms/#{@room.id}" end def show @room = Room.find(params[:id]) if Entry.where(user_id: current_user.id,room_id: @room.id).present? @messages = @room.messages @message = Message.new @entries = @room.entries else redirect_back(fallback_location: root_path) end end private def set_plan @plan = Plan.find(params[:id]) end end

room.rb

class Room < ApplicationRecord has_many :entries, dependent: :destroy has_many :messages, dependent: :destroy has_many :users has_many :plans end

views/rooms/index.html.erb

<div message-list> <h4>メッセージ一覧</h4> <% @my_rooms.each do |room| %> <p><%= render 'rooms', rooms: @rooms %></p> <% end %> </div>

質問

@anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?',@plan.user_id)
で掲載者のuserを入れたいのですが、user_idがnilの為、@plan = Plan.find(params[:id])でplan_idからplanを抜き出し、さらにそこからplan.user_idを抜き出して@anotherEntries のuser_idに代入させようとしたのですが、

ActiveRecord::RecordNotFound at /rooms Couldn't find Plan without an ID

と表示され,調べたところルーティングに関しての情報が多くあったのですが、
ルーティングを見ても問題ないと思います。
@plan = Plan.find(params[:id])でレコードから抜き出すやり方は正しいでしょうか?

!追記
routes.rb

Rails.application.routes.draw do devise_for :users, controllers: { registrations: 'users/registrations' } root to: 'toppages#index' get "users/show" => "users#show" devise_scope :users do get '/users', to: redirect("/users/sign_up") end resources :users do member do get :followings get :followers get :likes end collection do get :search end end resources :users resources :plans resources :relationships, only: [:create, :destroy] resources :favorites, only: [:create, :destroy] resources :messages, only: [:create] resources :rooms end

routes.rb

rooms GET /rooms(.:format) rooms#index POST /rooms(.:format) rooms#create new_room GET /rooms/new(.:format) rooms#new edit_room GET /rooms/:id/edit(.:format) rooms#edit room GET /rooms/:id(.:format) rooms#show PATCH /rooms/:id(.:format) rooms#update PUT /rooms/:id(.:format) rooms#update DELETE /rooms/:id(.:format) rooms#destroy

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

@plan = Plan.find(params[:id])でレコードから抜き出すやり方は正しいでしょうか?

そのコードが呼ばれていません。set_planメソッドは存在しますが、どこからも呼ばれていないようです。

投稿2022/10/05 01:46

maisumakun

総合スコア145121

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2022/10/05 02:01 編集

before_action :set_plan, only: [:index]を追加したのですが、 Couldn't find Plan without an IDと表示されました。 plan.user_idを@anotherEntriesに代入したいのですが、planを取得するやり方としては正しいですか?
maisumakun

2022/10/05 02:10

「どの」Planを取得したいのですか? (indexは全件をリストアップするためのアクションなので、エラーメッセージの通り、一般にidの値は流れて来ません)
退会済みユーザー

退会済みユーザー

2022/10/05 04:56 編集

current_userが入っているroomと一緒に入っている掲載者のuserのid。 つまりPlan(掲載)の中にあるuser_idを取得したいです。 user_id != ?',@plan.user_id ここの箇所が恐らく問題だと思うので、planを取得しなければならないと感じたのですが違いますでしょうか?
maisumakun

2022/10/05 04:59

@planは使わず、単にcurrent_user.idを入れる、というのは意図と違いますか?
退会済みユーザー

退会済みユーザー

2022/10/05 05:09

サイトを見て一覧作成を進めていたのですが、おそらく def index @currentEntries = current_user.entries myRoomIds = [] @currentEntries.each do |entry| myRoomIds << entry.room.id end @anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?',@plan.user_id) end @currentEntries = current_user.entries←自分のユーザー @anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?',@plan.user_id)←相手のユーザー という図式かなと思ったのですが、この考え方は違いますか?
maisumakun

2022/10/05 05:16

「相手のユーザー」を取りたいのであれば、条件は「自分以外」なので、current_user.idを入れればいいかと思います。
退会済みユーザー

退会済みユーザー

2022/10/05 05:28

なるほど! すみません。 もう1つ追加で質問なのですがその一覧にルームの名前と相手userの名前を表示させようとする際に、 その場合は例えば相手の名前を表示したい場合@anotherEntries.user.nameで大丈夫でしょうか?
maisumakun

2022/10/05 05:32

> @anotherEntries.user.nameで大丈夫でしょうか? いえ、名前の通り@anotherEntriesには複数入る可能性がありますので、1つずつ処理する必要があります。
退会済みユーザー

退会済みユーザー

2022/10/05 05:41

<% rooms.each do |room| %>で1つずつ処理させて @user.nameで表示しようかなとも考えたのですがそれでは処理されませんか?
maisumakun

2022/10/05 05:43

roomsとは何ですか? そして、roomに代入されるループをいくら回そうが、@userの中身は変化しません。
退会済みユーザー

退会済みユーザー

2022/10/05 06:07 編集

間違えました。。。 <% @anotherEntries.each do |room| %>です。
退会済みユーザー

退会済みユーザー

2022/10/05 06:07 編集

<% @anotherEntries.each do |entry| %> にしてみたところエラーはなくなり、user.nameの箇所のエラーで止まっているのですが、 上記の回し方は適切でしょうか? user.nameの箇所のエラーにundefined method `id' for #<ActiveRecord::Associations::CollectionProxy [#<Entry id: 22, user_id: 1, room_id: 24>]>と書いてあるのでentryで回すのが適切なのかと思ったのですが。。。
maisumakun

2022/10/05 06:04

もとの質問から離れていっているので、別に質問を立てて仕切り直ししたほうがいい気もします。
退会済みユーザー

退会済みユーザー

2022/10/05 06:08

回答ありがとうございます。 別の質問を立てることにします。 まずはエラーが治ったので一安心しました! 本当にありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問