Problem z Javascriptem

pytanie zadane 18 grudnia 2015 w JavaScript przez gab123 Mądrala (5,230 p.)

Mam problem z javascriptem wszystkie kody w nim napisanie nie sa wykonywane na stronie (mimo ze HTML i CSS działa) zrobiłam już 3 programy ale żaden się nie wyswietla domyślam się ze to mój blad a nie komputera ale nie wiem na czym polega wkleje jeden z kodow będę wdzieczna za pomoc.

<script type = "text/javascript">
var licznik = 1;

function zmienslajd () {

licznik ++ ; if(licznik>4) licznik = 1;
switch(licznik){
case 1:
document.getElementById("slider").innerHTML ="z1.jpg" ;
break ;
case 2 :
document.getElementById("slider").innerHTML="z2.jpg" ;
break ;
case 3:
document.getElementById("slider").innerHTML="z3.jpg" ;
break ;
case 4:
document.getElementById("slider").innerHTML="z4.jpg" ;
break ;
default:
document.getElementById("slider").innerHTML="z5.jpg" ;
}

setTimeout("zmienslajd()"5000) ;
}
</script>

<body onload = "zmianslajd();">

Dziekuje z góry za pomoc

3 odpowiedzi

odpowiedź 18 grudnia 2015 przez ArturS Początkujący (260 p.)

spróbuj odpalić to firefoxie, mi na chromie skrypty się nie odpalały, a poza tym chyba masz niepotrzebny średnik przy nazwie funkcji. Spróbuj tak:

<body onload = "zmianslajd()">

komentarz 18 grudnia 2015 przez gab123 Mądrala (5,230 p.)

sprawdzałam na wszystkich przeglądarkach jakie mam na komputerze (firefox, chrome, IE) na zadnej scripty nie dzialaja ): Ale i tak dzięki za pomoc (:

odpowiedź 18 grudnia 2015 przez writen Nałogowiec (29,060 p.)
edycja 18 grudnia 2015 przez writen

przyjrzyj się dokładnie 24 linijce.

komentarz 18 grudnia 2015 przez gab123 Mądrala (5,230 p.)

rzeczywiście brakuje przecinka. Dziękuje. Kod co prawdza caly czas niestety nie działa.

komentarz 18 grudnia 2015 przez writen Nałogowiec (29,060 p.)

ojejej.

Tak się to robi:

setTimeout(function() {
  	zmienslajd();
  },5000);

A licznik++; powinno być na końcu funkcji.

Wywal to: onload = "zmianslajd();"

Cały skrypt przenieś tuż przed </body>, a najlepiej umieść go w osobnym pliku js.

I wywołaj po prostu funkcję zmienslajd();

komentarz 18 grudnia 2015 przez gab123 Mądrala (5,230 p.)

Dziekuje ale caly czas nie dziala

odpowiedź 18 grudnia 2015 przez ShiroUmizake Nałogowiec (46,300 p.)

Funkcja setTimeout , wykona Ci tylko raz. Jeżeli chcesz, ciągłośc działanie funkcji, użyj funkcji setInterval. Czyli : (globalna) var id (lokalnie czyli w funkcji) id= setInterval (funckja, czas). Break używamy tylko do pętli. Jeżeli chcesz przerwać wykonanie switcha użyj np:return.

komentarz 18 grudnia 2015 przez writen Nałogowiec (29,060 p.)

Takie bzdury popisałeś, że głowa boli. W kodzie autora pytania funkcja zmienslajd będzie wykonywana rekurencyjnie co 5 sekund. Oczywiście gdyby nie było błędów.

return służy do zwracania wartości przez funkcję. Do zakończenia instrukcji warunkowych służy słówko break.

Za używanie zmiennych globalnych powinna być publiczna chłosta.

komentarz 18 grudnia 2015 przez gab123 Mądrala (5,230 p.)

Dziekuje, ale scripty caly czas się nie wykonują.

komentarz 18 grudnia 2015 przez ShiroUmizake Nałogowiec (46,300 p.)
edycja 18 grudnia 2015 przez ShiroUmizake

Teraz spojrzałem i brakuje przecinka w setTmeout :).

SetTimeout wykona się raz. Czyli załaduje stronę, wykona funkcje setTimeout i wyświetli.się pierwszy obrazek i zakończy funckję. No chyba, że ja żle zrozumiałem setTimeout. Jeżeli tak, proszę mnie wyprowadzić z błędu.
Technicznie sprawdziłem u siebie na switchu , który jest uruchamiany na onload. I wykonał się tylko raz.

https://developer.mozilla.org/pl/docs/Web/API/Window/setTimeout