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

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

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

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

Ruby on Rails 7

Ruby on Rails 7は、2021年12月に正式リリースされました。Ruby on Railsのバージョン7であり、フロントエンド開発環境を大幅に刷新。Node.jsを用いない構成がデフォルトになっています。

Q&A

解決済

2回答

423閲覧

eachからうまく取り出せない

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby

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

Ruby on Rails 7

Ruby on Rails 7は、2021年12月に正式リリースされました。Ruby on Railsのバージョン7であり、フロントエンド開発環境を大幅に刷新。Node.jsを用いない構成がデフォルトになっています。

0グッド

0クリップ

投稿2022/10/27 02:01

編集2022/10/28 07:57

前提

応募して作成されたDMをDM一覧で表示させようとしています。
テストで掲載者1,応募者2で複数人の応募とDM一覧をしっかりと表示されるかを試していたところ、応募者のDM一覧で、自身が応募していないDMまで表示されます。
当該userは2つ応募してDM一覧に2つだけ表示したいが、2つだけ表示されてはいるのですが、自分のDMと他人のDMの2つが表示され自身のDMの2つ目が表示されませんでした。

現在の一覧
DM-A :自身
DM-A: 他人

やりたい一覧表示
DM-A :自身
DM-B :自身

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

意図的にエラーを起こして、controllerに定義した変数にしっかり、取得出来ているか確認したところ、しっかり2つだけ取得出来ているのですが、eachの取り出し方が悪いのでしょうか?
イメージ説明

該当のソースコード

!すみませんusersと書かれていましたがroomsでした。

rooms_controller.rb

1def index 2 @pagy,@rooms = pagy(Room.all.order(id: "DESC").select(:id,:room_name,:plan_id), items:10) 3 @currentEntries = current_user.entries 4 myRoomIds = [] 5 6 @currentEntries.each do |entry| 7 myRoomIds << entry.room.id 8 end 9 10 @anotherEntries = Entry.order(id: "DESC").where(room_id: myRoomIds).where('user_id != ?',current_user.id) 11 12 end

_room.html.erb

1<div> 2 <% @rooms.zip(@anotherEntries) do |room,entry| %> 3 <% if entry.nil? %> 4 <% else %> 5 <div class="dm-list-box"> 6 <div class="dm-list-conent"> 7 <div class="dm-list-user-name"> 8 <h5>相手:</h5> 9 <p><%= link_to entry.user.name, user_path(entry.user), class: "text-decoration-none" %></p><br /> 10 </div> 11 <div class="dm-list-plan-name"> 12 <h5>掲載名:</h5> 13 <p><%= link_to room.room_name, plan_path(room.plan_id), class: "text-decoration-none" %></p> 14 </div> 15 </div> 16 <div class="dm-list-btn"> 17 <p><%= link_to 'メッセージ', room, class: "btn btn-outline-danger btn-lg" %></p> 18 </div> 19 </div><br /> 20 <% end %> 21 <% end %> 22 <div class="d-flex justify-content-center mb-2"> 23 <%== pagy_bootstrap_nav(@pagy) %> 24 </div> 25</div>

試したこと

一度entryをeachで取り出すのを止めてroomだけを一つずつ取り出したら全部のDMが表示されました。
恐らく2つ目からはentry.user.nameがうまく取り出せていないということが分かりました。
ですが上記の写真を見るとDMの相手の変数@anotherEntriesにはしっかりと2つだけ取得出来ています。ならばentry.user.nameは取得出来ていると思うのですが、eachで一つずつ取り出すときに変な取り出し方をしているのでしょうか?

!追記
rooms_controllerの変数

@pagy,@rooms = pagy(Room.all.order(id: "DESC").where(id: @currentEntries).select(:id,:room_name,:plan_id), items:10)

で@roomsを@currentEntriesのidと合致するものだけ絞ると、自身のDmだけを表示することが恐らくできたのですが、2つ目が表示されません。
今、原因を調べています。
何か手がかりが分かり次第、質問に追記させていただきます。

!追記
@roomsの中身を調べたところRoomが1つしか取り出せていませんでした。

>> @rooms => #<ActiveRecord::Relation [#<Room id: 1, room_name: "テスト1", plan_id: 1>]>

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

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

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

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

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

maisumakun

2022/10/27 02:06

モデル同士のリレーションがわからず回答しづらい状況です。そのあたりの説明も追加していただけませんでしょうか?
guest

回答2

0

ベストアンサー

@anotherEntriesだけを回したところとりあえず自身のDMだけを表示することができました。

<% @anotherEntries.each do |entry| %> <% if entry.nil? %> <% else %> <div class="dm-list-box"> <div class="dm-list-conent"> <div class="dm-list-user-name"> <h5>相手:</h5> <p><%= link_to entry.user.name, user_path(entry.user), class: "text-decoration-none" %></p><br /> </div> <div class="dm-list-plan-name"> <h5>掲載名:</h5> <p><%= link_to entry.room.room_name, plan_path(entry.room.plan_id), class: "text-decoration-none" %></p> </div> </div> <div class="dm-list-btn"> <p><%= link_to 'メッセージ', entry.room, class: "btn btn-outline-danger btn-lg" %></p> </div> </div><br /> <% end %>

投稿2022/10/28 08:08

退会済みユーザー

退会済みユーザー

総合スコア0

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

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

0

@rooms.zip(@anotherEntries)は、「@rooms@anotherEntriesに入ったものを同じ順に取り出していく」という動作です。本当にそれは意図した操作なのでしょうか?

投稿2022/10/27 02:06

maisumakun

総合スコア145201

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

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

maisumakun

2022/10/27 02:15

> 意図的にエラーを起こして、controllerに定義した変数にしっかり、取得出来ているか確認したところ、しっかり2つだけ取得出来ているのですが @roomsの中身はどうなっていますか?
退会済みユーザー

退会済みユーザー

2022/10/27 06:00

@roomsは全部のroomが入っていました! >> @rooms => #<ActiveRecord::Relation [#<Room id: 4, room_name: "テスト2", plan_id: 2>, #<Room id: 3, room_name: "テスト2", plan_id: 2>, #<Room id: 2, room_name: "テスト1", plan_id: 1>, #<Room id: 1, room_name: "テスト1", plan_id: 1>]>
退会済みユーザー

退会済みユーザー

2022/10/27 06:04

@roomsの中から@anotherEntriesのroom_idと合致するroomを一つずつ取り出していく思っていました。
退会済みユーザー

退会済みユーザー

2022/10/28 08:05

@rooms.zip(@anotherEntries)は@rooms(DMの名前とリンクを表示するため)、@anotherEntries(DM相手の名前を表示するため)という役割として記述しました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.47%

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

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

質問する

関連した質問