Czy ten kod zadziała?

pytanie zadane 18 lipca 2016 w JavaScript przez aken Nowicjusz (140 p.)

Cześć, jestem ultra- świeżakiem w programowaniu JS i mam taki mały problem.
Kwalifikacja E-14. Tworzenie aplikacji internetowych, 2014 str. 104 - 105

jest tam przykład, wklejam poniżej kod zadania, czy to jest poprawnie napisane? Bo nie działa mi zmienianie obrazków i nie wiem, co w tym jest źle... Chodzi o to, by zmieniały się automatycznie obrazki. Oczywiście mam swoje obrazki i podaję dokładnie ich nazwy i rozszerzenie, są one w tym samym folderze, co index.html w którym mam ten kod. Sprawdzone po kilkanaście razy wszystkie linie kodu i nie ma tam literówek, linia po linii identycznie jak w książce, tylko inne nazwy obrazków, więc proszę na ewentualne literówki nie zwracać uwagi, tylko czy to jest logicznie ok? A jeśli nie, to dlaczego? (w tym poniższym mogą jakieś literówki być, bo przepisuję go na szybko z książki, swoje zadanie mam na innym komputerze, do którego teraz nie mam dostępu dlatego nie przeklejam bezpośrednio)

<style type = "text/css">
#baner{
width: 300px;
height: 180px;
}
</style>

<script type = "text/javascript>
window.onLoad=Zmiana;
var Nr = 0;
function Zmiana(){
var obrazy = new Array ('z1.jpg', 'z2.jpg', 'z3.jpg);
Nr++;

if(Nr == obrazy.length){
Nr = 0;
}

document.getElementById('baner').src = obrazy[Nr];
setTimeout(Zmiana, 2000);
}

</script>

</head>

<body>
<div>
<img src = "z1.jpg" id ="baner" alt ="Reklama">
</div>
</body>

1	komentarz 18 lipca 2016 przez ScriptyChris Mędrzec (190,190 p.) Kod umieszczaj w specjalnych bloczkach. Poprawiłem troszkę Twój skrypt, aby działał - traktuj jako przykład/wskazówkę: http://codepen.io/ChrissP92/pen/EyQvxv