Giters

antoniolg / androidmvp

MVP Android Example

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

There is a loose coupling between View and Model in main package

alinhayati opened this issue · comments

commented

Thanks for your pioneering wonderful work on MVP in android.
After reviewing the code, I found something not consistent with the passive view MVP that you are implementing. The thing is that in your 'MainActivity' (View), an instance of 'FindInteractorImpl' (Model) is passed as an argument to the 'MainPresenter' constructor. This makes a coupling between View and Model, which is not admissible in passive view implementation on MVP, which rules out any such relation in the UML graph. Please correct me If I am wrong.