android10 / Android-CleanArchitecture-Kotlin

This is a movies sample app in Kotlin, which is part of a serie of blog posts I have written about architecting android application using different approaches.
https://fernandocejas.com/2018/05/07/architecting-android-reloaded/
4.64k stars 921 forks source link

Why MovieDetailsAnimator is not injected as a @Singleton? #83

Open minchaej opened 4 years ago

minchaej commented 4 years ago

First of all, thank you so much for this awesome repository.

I have a question about dependency injection: You inject MovieDetailsAnimator @Inject lateinit var movieDetailsAnimator: MovieDetailsAnimator at line 60, MovieDetailsFragment.kt.

And MovieDetailsAnimator is constructed through class MovieDetailsAnimator @Inject constructor()

Wouldn't it be better if MovieDetailsAnimator was constructed with @Singleton annotation? Something like: @Singleton class MovieDetailsAnimator @Inject constructor() It seems like there is no point on getting a new instance of MovieDetailsAnimator for every fragment.

Thank you! 👍