Discussion :

[hainaoui]

Bonjour à tous,
Je rencontre actuellement un problème lors de la lecture d'un nombre saisie par un utilisateur (via un formulaire d'une application web). En effet, l'utilisateur doit pouvoir rentrer un nombre décimal avec un point ou une virgule comme séparateur décimal.

Dans un premier temps, j'avais réalisé la méthode suivante pour réaliser la conversion de la saisie :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
	public static double parse(String priceformated) throws BusinessException {
		double result;
		try {
			DecimalFormat decimaleFormat = new DecimalFormat("0.00");
			Number number = decimaleFormat.parse(priceformated);
			result = number.doubleValue();
		} catch (ParseException ex) {
			throw new BusinessException(ex);
		}
		return result;
	}

Avec cette méthode, les saisies utilisant une virgule comme séparateur sont bien interprétées. Mais si on saisie "10.5" en paramètre, on récupère en sortie un double valant 10.0. Bref, la partie décimale est ignorée si on emploie le point comme séparateur.

J'ai donc essayé la méthode suivante :

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 static double parse(String priceformated) throws BusinessException {
		double result;
		try {
			char separateurVirgule = ',';
			NumberFormat nf = (priceformated.indexOf(separateurVirgule) == -1)?NumberFormat.getInstance(Locale.ENGLISH):NumberFormat.getInstance(Locale.FRENCH);
			Number number = nf.parse(priceformated);
//			DecimalFormat decimaleFormat = new DecimalFormat("0.00");
//			Number number = decimaleFormat.parse(priceformated);
			result = number.doubleValue();
		} catch (ParseException ex) {
			throw new BusinessException(ex);
		}
		return result;
	}

Avec cette nouvelle méthode, tout fonctionne très bien.

Par contre, je ne sais pas si c'est une bonne façon de faire. Je me demande s'il n'était pas possible dans la première méthode de définir deux séparateurs potentiels de décimales ('.' et ',') et laisser l'objet DecimalFormat parser ma chaine tout seul comme un grand.

Merci d'avance pour vos commentaires.

[Ivelios]

Tu peux récupérer ton nombre et le transformer en string.

Puis tu fait un petit

Code :

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

str.replace('.',',');

Retour en arrière, ça redevient un nombre.

TADAMMMM le tour est joué

[adiGuba]

Salut,


Le remplacement pourrait être une solution en effet !


Mais attention car selon la locale la virgule ou le point peuvent également être utilisé comme séparateur de millier ce qui viendrait tout gacher

Selon les pays/langue tu a 4 grands formats de représentation des nombres, par exemple :

• 1,234.56 (par exemple les pays anglophones)
• 1 234,56 (plusieurs pays européens dont la France, le Luxembourg, ...). A noter que généralement l'espace correspond à un espace insécable (code ascii 160 au lieu de 32).
• 1.234,56 (plusieurs pays européens dont l'Italie, l'Allemagne, l'Espagne, ...)
• 1'234.56 (en Suisse)

Et rien que cela pose problème pour le remplacement. Par exemple : 10,000 représent "dix milles" pour les anglophones, alors qu'on pourrait interpréter cela comme "dix" tout court en France...

Tout cela pour dire que cela peut être dangereux !!!


A la rigueur tu pourrais peut-être utiliser la locale du navigateur pour déterminer la locale à utiliser avec le DecimalFormat, ou simplement lui imposer un format précis !!!



Enfin il faut se méfier du parse() qui ne traite pas forcément toute la chaine : il s'arrête au premier caractère non-conforme sans générer d'erreur s'il arrive à obtenir un nombre (c'est pour cela que tu obtiens un 10.0 au lieu de 10.5).
Pour éviter cela tu dois vérifier que toute la chaine a bien été lu (voir ce message)

a++