Size: a a a

2017 January 10

AZ

Alexandr Zherebtsov in Dagger 2
если выбирать между классом с конструктором с 10 зависимостями и классом с 10 зависимостями филдами, то я выберу конструктор, так хотя бы эта проблема будет на виду)
источник

AZ

Alexandr Zherebtsov in Dagger 2
там несколько NotificationManager разных
источник

AZ

Alexandr Zherebtsov in Dagger 2
их можно в фасадик
источник

AZ

Alexandr Zherebtsov in Dagger 2
есть паттерны для рефакторинга такого
источник

DK

Dmitrii Korotovskii in Dagger 2
Не, посмотри там они за совершенно разное отвечают
источник

AZ

Alexandr Zherebtsov in Dagger 2
а если бы они по классу были бы размазаны зависимости, то сразу даже не увидишь
источник

DK

Dmitrii Korotovskii in Dagger 2
Получится фасадик-салат
источник

DK

Dmitrii Korotovskii in Dagger 2
Смысла в нем не особо ИМХО
источник

AZ

Alexandr Zherebtsov in Dagger 2
ну возможно)
источник

AZ

Alexandr Zherebtsov in Dagger 2
разбить на классы с более узким кругом задач нужно выходит
источник

DK

Dmitrii Korotovskii in Dagger 2
Этот класс отвечает за рассылку писем пользователям, учитывая их настройки периодичности
источник

AZ

Alexandr Zherebtsov in Dagger 2
но на практике конечно же не всегда single responsibility работает
источник

DK

Dmitrii Korotovskii in Dagger 2
Реально там $mailer и $templating надо сворачивать в отдельный менеджер
источник

DK

Dmitrii Korotovskii in Dagger 2
$pluralizer туда же. Все же это view слой.
источник

DK

Dmitrii Korotovskii in Dagger 2
Тогда будет более менее )
источник

AZ

Alexandr Zherebtsov in Dagger 2
источник

AZ

Alexandr Zherebtsov in Dagger 2
источник

AZ

Alexandr Zherebtsov in Dagger 2
@korotovskii собственно ваш вопрос
источник

AZ

Alexandr Zherebtsov in Dagger 2
про корни компоновок, не обращайте внимание что там левый фреймворк, андроид ведет себя точно также, в плане создания фрагмента/активити
источник

DK

Dmitrii Korotovskii in Dagger 2
Да. Локатор так себе идея. В книжке не врут )
источник