I'm new to RxJava and using this together with MVP architecture.
I've found a few examples on saving observables upon configuration changes using a retained fragment(still not sure if this is the best way to do it). The examples I've found though is handling observables directly on the Activity or Fragment, not from a Presenter.
So I experimented and set up this quick example(using only Reactivex's RxJava and RxAndroid lib) just to test, which seems to work fine. What this example does is:
- Initiates an activity with a headless retained fragment.
- Push button
- Presenter calls a FakeService for a delayed(5seconds) response observable.
- Presenter does .cache() on this observable.
- Presenter tells the view to retain this observable.
- View saves the observable in a retained fragment.
- Presenter subscribes to observable.
- User does a configuration change(device rotation). User can do this as many times as he wants.
- OnPause tells the Presenter's CompositeSubscription to clear and unsubscribes from all current subscriptions.
- Activity gets recreated and reuses the existing retained fragment.
- Activity's onResume checks if the retained fragment's stored observable is null.
- If not null, tells the Presenter to subscribe to it.
- The retained observable gets subscribed to, and because .cache was called, it just replays the result to the new subscriber without calling the service again.
- When the Presenter shows the final result to the view, it also sets the retained fragment's saved observable to null.
I'm wondering if I'm doing this properly, and if there's a more efficient or elegant way to handle configuration change when the observable's subscription is being handled in a Presenter?
Thanks for the feedback.
Based on this I've reached what I think is a cleaner solution, and I've updated my linked example with the changes.
With the new change; instead of passing the Observable from the Presenter to the Activity to the retainedFragment to be stored incase of a configurationChange event, I rather set the retainedFragment as a second "view" to the Presenter when it's created.
This way when onResume() happens after device rotation, I don't need to make the Activity do the ugly plumbing of passing the Observable from the retainedFragment back to the Presenter.
The Presenter can just interact with this second "view" directly and check for the retained observable itself and resubscribe if needed. The main Activity no longer needs to know about this observable. Suddenly it's a much simpler view layer.