Bug comparaison de champs

Version imprimable

Bonjour,

Je suis en train de m'arracher les cheveux, je ne comprends pas pourquoi mon code JS ne fonctionne pas dans mon formulaire ????
Voici le formulaire :

Code:

1
2
3
4
5
6
7
8
9
10
<form name="val" action="val.php" method="post" onsubmit="return verif()"><br/>
    <select name="NbrSalarie" >
        <option value=0></option>
        <option value=1    <?php echo $NbrS1; ?> >Moins de 10 </option>
        <option value=11   <?php echo $NbrS2; ?> >De 11 &agrave; 100</option> 
        <option value=101  <?php echo $NbrS3; ?> >De 101 &agrave; 1000</option> 
    </select>
    <input type="radio" name="MontantHT" value="5" checked="checked" />5&euro; <a href="#" onClick="popUp('membres')"><b>Membre</b></a> pour...<br/>
    <input type="radio" name="MontantHT" value="10" <?php echo $tarif2; ?> />10&euro; HT  <a href="#" onClick="popUp('membres')"><b>Membre</b></a> pour... <br/>
    <input type="radio" name="MontantHT" value="20" <?php echo $tarif3; ?>  />20&euro; HT  <a href="#" onClick="popUp('membres')"><b>Membre</b></a>  pour...<br/>

Coté JS

Code:

1
2
3
4
5
if((document.val.MontantHT.value=="5") && (document.val.NbrSalarie.value>'1')) {  
    alert("Impossible");
    document.val.MontantHT.focus();
    return false
}

Pourquoi lorsque la valeur MontantHT est égal à 5 et que le nb de salarié et plus plus grand que 1 la condition ne s’exécute pas ????
J'avais pensé que cela venait des cotes mais non je n'arrive pas a afficher Impossible !!!!

J'espère que vous allez pouvoir m'éclairer car je suis en mode brouillard, merci par avance.

18/02/2014, 07h20
sekaijin

C quoi ce php dans le HTML ?
Qu'est-on censé comprendre ?

A+JYT
18/02/2014, 07h33
l.laurent60

Ce code php est un selected="selected" conditionnel
18/02/2014, 08h30
Bovino

Code:

document.val.MontantHT.value

En dehors du côté quelque peu vintage de la notation, dans la masure où tu as trois champs différents ayant le nom MontantHT dans ton code, JavaScript est incapable de décider tout seul lequel il doit utiliser...
:faq: Comment récupérer l'état de cases à cocher ?
18/02/2014, 08h40
l.laurent60

Avant tout merci pour ton retour.
Je ne comprends pas

Code:

document.val.MontantHT.value

JS est bien capable d'apprécier le valeur passé donc

Code:

document.val.MontantHT.value==5

devrait bien fonctionné

et la comparaison de champs également, non ?

Code:

if((document.val.MontantHT.value=="5") && (document.val.NbrSalarie.value>'1')) {
18/02/2014, 08h41
SpaceFrog

merci de lire les règles incontournable d'utilisation de ce forum qui précisent que le code php n'est d'aucune utilité ici. Il faut nous coller le code html généré !!

ensuite pour ce qui est de ton code ...
hormis le fait de l'utilisation d'une syntaxe laxiste document.val.MontantHT.valuetes radios ont le même name pour fonctionner ensemble , MAIS ...

document.val.MontantHT qui devrait avec plus de rigueur être écrit
document.forms['val'].getElementsByName('MontantHT')ne retourne pas ce que tu penses et n'a pas de value.
Il s'agit en effet de la COLLECTION des éléments portant le nom 'MontantHT'
chacun de ses items étant un checkbox qui lui possède bien une propriété value.
l'exemple par la pratique : http://jsfiddle.net/xaKY5/

Il faut donc boucler sur la collection et tester le checked pour recpérer le value de celui qui est coché
Avec un minimum de recherche sur le forum tu aurais trouvé la réponse http://javascript.developpez.com/sou...rms#radiocheck
18/02/2014, 08h56
Bovino

Pour compléter ce que dit Spaffy.

Code:

document.val.MontantHT.value

Peut s'énoncer en français comme ceci : "Récupère la valeur du champ de formulaire ayant le nom MontantHT se trouvant dans le formulaire ayant le nom val du document".
Tu pourras noter qu'il n'est nulle part fait mention de la notion de coché (ce qui semble être la source de ton incompréhension), donc encore une fois, comme tu as trois champs ayant le nom MontantHT, JavaScript est incapable de savoir duquel tu lui parle !

Aussi dommage que cela puisse paraitre, il n'existe pas de méthode JavaScript permettant de récupérer uniquement les champs cochés.
D'autre part, c'est aussi l'une des raisons pour lesquelles la notation utilisée est maladroite et à éviter car sa valeur de retour dépend de la structure du document : soit un élément DOM, soit une collection.

Ceci dit, j'attire ton attention sur un point : tu montres un code qui ne fonctionne pas, donc soit tu viens de découvrir un bogue de JavaScript (ce qui est peu probable), soit ton code contient des erreurs. Je t'indique où se trouve l'erreur

Citation:

dans la masure où tu as trois champs différents ayant le nom MontantHT dans ton code, JavaScript est incapable de décider tout seul lequel il doit utiliser...

et plutôt que d'essayer de comprendre ce qu'il se passe (en s'appuyant sur le lien donné), tu préfères persister à croire que ton code est correct. Ce n'est habituellement pas comme ça qu'on corrige ses erreurs. ;)
18/02/2014, 09h17
l.laurent60

Effectivement c'est une incompréhension de part, je vous remercie pour vos conseils qui vont me permettre d'avancer. Je vais corriger la syntaxe et faire les contrôles que je souhaite coté serveur.

Merci à tous