[javascript] skrypt wyboru Opcje

[mari6274]

Mógłby mi ktoś powiedzieć, czy taki skrypt ma prawo działać(IMG:style_emoticons/default/questionmark.gif) Jesli nie to dlaczego, jeśli tak to co robie źle, że nie działa mi prawidłowo.

[HTML] pobierz, plaintext 
<p>
 
				<input  type='radio' name='typg' id='1' value='1' checked='checked'>
				<input  type='radio' name='typg' id='2' value='2'>
</p>
 
      <script>
 
            document.getElementById('d2').style.display = 'none';
            if(document.getElementById(1).checked) {
            document.getElementById('d1').style.display = 'block';
            document.getElementById('d2').style.display = 'none';
            }
            else if(document.getElementById(2).checked) {
            document.getElementById('d2').style.display = 'block';
            document.getElementById('d1').style.display = 'none';
            }
 
			</script>
 
<p>
			<div id='d1'>
 
 
              //skrypt JS
 
 
			</div>
			<div id='d2'>
 
              //Kod HTML
              //+
              //Skrypt JS
 
 
			</div>
</p>
[HTML] pobierz, plaintext 

[mortus]

Według specyfikacji wartość atrybutu id elementu HTML nie może być cyfrą, liczbą, a nawet nie może się cyfrą rozpoczynać. Problemem jest jednak nieprawidołowy argument metody getElementById, bo powinien on być objęty cudzysłowami lub apostrofami. Poza tym skrypt powinien się raczej znaleźć za div-ami d1 i d2, bo wcześniej te elementy mogą być niezdefiniowane (undefined). Jednak lepiej (zgodnie z dokumentacją HTML) będzie tak:

[HTML] pobierz, plaintext 
<p>
  <input type='radio' name='typg' id='typg_1' value='1' checked='checked'>
  <input type='radio' name='typg' id='typg_2' value='2'>
</p>
<p>
  <div id='d1'>
  </div>
  <div id='d2'>
  </div>
</p>
<script>
  document.getElementById('d2').style.display = 'none';
  if(document.getElementById('typg_1').checked) {
    document.getElementById('d1').style.display = 'block';
    document.getElementById('d2').style.display = 'none';
  } else if(document.getElementById('typg_2').checked) {
    document.getElementById('d2').style.display = 'block';
    document.getElementById('d1').style.display = 'none';
  }
</script>
[HTML] pobierz, plaintext