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

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

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

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

Ruby on Rails

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

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

Q&A

解決済

3回答

4567閲覧

Rails4でメッセージ送信先のユーザー名を表示したい (パート2)

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby

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

Ruby on Rails

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

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

0グッド

0クリップ

投稿2014/10/23 23:02

前回この「Rails4でメッセージ送信先のユーザー名を表示したい 」と質問させていただきました。
ご回答を元に以下の手順でユーザーメッセージアプリを作成いたしました。

lang

1rails g model User name:string message:string 2rails g model Message user_id:integer message:string post_to_id:integer

上記のようにUserモデルとMessageモデルを作成しました。
作成したそれぞれのモデルを以下のコードを追加しテーブルの関連付けをしました。

Userモデル

lang

1has_many :messages

Messageモデル

lang

1belongs_to :users 2belongs_to :user_id, class_name: "User",foreign_key: "user_id" 3belongs_to :post_to_id, class_name: "User",foreign_key: "post_to_id"

usersコントローラー内に以下のメソッドを追加(この「current_user」はすでにRuby on Railsのチュートリアルでログイン機能を実装済みのため作動するものとします。)

lang

1@send_messages = Message.where("user_id=?",current_user.id).preload(:post_to_id) if signed_in?

view側を以下のようにしました
app/views/users/message.html.erb

lang

1<% @send_messages.each do |msg| %> 2<li>送信日時:<%= msg.created_at %></li> 3<li>message内容:<%= msg.message %></li> 4<li>送り先id:<%= msg.post_to_id %></li> 5<li>送り先ユーザー名<%= <%= msg.post_to_id.name %> %></li> 6<% end %>

おかげさまで送り先のユーザー名を上記の手順で実装することに成功しました。
しかし今度はMessageモデルへデータが挿入できないというトラブルに遭遇してしまいました。

lang

1rails c

でコンソールを立ち上げ

lang

1message=Message.new 2message.user_id=1

と入力したところ

lang

1ActiveRecord::AssociationTypeMismatch:

とエラーが表示され、データを挿入できなくなってしまいました。

どうすればMessageモデルへデータを挿入できるようになりますでしょうか?
初歩的な質問で凝縮ですがよろしくお願いいたします。

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

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

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

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

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

guest

回答3

0

ベストアンサー

先にエラーの理由ですが、user_idが「ユーザ」を指すのか「ユーザID」を指すのかが重複して利用されていることが原因です。
Railsとしての、名前の使い方が間違っています。

Railsでは一般的にUserモデルと関連付けしたい場合は message.userUserのオブジェクトに、message.user_id でそのユーザのIDになるように設定します。

前に提示したとおり

class Message < ActiveRecord::Base belongs_to :user belongs_to :post_to, class_name: 'User' end

にするのがよいでしょう。


もしどうしてもuser_id, post_to_idUserモデルにアクセスしたいと思うのであれば、いくつか方法はあります。
でもRuby on Railsレールから外れる ことをするとかなり苦しむことになります。
全力でRailsの推奨方法に乗っかったほうがいいです。

推奨でないやり方は

  • DBに保存するカラム名に user_id_idpost_to_id_idを用いて重複を避ける。
  • write_attributeメソッドを使う

のどちらかを参考に探せば見つかると思います。

投稿2014/10/24 00:05

hello-world

総合スコア1342

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

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

退会済みユーザー

退会済みユーザー

2014/10/24 21:36

前回のご回答にありましたViewの「<%= User.post_to.name %>」のところで「undefined method `post_to' for #」エラーが表示されてしまいました。それで思考錯誤の結果このようなコードになりました。それと試しにrails cから値を入力してみましたがpost_to_idに何かを代入しようとするとエラーになってしまいます。。。 一人のユーザーが送信したメールと受信したメールのつながり...テーブルから見直して考えてみます。
退会済みユーザー

退会済みユーザー

2014/10/26 01:27

--追記します-- Userモデルへの関連づけをする部分を見直したところ解決しました。 Ruby on Railsチュートリアル第11章の考え方を元に書き直しました。 app/models/user.rb has_many :messages, foreign_key: "user_id", dependent: :destroy has_many :post_to_users, through: :messages, source: :post_to has_many :reverse_messages, foreign_key: "post_to_id", class_name: "Message", dependent: :destroy has_many :users, through: :reverse_messages, source: :user app/models/message.rb belongs_to :user, class_name: "User" belongs_to :post_to, class_name: "User" view側 <li>送り先ユーザー名:<%= msg.post_to.name %></li> 上記のコードで修正したところ解決できました。 Messageモデルへテストデータを代入することも出来ました。 たびたびご回答いただきありがとうございました。 おかげさまで実装する事が出来ました。
guest

0

質問の回答とは関係ないのですが、

lang

1class User < ActiveRecord::Base 2 has_many :messages 3end

があるので、

lang

1@send_messages = Message.where("user_id=?",current_user.id)

の代わりに

lang

1@send_messages = current_user.messages

と書けると思います。

投稿2014/10/24 01:00

hello-world

総合スコア1342

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

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

退会済みユーザー

退会済みユーザー

2014/10/24 21:39

>質問の回答とは関係ないのですが、 いえいえ、勉強になりますのでありがたいです。 おっしゃる通りこちらでも書けますね。 コンパクトですので、コードをこちらで修正しました。
guest

0

解決できるかは試していませんが、参考情報を紹介します。

...

@bukken.owner = @owner.id
と定義すると"ActiveRecord::AssociationTypeMismatch~"のエラーが発生してしまいます。

アソシエーションが定義されているので、明示的にオーナーIDを指定してはいけないようです。

@bukken.owner = @owner
...

...
belongs_to関連付けを設定する場合は、関連付けそのものの名前ではなく、外部キーの名前(上の例であればcity_id)を渡す必要があります。city_idではなくcityを渡すと、Person.newまたはPerson.updateにparamsハッシュを渡した時にActive RecordでActiveRecord::AssociationTypeMismatch: City(#17815740) expected, got String(#1138750)エラーが発生します。さらに、属性の編集のみを行なうフォームヘルパーについても注意が必要です。●ユーザーが外部キーを直接操作できてしまうとセキュリティ上の問題が生じる可能性があるため、十分注意してください。
...

アソシエーションを設定している場合、直接 各種の項目を設定するのは危険です。
アソシエーション設定により追加されるメソッドを通じて操作することが必要です。
参考:

投稿2014/10/25 00:32

katoy

総合スコア22324

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

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

退会済みユーザー

退会済みユーザー

2014/10/26 00:08

ご回答ありがとうございます。とても参考になりました。 リレーションの書き方のルールを理解できていなかったようです。 リレーションを書いているうちにuser_idなのかuserなのか、post_to_idなのかpost_toなのか分からなくなってきていましたが、ルールを理解しなければセキュリティにおいても危険と感じました。 ご回答いただいた情報とRuby on Railsのチュートリアル(http://railstutorial.jp/chapters/beginning?version=4.0#sec-install_git)の第11章を元にもう一度関連づけのあたりを整理してみます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問