so they re both `this` do I just cast `this` to that View wh

Question

so they re both `this` do I just cast `this` to that View when I need to use the view and use `this` as is when I want to refer to the context

menegatti · Answer

first of all the presenter should receive an instance of `BooksActivityView` instead of a context

oday · Answer

great

menegatti · Answer

then you don t need to pass the context

oday · Answer

I would ve except for the way the DB access is setup

oday · Answer

from an antonleiva article stuck out tongue

oday · Answer

obviously my case has differed

menegatti · Answer

let s say the view is not an activity then the cast from context would not work right

oday · Answer

ah yes

oday · Answer

it wouldn t

menegatti · Answer

so try to keep the presenter ignorant to which elements it is controlling

oday · Answer

great

menegatti · Answer

maybe it is a view maybe it is a fragment etc

oday · Answer

that s its principle anyways true

menegatti · Answer

so my first suggestion if to change `getBooks` to `getBookes view BooksActivityView context Context ` so you don t attach one to the other

oday · Answer

ah I was actually waiting to see if you had a fix to the DB access

menegatti · Answer

on a second step I d would try to move the DB handling layer to a different class so you don t need to pass a context to the presenter

oday · Answer

otherwise yea of course I can pass `this` and `this`

oday · Answer

to which layer regardless where you go it s tied to Context because of that line

menegatti · Answer

the rule of thumb is to try and keep your presenters android free i e without context or any other android specific class

menegatti · Answer

let s say you have a `DbHandler` which is `class DbHandler context Context `

menegatti · Answer

on the activity or using Dagger you can inject this DbHandler into the presenter

oday · Answer

true there is such a class

menegatti · Answer

which means the presenter would never need to know what `DbHandler` depends on

menegatti · Answer

and I think that helps with making the coroutine body a whole lot simpler

oday · Answer

yea let me see what to do about that I already have Dagger set up just didn t add DBHelper yet

menegatti · Answer

wink

oday · Answer

well I ve got that set up now but I m not sure how things are now replaced

oday · Answer

I am able to do dbHelper doAsync but inside no idea how to replicate the database use select

oday · Answer

alright well now it s like this <https hastebin com atalidefav js>

dave08 · Answer

< oday> Did you try thirtyinch mvp lib it helps in a lot of presenter wiring code

oday · Answer

never heard

dave08 · Answer

dave08 · Answer

I use it in most of my projects and it s helps a lot for converting messy old code into MVP which I did quite a few time