JS Funktion funktioniert nicht

Hey Leute,

ich habe in einem Element (#rightbottom) mehrere Elemente zu stehen, die ich per JS "ausklappen" will, sobald ein Radio-Button angeklickt wurde.
Das Problem ist, dass wenn ich nun auf den radio-Button klicke, absolut nichts passiert und der display nicht auf "none" gesetzt wird .. Wieso ? :s

Hier die function:

<script type="text/javascript">
                 function show(s)
                 {
                 if (document.getElementById(s).style.display == "block")
                    document.getElementById(s).style.display = "none";
                 else
                    document.getElementById(s).style.display = "none";
                  }
</script>

Radio-Button:

<input type="radio" onlick="show('one')">

Element ('one'):

<div id="rightbottom">
                         <h4>Reportstatus</h4>
                          <div id="one" style="display:block;">1. Grund waehlen (*)</div>
</div>

Da ich mir bisher nur die function in JS zusammengebastel bzw zusammengesucht habe,
ist meine Frage noch ob es das gleiche mit einem Absätze gibt ?
Jedoch hat ein Absatz ja keine ID und es ist kein Element, sowie in der function ..
Wie könnte ich diese Idee realisieren oder gibt es eine bessere Methode ?

Freue mich auf Antworten

lg

Ja mir ist der unnötige else-Fall auch aufgefallen. habe es ausgenommen aber es funktioniert nicht. was meinst du mit einem Alert ? wie gesagt ich bin mit js nicht vertraut :s
danke für deine Hilfe
Das lustige an der Sache ist, dass ich auf einer anderen Seite die gleiche function habe, jedoch nur von Display none zu Display block also andersrum als hier..das funktioniert -.-

Ja na klar, dank dir für den Link.
Aber was sagst du dazu, dass wenn die function andersrum ist also if style.display == 'none' ... style.display = 'block'
Wieso funktioniert diese function aber nicht andersrum, so wie in diesem Falle ? :s

lg und danke