Каким ещё способом, можно вывести уведомления?

Question

[Dimon3x]

Вывожу уведомления.

В таблицу notifications в дату добавляются данные, таким способом

public function toDatabase($notifiable) {
        return [
            'post_id' => $this->details['post_id'], // id поста
            'comment_id' => $this->details['comment_id'], //id комента
            'user_comment_id' => $this->details['user_comment_id'] // id юзера, который оставил коммент
        ];

Теперь мне надо вывести список уведомлений, что бы было видно, кто оставил комент,
и на какой пост, сделал это так

public function readNotification()
    {      
         $user = User::find(Auth::id());
       
         $allData = collect();
       
          foreach ($user->unreadNotifications as $notification) {
              $collect1 = collect(Post::find($notification->data['post_id']));
              $collect2 = collect(['user_comment' => Comment::find($notification->data['comment_id'])->text]);
              $collect3 = collect(['user_name' => User::find($notification->data['user_comment_id'])->name]);
             
              $allData->push($collect1
                      ->merge($collect2)
                      ->merge($collect3)
                      ->merge(['id_notif' => $notification->id]));
          }
         
         
       
         return view('read-notifications', [
            'allData' => $allData,
             ]);
    }

<section class="section main-content">
        <div class="container">
         
           @foreach ($allData as $data)
            <div class="card">
                <div class="card-body">
                  <h5 class="card-title">{{$data['user_name']}}, добавил комментраий к посту - {{$data['description']}}</h5>
                  <div class="card-text">{!!$data['user_comment']!!}</div>
                  <br>
                  <a href="/post/{{$data['id']}}" class="btn btn-primary">Перейти</a>
                  <a href="/mark-notif-read/{{$data['id_notif']}}" class="btn btn-primary">Отменить прочитанным</a>
                </div>
             </div>
           <br>
            @endforeach
           
        </div>
      </section>

Это нормальный способ? Какой есть по лучше?

Comments

[JhaoDa]

«по лучше» в каком плане? Что не устраивает в этом способе?

Рекомендую проверять вопрос после написания на ошибки и опечатки.

[Dimon3x]

JhaoDa, не устраивает манипуляция с коллекциями, говорят это плохая практика

$collect1 = collect(Post::find($notification->data['post_id']));
              $collect2 = collect(['user_comment' => Comment::find($notification->data['comment_id'])->text]);
              $collect3 = collect(['user_name' => User::find($notification->data['user_comment_id'])->name]);
             
              $allData->push($collect1
                      ->merge($collect2)
                      ->merge($collect3)
                      ->merge(['id_notif' => $notification->id]));

[JhaoDa]

Dimon3x, у тебя весь код — сборник плохих практик, а ты о коллекциях переживаешь...

[Dimon3x]

JhaoDa, почему весь?

[Dimon3x]

tommy-vercetti,

Так лучше?

$user = User::find(Auth::id());
       
 
          foreach ($user->unreadNotifications as $notification) {
               $aaa = Comment::with('post)
                      ->with('users')
                      ->where('post_id', $notification->data['post_id'])
                      ->where('user_id', $notification->data['user_comment_id'])
                      ->where('id', $notification->data['comment_id'])
                      ->get();
           
              dump($aaa);
             
         }

[теплый чай]

Dimon3x, da

[Alex Wells]

tommy-vercetti, единственная правильная реализация. Автор - гугли "проблема n+1"

[Dimon3x]

tommy-vercetti, Делаю так

$unreadNotifications = $user->unreadNotifications()
            ->with('users')
            ->get();

и мне выводит

Call to undefined relationship [users] on model [Illuminate\Notifications\DatabaseNotification].

[Dimon3x]

tommy-vercetti, не понимаю, как реализовать отношение Notification--->User

[JhaoDa]

Dimon3x, документацию читай, тебе же сказали.

[Dimon3x]

tommy-vercetti, у меня сейчас в папке Notifications, класс Comments расширяется от Notification, то есть мне надо его убрать и расширить от DatabaseNotification?

[Dimon3x]

tommy-vercetti, я самоучка, и двоечник по математике.

[JhaoDa]

Dimon3x, а где тут математика?

[Dimon3x]

JhaoDa, со сложной логикой в программировании, у меня проблемы