Discussion :

[anto2b]

Bonjour,

Je récupère des donnnées Json grâce à la classe JsonParser ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
public class JSONParser {
 
 
   public static JSONObject getJSONFromPost(String url) throws MalformedURLException, IOException, JSONException, ClientProtocolException, IllegalStateException {
       HttpGet httpPost = new HttpGet(url);
       return getJSONFrom(httpPost);
   }
 
   public static JSONObject getJSONFrom(HttpUriRequest req) throws ClientProtocolException, IOException, JSONException, IllegalStateException
   {
 
	DefaultHttpClient httpClient = new DefaultHttpClient();
 
 
	HttpResponse httpResponse = httpClient.execute(req);
 
 
       if (httpResponse.getStatusLine().getStatusCode()>=400) 
           throw new IOException(httpResponse.getStatusLine().getReasonPhrase());
 
	HttpEntity httpEntity = httpResponse.getEntity();
 
       String jsonStr = EntityUtils.toString(httpEntity);
 
       return new JSONObject(jsonStr);
   }
 
}

... dans mon Asyntask

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
private class MyTask extends AsyncTask<Void, Void, ArrayList<Video>> {
 
		private String URL_task = "";
		private JSONObject json = null;
		private String error = null;
                private boolean isRunnig = true;
 
 
                @Override
		protected ArrayList<Video> doInBackground(Void ...unused) {
 
			while (isRunnig) 
			{
 
 
			URL_task = PATH_TO_SCRIPT;
 
 
			try {
 
				json = JSONParser.getJSONFromPost(URL_task);
 
				JSONArray earthquakes = json.getJSONArray("videos");
 
				nb_articles = earthquakes.length();
 
				for (int i = 0; i < earthquakes.length(); i++) 
				{
					JSONObject e = earthquakes.getJSONObject(i);
					Video video = new Video(e.getString("id_video"));
					listVideos.add(video);
 
				}
 
			} catch (Exception ex) {
 
				error = "Couldn't retrive data !";
 
				Log.e("Json", error, ex);
 
			}
 
				isRunnig = false;
 
				if (isCancelled())
				{
				Log.e("asyntask", "isCancelled()");
				break;
				}
		    }
 
			return listVideos;
 
                   } 
 
		protected void onPostExecute(ArrayList<Video> result) {
 
			mTask = null;
 
			if(error != null)
			Toast.makeText(getActivity(), error, Toast.LENGTH_SHORT).show();
 
 
			else {
			           // TRAITEMENT NORMAL DE MA LISTE	
 
                               }
 
		} // onPostExecute();
 
 
		 @Override
		 public void onCancelled ()
		 {
			 isRunnig = false;
		 }
 
	}

Ma question est la suivante pourquoi quand j’exécute le code suivante pour fermer correctement mon asynctask:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
if (mTask != null)
mTask.cancel(true);

(où mTask est de type MyTask ) dans mon onDestroy() / onPause() ma requête HTTP continue de tourner. J'ai vue qu'il fallait utiliser un "abort" mais je ne vois pas l'utiliser dans mon cas. Merci d'avance pour votre aide.

[nicroman]

Je ne comprends pas trop l'interêt du "whille (isRuning)" vu qu'on ne passe dans le test que deux fois: la première au début (isRuning toujours vrai) la deuxieme à la fin (isRuning toujours faux), avec au passage un "break;" du coup inutile...

Il me semble qu'un simple:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
protected ArrayList<Video> doInBackground(Void ...unused) {
    if (!isCancelled()) {
            URL_task = PATH_TO_SCRIPT;
            try {
                 json = JSONParser.getJSONFromPost(URL_task);
                 JSONArray earthquakes = json.getJSONArray("videos");
                 nb_articles = earthquakes.length();
                 for (int i = 0; i < earthquakes.length(); i++)  {
                       JSONObject e = earthquakes.getJSONObject(i);
                       Video video = new Video(e.getString("id_video"));
                       listVideos.add(video);
                  }
             } catch (Exception ex) {
                    error = "Couldn't retrive data !";
                    Log.e("Json", error, ex);
             }
      }
      return listVideos;
}

Suffirait... D'autant que "onCancelled()" n'est appelé *que* quand le doInBackground est revenu ! Donc son utilité ici très relative !

Si il y a du "travail" à effectuer pour fermer la tâche, il suffit de surcharger "cancel" (attention aux synchronisations entre tâches quand même).

Pour le coup, ici, le traitement additionnel semble vraiment compter pour du beurre face à la requête HTTP, donc c'est elle qui doit être stopée.

Hors cette requête (comme toutes les requêtes IO) "attends" des données ("wait") 90% du temps. Donc bien utiliser cancel(true) pour immédiatement interrompre les attentes d'IO.

[anto2b]

Merci pour votre explication. Cependant comme vous le dites la méthode onCancelled() n'est appelé *que* lorsque le doInBackground est terminée donc je ne peux pas appeler mon "abort" dans onCancelled() car la requête se sera exécutée.

Si il y a du "travail" à effectuer pour fermer la tâche, il suffit de surcharger "cancel" (attention aux synchronisations entre tâches quand même).

De plus on ne peut pas surcharger la méthode cancel() de AsyncTask puisque c'est une final méthode.

Je ne vois vraiment pas comment arrêter ma requête HTTP dès le déclenchement du cancel(true)

[John.BE]

Rajouter des if(isRunnig)return ou break; entre +- chaque ligne de ton doInBackGround.

Tant que tu ne sors pas de ta procédure, il ira jusqu'au bout... même si tu finis ton activité.

Je ne sais pas si c'est propre mais bon...

[nicroman]

Le plus important est surtout d'appeler "cancel(true)"

Ou sinon si on "sait" qu'un a une tâche spécifique, faire une fonction qui fait "abort" (même si je n'en vois pas l'utilité) puis appele le cancel().