Discussion :

[Hyssgrif]

Bonjour,

Je souhaite realiser un simple calculecomposer essentiellement de divisions multiplication et addition avec des doubles.
Mais voila, le resultat obtenut est incoherent, je n'arrive pas a trouver l'erreur

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
		Map<String, Double> graphtemp = graphprob;
		double probtostay = 0;
		double probfromneighbourrooms = 0;
 
 
		for (Map.Entry<Room, List<Room>> graphindex : graph.entrySet())
		{//iterate all rooms to find prob for each
 
			probtostay = ((double) (1 / (graphindex.getValue().size() + 1))) * graphtemp.get(graphindex.getKey().getSpaceId());
 
			System.out.println("Space : " + graphindex.getKey().getSpaceId() + "  Size : " +(graphindex.getValue().size() + 1) + "  Value : " + 
			graphtemp.get(graphindex.getKey().getSpaceId()) + "  probtostay : " + probtostay);
                  }

Le resultat :
Space : Space1768 Size : 2 Value : 1.0 probtostay : 0.0

Or je devrais avoir: probtostay = 0.5
puisque de ce programme je fait: probtostay = (1/(Size+1))*Value
(Size initialement a 1)
Pourquoi est-ce different?
Merci a vous !

[joel.drigo]

Salut,

Le problème vient probablement de cette expression : (1 / (graphindex.getValue().size() + 1)) (il y a celle-là et une autre au moins).
Si size() est un int, cette division donne un int. En Java, un int divisé par un int donne un int (autrement dit 1/2 donne 0 !). Tu castes en double le résultat en double, donc ça en fait un double, mais trop tard : sa valeur int est 1, sa valeur double est dont 1.0. Il faut convertir le dividende ou le diviseur en double, avant l'opération. Une opération entre un int et un double donne un double, donc tu peux faire + 1.0 au lieu de + 1 et ça suffira. Tu peux aussi utiliser 1.0 comme dividende : (1.0 / (graphindex.getValue().size() + 1)) ou (1d / (graphindex.getValue().size() + 1)). S'il ne s'agit pas de litéral (comme 1), mais de résultat d'appel de méthode, ou de variable, comme graphindex.getValue().size(), tu peux caster en (double) : (1 / ((double)graphindex.getValue().size() + 1))

[Hyssgrif]

Pourtant j'ai fait le test separement d'une division d'un double par un int, ca marchait, sans ne mettre aucun cast

[Hyssgrif]

En effet, j'ai modifier en rajoutant le cast:

Code :

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

1
2
 
probtostay = ( (1.0 / ((double)(graphindex.getValue().size() + 1)))) * graphtemp.get(graphindex.getKey().getSpaceId());

Ca marche a present.

Merci a toi

[joel.drigo]

Pourtant j'ai fait le test separement d'une division d'un double par un int, ca marchait, sans ne mettre aucun cast

C'est ce que j'ai dit :

• double / double donne un double
• int / int donne un int
• int / double donne un double
• double / int donne un double