Discussion :

[cosmos38240]

Bonjour

Actuellement je souhaiterai valider une chaine de charactère avec la condition suivante :
string(a-z) + 0-9 + '-' + '_' autorisé.

J'ai donc utilisé :

Code :

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

var exp=new RegExp("[a-zA-Z0-9\-_]$");

Le problème c'est que les espaces et charactères comme le & fonctionnent ...

Merci

[TheGwy]

Salut, il y a deux soucis dans ton code :

1 - tu n'as pas mis dans ton expression l'ancre de départ (^), seulement l'ancre de fin ($). Du coup du moment que ta chaine de caractère se termine par un caractère valide, ton test passe. Peu importe si avant certains caractères étaient invalides, l'évaluateur de regex va juste faire du backtracking en démarrant un caractère plus loin jusqu'à obtenir un match.

2 - Tu utilise le constructeur RegExp au lieu de la syntaxe littérale, ce qui est très très moche ! Tu ne dois utiliser le constructeur uniquement quand tu construit une expression dynamiquement. Pourquoi donc ? car lorsque tu utilise le constructeur tu dois lui passer une String donc :

- tu peux faire une croix sur la coloration syntaxique
- tu cumules deux niveau d'échappement (la string et la regex) et pour peu que tu recherche des classes de caractères, des antislashs ou des quotes, tu as vite fait de te retrouver avec des échappements incompréhensibles avec des doubles, triples voir quadruples antislashs. Et la...


Donc pour résumer, en rajoutant l'ancre de départ et la bonne syntaxe cela te donne :

Code :

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

1
2
3
4
var str = "hello-world_-42_azerty";
var exp = /^[a-zA-Z0-9\-_]*$/;
 
console.log(exp.test(str));

que l'on peut simplifier en :

Code :

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

1
2
3
4
var str = "hello-world_-42_azerty";
var exp = /^[\w-]*$/;
 
console.log(exp.test(str));

[cosmos38240]

Parfait merci beaucoup pour tes explications.

Cela fonctionne parfaitement et c'est propre maitenant