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

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

ただいまの
回答率

87.77%

DB設計(Model定義)ミスの修正について

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,281

score 58

前提・実現したいこと

通知機能の実装をしています。
2つのお知らせを受け取り、そのお知らせを一覧ページで表示させようと思っています。

具体的には、
・トピックにて投稿した記事にコメントがついたらお知らせ
・タイムラインに投稿したものにコメントがついたらお知らせ
このようにしたいと思っていました。

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

一通りやってみたのですが、DB設計(Model定義)ミスで一覧ページで上手に表示することができません。
具体的には、
データの取得方法、eachメソッドについて
こちらで質問させていただいた内容です。

該当のソースコード

models/topic.rb

class Topic < ActiveRecord::Base
    validates :title, presence: true
    validates :content, presence: true

    # Topicモデルのレコードがuserモデルのレコードに属することを定義
    belongs_to :user

    # CommentモデルのAssociationを設定
    has_many :comments, dependent: :destroy
end


models/comment.rb[トピックに対するコメント]

class Comment < ActiveRecord::Base
  belongs_to :user
  belongs_to :topic

  has_many :notifications, dependent: :destroy
end

models/timeline.rb

class Timeline < ActiveRecord::Base
    validates :content, presence: true

    # Timelineモデルのレコードがuserモデルのレコードに属することを定義
    belongs_to :user
    # CommenttlモデルのAssociationを設定
    has_many :commenttls, dependent: :destroy
end

models/commenttl.rb[タイムラインに対するコメント]

class Commenttl < ActiveRecord::Base
  belongs_to :user
  belongs_to :timeline

  has_many :notifications, dependent: :destroy
end

models/notification.rb[お知らせのためのモデル]

class Notification < ActiveRecord::Base
  belongs_to :user
  belongs_to :comment
  belongs_to :commenttl
end

データベース

  create_table "comments", force: :cascade do |t|
    t.integer  "user_id"
    t.integer  "topic_id"
    t.text     "content"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  add_index "comments", ["topic_id"], name: "index_comments_on_topic_id", using: :btree
  add_index "comments", ["user_id"], name: "index_comments_on_user_id", using: :btree

  create_table "commenttls", force: :cascade do |t|
    t.integer  "user_id"
    t.integer  "timeline_id"
    t.text     "content"
    t.datetime "created_at",  null: false
    t.datetime "updated_at",  null: false
  end

  add_index "commenttls", ["timeline_id"], name: "index_commenttls_on_timeline_id", using: :btree
  add_index "commenttls", ["user_id"], name: "index_commenttls_on_user_id", using: :btree


  create_table "notifications", force: :cascade do |t|
    t.boolean  "read",         default: false
    t.integer  "user_id"
    t.integer  "comment_id"
    t.datetime "created_at",                   null: false
    t.datetime "updated_at",                   null: false
    t.integer  "commenttl_id"
  end

  add_index "notifications", ["comment_id"], name: "index_notifications_on_comment_id", using: :btree
  add_index "notifications", ["commenttl_id"], name: "index_notifications_on_commenttl_id", using: :btree
  add_index "notifications", ["user_id"], name: "index_notifications_on_user_id", using: :btree


  create_table "timelines", force: :cascade do |t|
    t.text     "content"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer  "user_id"
  end

  create_table "topics", force: :cascade do |t|
    t.string   "title"
    t.text     "content"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer  "user_id"
  end

  create_table "users", force: :cascade do |t|
   省略
  end

  add_foreign_key "comments", "topics"
  add_foreign_key "comments", "users"
  add_foreign_key "commenttls", "timelines"
  add_foreign_key "commenttls", "users"
  add_foreign_key "notifications", "comments"
  add_foreign_key "notifications", "commenttls"
  add_foreign_key "notifications", "users"
end

現在以上のようになっているのですが、どのように修正したらよろしいのでしょうか?

ご教授よろしくお願いします。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

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

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

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

Rails5なら Action Cableを利用して通知機能が簡単に実装できるのですが
Rails4でPusherを利用しているのですね。少し読み込みが足りませんでした、すみません

そうすると、下手にリファクタリングすると、修正箇所が膨大になりそうなので、前回の質問の回答を修正する形でいきましょう。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

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

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

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/03/16 01:49

    ありがとうございます。
    mokeさんの言う通り修正したら無事に表示できました。

    ですが、後1点だけ解決できない部分があって、
    mokeさんにご教授頂いたように記述するとリンクが
    <a href="/timelines/5?notification_id=45">投稿title</a>
    このように表示されてしまいリンクが生成されません。

    これは何が原因なんでしょうか?

    キャンセル

  • 2017/03/17 11:21

    .html_safe
    と末尾(ifの前につけて下さい)

    キャンセル

  • 2017/03/17 12:31

    ありがとうございました。
    html_safeメソッドを使用したら解決できました。

    まだまだ自分の知識量が少なくて分からないことが多いので、今後さらに頑張っていこうと思います。
    ありがとうございました!

    キャンセル

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

  • ただいまの回答率 87.77%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

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