Discussion :

[Ceubex]

Bonjour,

C'est un de ces fameux pb de cast de Integer à Double impossible mais j'ai une petite subtilité. En effet, je ne controle pas la variable d'entrée

Code :

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

HashMap<Integer, Double> action

qui à ma grande surprise peut renvoyer un Double ou un Integer (mais aussi parfois BigDecimal) à l'appel de

Code :

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

action.get(j)

Du coup je ne peux pas utiliser de fonction propre à Integer pour passer à Double, il faut que si c'est un Double on ne fait rien et si c'est un Integer on passe en Double. Bien évidemment j'ai le soucis de la mémoire RAM en tête puisque cela se fait dans une boucle qui peut beaucoup itérer.

Quels est la manière la plus intelligente de regler ce problème ?

Merci d'avance

[kolodz]

Peut-être que je n'ai pas compris le problème. Mais si on suppose qu'une variable "nombre" peut-être soit un integer, soit un double. Il est possible de résoudre le problème de cette manière :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
Double getDoubleForANumber(Integer number){
return new Double(number);
}
 
Double getDoubleForANumber(BigDecimal number){
return new Double(number);
}
Double getDoubleForANumber(Double number){
return number;
}

Sachant que la méthode utilisé sera toujours la plus restrictive au niveau du type.

Une autre manière serait de faire un test sur la classe de l'objet retourné :

Code :

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

1
2
3
4
5
6
7
Double valueToUse=null;
if( number isinstanceof Double){
   valueToUse=number;
}
else{
  valueToUse= new Double(number);
}

Cordialement,
K

[Ceubex]

Super ça fonctionne, merci à toi.
De manière à gérer mieux le fait qu'il peut s'agir d'un nombre d'un format que je ne controle pas j'ai pris ta deuxième fonction (un peu à ma sauce) :

Code :

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

((action.get(j) instanceof Double)?action.get(j):new Double(action.get(j)))

[Ceubex]

Par contre cela ne marche pas pour BigDecimal dommage, du coup je vais changer mon fusil d'épaule

[Logan Mauzaize]

Je comprends pas trop :

Code :

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

1
2
3
 
Map<Integer, Double> actions = ....;
actions.get(0);

Soit ça te renvoie un Double, soit tu te prends un ClassCastException.

Vu que tu veux manipuler indépendamment des Integer, Double et BigDecimal. J'utiliserai la classe abstraite Number.


PS : Il faut mieux utiliser la méthode Double.valueOf(double) plutôt que new Double(double)

[thelvin]

Je comprends pas trop :

Code :

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

1
2
3
 
Map<Integer, Double> actions = ....;
actions.get(0);

Soit ça te renvoie un Double, soit tu te prends un ClassCastException.

En fait, pas forcément.

Si tu fais ça, alors oui :

Code :

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

1
2
3
 
Map<Integer, Double> actions = ....;
Double action = actions.get(0);

Mais si tu fais ça, non :

Code :

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

1
2
3
4
5
6
 
Map untypedActions = new HashMap();
untypedAction.put(Integer.valueOf(0), new Date());
Map<Integer, Double> actions = untypedActions;
Object action = actions.get(0);
System.out.println(action);