AsyncTask e rotazione Activity!

[Orochi]

Come da titolo, come diavolo si gestisce la rotazione dello schermo se nell'activity c'è un AsyncTask in esecuzione?

Il codice della mia activity al momento è il seguente:
<br /> private AsyncTask<String, Void, String> loaderTask;<br /> <br /> public void onCreate(Bundle savedInstanceState) {<br /> super.onCreate(savedInstanceState);<br /> setContentView(R.layout.main);<br /> <br /> String url = "http://XXXXXXXXXX"; //UN URL QUALSIASI<br /> <br /> loaderTask = (AsyncTask<String, Void, String>)getLastNonConfigurationInstance();<br /> if(loaderTask == null) {<br /> loaderTask = new LoaderTask();<br /> }<br /> <br /> // VERIFICO LO STATO DEL TASK<br /> AsyncTask.Status state = loaderTask.getStatus();<br /> if(state == AsyncTask.Status.PENDING) {<br /> loaderTask.execute(url);<br /> } else if(state == AsyncTask.Status.FINISHED) {<br /> change(((LoaderTask)loaderTask).result);<br /> }<br /> }<br /> <br /> @Override<br /> public Object onRetainNonConfigurationInstance() {<br /> return loaderTask;<br /> }<br /> <br /> private void change(String file) {<br /> //qua gestisco il risultato dell'AsyncTask<br /> //...<br /> }<br /> <br /> private class LoaderTask exten<span class="forTooltip">ds</span> AsyncTask<String, Void, String> {<br /> public String result;<br /> <br /> pr<span class="forTooltip">ot</span>ected String doInBackground(String... urls) {<br /> String ret = null;<br /> <br /> int count = urls.length;<br /> for(int i = 0; i < count; i++) {<br /> //QUA ESEGUO IL TASK E SETTO IL RISULTATO IN RET<br /> //ret = ...<br /> }<br /> return ret;<br /> }<br /> <br /> @Override<br /> pr<span class="forTooltip">ot</span>ected void onPostExecute(String result) {<br /> this.result = result;<br /> // QUA VIENE CHIAMATO AL TERMINE DEL TASK<br /> // PER AGGIORNARE LO STATO DELL'ACTIVITY PADRE<br /> change(result);<br /> }<br /> }<br /> <br />

 

1

​

2

    private AsyncTask<String, Void, String> loaderTask;

3

​

4

    public void onCreate(Bundle savedInstanceState) {

5

        super.onCreate(savedInstanceState);

6

        setContentView(R.layout.main);

7

​

8

        String url = "http://XXXXXXXXXX"; //UN URL QUALSIASI

9

10

        loaderTask = (AsyncTask<String, Void, String>)getLastNonConfigurationInstance();

11

        if(loaderTask == null) {

12

            loaderTask = new LoaderTask();

13

        }

14

15

        // VERIFICO LO STATO DEL TASK

16

        AsyncTask.Status state = loaderTask.getStatus();

17

        if(state == AsyncTask.Status.PENDING) {

18

            loaderTask.execute(url);

19

        } else if(state == AsyncTask.Status.FINISHED) {

20

            change(((LoaderTask)loaderTask).result);

21

        }

22

    }

23

​

24

    @Override

25

    public Object onRetainNonConfigurationInstance() {

26

        return loaderTask;

27

    }

28

​

29

    private void change(String file) {

30

        //qua gestisco il risultato dell'AsyncTask

31

        //...

32

    }

33

​

34

    private class LoaderTask extends AsyncTask<String, Void, String> {

35

        public String result;

36

37

        protected String doInBackground(String... urls) {

38

            String ret = null;

39

​

40

            int count = urls.length;

41

            for(int i = 0; i < count; i++) {

42

                //QUA ESEGUO IL TASK E SETTO IL RISULTATO IN RET

43

                //ret = ...

44

            }

45

            return ret;

46

        }

47

​

48

        @Override

49

        protected void onPostExecute(String result) {

50

            this.result = result;

51

            // QUA VIENE CHIAMATO AL TERMINE DEL TASK

52

            // PER AGGIORNARE LO STATO DELL'ACTIVITY PADRE

53

            change(result);

54

        }

55

    }

56

​

57

​

Ho aggiunto la getLastNonConfigurationInstance() e onRetainNonConfigurationInstance() che dovrebbero servire a passare un oggetto da una activity che si sta distruggendo alla successiva che viene creata, ed in effetti dopo la rotazione quando la nuova activity passa dall'onCreate(), l'AsyncTask viene correttamente recuperato e ho potuto verificare che prosegue tranquillamente il lavoro.
Quello che non succede, e qui sta il problema, è che dopo che l'activity viene ricreata, l'AsyncTask quando chiama la sua onPostExecute() non invoca il metodo dell'activity padre, ho come l'impressione che sia ancora legato alla vecchia activity e pertanto l'activity resta sempre in attesa del termine del task!!!
Da notare che se ruoto di nuovo e il task è completato, ho aggiunto nella onCreate un if per verificare lo stato del task e se è già FINISHED, allora recupero il risultato direttamente dall'istanza dell'AsyncTask (altrimenti restava in attesa di una chiamata che ormai il task non farà più).

Qualche idea, suggerimento, consiglio su cosa fare per risolvere?
Grazie

[Orochi]

Forse faccio domande troppo difficili, ogni volta che chiedo qualcosa io non mi risponde mai nessuno

[Cymon]

Forse faccio domande troppo difficili, ogni volta che chiedo qualcosa io non mi risponde mai nessuno

Io ho pensato "Async task sarà una roba complicata, non ho voglia, a sto thread lascio rispondere Orochi"

[Orochi]

Sì, mi sa che mi toccherà rispondermi da solo

[Orochi]

Poi qualche giorno fa ho risolto! Non ho risposto (direi "mi sono risposto" ) prima perché non ho avuto molto tempo... non so se a qualcuno interessa, ma magari prima o poi servirà una roba simile.
In pratica il problema era quello che sospettavo, l'AsyncTask pare mantenere il riferimento alla vecchia activity, ma ho trovato come aggirare il problema cercando un po' su internet.

<br /> <br /> <b>static</b> class LoaderTask extends AsyncTask<String, Void, String> { <br /> public String result; <br /> public Activity activity;<br /> <br /> <b>void attach(Activity activity) {<br /> this.activity = activity;<br /> }<br /> <br /> void detach() {<br /> activity = null;<br /> }</b><br /> <br /> protected String doInBackground(String... urls) { <br /> String ret = null; <br /> <br /> int count = urls.length; <br /> for(int i = 0; i < count; i++) { <br /> //QUA ESEGUO IL TASK E SETTO IL RISULTATO IN RET <br /> //ret = ... <br /> } <br /> return ret; <br /> } <br /> <br /> @Override <br /> protected void onPostExecute(String result) { <br /> this.result = result; <br /> // QUA VIENE CHIAMATO AL TERMINE DEL TASK <br /> // PER AGGIORNARE LO STATO DELL'ACTIVITY PADRE <br /> <b>activity.</b>change(result); <br /> } <br /> } <br /> <br /> //MENTRE NELLA onCreate l'inzializzazione dell'AsyncTask adesso è così (risparmiatemi di ric<span class="forTooltip">op</span>iare di nuovo tutto ;)<br /> loaderTask = (LoaderTask)getLastNonConfigurationInstance();<br /> if(loaderTask == null) {<br /> loaderTask = new LoaderTask(this);<br /> }<b> else {<br /> loaderTask.attach(this);<br /> }</b><br /> <br /> // E PURE la onRetainNonConfigurationInstance E' CAMBIATA COSI'<br /> @Override<br /> public Object onRetainNonConfigurationInstance() {<br /> loaderTask.detach();<br /> return loaderTask;<br /> }<br /> <br />

50

 

1

​

2

​

3

 <b>static</b> class LoaderTask extends AsyncTask<String, Void, String> {  

4

     public String result;  

5

     public Activity activity;

6

7

     <b>void attach(Activity activity) {

8

          this.activity = activity;

9

     }

10

11

     void detach() {

12

          activity = null;

13

     }</b>

14

​

15

     protected String doInBackground(String... urls) {  

16

         String ret = null;  

17

18

         int count = urls.length;  

19

         for(int i = 0; i < count; i++) {  

20

             //QUA ESEGUO IL TASK E SETTO IL RISULTATO IN RET  

21

             //ret = ...  

22

         }  

23

         return ret;  

24

     }  

25

26

     @Override  

27

     protected void onPostExecute(String result) {  

28

         this.result = result;  

29

         // QUA VIENE CHIAMATO AL TERMINE DEL TASK  

30

         // PER AGGIORNARE LO STATO DELL'ACTIVITY PADRE  

31

         <b>activity.</b>change(result);  

32

     }  

33

 }  

34

​

35

//MENTRE NELLA onCreate l'inzializzazione dell'AsyncTask adesso è così (risparmiatemi di ricopiare di nuovo tutto ;)

36

        loaderTask = (LoaderTask)getLastNonConfigurationInstance();

37

        if(loaderTask == null) {

38

            loaderTask = new LoaderTask(this);

39

        }<b> else {

40

            loaderTask.attach(this);

41

        }</b>

42

​

43

// E PURE la onRetainNonConfigurationInstance E' CAMBIATA COSI'

44

    @Override

45

    public Object onRetainNonConfigurationInstance() {

46

        loaderTask.detach();

47

        return loaderTask;

48

    }

49

​

50

​

Riassumendo, ho aggiunto 2 metodi, attach e detach, che uso per agganciare o sganciare l'istanza corrente dell'activity dall'AsyncTask (così non ci sono manco memory leak) e la onPostExecute chiama il metodo direttamente dall'istanza corrente dell'activity.
Così funziona