Hacker News new | ask | show | jobs
by crla 3518 days ago
I also think the Activity handling of configuration changes was a mistake; it should have only ever just been the view that was recreated.

However, I don't think that in the end it's that horrible, once you overcome the learning curve.

If your process is killed and the user returns to the app, you have to be able to restore state from either the Activity's saved instance state or from some other form of persisted state anyway - whether the current approach to configuration changes was in use or not makes no difference here, it's due to the entire approach of the OS to killing and restoring processes as required (and I believe this same approach is used on iOS and for UWP Windows applications).

For handling of async results - once the owner of an async operation is not tied to a single instance of a view and is instead tied to the overall lifetime of that part of the application (e.g. retained Fragment, a Conductor controller, or pretty much any other reasonably modern view composition library), it's pretty trivial to handle. You're guaranteed that nothing else will run on the main thread for the duration of an Activity destroy/recreate cycle, so you don't need to worry about being in some unknown state. If the view was recreated due to a configuration change, rebind your view model/s when it attaches. If you're being torn down because the user left the retained Fragment/Controller/whatever the operation was tied to, you'll receive an indication of that and you can either send a cancellation signal (e.g. unsubscribe), or if that's not possible, have an isDestroyed check where you receive your result to ignore it. If you're on the back stack and still alive but have no view (i.e. view NPE), don't bind your model immediately - update your view model, and the same code you use to rebind on view reattach will kick in when the user returns and a new view is created and attached.

1 comments

I should add - I do think the learning curve is much worse than it should be, in part due to this.

It would be nice if Google provided some more recommendation and best practices & patterns around this. I have come across and had to work on plenty of code that handles this all very poorly.