Discussion :

[chipmunkette]

Bonjour, voila je souhaite faire une verification du mail si celui ci est renseigné lors de l'envoie d'un formulaire.

PRobleme il n'autorise pas le champ vide alors que l'email n'est pas obligatoire.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
function verifierMail
(champ) {
var str = champ.value;
var regexp = new RegExp("^[a-zA-Z0-9_\\-\\.]{3,}@[a-zA-Z0-9\\-_]{2,}\\.[a-zA-Z]{2,4}$", "g");
if
(!regexp.test(str)) {
alert("L'adresse e-mail n'est pas valide !");
champ.focus();
return false;
}
return true;
}

[Bovino]

Ben... il faut indiquer quelque part que si la valeur est vide, c'est bon aussi...

[chipmunkette]

un truc comme

Code :

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

1
2
 else if (str==""){
return true; }

[Bovino]

Pas besoin de else... tu peux simplement combiner les deux conditions dans le if avec && ou ||.

[chipmunkette]

si je combine les deux faut que je change la condition en :

Code :

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

1
2
3
4
5
6
7
 
(regexp.test(str) ||str=="" ) {
return true;
}alert("L'adresse e-mail n'est pas valide !");
champ.focus();
return false;
}

Sinon je voit pas ce que tu veux dire

[SpaceFrog]

Ou bien juste mettre le tout en facultatif

Code :

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

var regexp = new RegExp("^([a-zA-Z0-9_\\-\\.]{3,}@[a-zA-Z0-9\\-_]{2,}\\.[a-zA-Z]{2,4})?$", "g");

à noter que dans les classes il n'est pas besoin d'echapper pour la reg

Code :

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

var regexp = new RegExp("^([a-zA-Z0-9_.-]{3,}@[a-zA-Z0-9_-]{2,}\.[a-zA-Z]{2,4})?$", "g");

à condition de mettre le - en dernier afin d'éviter un range

[Watilin]

Conseil pour les prochaines fois : utiliser la forme littérale, c'est plus court, plus joli (selon moi) et y'a moins de caractères à échapper…

Au passage, quelques simplifications : [a-zA-Z0-9_] c'est la classe \w, donc [a-zA-Z0-9_\-\.] se simplifie en [\w.-] (et comme l'a dit SpaceFrog : dans une classe, pas besoin d'échapper le point, ni le tiret quand il est au début ou à la fin).
On peut encore simplifier un peu si on met le flag /i, ce qui permet de remplacer [a-zA-Z] par [a-z] tout court.

Code :

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

var regexp = /^[\w.-]{3,}@[\w-]{2,}\.[a-z]{2,4}$/gi;

Vu la vérification de l'extension (\.[a-z]{2,4}), il est certain que cette regexp ne validera pas toutes les adresses email possibles, surtout avec les plus de 1000 nouvelles extensions de domaine sorties récemment… (exemple, l'extension .paris a 5 lettres). Mais il a déjà été montré que la validation d'adresses email est une épreuve difficile

Pour rester dans le sujet, ça me paraît plus utile de mettre la condition « email vide » dans le if plutôt que dans la regexp, pour deux raisons :

• parce que ça se voit plus facilement à la lecture du code
• parce que le cas « pas d'email » mérite (selon moi) un traitement différent que le cas « l'email doit être vérifié ».