Discussion :

[Oluha]

Salut

Je viens de me rendre compte que mes regex n'étaient pas "bonnes".
Il m'en faut deux :

1) elle doit vérifier qu'il n'y a que des caractères numériques
J'avais fais :

Code :

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

var maReg = new RegExp("[0-9]+","");

Mais je viens de me rendre compte que si on tapais "222aaa" elle validait la chaine alors qu'il ne faudrait pas. Je ne comprend pas trop pourquoi

2) elle doit n'autoriser les valeurs numériques entières et décimales. Mais elle doit seulement accepter le point du pavé numérique et pas la virgule qu'il y a sous le point d'interrogation.
Jusqu'à maintenant j'utilisais la même qu'au dessus mais ca ne suffit pas et je ne vais pas trop comment faire

Merci pour votre aide

[Oluha]

euh en y repensant la 1ere ne me pose plus problème :

Code :

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

var maReg = new RegExp("^[0-9]+$","");

mais je veux bien un peu d'aide pour la seconde

[denisC]

1) elle doit vérifier qu'il n'y a que des caractères numériques
J'avais fais :

Code :

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

var maReg = new RegExp("[0-9]+","");

Mais je viens de me rendre compte que si on tapais "222aaa" elle validait la chaine alors qu'il ne faudrait pas. Je ne comprend pas trop pourquoi

Tu vérifies juste qu'il existe, quelque part dans ta chaine, une sous-chaine composée d'au moins un chiffre. (donc 222aaa convient car la sous-chaine 222 matche ton expression régulière).
Ajoute simplement:

Code :

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

1
2
 
 var maReg = new RegExp("^[0-9]+$","");

Et ça forcera la sous-chaine à commencer au début de la chaine (opérateur ^) et à terminer à la fin (opérateur $) donc à faire toute la chaine!!

2) elle doit n'autoriser les valeurs numériques entières et décimales. Mais elle doit seulement accepter le point du pavé numérique et pas la virgule qu'il y a sous le point d'interrogation.
Jusqu'à maintenant j'utilisais la même qu'au dessus mais ca ne suffit pas et je ne vais pas trop comment faire

Je dirais:

Code :

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

1
2
 
/^[\d]+(\.[\d]+)?$/

soit une chaine composée de chiffres (\d=0-9) suivi eventuellement d'un . et d'une chaine composée de chiffres.

Bonne chance

Edit : un peu lent, mais mon post était long

[Oluha]

merci !
Donc en gros le ? permet de dire que c'est un morceau facultatif si j'ai bien compris.
Je teste ça tout de suite

[Oluha]

bouah ca a pas l'air de marcher

Code :

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

1
2
3
4
var maReg3 = new RegExp("/^[\d]+(\.[\d]+)?$/","");
if (paye.search(maReg3) == -1) {
	erreur = erreur + "--> the amount paid must be numeric\n";
}

J'ai l'alert quand le montant vaut 39.95 ou 39

[siddh]

et si tu tentes un parseFloat dans un try catch ?

[Oluha]

euh je peux essayer, faut que je trouve comment marche le try catch lol

sinon je viens de tester ça mais ca ne fonctionne pas non plus, je capte pas pourquoi

Code :

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

var maReg3 = new RegExp("(^[0-9]+$)|(^[0-9]+\.[0-9]+$)","");

[siddh]

euh non meme, utilise isNaN :

Code :

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

if(isNaN(paye))

[Oluha]

oki, je vais tester ça quand j'aurais 2 min

[Oluha]

ca marche super ! Il fait bien la différence entre le point et la virgule.
Je ne connaissais pas cette fonction. D'ailleur quelqu'un aurait la liste des fonctions javascript et leurs explications ?

Merci

[denisC]

Donc en gros le ? permet de dire que c'est un morceau facultatif si j'ai bien compris.

C'est bien ça.

Bon, il suffit que je tourne le dos, et paf on critique mes solutions

Alors:

Code :

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

1
2
 
alert("39.95".search(/^[\d]+(\.[\d]+)?$/));

Ca renvoie 0 chez moi (IE et FFx), donc tu devais avoir une erreur ailleurs...

PS: tant que ta solution marche....

[siddh]

ca marche super ! Il fait bien la différence entre le point et la virgule.
Je ne connaissais pas cette fonction. D'ailleur quelqu'un aurait la liste des fonctions javascript et leurs explications ?

Merci

ben regardes en tete du forum, il me semble qu y a les references javascript
isNaN est une fonction de base de js ca veut dire is Not A Number

[Oluha]

Alors:

Code :

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

1
2
 
alert("39.95".search(/^[\d]+(\.[\d]+)?$/));

Ca renvoie 0 chez moi (IE et FFx), donc tu devais avoir une erreur ailleurs...

Euh 0 c'est pas false ?

[denisC]

Euh 0 c'est pas false ?

Euh, ben non, 0 c'est que le pattern commence au premier caractère de la chaine (ce qui est bien le but du jeu).

false c'est -1 (le pattern n'est pas trouvé).