Discussion :

[wilddances]

Pour faire les calculs demandés, j'utilise un Map dont les clés sont un Set d'Objets et les valeurs sont des double.
Dans ma méthode de calcul, je fournis un set pour lequel je dois calculer la valeur qui est égale à la somme
des valeurs des sets inclus dans le Set fourni comme paramètre.(c'est à dire que tout d'abord je dois parcourir les clés et
je vérifie à chaque fois si chaque valeur qui est un Set est inclus dans le set fourni si c'est inclus alors je dois récupérer
la valeur)

Pour ce faire j'ai écrit la méthode suivante:
Sachant que la classe EnsembleObjets est déjà implémentée ayant comme attribut un Set d'Objets

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
public Test() {
public Map<EnsembleObjets, Double>mapA;
 
public double Somme(EnsembleObjets ens){
		Double sum = 0.0;
		Set <EnsembleObjets>keysSet = mapA.keySet();
		Iterator <EnsembleObjets>it = keysSet.iterator();
 
		while (it.hasNext()){
			EnsembleObjets ensObj =it.next();
			if (ensObj.getEnsembleObjets().containsAll(ens.getEnsembleObjets()))
 
 
			{
			sum  = sum + mapA.get(ens.getEnsembleObjets());
			}
 
		}
		return sum;
 
	}
 
 
 
 
public static void main (String[] args){
 
		Test t = new T(); 
 
		EnsembleObjets e1 = new EnsembleObjets();
		e1.addObjets(new Singleton(new String("A")));
 
		FocalElement e2 = new FocalElement();
		e2.addObjets(new Singleton(new String("A")));
		e2.addObjets(new Singleton(new String("B")));
		e2.addObjets(new Singleton(new String("C")));
 
		t.addObjets(e1, new Double(0.6));
		t.addObjets(e2, new Double(0.4));
 
 
 
		System.out.println("Sum of {A} "+t.Somme(e1));
 
 
	}
}

un message d'erreur s'affiche:
Exception in thread main java.lang.NullPointerException

Je n'arrive pas à voir où est le problème.
Merci

[Nico02]

Utilises la balise code plz

Le problème peut venir de deux choses, soit tu n'as pas initialiser ton Map, soit tu essais d'accéder à un champ qui n'existe pas.

Au vu de ton code je ne vois pas la ligne qui instancie ton Map

Code :

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

map = new HashMap<EnsembleObjets, Double>();

Pourrais tu nous donner la ligne qui renvoi l'erreur ?

[tchize_]

ce code m'a l'air suspect

Code :

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

1
2
3
4
5
6
7
8
9
10
while (it.hasNext()){
			EnsembleObjets ensObj =it.next();
			if (ensObj.getEnsembleObjets().containsAll(ens.getEnsembleObjets()))
			
				
			{
			sum  = sum + mapA.get(ens.getEnsembleObjets());
			}
			
		}

Tu fais ça: si l'entrée A a le même contenu que l'entrée Y, alors récupérer la valeur à la clé Z. rien ne garanti donc qu'il y a quelque chose à cet endroit. Autrement dit, tu récupère la clé "BLEU" et t'en conclu qu'il faut aller chercher l'élément "ROUGE" dans la map, ca n'a pas de sens.

Il faudrait de toutes façons que tu utilise mieux que ca la map, c'est pas prévu pour être pracouru a chaque fois que tu y cherche quelque chose.

Il faut que EnsembleObjet définisse correctement les méthode equals et hashcode. Aussi n'oublie pas qu'une clé d'un Map DOIT être immutable. (Sa valeur ne peux pas changer)

[wilddances]

En fait l'erreur est signalée au niveau de cette ligne

Code :

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

sum  = sum + mapA.get(ens.getEnsembleObjets());

[Vakhyw]

Code :

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

mapA.get(ens.getEnsembleObjets())

retourne null : http://download.oracle.com/javase/6/...a.lang.Object)

En fait je crois comprendre que tu testes que tous les éléments d'un Set sont bien dans un autre, mais cela ne garantit pas que tu as ajouté le Set dans ta map, d'où le null.


Maintenant ton architecture me paraît bizarre. Tu pourrais nous expliquer ce que tu veux faire ? Par exemple pourquoi ne pas inclure une méthode "double getSum()" dans ta classe "EnsembleObjets" qui retournerait la somme de ces éléments (en la recalculant à chaque appel de la méthode ou en gardant un attribut "sum" que tu mettrais à jour à chaque modification de ton set ?) ? Des HashMap sur des Sets c'est souvent un peu bancal et couteux ...