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

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

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

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

Ruby

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

Q&A

解決済

1回答

4195閲覧

ambiguous column nameのエラーが表示される。

punchan36

総合スコア105

Ruby on Rails 5

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

Ruby

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

0グッド

0クリップ

投稿2020/12/20 07:06

編集2020/12/30 11:41

前提・実現したいこと

メッセージの送受信機能を実装しました。
現段階ではUserRoomの中間テーブルであるEntryidが作成された時間順でメッセージ一覧が表示されています。
これを、メッセージが送信された時間順で一覧を表示したいです。

リレーション関連

user.rb

ruby

1class User < ApplicationRecord 2has_many :messages, dependent: :destroy 3has_many :entries, dependent: :destroy 4has_many :rooms, through: :entries 5end

room.rb

ruby

1class Room < ApplicationRecord 2has_many :messages, dependent: :destroy 3has_many :entries, dependent: :destroy 4has_many :users, through: :entries 5end

entry.rb

ruby

1class Entry < ApplicationRecord 2belongs_to :user 3belongs_to :room 4end

message.rb

ruby

1class Message < ApplicationRecord 2belongs_to :user 3belongs_to :room 4end

rooms_controller.rb

このコントローラ内で定義したEntryをビュー側で繰り返し処理し、表示しています。

ruby

1class RoomsController < ApplicationController 2before_action :set_another_entries, only: [:index, :show] 3private 4def set_another_entries 5my_room_ids = @current_user.entries.pluck(:room_id) # ログイン中のユーザーの持つEntryを配列にする。 6# 上で定義した「ログイン中のユーザーの持つEntry」の内、自分以外の相手一覧を表示する。 7@another_entries = Entry.includes(:user, :room).where(room_id: my_room_ids).where('user_id != ?', @current_user.id).order(created_at: :desc) 8end 9end

試したこと

まず過去の質問を参考にしました。こちらの質問者さまは、私の場合で言うRoomを繰り返し表示されているので、Messageテーブルを結合(join)する事で解決されたようです。
ただ私の場合はEntryを繰り返し表示させており、messageテーブルとのリレーションが出来ていません。
ER図
ですのでそれを明示しようと考えました。
entry.rb

ruby

1class Entry < ApplicationRecord 2belongs_to :user 3belongs_to :room 4has_many :messages, through: :room #この行を追加。 5end

message.rb

ruby

1class Message < ApplicationRecord 2belongs_to :user 3belongs_to :room 4has_one :entry, through: :room #この行を追加。 5end

その上でMessageテーブルの結合(join)を試みました。
rooms_controller.rb

ruby

1class RoomsController < ApplicationController 2before_action :set_another_entries, only: [:index, :show] 3private 4def set_another_entries 5my_room_ids = @current_user.entries.pluck(:room_id) 6@another_entries = Entry.includes(:user, :room).where(room_id: my_room_ids).where('user_id != ?', @current_user.id) 7@another_entries = @another_entries.joins(:messages).order("messages.created_at DESC") 8end

しかし以下のエラーが発生しました。

ActiveRecord::StatementInvalid in Rooms#index Showing C:/hoge/app/views/rooms/index.html.erb where line #12 raised: SQLite3::SQLException: ambiguous column name: user_id: SELECT "entries".* FROM "entries" INNER JOIN "rooms" ON "rooms"."id" = "entries"."room_id" INNER JOIN "messages" ON "messages"."room_id" = "rooms"."id" WHERE "entries"."room_id" IN (?, ?, ?, ?) AND (user_id != 20) ORDER BY messages.created_at DESC Extracted source (around line #12): 10 <div class="message-items"> 11 12 <% @another_entries.each do |e| %> 13 <div class="message-item"> 14 15 <div class="message-left">

恐らくリレーションを与えた事でEntryMessageの両テーブルにuser_idが存在する事になり、どちらのテーブルの物かを判別する事が出来ない、と言った内容なのかと思います。
scopeを使ってそれを明示する(?)と言った記事を見つけたので試してみましたが、また同様にambiguous column nameのエラーが表示され上手くいきませんでした。
room.rb

ruby

1class Room < ApplicationRecord 2has_many :messages, dependent: :destroy 3has_many :entries, dependent: :destroy 4has_many :users, through: :entries 5scope :select_entry, -> (entry) { where('user_id = ?', entry) } #この行を追加。 6end

実はこの後で「messageの中身が存在するEntryのみを表示する」と言った記述もしていきたいので、出来ましたらEntryMessageの両テーブルはリンクさせた上で実装出来れば一番有難いです。
どなたかご助言を頂けませんでしょうか。

補足情報(FW/ツールのバージョンなど)

ruby 2.6.4p104
RubyGems 3.0.3
Rails 5.2.3

追記

その後、以下の2点を無事実装する事が出来ました。

  1. メッセージが送信された時間順でトークルーム一覧を表示する。
  2. messageの中身が存在しないEntryは表示しない。

こちらの質問をご参考頂ければと思います。

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

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

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

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

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

guest

回答1

0

ベストアンサー

where('user_id != ?', この部分をwhere('entries.user_id != ?', とtable名を明示するのが一番簡単かな。

投稿2020/12/24 03:31

winterboum

総合スコア23349

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

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

punchan36

2020/12/24 04:15

ご回答有難うございます! room.rbにて、 scope :select_entry, -> (entry) { where('entries.user_id = ?', entry) } と直して試してみましたが、再度上記と同じエラーが発生しました…。 これでもまだテーブルを明示出来ていないと言う事でしょうか…?
punchan36

2020/12/24 04:43

あ、rooms_controller.rb内の記述に関してですね。大変失礼致しました。 @another_entries = Entry.includes(:user, :room).where(room_id: my_room_ids).where('entries.user_id != ?', @current_user.id) @another_entries = @another_entries.joins(:messages).order("messages.created_at DESC") 上記のように記述する事で、エラーは取りあえず表示されなくなりました!有難うございます。 まだビューの繰り返し表示が少しおかしいのでまず試行錯誤してみます!
punchan36

2020/12/24 05:58

rooms_controller.rb内を上記の様に記述する事で、メッセージが作成された順に「メッセージの数の分だけ」エントリーが何度も表示されるようになりました。 各エントリー内の最新のメッセージが作成された順にエントリーを1つずつ表示したいのですが、コントローラ内でどう処理すれば良いでしょうか? @last_message = @another_entries.each do |lm| Message.find_by(id: lm.room.message_ids.last) end など作ってみましたが、joinする箇所にどう組み込んでいけば良いか分かりません…。
winterboum

2020/12/24 06:15

質問のタイトルと違う内容になっていくと、最適な回答者がhitしないから、新しい質問にしたほうが良いですよ
punchan36

2020/12/24 06:55

大変失礼致しました。 質問文の内容を最初から整理し、現状のコード・ER図に書き直しました。 可能でしたら引き続きご助言頂けますと有難いです。宜しくお願い致します。
punchan36

2020/12/27 08:01

当初質問させて頂いていたエラーはお陰様で解決致しましたので、希望の実装はまだ出来ておりませんがベストアンサーにさせて頂きます。有難うございました! 続きの実装に関しては別で質問させて頂くか、解決次第こちらに解決法を追記致します。 (質問文も当初書いてあったものに編集し直しました。失礼致しました。)
winterboum

2020/12/27 23:36

ごめんなさい 本職がちょっと追い込みになったのと、 あまり得意な分野ではないのと これ相当厄介なのですよ。。。。 ということで、手がついていませんでした
punchan36

2020/12/28 06:57

とんでもないです!いつも有難うございます。 「検索で得られた結果をArrayに変換してsort_by使う」など、他の方法も試してみようかと思います。 それかこちらの質問者さまの様に、そもそも中間テーブルであるEntryではなくRoom自体をビューで繰り返し表示する仕様に変える…など。 https://teratail.com/questions/272305 いずれにせよ、進展がありましたら追記致します!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問