regex qui ne fonctionne pas?

bonjour,
je cherche à tester les valeurs entre 0 et 24

Code:

---

1 2 3 4

var chiffres =new RegExp([01][0-9] | [2][0123]); var verif; verif = chiffres.test(champ.value); alert(verif);

---

j'ai essayé ce code mais il me renvoie faux
pour des valeurs comme 0 ou 1 ou 6 ou 15 ....
je ne comprend pas d'ou vient le pb
j'ai essayé avec la syntaxe suivante mais même pb

Code:

---

var chiffres =new RegExp([01][0-9] | [2][0123]);

---

merci pour votre aide

Je ne répond pas à la question mais pourquoi via regexp ?
pourquoi pas

Code:

---

if (0 <= val && val <= 24)

---

???

essais

var chiffres =new RegExp(^[01][0-9]$ | ^[2][0123]$);

je suis pas un pro de la syntaxe des expréssions réguliaires en js mais je pense qui faut grouper des blocks d'expréssion, comme tu l'a fait j'aurais tendance a dire kil cherche un 0 ou 1, suivi d'un chiffre ou d'un 2, suivi d'un chiffre compris entre 0 et 3

En plus simple

Code:

---

1 2	/^(\d|0\d|1\d|2[01234])$/

---

tu dois tester aussi tes debuts et fin de chaine et prendre en compte le cas ou ton chiffre fait 1 seul caractere.
Lien pour tester ses regexp

moi j'aurai fais plutot ça :
var chiffres =new RegExp(^[0-9]$ | ^[12][0123]$);
sinon pour les chiffres inférieurs à 10 tu obliges la forme 01,02,...
mais je ne suis pas un expert des RegExp
De plus j'ai eu des soucis avec test() alors je te conseillerais match() à la place

dac avec Mr N. et même dmarche: j'ai même pas cherché à comprendre ton masque, qui me paraît inutile;

Citation:

---

Envoyé par javatwister

dac avec Mr N. et même dmarche: j'ai même pas cherché à comprendre ton masque, qui me paraît inutile;

---

Vi je pense inutile pour ce traitement mais les expressions réguliaires sont plus que pratiques pour des traitements plus compliqué :) et vouloir en utiliser pour des traitements simples c'est bien pour commencer à apprendre :roll:

Citation:

---

Envoyé par leywos

Vi je pense inutile pour ce traitement mais les expressions réguliaires sont plus que pratiques pour des traitements plus compliqué :) et vouloir en utiliser pour des traitements simples c'est bien pour commencer à apprendre :roll:

---

+1
J'ai appris un truc de plus grace à ce thread, enfin surtout grace à fred777888999...

Néanmoins, ici on teste une plage de valeurs, les expressions régulières ne sont pas faites pour ca, elles sont faites pour "matcher" des patterns sur des chaines de caractères...
En utilisant pas les bons outils, on rend le code illisible, difficilement maintenable (ex : on a maintenant des float => immediat avec la plage de valeurs, prise de tete avec regexp -- enfin pour moi) et je suis sur que ca bouffe plus de ressources... :roll:

je devients fou....
j'ecris ca

Code:

---

var chiffres = new RegExp(^[01][0-9]$ | ^[2][0123]$);

---

le code javascript n'est pas compilé

jessaie avec ca

Code:

---

var chiffres = new RegExp("^[01][0-9]$ | ^[2][0123]$");

---

ca compile mais tous mes essais renvoie faux....

COMPILE ????? :cfou: (ça se voit que c'est vendredi...la fatigue)

Bien fait tu n'as qu'à pas utiliser les regexp pour un test de plage de valeur... non mais !

Essaie ceci :

Code:

---

var chiffres = /^[01][0-9]$ | ^[2][0123]$/;

---

:roll:

Et pourquoi que tu prends pas ça ?

Citation:

---

Envoyé par fred777888999

En plus simple

Code:

---

1 2	/^(\d|0\d|1\d|2[01234])$/

---

tu dois tester aussi tes debuts et fin de chaine et prendre en compte le cas ou ton chiffre fait 1 seul caractere.
Lien pour tester ses regexp

---

effectivement ca fonctionne....
vendredi......

Citation:

---

Envoyé par Mr N.

Et pourquoi que tu prends pas ça ?

Citation:

---

Envoyé par fred777888999

En plus simple

Code:

---

1 2	/^(\d|0\d|1\d|2[01234])$/

---

tu dois tester aussi tes debuts et fin de chaine et prendre en compte le cas ou ton chiffre fait 1 seul caractere.
Lien pour tester ses regexp

---

---

bon ca m'a pris 5mn, suffit de chercher un peu les gars qd meme

Citation:

---

chiffres = new RegExp(/^[0-9]$|^[12][0123]$/);
verif = chiffres.test(document.monForm.id_champ.value);
alert(verif);

---

surtout ne pas mettre d'espace ds ton expressions car l'espace est un caractères.

aprés l'expression est a ton gout, la ca marche pour les valeurs 1, 2, 3 ... 10, 11, ... 23, 24