faire apparaître un DIV avec bouton radio

Version imprimable

bonjour tout le monde!
voici mon problème:
j'ai deux boutons radio, et le premier est "checked" par défaut et ne change en rien l'apparence de mon formulaire.
je voudrais faire apparaître un DIV lorsque je clique sur le deuxième...
j'ai fait un essais avec le code suivant, mais évidement, cela ne fonctionne pas... :lol:
pouvez-vous me dire où je me suis planté... :roll:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 <!--code de le HEAD--!> function afficher(reservation) { document.getElementById("reservation").style.visibility = ""; } function cacher(reservation) { document.getElementById("reservation").style.visibility = "hidden"; } <!--code des boutons radio--!> <p id="Pnormal3"><input type="radio" name="motif" value="Renseignements" checked="checked" onClick="cacher(reservation)" />Renseignements</p> <p id="Pnormal3"><input type="radio" name="motif" value="Reservations" onClick="afficher(reservation)" />Réservation</p> <!--DIV à afficher--!> <div id="reservation" style="display: none"> </div>
merci d'avance!

Pas de quotes dans getElementById : c'est la variable que tu utilises
Code:

1 2 3 4 5 6 7 8 function afficher(reservation) { document.getElementById(reservation).style.visibility = ""; } function cacher(reservation) { document.getElementById(reservation).style.visibility = "hidden"; }
Des quotes ici car c'est une chaine que tu passes en parametre et un point-virgule à la fin, c'est plus propre :

Code:

onClick='afficher("reservation");'

Pour eviter de confondre parametre, chaine et variable, tu devrais plutot l'ecrire comme ça :
(c'est la même chose, y'a juste les noms de changé)
Code:

1 2 3 4 5 6 7 8 function afficher(elemID) { document.getElementById(elemID).style.visibility = ""; } function cacher(elemID) { document.getElementById(elemID).style.visibility = "hidden"; }
elemID pour "element ID" histoire d'être cohérent avec l'utilisation faite de ce parametre

Citation:
Envoyé par buzzkaido
Pas de quotes dans getElementById : c'est la variable que tu utilises
Code:

1 2 3 4 5 6 7 8 function afficher(reservation) { document.getElementById(reservation).style.visibility = ""; } function cacher(reservation) { document.getElementById(reservation).style.visibility = "hidden"; }
Des quotes ici car c'est une chaine que tu passes en parametre et un point-virgule à la fin, c'est plus propre :

Code:

onClick='afficher("reservation");'

Pour eviter de confondre parametre, chaine et variable, tu devrais plutot l'ecrire comme ça :
(c'est la même chose, y'a juste les noms de changé)
Code:

1 2 3 4 5 6 7 8 function afficher(elemID) { document.getElementById(elemID).style.visibility = ""; } function cacher(elemID) { document.getElementById(elemID).style.visibility = "hidden"; }
elemID pour "element ID" histoire d'être cohérent avec l'utilisation faite de ce parametre
merci!!!!!
ça a l'air si simple quand on a la solution! :oops:
deux heures que j'essaie (pas habitué du javascript...:lol: )
encore merci! ;)