JS - problem z timerem

Question

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());
        });

 

Comments

document.getElementsByClassName zwraca obiekt podobny do tablicy.

Czego nasłuchuje twoja funkcja elm.addEventListener(clock());

---

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());

---

Dlaczego poprostu nie odpalisz funkcjii clock()

Już przecież użyłeś DOMContentLoaded

---

Możesz sprecyzować?

---

@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

---

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

jest niepotrzebne. Wystarczy clock();

A DOMContentLoaded zostaw.

---

Gdy zostawiam DOMContentLoaded  wszystko się wykrzacza.

---

@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();
});

 

Answer 1

obstawiam że masz js przed body a element clock wtedy nie istanieje

Comments

Tak, ale przed odpaleniem skryptu czekam na załadowanie drzewa DOM, po za tym, nawet gdy przenoszę skrypt za body, to problem występuje.