Size: a a a

2020 January 03

MR

Maxim Rebitskiy in Laravel UA
У тебя используеться $fillable я так понял?
источник

РИ

Роман Игумнов in Laravel UA
Maxim Rebitskiy
У тебя используеться $fillable я так понял?
Да
источник

MR

Maxim Rebitskiy in Laravel UA
Ну тогда ладно
источник

MR

Maxim Rebitskiy in Laravel UA
Но все не вижу смысла тут прокидывать объект, если с него берешь только массив
источник

РИ

Роман Игумнов in Laravel UA
Maxim Rebitskiy
Но все не вижу смысла тут прокидывать объект, если с него берешь только массив
Пока не вьехал. Попробую переварить информацию
источник

РИ

Роман Игумнов in Laravel UA
Вернемся к SRP - public function allOrders()  это лишнее в данном классе или нет? Не понимаю что-то еще больше запутался. Наверное потому что буквально его понимаю
источник

MR

Maxim Rebitskiy in Laravel UA
Ну у тебя класс работы с заказом, почему лишнее? Пока оно не жирное используй там
источник

MR

Maxim Rebitskiy in Laravel UA
Слепое следование принципу единственной ответственности приводит к избыточной сложности приложения, его поддержки и тестированию. SRP стоит применять только тогда, когда это оправдано. Принцип SRP можно применить только в том случае, когда:

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

РИ

Роман Игумнов in Laravel UA
Maxim Rebitskiy
Слепое следование принципу единственной ответственности приводит к избыточной сложности приложения, его поддержки и тестированию. SRP стоит применять только тогда, когда это оправдано. Принцип SRP можно применить только в том случае, когда:

объекту класса становится позволительно слишком много;
доменная логика концентрируется только в одном классе;
любое изменение логики поведения объекта приводит к изменениям в других местах приложения, где это не подразумевалось изначально;
приходится тестировать, исправлять ошибки, компилировать различные места приложения, даже если за их работоспособность отвечает третья сторона;
невозможно легко отделить и применить класс в другой сфере приложения, так как это потянет ненужные зависимости.
Спасибо большое!
источник

РИ

Роман Игумнов in Laravel UA
Кажется я понимать начал!)
источник

MR

Maxim Rebitskiy in Laravel UA
Типа тут имееться виду что класс не должен работать только над созданием
источник

MR

Maxim Rebitskiy in Laravel UA
Такое может быть, но если там только один метод, в этом нет смысла
источник

MR

Maxim Rebitskiy in Laravel UA
В данном случае имеется виду что если класс работает с заказом, то он и должен с ним работать, а не и с заказом, и обрабатывать там клиента, создавать и тд
источник

РИ

Роман Игумнов in Laravel UA
Maxim Rebitskiy
В данном случае имеется виду что если класс работает с заказом, то он и должен с ним работать, а не и с заказом, и обрабатывать там клиента, создавать и тд
Я понял. Т.е тут четко Репозиторий отделить от Сервиса и уже будет хорошо.
источник

РИ

Роман Игумнов in Laravel UA
Вопрос еще имеет ли смысл тут отделять репозиторий от сервиса?
источник

MR

Maxim Rebitskiy in Laravel UA
Имеет ли тут смысл использовать репозиторий?)
источник

РИ

Роман Игумнов in Laravel UA
Maxim Rebitskiy
Имеет ли тут смысл использовать репозиторий?)
😂
источник

РИ

Роман Игумнов in Laravel UA
Maxim Rebitskiy
Имеет ли тут смысл использовать репозиторий?)
Ну тогда что выходит? Перенести это в контроллер?))
источник

MR

Maxim Rebitskiy in Laravel UA
Я не говорю что это верно, но вот как я использую сервис в данном случае, хотя тут нужен рефакторинг
https://github.com/eVentionSoftware/shop/blob/develop/core/Services/SettingsService.php
источник

РИ

Роман Игумнов in Laravel UA
Maxim Rebitskiy
Я не говорю что это верно, но вот как я использую сервис в данном случае, хотя тут нужен рефакторинг
https://github.com/eVentionSoftware/shop/blob/develop/core/Services/SettingsService.php
Спасибо!
источник