Discussion :

[Invité]

Salutation,
j'ai un souci qui doit être tout bête, mais cela fait tellement longtemps que je bute dessus, que j'arrive plus à analyser mon code objectivement.

voici le bout de code responsable de mon désarrois (a l'interieur d'un Activity):

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
 
	private OnClickListener showPreview = new OnClickListener() {
 
		public void onClick(View v) {
			wait = ProgressDialog.show(v.getContext(), null, "ok", true, false);
			runOnUiThread(new Runnable() {
 
				public void run() {
					URL url;
					try {
						url = new URL(bm.getImageURI());
						Bitmap bitmap = BitmapFactory.decodeStream(url.openStream());
				        preview.setImageBitmap(bitmap);
				        preview.setVisibility(ImageView.VISIBLE);
					} catch (MalformedURLException e) {
						Log.e("ERROR", e.getMessage());
					} catch (IOException e) {
						Log.e("ERROR", e.getMessage());
					}
					wait.dismiss();
				}
 
			});
		}
	};

Je clic sur le bouton magique, rien pendant quelque seconde ... et pouf, le thread se termine sans encombre.
... et attendez, où est passé ma ProgressBar ?
On dirais que le lancement du thread bloc l'affichage de la ProgressBar, c'est pourtant le but du Thread de pas bloquer ?
Si quelqu'un à une idée.
Merci, j'ai honte par avance ;-).

[sinok]

Le runOnUIThread ne doit contenir que les appels qui modifient les composant.

Dans ton cas tu dois créer un thread qui récupère l'image, et une fois l'image récupérée, appelle un runOnUIThread affectant l'image à la preview et rendant la preview visible.

[Sake_reflex]

Comme dit Sinok runOnUiThread permet de lancer un bout de code dans le Thread qui s'occupe de l'interface graphique de l'application.
ton code revient quasiment au même que:

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
 
	private OnClickListener showPreview = new OnClickListener() 
	{ 
		public void onClick(View v) 
		{
			wait = ProgressDialog.show(v.getContext(), null, "ok", true, false);
			URL url;
			try {
				url = new URL(bm.getImageURI());
				Bitmap bitmap = BitmapFactory.decodeStream(url.openStream());
				preview.setImageBitmap(bitmap);
				preview.setVisibility(ImageView.VISIBLE);
			} catch (MalformedURLException e) {
				Log.e("ERROR", e.getMessage());
			} catch (IOException e) {
				Log.e("ERROR", e.getMessage());
			}
			wait.dismiss();
		}
	};

[Invité]

Merci, j'ai pondu un truc diaboliquement affreux pour tester ton conseil sinok:

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
	private OnClickListener showPreview = new OnClickListener() {
 
		public void onClick(View v) {
			wait = ProgressDialog.show(v.getContext(), null, "ok", true, false);
			Thread thread = new Thread(new Runnable(){
 
				public void run() {
					URL url;
					try {
						url = new URL(BMDesktop.getInstance(getApplicationContext()).getImageURI());
						bitmap = BitmapFactory.decodeStream(url.openStream());
						runOnUiThread(new Runnable(){
 
							public void run() {
						        preview.setImageBitmap(bitmap);
						        preview.setVisibility(ImageView.VISIBLE);
								wait.dismiss();					
							}
 
						});
					} catch (MalformedURLException e) {
						Log.e("ERROR", e.getMessage());
					} catch (IOException e) {
						Log.e("ERROR", e.getMessage());
					}
				}
 
			});
			thread.run();
		}
	};

Résultat : toujours pas de ProgressDialog en vue.
Pourtant, j'ai bien un Thread pour récupérer le bitmap et un runOnUiThread pour gérer l'affichage ?
Je me perds dans les scope là.

[Sake_reflex]

Utilise la fonction start et non run pour lancer le Thread

En appelant la fonction run tu exécutes juste ta fonction sans lancer le Thread.
start lance le thread et s'occupe d'appeler la fonction run que tu as défini.

Edit: un conseil, renseigne toi du côté des AsyncTasks pour rendre le code plus joli :
http://developer.android.com/referen...AsyncTask.html

[Invité]

Lol, heureusement que j'avais honte d'avance :-D.
En tout cas un grand merci à vous 2 !

le code corrigé au cas où :

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
 
	private OnClickListener showPreview = new OnClickListener() {
 
		public void onClick(View v) {
			wait = ProgressDialog.show(v.getContext(), null, "ok", true, false);
			Thread thread = new Thread(new Runnable(){
 
				public void run() {
					URL url;
					try {
						url = new URL(BMDesktop.getInstance(getApplicationContext()).getImageURI());
						bitmap = BitmapFactory.decodeStream(url.openStream());
						runOnUiThread(new Runnable(){
 
							public void run() {
						        preview.setImageBitmap(bitmap);
						        preview.setVisibility(ImageView.VISIBLE);
								wait.dismiss();					
							}
 
						});
					} catch (MalformedURLException e) {
						Log.e("ERROR", e.getMessage());
					} catch (IOException e) {
						Log.e("ERROR", e.getMessage());
					}
				}
 
			});
			thread.start();
		}
	};