test de validité d'un calcul en bigdecimal

Version imprimable

hi all,

J'ai crée une fonction en BigDecimal, qui prend des éléments en String (mais convertit en bigdecimal dans la fonction) depuis un formulaire.

Le calcul fonctionne et tout, mais mon problème survient au niveau du test d'erreur. C'est à dire savoir si le calcul a fonctionné ou pas.

voici le code :
Code:

1 2 3 4 5 6 7 discountCurrency = simulationVehiclesForm.getDiscountValue(); System.out.println(simulationVehiclesForm.getPrice()); System.out.println(discountCurrency); BigDecimal resultCalc = currencyCalcul(simulationVehiclesForm.getPrice(), discountCurrency); System.out.println("calcul résultat: "+resultCalc); return null;
et j'aimerais faire un truc du style :
Code:

1 2 3 4 5 6 if(!resultCalc){ errors.add ... }else { return null; }
Comme cela c'est pas possible, mais je ne sais pas s'il y a une méthode ou un technique dédié :(

Merci d'avance pour vos réponses !

05/03/2008, 14h08
Iyoiyo

Il faudrai le code de la fonction currencyCalcul(...). Si j'ai bien compris, c'est elle qui fait le boulot donc c'est elle qui doit savoir si le calcul c'est bien effectué.

Tout d'abord merci pour la réponse, et puis

La voici :

Code:

1
2
3
4
5
6
7
8
9
 
public BigDecimal currencyCalcul(String priceForm, String discountCurrency) {
	BigDecimal price;
	BigDecimal priceTmpForm 	= new BigDecimal(priceForm);
	BigDecimal discountTmpForm 	= new BigDecimal(discountCurrency);
 
	price = priceTmpForm.subtract(discountTmpForm);
	return price;
}

Si j'ai bien compris ton raisonnement, il faudrait que je fasse dans la fonction :

Code:

1
2
3
4
5
6
 
if(!price) {
  error.add...
}else {
  return price;
}

non ?

Citation:
Envoyé par mLk92
Tout d'abord merci pour la réponse, et puis

La voici :
Code:

1 2 3 4 5 6 7 8 9 public BigDecimal currencyCalcul(String priceForm, String discountCurrency) { BigDecimal price; BigDecimal priceTmpForm = new BigDecimal(priceForm); BigDecimal discountTmpForm = new BigDecimal(discountCurrency); price = priceTmpForm.subtract(discountTmpForm); return price; }
Si j'ai bien compris ton raisonnement, il faudrait que je fasse dans la fonction :
Code:

1 2 3 4 5 6 if(!price) { error.add... }else { return price; }
non ?
Bah... Comment dire... C'était l'idée mais la on risque d'aller loin...

Mais en gros, si tes fonctions ne sont pas prévues pour retourner une valeur null ou émettre une exception lorsque le calcul c'est mal passé, je vois pas comment toi au bout de la chaîne tu vas pouvoir le faire... A moins que je n'ai pas compris quelque chose...

05/03/2008, 14h34
mLk92

Eh bien, je veux bien leur faire retourner une exception en cas d'erreur, mais où doit je mettre l'exception, dans la fonction même ? ou après son appel ?

Et surtout quel type d'exception ?
05/03/2008, 14h38
Iyoiyo

Citation:

Envoyé par mLk92

Eh bien, je veux bien leur faire retourner une exception en cas d'erreur, mais où doit je mettre l'exception, dans la fonction même ? ou après son appel ?

Et surtout quel type d'exception ?

D'abord j'aimerai savoir quel erreur pourrai renvoyer la ligne :

Code:

price = priceTmpForm.subtract(discountTmpForm);

La valeur de priceTmpForm est null ? Celle de discountTmpForm ? Ou autre chose...

eh bien, lors de la saisie du formulaire si l'utilisateur met une "," au lieu d'un ".", ma fonction foire complètement, du coup il me faut faire quelque chose dans ce genre de cas, voilà ce que j'ai fait, vous me direz si c'est convenable :
Code:

1 2 3 4 5 6 7 8 9 try { BigDecimal resultCalc = currencyCalcul(simulationVehiclesForm.getPrice(), discountCurrency); System.out.println("calcul résultat: "+resultCalc); return null; } catch (RuntimeException e) { // TODO Auto-generated catch block e.printStackTrace(); }

Citation:
Envoyé par mLk92
eh bien, lors de la saisie du formulaire si l'utilisateur met une "," au lieu d'un ".", ma fonction foire complètement, du coup il me faut faire quelque chose dans ce genre de cas, voilà ce que j'ai fait, vous me direz si c'est convenable :
Code:

1 2 3 4 5 6 7 8 9 try { BigDecimal resultCalc = currencyCalcul(simulationVehiclesForm.getPrice(), discountCurrency); System.out.println("calcul résultat: "+resultCalc); return null; } catch (RuntimeException e) { // TODO Auto-generated catch block e.printStackTrace(); }
Eh ben ça a pas l'air trop mal. Deux choses :

D'abord, sécurisé un peu mieux les types de données que tu rentres dans le formulaire. Je ne sais pas quel techno tu utilises pour ça, mais en SWING il y a des methodes de JTextField qui n'autorise que l'entré de valeur correctes par exemple.
Ensuite peut-etre n'encadrer avec try que l'appel a la fonction currencyCalcul(...) pour pouvoir continuer le code par exemple.

Ca marche impec !!

Voici ce que j'ai mis !

Code:

1
2
3
4
5
6
7
8
9
10
11
 
try {
	BigDecimal resultPercent = percentCalcul(simulationVehiclesForm.getPrice(), discountPercent);
	System.out.println("calcul résultat: "+resultPercent);
	return null;
} catch (RuntimeException e) {
	// TODO Auto-generated catch block
	errors.add("errorCalculPriceVeh", new ActionMessage("settingsVeh.calc.error"));
	e.printStackTrace();
	return errors;
}

J'utilise Struts, mais là ca marche correctement :p

Merci pour tes réponses !
@+