Discussion :

[lenstoi]

slt, voilà, mon problème c'est que j'utilise ce code pour verifier la validité ou l'existance de la donnée : adresse mail dans mon formulaire, mais rien ne se passe !!!
voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
function test_mail(email) {
var parobase;
	parobase=email.indexOf("@");
	if (parobase == -1) return false;
var point;
	point=email.lastIndexOf(".");
	if ((point == -1) || (point < arobase)) return false;
	if (email.value.length < 1) return false;
return true;
}

et un peu en bas dans mon code, j'appelle la fonction :

Code :

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

1
2
3
4
5
6
 
if (!test_mail(document.mail.value)) {
	alert('Veuillez verifier votre E-MAIL');
	document.mail.focus();
	return false;
	}

merci de me dire où elle est l'erreur !

[Bisûnûrs]

http://developpez.net/forums/showthread.php?t=288611

[lenstoi]

ca marche très bien, merci à toi

[Tellounet]

Je sais que le sujet est dépassé mais je cherchais justement ça.
En lisant le code quelque chose m'a sauté aux yeux:

Code :

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

1
2
3
var parobase;
...
if ((point == -1) || (point < arobase)) return false;

Bon, et moi je m'en vais lire le lien donné par bisûnûrs

[SpaceFrog]

Cette regExp me donne toute satisfaction pour les emails :

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");

[Tellounet]

Je n'en doute pas un instant, mais après quelques tests je reste bloqué.
Ca vient probablement de l'appel de ma fonction puisque je n'ai aucun alert qui s'affiche.

ma fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<script type="text/javascript">
function Verifmail(monmail){
var expressionReguliere = new RegExp("^[a-zA-Z0-9_-.]+@[a-zA-Z0-9-]{2,}[.][a-zA-Z]{2,3}$")
if (!expressionReguliere.test(valeurDeTonChampsEmail))
{
	alert('mauvaise adresse');
}
else 
{
	alert('bonne adresse');
}	
 
</script>

mon appel, à l'intérieur de la fonction de vérification des champs vides (cf la semaine dernière ^^ ):

Code :

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

1
2
3
4
5
6
7
8
...
             if(AllIsOk==0){
	alert('envoi');
	Verifmail(document.forms['Formulo'].elements['mail'].value);
	return true;}
		else{
	alert('veuillez remplir tous les champs');
	return false}

lorsque je remplis tous mes champs mais que je mets une adresse invalide il me passe mon alert('envoi'), envoie mon mail et c'est tout.
je pense que je devrais touver plus simple que document.forms['Formulo'].elements['mail'].value à entrer dans la fonction

[SpaceFrog]

tu verifies le mail dans une autre fonction de vérification apèrs le allisOK ???
pourquoi ne pas l'inclure directement et incrémenter si le test n'est pas bon ???

[Tellounet]

Hé bien je la différencie du allisok pour pouvoir mettre l'accent sur le fait que le mail est pas valide, puisque le champ est rempli.

J'ai d'abord essayé de l'inclure dans la même fonction, à l'endroit où maintenant je l'appelle, mais ça foirait le reste et ça ne vérifiait plus le remplissage des champs.
Note: j'avais déclaré le var regexp au début de la fonction, avant les tests du allisok

[SpaceFrog]

un exemple de focntion de validation de formulaire ...

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
 
function ValidForm(Myform){
 
var ErrMsg="";
var SendFocus;
 
//Civilité:
var Input_Civ=0;
for(i=0;i<3;i++){
				Input_Civ+=(Myform.elements['Civil'][i].checked)?1:0;
				}
ErrMsg+=(Input_Civ==0)?"Vous n'avez pas saisi la civilté. \n":"";				
 
 
//Nom
if(!isNaN(Myform.elements['Nom'].value) || Myform.elements['Nom'].value.length<2) { ErrMsg+="Votre Nom n\'est pas saisi ou est incorrect. \n" ;
 
																					SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['Nom'];
																						}
 
//RaisonSociale
if(!isNaN(Myform.elements['NomSoc'].value) || Myform.elements['NomSoc'].value.length<2){ErrMsg+="Vous n\'avez pas (ou mal) saisi la Raison Sociale. \n";
																						SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['NomSoc'];
																							}
 
//Adr1
if(!isNaN(Myform.elements['Adr1'].value) || Myform.elements['Adr1'].value.length<4){ErrMsg+="Vous n\'avez pas (ou mal) saisi votre adresse. \n";
																					SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['Adr1'];}
 
 
//Ville
if(!isNaN(Myform.elements['Ville'].value) || Myform.elements['Adr1'].value.length<2){ErrMsg+="Vous n\'avez pas (ou mal) saisi votre ville. \n";
																					SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['Ville'];}
 
//Code Postal
if(Myform.elements['Pays'].options[Myform.elements['Pays'].selectedIndex].value=="FRA"){
	var CP= Myform.elements['CP'].value;
	var reg = new RegExp(/^((0[1-9])|([1-8][0-9])|(9[0-8]))[0-9]{3}$/);
	if(!reg.test(CP)){ ErrMsg+="Vous n\'avez pas (ou mal) saisi votre Code Postal. \n";
						SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['CP'];	}
						}
 
//Pays
if(Myform.elements['Pays'].value==-1){ErrMsg+="Vous n\'avez pas selectionné votre pays. \n" ;
									SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['Pays'];	}
 
 
//Telephone
 var tel= Myform.elements['Tel'].value;
 var reg = new RegExp(/[0-9]{2}[\W][0-9]{2}[\W][0-9]{2}[\W][0-9]{2}[\W][0-9]{2}/);
 
 if (!reg.test(tel)){ErrMsg+="Veuillez saisir votre Nr de téléphone au format 00.00.00.00.00.00 \n";
					SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['Tel'];}
 
 
//Email
var email = Myform.elements['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(email)){ ErrMsg+="Votre adresse E-mail semble erronnée. \n" ;
					SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['Mail'];}
 
 
//Activité
if(Myform.elements['Activ'].value==-1){ErrMsg+="Vous n\'avez pas selectionné votre activité. \n" ;
										SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['Activ'];}
 
if(Myform.elements['Activ'].value=="Autre" && Myform.elements['AutreActiv'].value.length<3){ErrMsg+="Vous n\'avez pas précisé l\'activité autre. \n";
																							SendFocus=(SendFocus!=undefined)?SendFocus:Myform.elements['AutreActiv'];}
 
 
//RESULTAT																							
if(ErrMsg.length>0){ alert(ErrMsg);
					 SendFocus.focus();	
					 return false;}
 
else{ return true;}
 
}

[Tellounet]

C'est tout de même étrange...

Je passe ma vérification de mail dans la vérification de remplissage, sans appeler d'autre fonction, et du coup ça ne vérifie plus rien alors que ces traitements sont écrits et devraient être exécutés avant la vérification du mail.

Même les alert ne passent plus, comme si la fonction n'était pas appelée. Pourtant c'est elle qui autorise le submit de mon formulaire, et justement le mail part

[Tellounet]

Cette simple ligne dans mon code:

Code :

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

Var monmail=document.forms['Formulo'].elements['mail'].value;

empêche la fonction de s'exécuter...

Pourtant document.forms['Formulo'].elements['mail'].value est copié-collé depuis un peu plus haut, dans les tests de remplissage qui eux fonctionnent (seuls).
Bien évidemment mon formulaire et mon champ ont le bon nom



edit: oui, sans la majuscule à var ça ira mieux en effet... stupid me...

[Tellounet]

Bon, maintenant tout est exécuté sauf:

Code :

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

1
2
3
4
5
6
7
var expressionReguliere = new RegExp("^[a-zA-Z0-9_-.]+@[a-zA-Z0-9-]{2,}[.][a-zA-Z]{2,3}$");
								if (!expressionReguliere.test(nommail)){
									alert('mauvaise adresse');
									return false;}
								else {
									alert('bonne adresse');
									return true;}

Et le pire c'est que je m'aperçois que j'avais oublié le { après le else. JE la rajoute et... la fonction n'est plus exécutée. Mouarf!


edit: une pause, je vais faire une pause, avaler on bon litre de café et relire attentivement tout mon code, parce que je fais avant tout des étourderies (majuscules, quotes, dyslexie tactile...)
C'était monmail et pas nommail...

ceci étant ça ne marche pas mieux...