Повтор запроса и его кеширование?

Question

[Analka]

Всем привет. Возник вопрос

у меня часто повторяется вот этот запрос

Article::find($id);

я его хочу закешировать, но как быть, если он сидит в нескольких сервисах и контроллерах?

Я думал вынести его в класс ArticleQuery.php и там хранить запросы по модели Article

class ArticleQuery 
{
   public function getById(int $id) {
      Cache::remember('article_'.$id, $minutes, function () {
        return Article::find($id);
     });
   }
}

и вызывать уже типа
$articleQuery->getById($id)

Как кто решает такие проблемы?

Answer 1

[pLavrenov]

Это решается вот так

По сути это слой получения данных. в контроллерах надо будет получать из репозитория и тогда если понадобится изменить способ получения записей (или например ключ кэша) достаточно будет поменять только в репозитории.

И в твоем коде ошибка. ID не передается в колбэк а $minutes вообще не существует.

Comments

[Analka]

пример я накидал так, но опять же, я не собираюсь уходить с eloquent, поэтому даже и не знаю, нужен ли repository паттерн + с репо нужно возвращать массив, а я везде модели использую, вот и делема

Но по сути ArticleQuery у меня и есть репо, только на получение данных

[pLavrenov]

Analka, это не уход от Eloquent. Не нужно переписывать стандартные методы в репозитории. В интернетах пишут эту ерунду всякие Симфонисты.

Можешь рассматривать репозиторий как API, это методы получения определенных данных. Например получить запись по ID (пусть будет getArticleById) и тогда в любом месте проекта ты получаешь запись по ID вот так $articleRepository->getArticleById(5) а не обычным методом повторения одного и того же. А почему оно должно возвращать не коллекцию а массив вообще не понятно. Возвращай коллекцию.

Вот готовый пример метода репозитория.

public function getArticleById(int $id) : Article
{
      return Cache::remember('article_' . $id, $minutes, function () use ($id) {
         return Article::find($id);
     });
}
public function getArchiveArticles() : Collection
{
      return Cache::remember('articles_archive' , $minutes, function () {
         return Article:: // Тут условие что такое архивные записи
     });
}