データの取得方法、eachメソッドについて

Question

###前提・実現したいこと
rails4 ローカル環境。

現在Pusherを使用し通知機能を実装しました。
通知機能自体は無事実装することができました。
teratailでいうと、ヘッダーのベルマークにお知らせのカウントが表示されるようにはできました。

そして、通知されたものを一覧ページで表示させたいと思ったのですが、一覧ページでのデータの取得方法が分かりません。

受け取るお知らせは「トピック」「タイムライン」の２つあり、どちらもNotificationモデルとアソシエーションしてあります。

現在記述しているコードはこちらです。

notifications_controller.rb

  def index
    # 自分の通知のみ取得、通知は新しい順に並び替える。
    @notifications = Notification.where(user_id: current_user.id).order(created_at: :desc)
  end

notifications/index.html.erb

<h2>お知らせ</h2>
 <% @notifications.each do |notification| %>
   <p>
       <%= notification.comment.user.try(:name) %>さんが
       あなたの投稿（<%= link_to "#{notification.comment.topic.title}", topic_path(notification.comment.topic, notification_id: notification.id) %>）にコメントしました。
   </p>
   <p>
       <%= notification.commenttl.user.try(:name) %>さんが
        あなたの投稿（<%= link_to "#{notification.commenttl.timeline.content}", timeline_path(notification.commenttl.timeline, notification_id: notification.id) %>）にコメントしました。
   </p>
 <% end %>

###発生している問題・エラーメッセージ
現在のnotifications/index.html.erbでの記述では

ActionView::Template::Error (undefined method `user' for nil:NilClass):

とエラーになってしまいます。

トピックとタイムラインどちらかだけのデータなら取得して表示はできるのですが、どちらのお知らせも表示する方法が分かりません。

こういう場合はそもそもeachメソッドではなく違うメソッドを使ったりするのでしょうか？

説明が分かりづらい点もあるかとは思いますが、ご教授のほどよろしくお願いします。

###補足
notificationsテーブル

  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

下記のようにトピック、あるいはタイムラインのみを取得すると、どちらかのデータは正常に表示はできます。
下記の場合はトピックに関するお知らせのみ表示されます。

notifications/index.html.erb

<h2>お知らせ</h2>
 <% @notifications.each do |notification| %>
   <p>
       <%= notification.comment.user.try(:name) %>さんが
       あなたの投稿（<%= link_to "#{notification.comment.topic.title}", topic_path(notification.comment.topic, notification_id: notification.id) %>）にコメントしました。
   </p>
 <% end %>

Answer 1

DB設計(Model定義)ミスかと思います。
とりあえず

<h2>お知らせ</h2>
 <% @notifications.each do |notification| %>
       <%= (content_tag :p ,"#{notification.comment.user.try(:name)}さんが
       あなたの投稿#{<%= (link_to notification.comment.topic.title, topic_path(notification.comment.topic, notification_id: notification.id)}）にコメントしました。") if notification.comment.present? %>
       <%= (content_tag :p ,"#{notification.commenttl.user.try(:name)}さんが
        あなたの投稿（#{<%= (link_to notification.commenttl.timeline.content, timeline_path(notification.commenttl.timeline, notification_id: notification.id)}）にコメントしました。") if notification.commenttl.present? %>
 <% end %>

というふうにすれば、とりあえず思い通りの結果が表示されると思います。
あまりスマートではないですが。

もし必要なら、別質問を立て、各モデルを晒してくださればリファクタリングのお手伝いもできますが。。

Comments

[lapi]

ご回答ありがとうございます。
確かにmokeさんのご指摘通り修正したら表示はできました。

お知らせがない場合は空の値で表示されてしまい思うようにはいきません。

別質問を立て、各モデルを記載させて頂きますので、ぜひリファクタリングのお手伝いして頂けると嬉しいです。

別質問立て終わりましたら、再度こちらでコメントさせて頂きます。

[moke]

一応レイアウトに関してはp　tagをどうにかすればいいので
回答修正するので再度試してみてください

[lapi]

回答修正までありがとうございます。
再度そちらでも試しています。

https://teratail.com/questions/69071?modal=q-comp
こちらの方で新しく質問を立てましたので、お時間があるときに改めてご教授いただけると嬉しいです。

よろしくお願いします。

Answer 2

userメソッドが定義されていないというエラーが出ています。
データベースの構造がわからないので具体的なことは何も言えないのですが、下記の部分が間違っているのではないでしょうか？

notification.comment.user

Comments

[lapi]

ご回答ありがとうございます。
データベースのnotificationsテーブルに関して追記させて頂きました。

現状、トピックあるいはタイムラインのどちらかのみのお知らせを取得する場合は
notification.comment.user
で取得できるのですが、２つのお知らせを取得すると質問したようなエラーになってしまいます。