Как правильно проверять находится ли пост в закладках пользователя?

Question

[root09]

Есть страница где выводятся посты, есть пользователь, у пользователя есть модель Bookmark, в которой хранятся закладки на посты. (user_id, post_id)
На странице со списком с постами нужно показывать кнопку добавить/удалить закладку на пост в зависимости от того есть ли в закладках или нет.

Как это правильно сделать?
Сейчас у меня посты выводятся через foreach и я просто внутрь добавил проверку, но это дает нагрузку.

@foreach($posts as $post) 
     <div class="post">
         @if(Auth::check() && isset(Auth::user()->bookmarkCached[$post->id]))
             <button>Удалить закладку</button>
        @else
            <button>Добавить закладку</button>
        @endif
     </div>
@endforeach

bookmarkCached() - это мутатор в котором находится связь bookmarks кэшированная и переведенная в массив с id постами.

Как это правильно сделать и оптимизировать?

В модели User

public function bookmarks()
    {
        return $this->hasMany('App\Models\Bookmark');
    }
   
   public function getBookmarkCachedAttribute()
    {
        return Cache::remember('bookmarks_cached_list_'.$this->id, 3600, function () {
            $arr = $this->bookmarks->toArray();
            $new_arr = [];
            foreach ($arr as $item)
            {
                $new_arr[] = $item['post_id'];
            }
            return $new_arr;
        });
    }

Смотрю через debugbar ларавела, без этих функций в разделе timeline графа application 200ms скорость, а с этими функциями 450ms, я так понимаю это очень плохо?

Comments

[N]

bookmarkCached() - это мутатор в котором находится связь bookmarks кэшированная и переведенная в массив с id постами.

Код оттуда можно в студию?

[root09]

N, добавил

[N]

root09,

Сейчас у меня посты выводятся через foreach и я просто внутрь добавил проверку, но это дает нагрузку.

Как это проверяли? Что именно это место "грузит"....

Щас другой вариант решения данной задачи напишу...без кэша через мутаторы...сравните...

[N]

root09, У постов есть отношение "обратное" ? Ну т.е. к каждому посту список юзеров его добавивших?

[root09]

N, Нет но могу сделать. По поводу нагрузки через дебагбар смотрел ларавела, там в разделе application без проверок 200ms, а с ними 450ms

[root09]

N,
когда убираю проверки из blade то в 2 раза почти уменьшается время в application

[N]

root09, В ответах написал другой вариант...без всякого кэширования...но его тоже можно придумать, но пока проверь: как с таким вариантом будет работать...

Сколько вообще примерно постов и "закладок" к ним?

[N]

когда убираю проверки из blade то в 2 раза почти уменьшается время в application

Только тот if, который указан в вопросе?

[root09]

N,

Только тот if, который указан в вопросе?

да.

Сколько вообще примерно постов и "закладок" к ним?

на странице по 50 постов

все спасибо, ваш вариант идеален, время в application не увеличивается, столько же сколько и без проверок, правда теперь дополнительный запрос в базу, но думаю это намного лучше чем было.

[N]

root09,

правда теперь дополнительный запрос в базу, но думаю это намного лучше чем было.

Так у Вас в мутаторе тоже доп. запрос при первом обращении каждого юзера и каждый час после последнего обращения))

[root09]

N, а как лучше проверять в таком случае на наличие закладки на странице с самой новостью?

1. Через связь bookmark_users это для каждого пользователя будет создаваться запрос.
2. Засунуть список всех пользователей у которых есть закладка у данной новости в кэш, но этот вариант плох тем, что у новости могут быть десятки тысяч закладок.
3. Через мой старый вариант с кэшем закладок в модели пользователя, но там непонятно от чего нагрузка. Плюс у пользователя тоже может быть около тысячи закладок.

[N]

а как лучше проверять в таком случае на наличие закладки на странице с самой новостью?

Не совсем понял структуру проекта...

Через связь bookmark_users это для каждого пользователя будет создаваться запрос.

Тут тоже не понял...у нас только "текущий" юзер же фигурирует...будет всего два запроса:
1. Получение постов
2. Получение текущего юзера к каждому посту, если он добавил в "закладки"...

Если же речь идёт о выводе постов с простым условием "есть ли этот пост у кого-то в закладках", то можно сделать так:

<?php

$posts = Post::with(['bookmark_users'])
    ->get();

foreach ($posts as $item) {
    
    if($item->bookmark_users->count() > 0) {
        
        // Пост добавлен в закладки одним или несколькими юзерами
        
    }else{
        
        // Никто не добавлял в закладки данный пост
        
    }
    
}

[N]

Кажется понял...Пост = Новость? Если да, то можно так же аналогично:

$CURRENT_USER_ID = \Auth::user()->id;

$rsPost = Post::with([
    'bookmark_users' => function($q) use($CURRENT_USER_ID) {
            $q->where('id', $CURRENT_USER_ID);
        }
    ])
    ->where('id', 'ID_НОВОСТИ')
    ->first();

if($rsPost) {

    if($rsPost->bookmark_users->count() > 0) {
        
        // Удалить
        
    }else{
        
        // Добавить
        
    }

}

[N]

root09, Во всех моих предложенных вариантах будет всегда 2 запроса...не больше - не меньше...
В Вашем случае, думаю, вполне всё будет нормально в плане производительности...

[root09]

N, спасибо. я просто думал возможно есть какой-нибудь вариант с кэшем, потому что например на странице с одним постом есть блок "похожих постов", блок "популярные посты" где тоже нужно выводить закладки, а это уже не один запрос получается.

ну ничего страшного, запросы вроде не тяжелые не должно быть проблем

[N]

root09, Думаю, пока рано об этом думать...получается всего 6 запросов для формирования "детальной" страницы поста(новости)...

[N]

root09, P.S.: Не моя цитата, но тут уместна:

Главное не количество, а качество.

Answer 1

[N]

В модели постов нужно добавить отношение с привязкой к юзерам, которые их добавили...в моём коде это bookmark_users...

<?php

$CURRENT_USER_ID = \Auth::user()->id;

$posts = Post::with([
    'bookmark_users' => function($q) use($CURRENT_USER_ID) {
            $q->where('id', $CURRENT_USER_ID);
        }
    ])
    ->get();

foreach ($posts as $item) {
    
    if($item->bookmark_users->count() > 0) {
        
        // Удалить
        
    }else{
        
        // Добавить
        
    }
    
}

Comments

[jazzus]

для подсчета отношений есть withCount и тогда не придется считать в цикле

[N]

jazzus, Да. Но я тут саму логику показал...по сути with заменить на withCount...
Думаю автор заглянет в документацию и поймёт...

P.S.: Я отвечал в контексте кол-ва запросов к БД...

Answer 2

[jazzus]

$posts = Post::withExists(['bookmarks' => function ($query) {
    $query->where('user_id', Auth::id());
}])
    ->get();

во вьюхе проверять
$post->bookmarks_exists