Discussion :

[Guittoon]

Bonjour à tous,
Je n'arrive pas à me dépétrer de cette erreur NetworkOnMainThreadException. Biensur, je sais qu'il ne faut pas faire du Network depuis le mainThread. J'ai mis mon code de test dans un Thread, et cela fonctionne.

Mais en fait je dois communiquer avec une api SOAP web, et donc effectuer plusieurs requêtes à la suite, et ce, de façons conditionnelles. Ex :

if (! identified) {
.. httpRequest1 : login
} else {
.. if (currentDB == "toto") {
.... httpRequest 2;
.... httpRequest 3;
}

donc si je commence a jouer avec des Handler et des Runnables, je vais en faire pleins... c'est pourri !

Je ne vois pas comment faire...

Pouvez-vous m'aider ?

[Simon MARQUIS]

Une solution simple est d'utiliser les AsyncTask.
Elles séparent bien les tâches à effectuer dans le UiThread et celles à faire en arrière plan.

La méthode doInBackground comme son nom l'indique, est l'endroit où tu peux faire tes requêtes web.
Et onPostExecute te permet de modifier et d'intérragir avec le Thread graphique lorsque ta tâche est finie.

[Guittoon]

Merci pour ta réponse rapide. J'ai essayé sans succès :

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
		tv = (TextView) findViewById(R.id.textView1);
		button = (Button) findViewById(R.id.button1);
		button.setOnClickListener(new View.OnClickListener() {
			public void onClick(View v) {
				class Sess extends AsyncTask<Void, Integer, Long> {
					protected Long doInBackground(Void... arg0) {
						linkSession.isSessionOpened();
						linkSession.executeBatch(false, null, null);
						return null;
					}
 
					protected void onPostExecute(Long result) {
						tv.setText(linkSession.getSoapResponse());
					}
				}
				new Sess().execute();
			}
		});

Mon isSessionOpened construit simplement la requête SOAP dans un String.
Mon executeBatch effectue le post HTTP de la requête sur le serveur, c'est à ce moment que çà plante :

Code :

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

06-11 14:16:40.035: E/AndroidRuntime(2189): Caused by: java.lang.RuntimeException: Can't create handler inside thread that has not called Looper.prepare()

C'est pas très propre, désolé, pour le moment je valide le process avant de faire propre.

[Simon MARQUIS]

En effet, ce n'est pas propre, et ça te fait faire des erreurs
La classe devrait être définie à l'exterieur du onClick.

[Guittoon]

Oui, mais j'ai beau la sortir du onClick... ça plante toujours :

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
public class MainTestSOAP extends Activity {
	private TextView tv;
	private Button button;
	private TLinkSession linkSession = new TLinkSession();
 
	class Sess extends AsyncTask<Void, Integer, Long> {
		protected Long doInBackground(Void... arg0) {
//			Looper.prepare();
			linkSession.isSessionOpened();
			linkSession.executeBatch(false, null, null);
			return null;
		}
 
		protected void onPostExecute(Long result) {
			tv.setText(linkSession.getSoapResponse());
		}
	}
 
	@Override
	protected void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		setContentView(R.layout.activity_main_test_soap);
 
		tv = (TextView) findViewById(R.id.textView1);
		button = (Button) findViewById(R.id.button1);
		button.setOnClickListener(new View.OnClickListener() {
			public void onClick(View v) {
				new Sess().execute();
			}
		});
	}
}

J'ai tenté de mettre le Looper.prepare(); mais je ne vois pas pourquoi ni comment cela fonctionne...

[nicroman]

Non non le Looper.prepare() permet de créer une pile de message, chose que le thread UI a déjà.

Là on dirait que le onClick() est reçu dans un autre thread que celui de l'UI, ce qui est très étrange !

On peut avoir la stacktrace complète ? (et pas seulement l'exception) ?

[Guittoon]

Voilà :

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
06-11 14:58:10.079: W/dalvikvm(3176): threadid=11: thread exiting with uncaught exception (group=0x40a71930)
06-11 14:58:10.096: E/AndroidRuntime(3176): FATAL EXCEPTION: AsyncTask #1
06-11 14:58:10.096: E/AndroidRuntime(3176): java.lang.RuntimeException: An error occured while executing doInBackground()
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at android.os.AsyncTask$3.done(AsyncTask.java:299)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:352)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at java.util.concurrent.FutureTask.setException(FutureTask.java:219)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at java.util.concurrent.FutureTask.run(FutureTask.java:239)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:230)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1080)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:573)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at java.lang.Thread.run(Thread.java:856)
06-11 14:58:10.096: E/AndroidRuntime(3176): Caused by: java.lang.RuntimeException: Can't create handler inside thread that has not called Looper.prepare()
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at android.os.Handler.<init>(Handler.java:197)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at android.os.Handler.<init>(Handler.java:111)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at com.gringo.soapproxy.HTTPConnector.<init>(HTTPConnector.java:23)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at com.gringo.soapproxy.TLinkSession.executeBatch(TLinkSession.java:19)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at com.gringo.soapproxy.MainTestSOAP$2$Sess.doInBackground(MainTestSOAP.java:44)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at com.gringo.soapproxy.MainTestSOAP$2$Sess.doInBackground(MainTestSOAP.java:1)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at android.os.AsyncTask$2.call(AsyncTask.java:287)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	at java.util.concurrent.FutureTask.run(FutureTask.java:234)
06-11 14:58:10.096: E/AndroidRuntime(3176): 	... 4 more
06-11 14:58:12.326: I/Process(3176): Sending signal. PID: 3176 SIG: 9

[nicroman]

Ben voilà... on a la réponse.... com.gringo.soapproxy

Je ne sais pas d'ou ca vient, mais le truc crée déjà un handler (probablement pour lancer un thread ?)... à mon avis il nous manque des infos sur la librairie soap utilisée, sa documentation (si il y a un handler, la gestion des threads doit être intégrée), etc...

[Guittoon]

You good men !

Suffisait de lire les logs en fait... pauvre de moi.
com.gringo c'est moi, c'est mon api SOAP. En fait j'avais fait 2 méthodes pour faire la requête : une synchrone et une asynchrone. La version asynchrone utilise un handler, que j'avais instancié dans la classe et non dans la méthode... oups.

J'ai encore pas mal de questions futures, mais merci beaucoup pour aide !