DaggerAndroidInjector icon indicating copy to clipboard operation
DaggerAndroidInjector copied to clipboard

MVVM issue

Open gaara87 opened this issue 8 years ago • 2 comments

From whatever i've done and read, you have claimed that you are following the MVVM pattern, but I think its incorrect for the ViewModel to contain a reference of the View.

gaara87 avatar May 04 '17 20:05 gaara87

I like my approach because it's easy to write Unit Tests for it. Else I'll have a bit more logic stuck in View. This is my approach.

rakshakhegde avatar May 05 '17 05:05 rakshakhegde

I tend to agree with @gaara87 Here. This is not an ideal use of the MVVM paradigm. UI stuff should never touch the ViewModel.

RJ-Clegg avatar Jul 20 '17 07:07 RJ-Clegg