Discussion :

[noinneh]

Bonjour les gens
je dois être une de ces innombrables victimes face à l'objet regexp, alors voici mon script:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<script language="JavaScript">
function Test()
{
var re=new RegExp("^[a-zA-Z0-9\-_]+[a-zA-Z0-9\.\-_]*@[a-zA-Z0-9\-_]+\.[a-zA-Z\.\-_]{1,}[a-zA-Z\-_]+","i");
var mail = document.form.mail.value;  //document.form[0].mail.value marche pas
if(!re.test('mail'))
    {
    alert ('email invalide');
    return false;
    }
return true;
}
</script>

mon appel de script:

Code :

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

1
2
3
4
<FORM name= form method="post" action='http://localhost/infotem/inscription_verif.php'> 
mail:<BR><INPUT type=text name=mail maxlength=40 ><BR>
<INPUT type=submit value=envoyer onClick="return Test()"><BR>
</FORM>

et pour finir les symptomes:
je peux mettre ce que je veux dans l'email l'action de ma form est exécuter et aucun message d'alerte.

voila,

A votre bon coeur messieurs dames.

[necronick]

Je pense qu'il faut des " de part et d'autre de ton expression régulière :

Code :

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

var re=new RegExp("^[a-zA-Z0-9\-_]+[a-zA-Z0-9\.\-_]*@[a-zA-Z0-9\-_]+\.[a-zA-Z\.\-_]{1,}[a-zA-Z\-_]+","i");

et également autour du message d'erreur

Code :

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

alert("email invalide");

[noinneh]

désolé necronick rien n'y fait mais merci quand même

code édité

[necronick]

hmmm.. le reste du code m'a l'air correct
j'ajouterais bien un alert(mail); juste avant le test pour voir si on passe bien dans la fonction et si la variable mail contient bien ce que l'utilisateur a saisi.

[noinneh]

tu viens de confirmer un doute que j'avais mon script n'a pas l'air d'être parcouru
nouveau code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<script language="JavaScript">
function Test()
{
var mail = document.form.mail.value;
alert('mail');
var re=new RegExp("^[a-zA-Z0-9\-_]+[a-zA-Z0-9\.\-_]*@[a-zA-Z0-9\-_]+\.[a-zA-Z\.\-_]{1,}[a-zA-Z\-_]+","i");
if(!re.test('mail'))
{
alert('true');
return false;
}
}	
</script>

et aucune alerte en vue

[noinneh]

correction le script est bien parcouru, j'avais effacé une accolade de trop mais c'est corrigé, cela dit il me donne toujours la même réponse.

[necronick]

Peut-etre que le probleme vient du coté de ton code HTML :

Code :

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

1
2
3
4
<FORM name= form method="post" action='http://localhost/infotem/inscription_verif.php'>
mail:<BR><INPUT type=text name=mail maxlength=40 ><BR>
<INPUT type=submit value=envoyer onClick="return Test()"><BR>
</FORM>

1. il vaut mieux mettre les controles dans l'événement onsubmit de la balise form plutot que sur un onclick : 'est pls logique, et l'utilisateur pourra ainsi soumettre le formulaire aussi en appuyant sur entrée (ce qui est parfois plus pratique si le formulaire est important que de cliquer sur un bouton)

2. Peut-etre qu'appeler une balise <form> "form" peut poser des problemes, mais je n'en suis pas sur.

3. il vaut mieux mettre les attributs entre ", et les noms des balises en minuscules c'est plus propre

[noinneh]

a y est ça marche
correction finale

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
16
17
<script language="JavaScript">
function Test()
{
var mail = document.form.mail.value;
var re=new RegExp("^[a-zA-Z0-9\-_]+[a-zA-Z0-9\.\-_]*@[a-zA-Z0-9\-_]+\.[a-zA-Z\.\-_]{1,}[a-zA-Z\-_]+","i");
if(!re.test(mail))
{
alert('false');
return false;
}
else
{
alert('true');
return false;
}
}	
</script>

je fesais re.test('mail') au lieu de re.test(mail)