• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

JS - problem z timerem

VPS Starter Arubacloud
0 głosów
289 wizyt
pytanie zadane 8 maja 2018 w JavaScript przez TheFunny Gaduła (3,420 p.)
edycja 8 maja 2018 przez TheFunny

Witam,

mam problem z prostym timerem. Dostaję powiadomienie o błędzie:

ReferenceError: clock is not defined

Nie mam pomysłu co może być źle.

 document.addEventListener("DOMContentLoaded", function() {

            const elm = document.getElementsByClassName("clock");

            function clock() {
                
                let now = new Date();
                let hours = now.getHours();
                let min = now.getMinutes();

                if (hours < 10) hours = "0" + hours;
                if (min < 10) min = "0" + min;

                console.log(hours+":"+min);
                elm[0].innerHTML = hours + ":" + min;
                setTimeout("clock()", 1000);
            }

            elm[0].addEventListener("load", clock());
        });

 

1
komentarz 8 maja 2018 przez pablop76 VIP (123,460 p.)
document.getElementsByClassName zwraca obiekt podobny do tablicy.

Czego nasłuchuje twoja funkcja elm.addEventListener(clock());
komentarz 8 maja 2018 przez TheFunny Gaduła (3,420 p.)
Zauważyłem w czym tkwił problem z wywołaniem, ten kłopot z głowy.

Teraz ma to formę elm[0].addEventListener("load", clock());
1
komentarz 8 maja 2018 przez pablop76 VIP (123,460 p.)
Dlaczego poprostu nie odpalisz funkcjii clock()

Już przecież użyłeś DOMContentLoaded
komentarz 8 maja 2018 przez TheFunny Gaduła (3,420 p.)
Możesz sprecyzować?
komentarz 8 maja 2018 przez TheFunny Gaduła (3,420 p.)

@pablop76,

const elm = document.getElementsByClassName("clock");
   
            function clock() {
                   
                let now = new Date();
                let hours = now.getHours();
                let min = now.getMinutes();
   
                if (hours < 10) hours = "0" + hours;
                if (min < 10) min = "0" + min;
   
                console.log(hours+":"+min);
                elm[0].innerHTML = hours + ":" + min;
                setTimeout("clock()", 1000);
            }
   
               elm[0].addEventListener("load", clock());

Darowałem sobie ładowanie DOM, dałem na koniec body, teraz działa, dzięki za chęć pomocy

komentarz 8 maja 2018 przez pablop76 VIP (123,460 p.)
To elm[0].addEventListener("load", clock());

jest niepotrzebne. Wystarczy clock();

A DOMContentLoaded zostaw.
komentarz 8 maja 2018 przez TheFunny Gaduła (3,420 p.)
Gdy zostawiam DOMContentLoaded  wszystko się wykrzacza.
komentarz 9 maja 2018 przez pablop76 VIP (123,460 p.)

@TheFunny,

document.addEventListener("DOMContentLoaded", function() {

const elm = document.getElementsByClassName("clock");
function clock() {

let now = new Date();
let hours = now.getHours();
let min = now.getMinutes();

if (hours < 10) hours = "0" + hours;
if (min < 10) min = "0" + min;
elm[0].innerHTML ="godz: " + hours + ":" + min;
setTimeout(clock, 1000);
}
clock();
});

 

1 odpowiedź

0 głosów
odpowiedź 8 maja 2018 przez hoktaur Pasjonat (22,250 p.)
obstawiam że masz js przed body a element clock wtedy nie istanieje
komentarz 8 maja 2018 przez TheFunny Gaduła (3,420 p.)
Tak, ale przed odpaleniem skryptu czekam na załadowanie drzewa DOM, po za tym, nawet gdy przenoszę skrypt za body, to problem występuje.

Podobne pytania

0 głosów
1 odpowiedź 194 wizyt
pytanie zadane 14 października 2017 w JavaScript przez Adrian1999 Nałogowiec (34,570 p.)
0 głosów
0 odpowiedzi 224 wizyt
pytanie zadane 31 marca 2018 w JavaScript przez KrosFire Nowicjusz (210 p.)
0 głosów
1 odpowiedź 1,987 wizyt
pytanie zadane 3 czerwca 2017 w JavaScript przez Eryk Chojnacki Obywatel (1,420 p.)

93,078 zapytań

142,041 odpowiedzi

321,445 komentarzy

62,422 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj

Wprowadzenie do ITsec, tom 2

Można już zamawiać tom 2 książki "Wprowadzenie do bezpieczeństwa IT" - będzie to około 650 stron wiedzy o ITsec (17 rozdziałów, 14 autorów, kolorowy druk).

Planowana premiera: 30.09.2024, zaś planowana wysyłka nastąpi w drugim tygodniu października 2024.

Warto preorderować, tym bardziej, iż mamy dla Was kod: pasja (użyjcie go w koszyku), dzięki któremu uzyskamy dodatkowe 15% zniżki! Dziękujemy zaprzyjaźnionej ekipie Sekuraka za kod dla naszej Społeczności!

...