Thread ou AsyncTask ?

Version imprimable

12/03/2012, 10h19
Drlord

Thread ou AsyncTask ?

Bonjour,

Après avoir lu l'article http://mathias-seguy.developpez.com/...async_memleak/ sur les thread et asyncTask, je me sens un peu perdu.

Je vous explique mon application ainsi que mon problème.

Mon application fait appel à un webservice sachant que je fais faire appel à une 50ène de méthodes.
Pour presque toutes ces méthodes, le résultat de l'appel au webservice, va avoir un impact sur l'IHM de l'application (affichage de toast, lancement de nouvelle activity...).
Il y a une méthode qui est lancé à partir d'un service car elle doit être effectué toutes les 5 secs.

Au niveau de mon architecture, j'ai une classe "Webservice" où il y a une méthode par chaque requête qui prend comme paramètre, une classe correspondant à la requête et retourne une classe correspondant à la réponse. (Je sais pas si j'ai été bien clair :?)

Pour revenir à ma problématique, j'ai d'abord penser utiliser un thread pour chaque appel à ma classe "Webservice". Mais après avoir lu l'article cité plus haut, je me demande si utiliser un asyncTask ne serai pas plus intéressant...

Drlord
13/03/2012, 16h07
Drlord
Bonjour,

J'avance un peu dans mon application.
Par contre j'ai un réel problème avec les Threads, je ne dois pas bien tout comprendre dans leur fonctionnement.
Code:

1 2 webServiceManager.request("Authentification", aut); String response = webServiceManager.Authentification;
La méthode "request" de mon manager, gère les différentes requête à mon "webservice".
Et après je voudrais récupérer la réponse "Authentification" qui a été mise à jour.
La valeur récupérer ne correspond pas à la requête, ce qui est normal.
Il y aurai la possibilité d'implémenter un handler, mais si j'ai 3 ou 4 requêtes dans une activity, je ne vais pas implémenter autant d'handler?

Et puis mon thread n'est pas dans mon activity, donc pas possible d'utiliser un handler.

Je suis un peu pommé dans tout ça, j'ai sans doute dû ne pas tout comprendre comment implémenter correctement un thread...

Merci.
Drlord.

Vu les messages je pense que tu as bien compris ce qu'était un Thread.
Le probleme c'est qu'on ne peut pas modifier l'UI dans un autre thread que celui de l'UI (le "Main Thread") et on ne peut pas faire d'appels "lents" (réseaux) dans le thread de l'UI.

Donc effectivement, tout appel à un webservice doit se faire dans un thread à part...

Un "Handle" est un objet capable d'executer du code dans un autre thread (par le truchement de messages) il faut que le thread soit bien sur preparé pour cela, et ca tombe bien, c'est le cas du main-thread de l'UI...

Donc l'idée est de créer un Handle dans le thread de l'UI, créer un Thread pour la requete, et lui passer ce handle... A la fin de la requete, le handle est appelé avec la réponse du service...

Par exemple:
Code:

1 2 3 4 5 6 7 8 9 new Thread(new Runnable() { void run() { ... appel au webservice handle.execute(new Runnable() { ... appel au thread UI avec la réponse }); } }).run();
Le probleme de ce code, c'est qu'à moins de faire un classe dédiée d'appels au webservice qui implémente Runnable, il est très compliqué de tout gérer, surtout la durée de vie des objets.
Dans les cas ou la communication se résume entre le thread UI et une opération simple, il vaut alors largement mieux passer par un AsyncTask qui s'occupera:

De gérer le passage des paramètres
De faire l'appel dans un thread à part (doInBackground())
De faire l'appel retour dans le thread de l'UI (doPostExecute())
De gérer le "cancel" eventuel.
Code:

1 2 3 4 5 6 7 8 9 10 MyWebserviceRequest myRequest = ...; new AsyncTask<MyWebserviceRequest,MyWebserviceResponse,Void>() { public MyWebserviceResponse doInBackground(MyWebserviceRequest ... params) { return params[0].execute(); } public void doPostExecute(MyWebserviceResponse resp) { MonActivity.this.onWebserviceResponse(resp); } }).execute(myRequest);

Merci pour tes explications.
Dans mon service, j'ai essayer les 2 méthodes.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
@Override
public int onStartCommand(Intent intent, int flags, int startId) {
	super.onStartCommand(intent, flags, startId);
	Log.d(TAG, "onStart");
	timer.scheduleAtFixedRate(new TimerTask() {
		public void run() {
			// setConnexionTime();
			new MyAsyncTask().execute();
		}
	}, 0, 5000);
	return START_NOT_STICKY;
}

Mon AsyncTask:

Code:

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
class MyAsyncTask extends AsyncTask<Void, Void, String> {
 
		@Override
		protected String doInBackground(Void... params) {
			// Requête de set connection time
			SetconnexionTime sct = new SetconnexionTime();
			sct.userT = infosSession.idUser;
			return webservice.SetconnexionTime(sct);
		}
 
		@Override
		protected void onPostExecute(String result) {
 
			try {
				result = AESEncrypter.decryptTextFromBase64(result);
			} catch (AESEncrypterException e) {
				e.printStackTrace();
				Log.e(TAG, "AESEncrypterException: " + e.getMessage());
				error--;
			}
			Log.d(TAG, "result: " + result);
			if (result.equals("1"))
				error = maxError;
			else
				error--;
			if (error == 0) {
				errorManager.showToastShort(getBaseContext(), getString(R.string.set_connexion_time_error));
			}
		}
	}

Mon thread:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
public void setConnexionTime() {
		Log.d(TAG, "setConnexionTime");
 
		new Thread(new Runnable() {
 
			@Override
			public void run() {
				// Requête de set connection time
				SetconnexionTime sct = new SetconnexionTime();
				sct.userT = infosSession.idUser;
				String result = webservice.SetconnexionTime(sct);
				// Envoie de la réponse au handler
				Message myMessage = myHandler.obtainMessage();
				Bundle messageBundle = new Bundle();
				messageBundle.putString("response", result);
				myMessage.setData(messageBundle);
				myHandler.sendMessage(myMessage);
			}
		}).start();
	}

Mon handler:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
Handler myHandler = new Handler() {
 
		@Override
		public void handleMessage(Message msg) {
			String result = msg.getData().getString("response");
			try {
				result = AESEncrypter.decryptTextFromBase64(result);
			} catch (AESEncrypterException e) {
				e.printStackTrace();
				Log.e(TAG, "AESEncrypterException: " + e.getMessage());
				error--;
			}
			Log.d(TAG, "result: " + result);
			if (result.equals("1"))
				error = maxError;
			else
				error--;
			if (error == 0) {
				Log.e(TAG, getString(R.string.set_connexion_time_error));
				errorManager.showToastShort(getBaseContext(), getString(R.string.set_connexion_time_error));
			}
		}
	};

Tout se passe bien quand je fais ma requête avec mon thread + handler.
Par contre j'ai une erreur quand j'utilise mon AsyncTask:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
03-14 08:12:09.455: W/dalvikvm(8042): Exception Ljava/lang/RuntimeException; thrown during Landroid/os/AsyncTask;.<clinit>
03-14 08:12:09.455: W/dalvikvm(8042): threadid=7: thread exiting with uncaught exception (group=0x4001d800)
03-14 08:12:09.475: E/AndroidRuntime(8042): FATAL EXCEPTION: Timer-0
03-14 08:12:09.475: E/AndroidRuntime(8042): java.lang.ExceptionInInitializerError
03-14 08:12:09.475: E/AndroidRuntime(8042): 	at com.camka.service.VamService$2.run(VamService.java:57)
03-14 08:12:09.475: E/AndroidRuntime(8042): 	at java.util.Timer$TimerImpl.run(Timer.java:289)
03-14 08:12:09.475: E/AndroidRuntime(8042): Caused by: java.lang.RuntimeException: Can't create handler inside thread that has not called Looper.prepare()
03-14 08:12:09.475: E/AndroidRuntime(8042): 	at android.os.Handler.<init>(Handler.java:121)
03-14 08:12:09.475: E/AndroidRuntime(8042): 	at android.os.AsyncTask$InternalHandler.<init>(AsyncTask.java:421)
03-14 08:12:09.475: E/AndroidRuntime(8042): 	at android.os.AsyncTask$InternalHandler.<init>(AsyncTask.java:421)
03-14 08:12:09.475: E/AndroidRuntime(8042): 	at android.os.AsyncTask.<clinit>(AsyncTask.java:152)
03-14 08:12:09.475: E/AndroidRuntime(8042): 	... 2 more

Pour infos voici le code de la ligner 57: new MyAsyncTask().execute();

Je ne vois pas trop ce que j'ai louper dans la construction de mon AsyncTask...

Merci.
Drlord.

15/03/2012, 01h24
nicroman

Tu le fais bien dans le thread de l'UI ?

Parce que new Handle() ne devrait pas marcher à cet endroit là non plus

(il n'y pas de mystère hein ! AsyncTask utilise un handle ! :) )
15/03/2012, 08h13
Drlord

Tout le code que j'ai mis est exécuter dans un service....
Sachant qu'un service n'a pas d'UI même ma méthode avec un thread ne devrait pas fonctionner il me semble.
15/03/2012, 09h20
Hizin
L'erreur vient du fait que tu lances l'AsyncTask dans une autre tâche.

Voici l'arbo pour les appels :
Code:

1 2 3 4 main thread TimerTask MyAsyncTask
Tu demandes explicitement un traitement devant se faire dans le main thread dans un autre thread, et c'est la raison de l'erreur pour MyAsyncTask.

L'arbo devrait être ainsi :
Code:

1 2 3 main thread MyAsyncTask
Si tu veux utiliser les propriétés d'une TimerTask (ce que je pense, sinon, tu ne l'aurais pas utilisé), je te conseils de faire hériter "MyAsyncTask" de "TimerTask".
15/03/2012, 09h28
Drlord

Je vois l'idée par contre je vois pas comment je peut faire hériter MyAsyncTask de TimerTask, vu qu'il hérite déjà de AsyncTask?
15/03/2012, 09h35
Hizin

J'ai loupé la hiérarchie de classes quand j'ai regardé la doc de la TimerTask moi.

Je te conseil de remplacer passer l'héritage de MyAsyncTask de AsyncTask à TimerTask.

Bon, ça fait que tu passes de l'utilisation d'une AsyncTask à l'utilisation d'un Thread, ce qui alourdit un peu la mise en oeuvre, mais qui ne devrait normalement rien changer.