Discussion :

[Katachana]

bonjour,


je souhaiterais supprimer les doublons au niveau des valeurs et des clefs dans une hasmap lors de l'enregistrement des données :

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
	public void ajouterTacheConges(String nom,String jour) throws ParseException{
		ArrayList<Interval> lst = (ArrayList<Interval>)mapConges.get(nom);
		if(lst==null){
			lst= new ArrayList<Interval>();	
			mapConges.put(nom,lst);
			   		}
		Interval j =new Interval() ;
		if(!"".equals(jour)){
			if(lst.contains(jour)==false){
		j.setJour(jour);
		lst.add(j); 	
		mapConges.put(nom,lst);}}
 
 
	}

la fonction est appelé sur toutes les tâches d'un projet, et a chaque tache est associé, par ressource (s'il y en a plusieurs), des dates de conges.


Merci d'avance!

[djo.mos]

les doublons au niveau des valeurs et des clefs dans une hasmap

! A la rigueur, si tes clés étaient des objets personnels qui n'implémentent pas du tout ou correctement equals() et hashcode(), il se peut qu'un Map contienne des clés en doublons, mais dans ton cas, tu utilises des String comme clés, alors, rassures toi : il n'y aura jamais de clés en doublons dans ton Map !

[Rayndell]

Tu ne peux pas avoir de clés doublons dans une Map. En revanche, pour les valeurs, tu devras vérifier à l'insertion d'une valeur qu'elle n'est pas déjà dans la Map, avec la méthode containsValue.

[CyberChouan]

info complémentaire:

Lorsque tu effectues un "put(cle, valeur);" , si ta clé existait déjà dans la HashMap, ta nouvelle valeur ECRASE la précédente valeur enregistrée pour cette clé: tu perds donc définitivement cette premiere valeur.

Ensuite tout dépend de ce que tu veux faire. Je me suis déjà servi de ce fonctionnement car le dernier objet inséré pour une clé donné était la valeur la plus à jour.

[Katachana]

Merci pour vos réponses. En fait ce que je souhaite faire, c'est pour chaque clé (qui correspond a un nom (string)) vérifiér a chaque fois s'il contient déja la date avant de l'ajouter ou de la refuser.


J'ai essayer comme cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
	public void ajouterTacheConges(String nom,String jour) throws ParseException{
		ArrayList<Interval> lst = (ArrayList<Interval>)mapConges.get(nom);
		if(lst==null){
			lst= new ArrayList<Interval>();	
			mapConges.put(nom,lst);
			   		}
		Interval j =new Interval() ;
		if(!"".equals(jour)){	
			if(!mapConges.containsValue(jour)){
		j.setJour(jour);
		lst.add(j); 		
		mapConges.put(nom,lst);}

Mais j'ai pas le résultat escompté...

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
NOM : JAMES
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18
 -- Jour d'absence : 2007-05-17
 -- Jour d'absence : 2007-05-18

[adiGuba]

Salut,

Code :

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

1
2
3
4
5
6
7
 
		Interval j =new Interval() ;
		if(!"".equals(jour)){	
			if(!mapConges.containsValue(jour)){
		j.setJour(jour);
		lst.add(j); 		
		mapConges.put(nom,lst);}

Tu ajoutes des objets de type Interval dans ta liste, mais tu recherches des String !
De plus tu compares avec les données de la Map (qui sont des ArrayList) au lieu de rechercher dedans

Je pense que tu t'es un peu embrouillé !


si tu veux que contains() fonctionne il faut déjà que tu compares des objets de type compatible, et ensuite il ne faut pas oublier d'implémenter la méthode equals() dans ton type Interval (et hashCode() par la même occasion) -- voir la FAQ pour plus de détail.

Ensuite ton code pourra ressembler à ceci :

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
	private Map<String, List<Interval>> mapConges;
 
	public void ajouterTacheConges(String nom, String jour) throws ParseException {
		ArrayList<Interval> lst = (ArrayList<Interval>) mapConges.get(nom);
		if (lst == null) {
			lst = new ArrayList<Interval>();
			mapConges.put(nom, lst);
		}
		if (jour!=null && !"".equals(jour)) {
			Interval j = new Interval();
			j.setJour(jour);
			if (!lst.contains(j)) {
				lst.add(j);
			}
		}
	}

Et à la rigueur tu pourrais utiliser directement un Set à la place de la List, car elle intègre la notion d'unicité

a++

[Katachana]

Merci pour les explications, je comprend mieux mes erreurs maintenant. Par contre j'obtiens toujours le meme résultat en utilisant ta méthode.....

[adiGuba]

Fais voir le code de ta méthode equals()

a++