Нужно ли делать DI для простых Kotlin объектов?

Question

[Tsudzukeru]

Есть Rx метод, внутри которого я делаю некоторое преобразование ответа от сервера с помощью kotlin
object - RawMediaPreviewResponseConverter .

...
.subscribe({
          val mediaPreviewResponse = RawMediaPreviewResponseConverter.getMediaPreviewResponse(it)
          _downloadedMediaPreviewsResponse.postValue(mediaPreviewResponse)
          _networkState.postValue(NetworkState.LOADED)
                    }
...

Если я использую DI (Dagger) - это значит мне нужно прям для всех зависимостей делать DI? Даже для таких маленьких классов как мой RawMediaPreviewResponseConverter? В нем всеголишь два метода и больше ничего нет.

Answer 1

[Денис Загаевский]

Вообще - да, нужно всё внедрять. Давай приведу пример. Вот у тебя какой-то медиа превью конвертер. Не знаю, чем он занимается, но предположим, он форматирует некое превью, фото или видео, так? В этом случае может понадобиться использовать контекст для получения размеров экрана, тк превью на разных экранах может иметь разные размеры. Вот тут-то di и поможет.

Comments

[Tsudzukeru]

Денис, а если DI только усложняет код? Например у меня есть фргмент с RecyclerView.
Этот RecyclerView я инициализирую с помощью findViewById(), но так же мне нужно для него создать адаптер.

Адаптер можно внедрить, а можно создать руками.
Внутри адаптера нужно хранить ссылку на viewModel, чтобы дергать ее методы. Получается мне нужно инджектить ViewModel в Adapter, если я буду делать DI адаптера в фрагмент. А раз так, то нужно так же передавать ссылку на фрагмент. Получается примерно вот такой код:

@PreviewScope
class MediaPreviewAdapter @Inject constructor(
    val providerFactory: ViewModelProviderFactory,
    val picasso: Picasso,
    val searchParams: SearchParams,
    val previewFragment: PreviewMediaFragment
) :
    RecyclerView.Adapter<RecyclerView.ViewHolder>() {

    val viewModel =
        ViewModelProviders.of(previewFragment, providerFactory)
            .get(PreviewMediaViewModel::class.java)

...

}

Ну и компонент выглядит вот так:

@PreviewScope
@Subcomponent(modules = [ViewModelFactoryModule::class, PreviewMediaViewModulesModule::class, PreviewModule::class])
interface PreviewComponent {

    @Subcomponent.Factory
    interface Factory {
        fun create(@BindsInstance fragment: PreviewMediaFragment): PreviewComponent
    }

    fun inject(previewMediaFragment: PreviewMediaFragment)
}

В фрагменте делаю внедрение в onAttach() вот так:

override fun onAttach(context: Context) {
        super.onAttach(context)
        Log.i(TAG, "onAttach: ")
        (requireActivity().application as BaseApplication).appComponent.getPreviewComponent()
            .create(this).inject(this)
    }

Код работает, но я не знаю норм это или нет?
Просто если руками создать адаптер - код выглядит гораздо проще.

[Денис Загаевский]

Tsudzukeru, почему бы ViewModel тоже не держать в графе? В модуле сделай Provides- метод, в который прими фабрику и фрагмент, а верни вьюмодель. Это с точки зрения DI.
А с точки зрения архитектуры, конечно, адаптер не должен знать ни про какую ViewModel. У него должен быть метод для обновления списка элементов и всё, а работа с вьюмоделью должна быть вне его.

Answer 2

[Alex Wells]

Ну, если у твоего конвертера нет зависимостей, он не open и не интерфейс - то можно и object, разницы вообще никакой.