Discussion :

[scoubi77]

Bonsoir à tous,

Ayant modifié sur mon livre d'or la partie du onsubmit j'ai perdu la vérification de mes deux caractères, voici mon code :

Code html :

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

1
2
<form method=post action=writeguest.php 
 onsubmit="return ValidBotBoot(); if ( this.nom.value.length < 2 ) { return false ;}  " >

le return Validbotboot ce fait bien mais plus la vérification du nombre de caractères avant l'envoi.
J'ai donc essayé de modifier comme ceci :

Code html :

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

1
2
<form method="post" action="writeguest.php"
 onsubmit="return (ValidBotBoot()&& 'if ( this.nom.value.length < 2 ) { return false ;}')  " >

Pourriez-vous me dire où j'ai faux !
Par avance merci

Cordialement scoubi

[Bovino]

A partir du moment où tu fais un return, les instructions suivantes ne sont pas interprétées...

EDIT
Quant à la deuxième syntaxe...

[scoubi77]

Merci à vous Mr Bovino pour avoir modifié mon titre je ne savais pas quoi mettre comme balise

[scoubi77]

faudrait-il dans ce cas le mettre avant le return ?
Comme ceci ?

Code :

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

1
2
<form method=post action=writeguest.php 
 onSubmit=" if ( this.nom.value.length < 2 ) { return false ;} ; return ValidBotBoot();" >

ou une solution alternative me permettant de conserver la vérification du nombre de caractères et la validation de la fonction Valibotboot ?
Merci par avance pour cette réponse

Cordialement scoubi

[Bovino]

Tu peux garder ton && logique, mais en le rectifiant...
Dans ce que tu montres, tu fais un && entre le résultat d'une fonction et une chaine de caractère. Or l'équivalent booléen d'une chaine de caractères non vide est toujours true, surtout, le if ne sert à rien.

Code :

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

onsubmit="return (ValidBotBoot()&& this.nom.value.length > 1)"

[scoubi77]

je comprend mais il n'est plus nécessaire dans ce cas de faire un return false ?

Si non je pensais écrire cette condition dans une fonction et l'appeler comme ma fonction ValidBotBoot ? mais le premier return annule t'il les suivants ?
Après test cela ne fonction pas car si je n'écris rien dans mes champs le message ce poste quand même

Merci

[Bovino]

Le résultat d'un && logique est obligatoirement un booléen...
Donc quand tu fais un &&, tu renvoies toujours true ou false.
Quant à externaliser ton test, c'est exactement pareil :

Code :

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

onsubmit="return fonction1() && fonction2()"

Enfin, oui, c'est le but de l'instruction return de stopper l'exécution d'une fonction et de renvoyer le résultat passé en opérande.
Encore faut-il comprendre qu'un attribut d'événement HTML comme onsubmit crée en fait une fonction anonyme dont le code est celui passé en valeur de l'attribut.

[scoubi77]

Si j'ai bien compris la réponse, même si je crée une fonction cela ne fonctionnera pas en utilisant par exemple :

Code :

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

onsubmit="return ValidBotBoot()&& verif()"

avec par exemple pour verif :

Code :

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

1
2
3
4
5
6
7
8
 <script type="text/javascript">
 
 function verif(){
if this.nom.value.length <  2
{ return false ;}
}
 
 </script>

[scoubi77]

ça marche pas snif

[NoSmoking]

Bonjour,

ça marche pas snif

voilà quelque chose de hautement instructif

Pose toi quand même la question de ce que vaut this dans ta fonction.

[scoubi77]

Bonsoir NoSmoking,

si le champs nom est inferieur à 2 le résultat est " false "

[scoubi77]

Je vais passer par l'instruction getElementById

Code :

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

1
2
3
4
5
6
7
8
9
 
 
function verif(){
    if(document.getElementById('nom').value == ""){
        alert ('Vous avez oublié de saisir votre nom!');
        return false;
    }
	return true;
}

[NoSmoking]

Certes c'est ce que tu souhaites, mais même en y pensant très fort le this fait référence à l'objet window donc ce n'est pas ce que tu cherches donc ...(voir le PS après relecture)

Code :

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

1
2
3
4
5
6
7
8
9
function verif(){
  var oElem = document.getElementById('id_du_champ_a_tester');
  if( oElem.value.length <  2){
    return false ;
  }
  else{
    return true;
  }
}

voilà la bonne syntaxe, non simplifiée, en mettant les parenthèses qui vont bien.

PS: je pense d'ailleurs que this.nom doit pointer quand même sur le bon élément même si cette syntaxe me pique les yeux.

[Bovino]

Code :

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

1
2
3
4
 function verif(){
if this.nom.value.length <  2
{ return false ;}
}

C'est quoi cette syntaxe pour le if ?

Sinon

Code :

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

1
2
3
 function verif(){
return document.getElementById('id_du_champ_a_tester').value.length > 1;
}

me semble amplement suffisant...

[scoubi77]

Et bien cela ne fonctionne pas, pourtant j'ai bien l'id nom comme cela :

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
<tr><td><FONT FACE="monotype corsiva" size="4"  color="white">Nom :</FONT></td>
    <td><input type=text name="nom" id="nom" size=25 ></td></tr>
 
 <script type="text/javascript">
 
function verif(){
  var oElem = document.getElementById("nom");
  if( oElem.value.length <  2){
    return false ;
  }
  else{
    return true;
  }
}
 
 </script>
<form method="post" action="writeguest.php"
 onsubmit="return ValidBotBoot()&& verif()">

même si le champ nom est vide le message ce poste !!!

je suis attaqué par un robo spameur qui malgré une case à cocher en jquery ( fonctionnelle ) et une addition aléatoire en javascript à compléter, un champ vide caché, ce robot arrive à me poster des cochonneries !
je ne sais plus quoi faire pour éviter cela !

[scoubi77]

Je vais changer de concept, vue que le dernier robot spammeur utilise toujours la même adresse mail
je pense interdire l'envoi du message si dans le champ form1 la fameuse adresse email est remplie

J'ai donc pensé à :

Code :

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

1
2
3
function valider()
if(document.form1.Mail.value == "@exemple.fr")
  return false;

Sauf que au lieu de bloquer l'envoi, j'ai simplement l'adresse qui est effacée, voir toutes les adresse même, mais le message est posté

[NoSmoking]

Code :

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

return document.form1.Mail.indexOf("@exemple.fr")<0; // return false si contenue donc pas de soumission du formulaire

autant bloquer coté serveur.