Ребята всем привет. Может вопрос не в тему, но задам. Есть код https://codeshare.io/5gj3wB  хочу понять первый принцип SOLID (Single Responsibility Principle)
Можете помочь просто черкануть что лишнее в этом классе и что нужно вынести в отдельный?

Например я вынес создание объекта в отдельный класс в методе public function orderRetailStore($request)
вот таким образом CreateOrder::createOrder($request); Верно ли сделал? Что еще лишнее. Помогите плиз понять

Помойму ты пытаешься юзать все не понимая зачем это все. Зачем тебе фабрика для репозитория? Почему репозиторий называется сервисным классом у тебя?

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

Вопрос как раз о том как верно это все разделить чтобы соблюсти принцип SRP

Тут ты пытаешься реализовать фабрику

Твой сервисный класс чисто работает с базой

Соответственно это ещё абстракция над элоквентом

То есть репозиторий для элоквента

Да так и есть!

Понимаю

Сервисный класс реализовывает бизнес логику

В этом есть смысл только в том случае если ты уверен что в будущем откажешься от элоквента

Но зная о том что ты будешь юзать доктрину и продолжать юзать элоквент - нелогично выходит. То есть по факту ты делаешь лишнюю абстракцию

Я тоже в свое время таким упоролся, на каком-то портале видел и думал что так и надо, кроме как головной боли ничего не принесло

Т.е выносить создание фабрики имеет только смысл если я в будущем думаю сменить Элоквент?

Нет, использование репозитория вместе с элоквентом имеет смысл только в том случае если ты думаешь отказаться от элоквента в будущем

Тогда я запутался вообще.... ))))

Репозиторий - абстракция над бд

Элоквент - абстракция над бд