Home/ Questions/Q 8437353
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-06-10T07:24:24+00:00

I have a few Activities implement an Interface ( called NetworkListener). When requesting information

  • 0

I have a few Activities implement an Interface ( called NetworkListener). When requesting information from a remote server, they spawn an instance of a class (called NetworkRequest). NetworkRequest is a threaded class that sends requests to the server, waits for the response and sends it back to the requesting NetworkListener object. It could take up to 10 seconds to return data back to the NetworkListener instance. During that time, it is quite possible that that Activity or Fragment no longer exists, and a lot of the actions based on the server call
are no longer valid and will throw NPEs, such as:

((TextView) findViewById(R.id.server_status)).setText("Server response: " + response.getMsg());

What’s the best way to ensure that the requesting class is still alive and able to update UI?

Currently at the top of every method that receives calls back from the server I have:

if (getWindow() == null) {  // or getView()/getActivity() if Fragment
            finish();
            return;
        }

This seems inefficient and redundant. Is there a better method to what I’m trying to accomplish?

Thanks!

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    You can move the logic checking state of activity / fragment to the NetworkRequest. Add isActive() method to NetworkListener interface.
    NetwrokRequest sends response to NetworkListener only if the latter is active.

    You can set activity / fragment activeness in onCreate() -> active and onDestroy() -> inactive.

    Just be wary of activity state changes like when rotating screen. Then, you get a new activity which doesn’t get server’s response, which might not be what you want.

    • 0