Vérif de formulaire : marche pas !

Version imprimable

Bonjour à tous,

Je refais un formulaire de contact avec captchat et comme je suis nul en JS, je réclame votre indulgence !

Le test sur le captchat se fait bien, l'ortho du mail également.

Le prb est que si j'arrive à vérifier le mail et les champs, il n'y a plus aucune vérification sauf le captchat, qui est indépendant.

Je vous mets le code, vous y verrez plus clair :
Script mis entre les Head :

Code:

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
<script language="JavaScript">
<!--
function verification() 
{
   adresse = document.form1.email.value;
   var place = adresse.indexOf("@",1);
   var point = adresse.indexOf(".",place+1);
   if ((place > -1)&&(adresse.length >2)&&(point > 1))
        {
      return true;
       }
   else {
      alert('Adresse e-mail non valide !');
       return false;
        }
		{
 
    if(document.getElementById('nom').value == ""){
        alert ('Vous avez oublié de saisir un champ!!!!');
        return false;
        }
/*
 
    if(document.getElementById('sujet').value == ""){
        alert ('Vous avez oublié de saisir un champ!!!!');
        return false;
        }
 
    if(document.getElementById('message').value == ""){
        alert ('Vous avez oublié de saisir un champ!!!!');
        return false;
        }
*/
}
//-->
</script>

Script mis entre les Body :

Code:

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
<form name="form1" action="verifier.php" method="post" onsubmit="return verification()">
<table border="1" align="center" cellpadding="0" cellspacing="10">
  <tr>
    <td align="right">Nom *</td>
    <td><input name="nom" id="nom" type="text" size="45"></td>
  </tr>
  <tr>
    <td align="right">e-Mail* </td>
    <td><input name="email" type="text" size="45"></td>
  </tr>
<tr><td>&nbsp;</td></tr>
  <tr>
    <td align="right">Sujet *</td>
    <td><input name="sujet" id="sujet" type="text" size="45"></td>
  </tr>
  <tr>
<tr><td>&nbsp;</td></tr>
    <td align="right" valign="top">Message *</td>
    <td><textarea name="Message" id="message" cols="40" rows="4"></textarea></td>
  </tr>
<tr><td>&nbsp;</td></tr>
  <tr>
	<td align="right" valign="top">
        Anti-spam <br><br>
	Recopiez ces 5 lettres ici
    </td>
 
	<td valign="top">
	 <img src="cryptographp.php"><br>
	 <input class=champtext name="code" type="text" size="17"><br><br>
	 <input class=champtext type="submit" name="Submit" value="Envoyer"> 
     <input type="reset" value="Annuler">
    <td>
 
  </tr>
</table>
</form>

Je ne vois vraiment pas d'où ça peut venir. Pourtant, je pense avoir mis les bonnes syntaxes.

Est-ce que qq pourrait m'aider ?

Merci d'avance.

07/12/2007, 09h56
d-Rek
Pour tester la validité d'un mail :
Code:

1 2 3 4 var reg1 = /^[a-z0-9\-_\.]+@[a-z0-9\-_\.]+\.[a-z]{2,5}$/i; if(email.match(reg1)){ ... }
Je suis pas certain de document.getElementById('nom').value, utilise plutôt document.form1.nom.value

Pour débugger, fait plutôt :

Code:

onsubmit="alert(verification()); return false;"
07/12/2007, 10h00
jln13

salut,

tu as juste ouvert une parenthese en trop ^^

Citation:

else {
alert('Adresse e-mail non valide !');
return false;
}
{

if(document.getElementById('nom').value == ""){
alert ('Vous avez oublié de saisir un champ!!!!');
return false;
}
07/12/2007, 11h41
jlb59
Citation:
Envoyé par d-Rek

Pour tester la validité d'un mail :

Code:

1 2 3 4 var reg1 = /^[a-z0-9\-_\.]+@[a-z0-9\-_\.]+\.[a-z]{2,5}$/i; if(email.match(reg1)){ ... }

Je suis pas certain de document.getElementById('nom').value, utilise plutôt document.form1.nom.value

Pour débugger, fait plutôt :

Code:

onsubmit="alert(verification()); return false;"
Bon, pour le mail, j'ai rien changé pour l'instant puisque c'est OK. Et puis, comme je l'ai dit, je suis nul en JS. De ce fait, je n'ose pas trop m'aventurer sans savoir ce que je dois faire ! ;)
Par contre, pour savoir si un champs a été saisi, j'ai fait comme tu as dis et le résultat est le même, il n'y a pas de vérif. des champs.
C'est bien comme ça qu'il fallait faire ?
Code:

1 2 3 if(document.form1.nom.value == "") alert ('Vous avez oublié de saisir le champ "Nom" !!!!'); }
C'est énervant quand ça marche pas. Grrrr!!!
07/12/2007, 11h41
jlb59

Citation:

Envoyé par jln13

salut,

tu as juste ouvert une parenthese en trop ^^

Oui, je l'avais vu, mais ça marche toujours pas !!!!
07/12/2007, 14h33
jln13

Salut,

voyons si j'ai bien compris ton pb ..
Tu mets ca :

Citation:

if ((place > -1)&&(adresse.length >2)&&(point > 1))
{
return true;
}
else {
alert('Adresse e-mail non valide !');
return false;
}
// autres tests ....

Donc :
- dans le cas ou ton mail est valide : tu sors "return true"
- dans le cas ou le mail n'est pas valide : tu sors "return false"

Donc tu depasse jamais ces deux tests... Du coup ta verification des champs marche mais c'est juste que tu ne peux pas arriver a eux avec ton code...

Citation:

Envoyé par jln13

Salut,

voyons si j'ai bien compris ton pb ..
Tu mets ca :
....
.....

Donc :
- dans le cas ou ton mail est valide : tu sors "return true"
- dans le cas ou le mail n'est pas valide : tu sors "return false"

Donc tu depasse jamais ces deux tests... Du coup ta verification des champs marche mais c'est juste que tu ne peux pas arriver a eux avec ton code...

C'est logique ce que tu dis !
Mais je dois faire quoi ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 ... adresse = document.form1.email.value; var place = adresse.indexOf("@",1); var point = adresse.indexOf(".",place+1); if ((place > -1)&&(adresse.length >2)&&(point > 1)){ return true; } else { alert('Adresse e-mail non valide !'); return false; } ...
Donc, ce serait le "return false" qui bloquerait la poursuite de la vérification ?

Mais si je l'enlève (j'ai essayé), is no good ! Je teste les autres champs (boite de msg de js), mais ça envoi quand même !!!! :(

Tu dois faire un choix.

Soit tu decides de t'arreter a la premiere erreur :
=> des que tu rencontres une erreur tu renvoies false. Si l'utilisateur fait plusieurs erreurs de saisie tu n'afficheras que la premiere erreur dans ton message.

Soit tu decides d'afficher toutes les erreurs :
=> dans ce cas tu dois utiliser un booleen que tu met a false des que tu rencontre une erreur et des que tu rencontre une erreur tu la stocke dans une liste d'erreurs.

Dans tous les cas, il ne faut pas que tu renvoies true quand le test est valide. Tu renvoies true a la fin de ta fonction si ton booleen indique que tout va bien.

En gros tu peux adapter de cette maniere :
Code:

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  </script>

Citation:
Envoyé par jln13
Tu dois faire un choix.

Soit tu decides de t'arreter a la premiere erreur :
=> des que tu rencontres une erreur tu renvoies false. Si l'utilisateur fait plusieurs erreurs de saisie tu n'afficheras que la premiere erreur dans ton message.

Soit tu decides d'afficher toutes les erreurs :
=> dans ce cas tu dois utiliser un booleen que tu met a false des que tu rencontre une erreur et des que tu rencontre une erreur tu la stocke dans une liste d'erreurs.

Dans tous les cas, il ne faut pas que tu renvoies true quand le test est valide. Tu renvoies true a la fin de ta fonction si ton booleen indique que tout va bien.

En gros tu peux adapter de cette maniere :
Code:

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  </script>
je ne connaissais la variable "Bool", c'est pratique en tout cas.

Ta solution fonctionne bien, mais quand j'arrive au captchat, si je ne saisis rien, je suis renvoyé, après vérif. de ce champ, vers une page d'erreur
Page de vérification du captchat :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 <?php $cryptinstall="./cryptographp.fct.php"; include $cryptinstall; ?> <html> <?php if (chk_crypt($_POST['code'])) include("thank.html"); else include("erreur.html"); ?> ?> </html>
cryptographp.fct.php
Code:

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 <?php if(session_id() == "") session_start(); $_SESSION['cryptdir']= dirname($cryptinstall); function dsp_crypt($cfg=0,$reload=1) { // Affiche le cryptogramme echo "<table><tr><td><img id='cryptogram' src='".$_SESSION['cryptdir']."/cryptographp.php?cfg=".$cfg."&".SID."'></td>"; if ($reload) echo "<td><a title='".($reload==1?'':$reload)."' style=\"cursor:pointer\" onclick=\"javascript:document.images.cryptogram.src='".$_SESSION['cryptdir']."/cryptographp.php?cfg=".$cfg."&".SID."&'+Math.round(Math.random(0)*1000)+1\"><img src=\"".$_SESSION['cryptdir']."/images/reload.png\"></a></td>"; echo "</tr></table>"; } function chk_crypt($code) { // Vérifie si le code est correct include ($_SESSION['configfile']); $code = addslashes ($code); $code = str_replace(' ','',$code); // supprime les espaces saisis par erreur. $code = ($difuplow?$code:strtoupper($code)); switch (strtoupper($cryptsecure)) { case "MD5" : $code = md5($code); break; case "SHA1" : $code = sha1($code); break; } if ($_SESSION['cryptcode'] and ($_SESSION['cryptcode'] == $code)) { unset($_SESSION['cryptreload']); if ($cryptoneuse) unset($_SESSION['cryptcode']); return true; } else { $_SESSION['cryptreload']= true; return false; } } ?>
Ce fichier ne sert à vérifier que le captchat. Il est totalement indépendant du formulaire.

J'ai essayé plusieurs formulaire anti-spam, mais j'avais beaucoup d'erreur (pas d'image, erreurs de syntaxes, etc.), et les auteurs ne m'ont jamais répondu lorsque je leur ai demandé de l'aide !

Avec ce que tu m'as donné, je vais essayer de faire qq chose, mais sans conviction.

Merci encore de ton aide qui m'a un peu éclairé. Je dis un peu parce que c'est de l'hébreu pour moi.

A bientôt.