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

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

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

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

Ruby on Rails 7

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

解決済

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

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby

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

Ruby on Rails 7

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

2回答

0グッド

0クリップ

188閲覧

投稿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>]>

以下のような質問にはグッドを送りましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

グッドが多くついた質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

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

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

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

下記のような質問は推奨されていません。

  • 間違っている
  • 質問になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

適切な質問に修正を依頼しましょう。

maisumakun

2022/10/27 02:06

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

回答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

総合スコア141324

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

下記のような回答は推奨されていません。

  • 間違っている回答
  • 質問の回答になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

このような回答には修正を依頼しましょう。

回答へのコメント

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相手の名前を表示するため)という役割として記述しました。

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

ただいまの回答率
86.12%

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

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

質問する

関連した質問

同じタグがついた質問を見る

Ruby

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

Ruby on Rails 7

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