problème d'insertion dans une listItem + android

Version imprimable

Bonjour,
Je veux lister les images et les videos de la sdcard dans une listView comme le montre ce bout de code

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
 File rep = new File("/sdcard");
         String [] liste;
         liste=rep.list();
        listSD = (ListView) findViewById(R.id.listSD);
		  	ArrayList<HashMap<String, String>> listItem = new ArrayList<HashMap<String, String>>();
		  	listItem.clear();
		  	HashMap<String, String> mapvid;		  	
		    mapvid = new HashMap<String, String>();
		    mapvid.clear();
 
		    HashMap<String, String> mapimg;
		    mapimg = new HashMap<String, String>();
		    mapimg.clear();
		    int i=0;
			   for( i=0 ;i<liste.length;i++)
			{if(liste[i].endsWith(".jpg"))
			{ mapimg.put("imgelt", String.valueOf(R.drawable.imgimg));     
		      mapimg.put("nomelt",liste[i]);
		       listItem.add(mapimg);}
			if(liste[i].endsWith(".3gp"))
			{ mapvid.put("imgelt", String.valueOf(R.drawable.imgvid));     
			     mapvid.put("nomelt",liste[i]);
			     listItem.add(mapvid);}     
			}
		   SimpleAdapter mSchedule = new SimpleAdapter (this.getBaseContext(), listItem, R.layout.affichnomeltsd,
			 new String[] {"imgelt","nomelt"}, new int[] {R.id.imgelt,R.id.nomelt});
			 //On attribut à notre listView l'adapter que l'on vient de créer
		     listSD.setAdapter(mSchedule);

Mon problème se manifeste dans le nom des videos et des images.
La listItem lors du .add change le nom de ces composants avec le dernier nom.
Tout les images sont nommés le nom de la dernière image et tous les vidéo sont nommés le nom de la dernière vidéo.
Votre aide s'il vous plait

09/05/2012, 15h02
Ryu2000
C'est parce que tu utilises des HashMap.
Une ArrayList d'HashMap c'est n'importe quoi, soit dit en passant.
Ce serait beaucoup mieux de créer sa classe avec imgelt et nomelt et d'avoir une ArrayList de cette classe, mais enfin bon...

Bon j'y connais pas grand chose en HashMap, mais à mon avis tu dois faire :
Code:

1 2 3 4 5 for (...) { monHashMap = new HashMap <String, String> (); monHashMap.put("maClé", maValeur); monArrayList.add(monHashMap); }
Il faut que tu créer un nouveau HashMap à chaque fois.
Ça doit être ça le problème.
09/05/2012, 15h19
MrDuChnok

Bonjour,

Très difficile de relire ton code sans indentation...
De ce que je vois tu as une erreur d’algorithmie tu ajoutes dans ta hashtable toujours la même instance de ton hashmap.
Et vue que ce hashmap tu le modifie à chaque fois que tu trouves une image (respectivement une vidéo), le nom de ton image (resp. vidéo) est à chaque fois changé...

edit : oups trop long à répondre ;)

Merci à tous,
J'ai résolue le problème :ccool:qui était causé par l'instanciation

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
 
File rep = new File("/sdcard");
String [] liste;
liste=rep.list();
listSD = (ListView) findViewById(R.id.listSD);
ArrayList<HashMap<String, String>> listItem = new ArrayList<HashMap<String, String>>();
       	listItem.clear();
        int i=0;
      for( i=0 ;i<liste.length;i++)
      {if(liste[i].endsWith(".jpg"))
      { HashMap<String, String> mapimg;
	mapimg = new HashMap<String, String>();
	mapimg.clear();
	mapimg.put("imgelt", String.valueOf(R.drawable.imgimg));     
	mapimg.put("nomelt",liste[i]);
	listItem.add(mapimg);}
 
	if(liste[i].endsWith(".3gp"))
	   {HashMap<String, String> mapvid;		  	
	    mapvid = new HashMap<String, String>();
	    mapvid.clear();
            mapvid.put("imgelt", String.valueOf(R.drawable.imgvid));     
	    mapvid.put("nomelt",liste[i]);
	    listItem.add(mapvid);}     
			}
 
SimpleAdapter mSchedule = new SimpleAdapter (this.getBaseContext(), listItem, R.layout.affichnomeltsd, new String[] {"imgelt","nomelt"}, new int[] {R.id.imgelt,R.id.nomelt});
 //On attribut à notre listView l'adapter que l'on vient de créer
		     listSD.setAdapter(mSchedule);

Tu devrais présenter ton code de façon plus lisible, genre comme ça :

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
File rep = new File("/sdcard");
String [] liste;
liste=rep.list();
listSD = (ListView) findViewById(R.id.listSD);
ArrayList<HashMap<String, String>> listItem = new ArrayList<HashMap<String, String>>();
listItem.clear();
int i=0;
for( i=0 ;i<liste.length;i++){
	if(liste[i].endsWith(".jpg")){ 
		HashMap<String, String> mapimg;
		mapimg = new HashMap<String, String>();
		mapimg.clear();
		mapimg.put("imgelt", String.valueOf(R.drawable.imgimg));     
		mapimg.put("nomelt",liste[i]);
		listItem.add(mapimg);
	}
	if(liste[i].endsWith(".3gp")){
		HashMap<String, String> mapvid;		  	
		mapvid = new HashMap<String, String>();
		mapvid.clear();
		mapvid.put("imgelt", String.valueOf(R.drawable.imgvid));     
		mapvid.put("nomelt",liste[i]);
		listItem.add(mapvid);
	}     
}
SimpleAdapter mSchedule = new SimpleAdapter (this.getBaseContext(), listItem, R.layout.affichnomeltsd,new String[] {"imgelt","nomelt"}, new int[] {R.id.imgelt,R.id.nomelt});
//On attribut à notre listView l'adapter que l'on vient de créer
listSD.setAdapter(mSchedule);

C'est plus lisible.