Reputation: 1528
What I would like to do is unsubscribe from events at the moment the ViewModel is no longer needed. I tried implementing IDisposable but nobody calls Dispose(), not Xamarin.Forms nor Prism.Forms.
We have an app created with Xamarin.Forms. We use Prism.Forms to do MVVM. When navigating to a new page (push on stack) Prism.Forms wires the ViewModel to the Page. When navigating back (pop from stack) the ViewModel gets GarbageCollected after a while.
The problem is however that at a point in time we have a couple of the same type of ViewModels with subscriptions to events that are not bound to a View. When the events fire all these ViewModels start doing their thing. So I am looking for a way to unsubscribe at the moment the subscription is no longer needed.
Does anyone have a solution?
Upvotes: 11
Views: 9894
Reputation: 14813
IDestructible
or INavigationAware
BaseViewModel
sample from Prism).
Depending on your object lifecycle :
Destroy
method of IDestructible
interface.OnNavigatedFrom
/OnNavigatedTo
methods on INavigationAware
interface.Bonus:
IDestructible
can be implemented also by the View (and it will called accordingly by Prism when the view is destroyed).
Note:
While the solution above using OnAppearing
/OnDisappearing
works, it induces that ViewModel will depends on a call from View for managing its lifecycle (not clean). Moreover these methods don't exist on ContentView
.
Upvotes: 4
Reputation: 11787
You can make sure the Dispose()
is called in the OnDisappearing()
event of the View, if you want to ensure that ViewModel is not present anymore in the memory than the view.
It is better if you care only about subscribe and unsubscribe of events, then to do it in the OnAppearing()
and OnDisappearing()
. In that case you will be sure of no event handlers been present on the viewmodel once view is not visible.
Upvotes: 8