Discussion :

[Invité]

Bonsoir à tous,

J'ai un problème sur lequel je bloque depuis de nombreuses heures et qui je pense ne doit pas être très compliqué à résoudre, je m'explique:

J'ai 2 activités,
- La première me permet d'accéder à la deuxième en appuyant sur un bouton.

- Ma deuxième activité est une ListView qui comporte une image (téléchargée sur un serveur HTTP), un titre et une description. Cette ListView est affichée à l'aide d'un adaptateur personnalisé que j'ai crée dérivant de BaseAdapter.

Le problème c'est que dans la méthode getView() de cet adaptateur personnalisé je fais des requêtes HTTP afin de récupérer les images, ce qui est un processus assez lent. Je voudrai donc mettre une boite de dialogue de type ProgressDialog afin d'indiquer à l'utilisateur de patienter un peu avant l'affichage de la liste mais je n'y arrive pas...

Toutes mes tentatives (à l'aide de thread etc..) ont échoué : la boite de dialogue n'apparaissait jamais ou que très rapidement après un long écran noir (celui pendant lequel je voudrai qu'elle s'affiche ...).

J'espère avoir été clair et vous remercie d'avance pour votre aide

[hotcryx]

Tu devrais employer un AsyncTask et coder ton progress dans les onPreExecute, onPostExecute, onProgressUpdate

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
private class BigCalcul extends AsyncTask<Void, Integer, Void>
{
 
	@Override
	protected void onPreExecute() {
		super.onPreExecute();
		Toast.makeText(getApplicationContext(), "Début du traitement asynchrone", Toast.LENGTH_LONG).show();
	}
 
	@Override
	protected void onProgressUpdate(Integer... values){
		super.onProgressUpdate(values);
		// Mise à jour de la ProgressBar
		mProgressBar.setProgress(values[0]);
	}
 
	@Override
	protected Void doInBackground(Void... arg0) {
 
		int progress;
		for (progress=0;progress<=100;progress++)
		{
			for (int i=0; i<1000000; i++){}
			//la méthode publishProgress met à jour l'interface en invoquant la méthode onProgressUpdate
			publishProgress(progress);
			progress++;				
		}	
		return null;
	}
 
	@Override
	protected void onPostExecute(Void result) {
		Toast.makeText(getApplicationContext(), "Le traitement asynchrone est terminé", Toast.LENGTH_LONG).show();
	}
}

[Invité]

Merci de ta réponse, je vais essayer et te tient au courant

[Invité]

Alors, grâce à AsyncTask j'arrive à faire en sorte que ma liste s'affiche avec une image qui indique le chargement pour chaque item le temps que la bonne image ai été téléchargé.

Cependant, j'ai un bug que je n'arrive pas à résoudre --' : durant le chargement de chacune des images, parfois, l'image d'attente est remplacée par une succession de plusieurs images qui correspondent aux images qui doivent s'afficher sur les autres item de la liste (en cours de téléchargement).
De plus, l'image qui doit se trouver à l'item N°i se trouve quelque fois à l'item N°j ..

J'espère avoir été assez clair , merci d'avance de votre aide.

[hotcryx]

Ca me fait penser aux problèmes parfois avec les ressources files, de string décalées, d'un composant à l'autre, si tu le testes avec l'émulateur.

Dans ce cas, je réinstalle l'applic (désinstalle et réinstalle).

[Invité]

Je viens de désinstaller/réinstaller l'application et le problème est toujours le même
Voici le code de mon adaptateur qui fait appel à la fonction de traitement asynchrone (je pense que le problème vient de là) :

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
 
public class BonsPlansAdapter extends BaseAdapter{
	protected List<Societe> entreprise;
	protected LayoutInflater inflater;
	protected static Context context;
 
	public BonsPlansAdapter(Context context, List<Societe> entreprise)
	{
		this.entreprise = entreprise;
		inflater = LayoutInflater.from(context);
		this.context = context;
	}
 
	public int getCount() {
		return entreprise.size();
	}
 
	public static Context getContext()
	{
		return context;
	}
 
	public Object getItem(int position) {
		return entreprise.get(position);
	}
 
	public long getItemId(int position) {
		return position;
	}
 
	static class ViewHolder {
		  public ImageView img;
		  public TextView titre;
		  public TextView description;
		}
 
	public View getView(int position, View convertView, ViewGroup parent) {
		  ViewHolder holder = null;
 
		  // Si la vue n'est pas recyclée
		  if(convertView == null) {
			holder = new ViewHolder();
 
		    // On récupère le layout
		    convertView  = inflater.inflate(R.layout.affichageitem, null);
 
		    // On place les widgets de notre layout dans le holder
        	holder.titre = (TextView) convertView.findViewById(R.id.titre);
		    holder.description = (TextView) convertView.findViewById(R.id.description);
		    holder.img = (ImageView) convertView.findViewById(R.id.img);		        
 
		    // puis on insère le holder en tant que tag dans le layout
		    convertView.setTag(holder);
		  } else {
 
			  // Si on recycle la vue, on récupère son holder en tag
		    holder = (ViewHolder)convertView.getTag();
		  }
 
		// On place dans le holder les informations sur la societe
		holder.img.setImageResource(R.drawable.chargement);				
		holder.description.setText(entreprise.get(position).getDescriptionBonPlan());
	    holder.titre.setText(entreprise.get(position).getNom()); // Insertion d'une image d'attente (pendant le telechargement)
 
	    // Telechargement de l'image qui sera affiché après
	    DownloadImageTask telecharger = new DownloadImageTask(holder.img);
		telecharger.execute(entreprise.get(position).getLogo());
 
	    return convertView;
	}

Merci pour ton aide

[nicroman]

Il faudrait le code du DownloadImageTask

[Invité]

Le voici :

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
79
80
81
82
83
84
85
86
87
88
 
public class DownloadImageTask extends AsyncTask<String, Drawable, Drawable> {
 
	private static HashMap<String, Drawable> listImages = new HashMap<String, Drawable>();	
	private final WeakReference<ImageView> imageViewReference;
 
	public DownloadImageTask(ImageView imageView){
		super();
		imageViewReference = new WeakReference<ImageView>(imageView);
	}
 
		@Override
		protected Drawable doInBackground(String... url) {
 
			Drawable d = loadImageFromUrl(url[0]);
 
			return d;
 
		}
 
		@Override
		protected void onProgressUpdate(Drawable...d){
 
 
		}
 
 
 
 
 
		@Override
		protected void onPostExecute(Drawable d) {
 
 
	        if (imageViewReference != null) {
	            ImageView imageView = imageViewReference.get();
	            if (imageView != null) {
	            	if(d != null)
	            		imageView.setImageDrawable(d);
	            }
	        }
 
		}
 
 
 
 
 
 
	public static Drawable loadImageFromUrl(String url) {
 
		  if (!listImages.containsValue(url)) {
		    InputStream inputStream;
		    try 
		    {
		      inputStream = new URL(url).openStream();
		    } 
		    catch (IOException e) 
		    {
		      throw new RuntimeException(e);
		    }
 
		    Drawable dr = null;
 
		    try
		    {
		    dr = Drawable.createFromStream(inputStream, "src");
		    }
		    catch(Exception e)
		    {
 
		    }
		    if(dr != null)
		    	listImages.put(url, dr);
 
		   return dr;
 
		  } else {
 
		     return listImages.get(url);
 
		  }
 
	}
 
 
 
}

[Invité]

Une idée? Voyez vous une erreur dans ce que j'ai fait ?

[hotcryx]

final avec weakreference?!

[Rokia88]

tu peux essayer de mettre dans onProgessupdate le code d'affichage de la vue qui doit s'afficher le temps de téléchargement de la photo, et ensuite , tu crées une sorte de Handler qui permet d'informer que le traitement de téléchargement est terminé, et donc à ce moment là la méthode qui affiche la photo peut être appelée. car d'après la description de bug que tu nous a donné, j'ai l'impression que l'affichage de la photo interfère avec ta fenêtre d'attente.

[Invité]

Erreur d’inattention hotcryx, je viens d'essayer Rokia88 mais le problème est toujours là...

Je ne comprend vraiment plus rien là --'

[hotcryx]

J'espère que tu as adapté ceci

Code :

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

1
2
3
4
5
6
protected Drawable doInBackground(String... url) {
 
	Drawable d = loadImageFromUrl(url[0]);
 
	return d; 
}

ouille un catch vide dans loadImageFromUrl:

Code :

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

1
2
3
4
catch(Exception e)
{
 
}

[Invité]

Oui oui je l'ai adapté
Je ne comprend vraiment pas pourquoi ça beug

[Rokia88]

je sais pas si la proposition que je vais te donner résoudra ton bug, mais par expérience, les inputstream produisent des comportement incompréhensibles. donc , je te propose d'ajouter un finally à ta méthode loadImageFromUrl pour fermer le inputstream que tu as ouvert en début de la méthode loadImageFromUrl.

[Hizin]

Bon, déjà, PAS DE CATCH VIDE. Tu logs quelques part, tu fais quelque chose d'utile, mais pas d'exception silencieuse. Si c'est la, c'est pour une raison.

Ensuite, pour ton souci :
Tu les vois en multiple car tes ImageView sont réutilisées entre-temps.

Situation de départ : tu affiches ta ListView, les X premiers items sont là, les images correspondantes sont en train d'être téléchargées et seront affichées dans les ImageView passés en paramètre.

Situation suivante : le téléchargement des images est en cours, tu fait un peu défiler ta liste de 3 items. Les 3 ImageView de départ sont remplacées par 3 autres en apparence, mais en réalité, ce sont les même qui sont réutilisées. Les 3 nouveaux items lancent le téléchargement de leurs images respectives.

Situation semi-finale : Tu n'as plus bougé ton UI. Le téléchargement des premiers items est finis. Tu t'aperçois maintenant que tes 3 nouveaux items obtenus précédemment ont des images qui ne sont pas les bonnes.

Situation finale : les derniers items finissent leur téléchargement, les 3 "nouveaux" items ont bien les bonnes images.

Bilan : ne pas passer l'ImageView, celle-ci étant réutilisée. Il faut lui passer le holder, le mettre à jour avec l'image puis dire gentiment au ArrayAdapter que les données ont un peu changé dans le holder et qu'il doit se redessiner (via notifyDataSetChanged()).

Doc : http://developer.android.com/referen...tChanged%28%29
Remarque : c'est pas simple à expliquer ce machin ...

[Invité]

Concernant le catch vide c'est corrigé.

Hizin je viens d'implémenter une solution qui ressemble à ce que tu m'as expliqué et qui fonctionne : donc merci à tous pour votre aide très précieuse !

Et passez de joyeuses fêtes