Malgre return false, mon script continue à s'exécuter

**Arzawe** · 06/10/2011, 10h44

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 :

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
<!--
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

**SpaceFrog** · 06/10/2011, 10h48

on peut voir le onsubmit du form ?

**Arzawe** · 06/10/2011, 10h57

Biensûr,

Voici la totalité du script html :

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
<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 :

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

**SpaceFrog** · 06/10/2011, 11h03

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

Sinon autre conseil:

passe plutot par le selecteur id

Code :

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

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

ou au moins une syntaxe plsu stricte:

Code :

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

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

**Arzawe** · 06/10/2011, 11h30

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

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
<!--
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

**SpaceFrog** · 06/10/2011, 11h30

Code :

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

getElementById

**Bovino** · 06/10/2011, 11h34

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.

**Arzawe** · 06/10/2011, 11h38

Non, même avec les rectifications orthographiques ça ne change rien.

**SpaceFrog** · 06/10/2011, 11h38

un

Code :

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

 switch(true)

**Arzawe** · 06/10/2011, 11h42

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 :

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

switch(true)

au début ou à la fin ?

Merci de ta patience.

**Bovino** · 06/10/2011, 11h42

Envoyé par SpaceFrog;

Code :

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

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

Envoyé par Arzawe

Code :

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

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

Trouve les différences !

**SpaceFrog** · 06/10/2011, 11h44

quand le crochet part en thèse ...

**Arzawe** · 06/10/2011, 11h56

J'ai bien remplacé les crochets par les parenthèses.

Voici où en est le code

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
 
<!--
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

**SpaceFrog** · 06/10/2011, 12h00

Code :

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

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

**Arzawe** · 06/10/2011, 12h51

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 :

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

**SpaceFrog** · 06/10/2011, 12h54

Code :

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

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;
}