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.