Malgre return false, mon script continue � s'ex�cuter

Version imprimable

Bonjour,

J'ai r�alis� un formulaire avec une v�rification sur la saisie de certains champs. Pour ce la j'ai cr�� le script suivant :

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
38
39
40
41
42
43
44
45
46
<!--
function verif()
{
 
if(document.contact_part.nom.value == "")  {
   alert("Veuillez entrer votre nom svp");
   document.contact_part.nom.focus();
   return false;
  }
   else
 
if(document.contact_part.prenom.value ==""){
alert("Merci de saisir votre prénom.");
document.contact_part.prenom.focus();
}
else
 
 if(document.contact_part.telephone.value == "") {
   alert("Merci de saisir cos coordonnées téléphoniques.");
   document.contact_part.telephone.focus();
   return false;
  }
   else
 
 if(document.contact_part.email.value == "") {
   alert("Veuillez entrer votre adresse electronique svp");
   document.contact_part.courriel.focus();
   return false;
  }
 
 if(document.contact_part.email.value.indexOf('@') == -1) {
   alert("Ce n'est pas une adresse electronique, verifiez la svp");
   document.contact_part.courriel.focus();
   return false;
  }
else
 
if(document.contact_part.texte.selectedIndex == 0){
alert("Merci de choisir un sujet de message");
document.contact_part.sujet.focus();
return false;
}
return true;
  }
 
//-->

Le probl�me est que si je ne saisis rien dans le champ "Nom" par exemple, l'erreur est bien signal�e, mais le formulaire est quand m�me envoy�.
Et malgr� de nombreuses recherches, je n'arrive pas � trouver pourquoi ?
Est-ce que l'un de vous saurait me dire ce qu'il manque SVP ?

Merci d'avance

Arzawe

06/10/2011, 10h48
SpaceFrog

on peut voir le onsubmit du form ?

Biens�r,

Voici la totalit� du script html :

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
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
<form name="contact_part" method="POST" action="traitement.php"
onsubmit="return verif()">
 
  <table>
    <tbody>
      <tr>
        <td><label>Votre nom (*) :*</label></td>
        <td><input name="nom" id="nom" type="text" size="50" /></td>
      </tr>
<tr>
        <td><label>Votre prénom (*) :*</label></td>
        <td><input name="prenom" id="nom" type="text" size="50" /></td>
      </tr>
      <tr>
        <td><label>Votre société :*</label></td>
        <td><input name="societe" type="text" id="societe" size="50" /></td>
      </tr>
      <tr>
        <td><label>Sujet du message (*) :</label></td>
        <td>
          <select name="sujet">
            <option value="choix" selected="selected">Sélectionner un
            thème</option>
            <option value="Question sur le Portage Salarial">Questions sur le Portage Salarial
            </option>
            <option value="Question sur le site">Questions sur le site</option>
            <option value="Demande d'inscription">Demande d'inscription</option>
            <option value="Pour notre Agent Local Antilles-Guyane">Pour notre Agent Général
            Antilles-Guyane</option>
            <option value="CNIL et Droit d'acc&eagrave;s">CNIL Droit d'accès</option>
          </select>
           </td>
      </tr>
      <tr>
        <td><label>Votre téléphone (*) :*</label></td>
        <td><input name="telephone" type="text" id="telephone" size="50" /></td>
      </tr>
      <tr>
        <td><label>Votre courriel (*) :</label></td>
        <td><input name="courriel" type="text" id="courriel" size="50" /></td>
      </tr>
      <tr>
        <td><label>Votre message (*) :</label></td>
        <td><textarea name="message" cols="40" rows="12"
          id="texte/plain">Inscrivez ici votre message.</textarea><br />
          (*) Mentions obligatoires</td>
      </tr>
      <tr>
        <td></td>
        <td>
          <table width="100%">
            <tbody>
              <tr>
                <td>*</td>
                <td></td>
              </tr>
              <tr>
                <td>
                  <input name="annuler" type="reset" value="Annuler" /></td>
                <td>
                  <input name="soumettre" type="submit" value="Envoyer" /></td>
              </tr>
            </tbody>
          </table>
        </td>
      </tr>
    </tbody>
  </table>
</form>

et voici m�me le script 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
38
39
40
41
<?php
$nom=$HTTP_POST_VARS['nom']; 
$prenom=$HTTP_POST_VARS['prenom'];
$telephone=$HTTP_POST_VARS['telephone'];
$mail=$HTTP_POST_VARS['courriel']; 
$objet=$HTTP_POST_VARS['sujet']; 
$message=$HTTP_POST_VARS['message']; 
 
/////voici la version Mine
$headers = 'Content-Type: text/plain; charset="iso-8859-1"'."\n"; 
$headers .='Content-Transfer-Encoding: 8bit';
 
//////ici on détermine le mail en format text
$headers .= "Content-type: text/plain; charset=iso-8859-1\r\n";
 
////ici on détermine l'expediteur et l'adresse de réponse
$headers .= "From: $nom $prenom<$mail>\r\nReply-to : $nom <$mail>\nX-Mailer:PHP";
 
$subject="$objet"; 
 
if($objet == "Pour notre Agent Local Antilles-Guyane")
{
$destinataire="adresse@mail2";
}
else
$destinataire="adresse@mail1"; 
$body="$message"; 
 
 
 
$msg  = 'Bonjour,'."\r\n\r\n";
$msg .= 'Ce mail a été envoyé depuis http://www.dorhel.fr par '.$nom.' '.$prenom."\r\n\r\n";
$msg .= 'Ses coordonnées téléphoniques : '.$telephone."\r\n\r\n";
$msg .= 'Courriel : '.$mail."\r\n\r\n";
$msg .= 'Voici le message qui vous est adressé :'."\r\n";
$msg .= "\r\n\r\n";
$msg .= stripslashes($message)."\r\n";
$msg .= "\r\n\r\n";
 
mail($destinataire, $subject,$msg, $headers);
?>

06/10/2011, 11h03
SpaceFrog

tu as un souci avec les id ...
tu as deux fois id="nom"

Sinon autre conseil:

passe plutot par le selecteur id

Code:

if(document.getElementById('nom').value == "") {

ou au moins une syntaxe plsu stricte:

Code:

if(document.forms['contact_part'].elements['nom'].value == "") {

J'ai bien modifi� mon deuxi�me champ 'nom' en prenom et j'ai modifi� le script de la fa�on suivante:

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
38
39
40
41
42
43
44
45
46
<!--
function verif()
{
 
if(document.getElementbyId['nom'].value == "")  {
   alert("Veuillez entrer votre nom svp");
   document.contact_part.nom.focus();
   return false;
  }
   else
 
if(document.fgetElementbyId['prenom'].value ==""){
alert("Merci de saisir votre prénom.");
document.contact_part.prenom.focus();
}
else
 
 if(document.getElementById['telephone'].value == "") {
   alert("Merci de saisir vos coordonnées téléphoniques.");
   document.contact_part.telephone.focus();
   return false;
  }
   else
 
 if(document.getElementById['email.value'] == "") {
   alert("Veuillez entrer votre adresse electronique svp");
   document.contact_part.courriel.focus();
   return false;
  }
 
 if(document.getElementById['email'].value.indexOf('@') == -1) {
   alert("Ce n'est pas une adresse electronique, verifiez la svp");
   document.contact_part.courriel.focus();
   return false;
  }
else
 
if(document.getElementById['texte'].selectedIndex == 0){
alert("Merci de choisir un sujet de message");
document.contact_part.sujet.focus();
return false;
}
return true;
  }
 
//-->

Et l�, il ne fait m�me plus le test

06/10/2011, 11h30
SpaceFrog

Code:

getElementById
06/10/2011, 11h34
Bovino

Je te conseillerais aussi de supprimer tous tes else...
Ils ne servent strictement � rien sinon � complexifier inutilement le code...
Si tu es rentr� dans le if, alors le return te fait sortir de la fonction, sinon, tu peux continuer l'enchainement de tes tests.
06/10/2011, 11h38
Arzawe

Non, m�me avec les rectifications orthographiques �a ne change rien.
06/10/2011, 11h38
SpaceFrog

un

Code:

switch(true)

:mouarf:
06/10/2011, 11h42
Arzawe

Je te remercie pour ton aide, mais ceci est mon premier code en js, et j'apprends sur le tas.
O� est-ce que je mets

Code:

switch(true)

au d�but ou � la fin ?

Merci de ta patience.
06/10/2011, 11h42
Bovino

Citation:

Envoy� par SpaceFrog;

Code:

if(document.getElementById('nom').value == "") {

Citation:

Envoy� par Arzawe

Code:

if(document.getElementbyId['nom'].value == "") {

Trouve les diff�rences ! :mrgreen:
06/10/2011, 11h44
SpaceFrog

quand le crochet part en th�se ...

J'ai bien remplac� les crochets par les parenth�ses.

Voici o� en est le code

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
 
<!--
function verif()
{
 
if(document.getElementById('nom').value == "")  {
   alert("Veuillez entrer votre nom svp");
   document.contact_part.nom.focus();
   return false;
  }
   else
 
if(document.getElementById('prenom').value ==""){
alert("Merci de saisir votre prénom.");
document.contact_part.prenom.focus();
}
else
 
 if(document.getElementById('telephone').value == "") {
   alert("Merci de saisir vos coordonnées téléphoniques.");
   document.contact_part.telephone.focus();
   return false;
  }
   else
 
 if(document.getElementById('email').value == "") {
   alert("Veuillez entrer votre adresse electronique svp");
   document.contact_part.courriel.focus();
   return false;
  }
 
 if(document.getElementById('email').value.indexOf('@') == -1) {
   alert("Ce n'est pas une adresse electronique, verifiez la svp");
   document.contact_part.courriel.focus();
   return false;
  }
else
 
if(document.getElementById('texte').value == "Inscrivez ici votre message."){
alert("Merci de saisir votre demande.");
document.contact_part.sujet.focus();
return false;
}
if(document.getElementById('texte').value == ""){
alert("Merci de saisir votre demande");
document.contact_part.sujet.focus();
return false;
}
return true;
}
 
//-->

Ca marche bien pour le nom, mais pour le pr�nom, le message s'affiche, puis le script continue � s'�x�cuter.

Si vous le voulez voici le lien auquel vous pourrez voir le formulaire et l'essayer.
http://dorhel.fr/essai/Nouveau.html

Code:

1
2
3
document.getElementById('nom').value == ""
 
document.contact_part.nom.focus();

:roll:

Franchement merci beaucoup. Je n'avais pas pens� � faire les modifs sur le focus.

Il a juste maintenant la v�rif sur le champ message qui ne fonctionne pas : j'aurais souhait� une alerte si la case est vide ou bien si la case contient uniquement le message par d�faut.

Et je cherche comment faire une v�rification sur la liste de choix.

Le code actuel :

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
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
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
 
<html>
<head>
<!--
function verif()
{
 
if(document.getElementById('nom').value == "")  {
   alert("Veuillez entrer votre nom svp");
   document.getElementById('nom').focus();
   return false;
  }
   else
 
if(document.getElementById('prenom').value ==""){
alert("Merci de saisir votre prénom.");
document.getElementById('prenom').focus();
return false;
}
else
 
 if(document.getElementById('telephone').value == "") {
   alert("Merci de saisir vos coordonnées téléphoniques.");
   document.getElementById('telephone').focus();
   return false;
  }
   else
 
 if(document.getElementById('courriel').value == "") {
   alert("Veuillez entrer votre adresse electronique svp");
   document.getElementById('courriel').focus();
   return false;
  }
 
 if(document.getElementById('courriel').value.indexOf('@') == -1) {
   alert("Ce n'est pas une adresse electronique, verifiez la svp");
   document.getElementById('courriel').focus();
   return false;
  }
else
 
if(document.getElementById('message').value == "Inscrivez ici votre message."){
alert("Merci de saisir votre demande.");
document.getElementById('message').focus();
return false;
}
if(document.getElementById('message').value == ""){
alert("Merci de saisir votre demande");
document.getElementById('message').focus();
return false;
}
return true;
}
 
//-->
 
 
  </script>
</head>
 
<body>
 
<form name="contact_part" method="POST" action="traitement.php"
onsubmit="return verif()">
 
  <table>
    <tbody>
      <tr>
        <td><label>Votre nom (*) :*</label></td>
        <td><input name="nom" id="nom" type="text" size="50" /></td>
      </tr>
<tr>
        <td><label>Votre prénom (*) :*</label></td>
        <td><input name="prenom" id="prenom" type="text" size="50" /></td>
      </tr>
      <tr>
        <td><label>Votre société :*</label></td>
        <td><input name="societe" type="text" id="societe" size="50" /></td>
      </tr>
      <tr>
        <td><label>Sujet du message (*) :</label></td>
        <td>
          <select name="sujet">
            <option value="choix" selected="selected">Sélectionner un
            thème</option>
            <option value="Question sur le Portage Salarial">Questions sur le Portage Salarial
            </option>
            <option value="Question sur le site">Questions sur le site</option>
            <option value="Demande d'inscription">Demande d'inscription</option>
            <option value="Pour notre Agent Local Antilles-Guyane">Pour notre Agent Général
            Antilles-Guyane</option>
            <option value="CNIL et Droit d'acc&eagrave;s">CNIL Droit d'accès</option>
          </select>
           </td>
      </tr>
      <tr>
        <td><label>Votre téléphone (*) :*</label></td>
        <td><input name="telephone" type="text" id="telephone" size="50" /></td>
      </tr>
      <tr>
        <td><label>Votre courriel (*) :</label></td>
        <td><input name="courriel" type="text" id="courriel" size="50" /></td>
      </tr>
      <tr>
        <td><label>Votre message (*) :</label></td>
        <td><textarea name="message" cols="40" rows="12"
          id="texte/plain">Inscrivez ici votre message.</textarea><br />
          (*) Mentions obligatoires</td>
      </tr>
      <tr>
        <td></td>
        <td>
          <table width="100%">
            <tbody>
              <tr>
                <td>*</td>
                <td></td>
              </tr>
              <tr>
                <td>
                  <input name="annuler" type="reset" value="Annuler" /></td>
                <td>
                  <input name="soumettre" type="submit" value="Envoyer" /></td>
              </tr>
            </tbody>
          </table>
        </td>
      </tr>
    </tbody>
  </table>
</form>
</body>
</html>

Code:

1
2
3
4
5
if(document.getElementById('message').value == "Inscrivez ici votre message." || document.getElementById('message').value == ""){
     alert("Merci de saisir votre demande.");
     document.getElementById('message').focus();
     return false;
}

06/10/2011, 13h05
Arzawe

J'ai fais un copier/coller de ta r�ponse, mais non, �a passe au travers.
Je cherche toujours de mon c�t�, mais je s�che.
06/10/2011, 15h42
Arzawe

Merci beaucoup � vous deux.
Je me prosterne devant votre savoir :lahola:

L'essentiel de mon script fonctionne.

Pour les deux points qui restent, je vais continuer � creuser.:marteau:

A bient�t.