Discussion :

[ensinienne]

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

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
 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

[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 :

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

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.

[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

[ensinienne]

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

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
 
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);

[Ryu2000]

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

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
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.