JavaScript działa,, ale według wykładowcy jest błąd Opcje

[ksenonlogin]

Witam

Skrypt ma za zadanie podmieniać element <div></div> po wybraniu odpowiedniego elementu z listy (<option></option>).
Wszystko ładnie działa (przykład - konrad-lewczuk.pl), jednak według wykładowcy w JavaScript jest jakiś błąd.

Niżej zamieszczam java script:

[HTML] pobierz, plaintext 
<script type="text/javascript">
<!-- // <![CDATA[
function akcja(zmienna) {
                          var ile = document.getElementById('wybrana').options.length;
                          for(var i=1; i<=ile; i++) {
                                                      var jaka = 'iddiv'+i;
                                                      if(jaka != zmienna) {
                                                                           document.getElementById(jaka).style.display = 'none';
                                                                          }
                                                      else if(jaka == zmienna) {
                                                                                document.getElementById(jaka).style.display = 'block';
                                                                               }
                                                     }
                        }
// ]]> -->
</script>
[HTML] pobierz, plaintext 

Niżej zamieszczam HTML:

[HTML] pobierz, plaintext 
<form id="akcja" action="#">
<p><select name="wybrana" id="wybrana" onchange="akcja(this.value);">
<option value="iddiv1">Imię, Nazwisko</option>
<option value="iddiv2">Tok nauki</option>
<option value="iddiv3">Telefon</option>
<option value="iddiv4">Email</option>
<option value="iddiv5">Twoja zmienna</option>
</select></p>
</form>
 
<div id="iddiv1">Konrad Lewczuk</div>
<div id="iddiv2">Informatyka, Nr albumu: 7611</div>
<div id="iddiv3">Tel: 531447823</div>
<div id="iddiv4">kontakt@konrad-lewczuk.pl</div>
[HTML] pobierz, plaintext 

Niżej zamieszczam CSS:

[HTML] pobierz, plaintext 
#iddiv1 {
display: block;
}
#iddiv2,#iddiv3,#iddiv4,#iddiv5 {
display: none;
}
[HTML] pobierz, plaintext