Forum PHP.pl

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 

jakiś to wykładowca nie wie jaki jest błąd??

ja tu za bardzo nie widzę błędu - mogę jedynie napisać drobne uwagi...

wszystko zależy co dokładnie miałeś zrobić - jeśli podmianę elementu drzewa DOM potraktować dosłownie to nic nie podmieniasz, po prostu ukrywasz pozostałe elementy a pokazujesz inny który de fakto tam był i jest...

następne ale to tylko szczegół i nie jest tutaj błędem, a jedynie jeśli ogólnie znasz strukturę HTML'a na którego piszesz funkcję a nie masz wpływu na jej poprawną strukturę to mogłeś sprawdzić, czy wyszukało Ci elementy o zadanym id by następne metody w razie braku danego obiektu nie spowodowały błędu...

var el;
el = document.getElementById(jaka);
el && el.style.display = 'none'; // lub if(el) el.style.display = 'none';

@zegarek84: Zamiast masy IF-ów nie dających żadnej kontroli nad błędami lepiej skorzystać z fundamentalnych elementów JS - wyjątków:

[JAVASCRIPT] pobierz, plaintext 
try {
  document.getElementById(jaka).style.display = 'none';
} catch (e) {
  // jakas reakcja na błąd
}
[JAVASCRIPT] pobierz, plaintext