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.

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

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

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

**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();

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

JavaScript

Vue hybride

Discussions similaires

Partager

Partager