Под каждого поставщика не нужно, наоборот репозиторий делается для того чтобы сводить все в 1 поток при необходимости

нет, не так

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

У него же агрегатор

ну смотри у него есть несколько источников newsApi1, newsApi2, newsApi3

Ему требуется 1 метод - getNewsFeed из всех 3х источников

ближе склоняюсь к идеии Кости, один репозиторий, который будет сводить все в 1 поток если есть такая необходимость

репозиторий про другое

Называй это серсивом

ну де-факто да, да вот только репозиторий у меня тоже будет

хех

вот согласно статье кодить буду

http://dfg.gd/blog/decoupling-your-code-in-laravel-using-repositiories-and-services

делайте что хотите

Ну ты б показал пример

я вообще-то сейчас реализую и закину сюда, если захотите посмотрите скажите как вам

Не в контроллере же все сводить

ну понятно в контроллер, только нужные данные выбрасывать

в принципе идея здравая вынести все в отдельные сервисы

для следующего рассширения хорошо

для следующего рассширения хорошо

У каждого сервиса свои настройки

Тут иначе вообще никак, если конечно не хардкодить

я с лаварелем серьезно начал работать только сейчас, когда работаешь с базой только вообще все просто

забиндил данные в духе Product::getAll() во вьюху и все

Палю тему: - Пилим сервисы которые умеют работать с внешними API - Пилим сервис который объеденяет эти сервисы - Пилим репозиторий который будет складывать данные куда нам нужно Делаем так: - Получаем данные из всех API и приводим их к одному формату. - Передаем полученные данные в репозиторий который их сохранит.

Ну а мы про что?

Вопрос только зачем сервис который всех объединяет

Я хз про что вы

Вопрос только зачем сервис который всех объединяет

Ему же нужно

Ему требуется 1 метод - getNewsFeed из всех 3х источников

Ладно я фронтовик же)

Вообще агрегатор нужен если ему нужно в одном месте сразу все получать.

Если нет, то и агрегатор не надо

Хай гайс)) Вопрос в лоб))

га

Как можно упростить?

$tmp = DB::table('users') ->join('pivot_user_group', 'users.id', '=', 'pivot_user_group.user_id') ->join('groups', 'pivot_user_group.group_id', '=', 'groups.id') ->where('pivot_user_group.user_id', '=', $user->id) ->where('pivot_user_group.group_id', '=', $group_id) ->where('pivot_user_group.status', '=', '0') ->where('users.is_delete', '=', '0') ->where('pivot_user_group.is_delete', '=', '0') ->where('groups.is_delete', '=', '0') ->count();

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

по идеии такой вот херни возникать не должно

база до жути нормализованная))

отдельно список пользователей

отдельно список групп

отдельно таблица с их связями

запрос норм работает)) только вот страшный он очень))

оооо, заметил в таблице пользователей нету колонки is_delete

запрос на одну строчку короче)

$tmp = DB::table('users') ->join('pivot_user_group', 'users.id', '=', 'pivot_user_group.user_id') ->join('groups', 'pivot_user_group.group_id', '=', 'groups.id') ->where('pivot_user_group.user_id', '=', $user->id) ->where('pivot_user_group.group_id', '=', $group_id) ->where('pivot_user_group.status', '=', '0') ->where('pivot_user_group.is_delete', '=', '0') ->where('groups.is_delete', '=', '0') ->count();

чет ты не то делаешь

😂😂 привет нормализация, только заговорили о тебе))

шучу

Как ты думаешь где тут косяк зарыт?

Связи нормальные пропиши

И используй их в модели

И не будет ебатни с querybuilder

is_delete -> в ларавел можно использовать soft deletes, он сам будет проверять

Эт и есть querybuilder

Эт и есть querybuilder

И НЕ будет ебатни с querybuilde

Пропиши в модели hasMany свзять и всё.

Оно тебе само соберет запрос

а при soft deletes, отмеченные deleted_at, отдельно нужно потом сортировать в запросе ->select()... или ->count()...

оно автотоматом добавит where deleted_at = null

Если SoftDeletsTrait заюзають в модели

Если SoftDeletsTrait заюзають в модели

А когда билдером делаешь запрос он сможет Soft Delets автоматом включить?

билдером самому надо

аа, ок

А когда билдером делаешь запрос он сможет Soft Delets автоматом включить?

withTrashed() метод добавляешь, тогда в коллекцию удаленные модели включатся

ок

спс

@ в этом чате принято отвечать (reply) на сообщение пользователя, чтобы его поблагодарить.

onlyTrashed() если хочешь получить только удаленные модели

читаю документацию по ларе раздел про отношения

уже раз двадцаты говорю "Ух,.. тыыы"

супруга ругается, "Ты че сново камасутру смотришь?"

onlyTrashed() если хочешь получить только удаленные модели

спс

Спасибо (+1) для @vsbeats принято! Текущая карма +2

билдером самому надо

спс

Спасибо (+1) для @Denormalization принято! Текущая карма +16

Ток withTrashed это метод трейта