Wstrzymywanie czasu w zegarku

Cześć. Jestem kompletną świeżynką jeśli chodzi o JS, ale na początku sobie założyłem, że zrobię projekt zegarka z wyświetlaniem aktualnej godziny i daty z możliwością spauzowania odliczania czasu. Sam zegarek działa, ale niestety nie działa funkcja wstrzymywania tego czasu, po wciśnięciu w button, który za to ma odpowiadać. Przeszukiwałem różne źródła, ale niestety wszystkie się koncentrują na działaniu takiej funkcji w odniesieniu funkcji uruchamianie zegarka na żądanie, a nie jak u mnie (po załadowaniu strony).

let displayCurrentTimeAndDate = () => {

let currentTime = new Date()

let secundes = currentTime.getSeconds()
if (secundes<10) {
    secundes = '0' + secundes
}

let minutes = currentTime.getMinutes()
if (minutes<10) {
    minutes = '0' + minutes
}

let hours = currentTime.getHours()
if (hours<10) {
    hours = '0' + hours
}

let year = currentTime.getFullYear()

let mounth = currentTime.getMonth()
if (mounth<10) {
    mounth = '0' + (mounth + 1)
}

let day = currentTime.getUTCDate()
if (day<10) {
    day = '0' + day
}


document.querySelector('#clock').innerHTML = hours + ':' + minutes + ':' + secundes + ' |' + ' dnia ' + day + '-' + mounth + '-' + year

// setTimeout('displayCurrentTimeAndDate()',1000)

}

Window.onload = setInterval(displayCurrentTimeAndDate,1000)


displayCurrentTimeAndDate()


let timerId = setInterval(displayCurrentTimeAndDate(), 1000)


let stopStartTime = document.querySelector('#button')


stopStartTime.addEventListener('click',  
    clearInterval(timerId)

)

I właśnie nie działa ta część kodu:

stopStartTime.addEventListener('click',  
    clearInterval(timerId)

Proszę o jakieś sugestie, podpowiedzi.

Deklaracje timerID i stopStartTime też powinieneś umieścić w funkcji windows.onload (zwróć uwagę, że masz literówkę i napisałeś window.onload zamiast window.onload co też stwarza różnicę.

window.onload = () => {
    let timerId = setInterval(displayCurrentTimeAndDate, 1000)
    let stopStartTime = document.querySelector('#button')
    stopStartTime.addEventListener('click', () => clearInterval(timerId))
}

powyższe rozwiązanie nie jest jedyne, ale chodzi o to, że generalnie masz kilka błędów deklaracjami i kolejnością;

• przypisanie setInterval bezpośrednio do window.onload
• błędnie zadeklarowany timerid: powinien być przypisany do setInterval odnoszącego się do displayCurrentTimeAndDate zamiast do displayCurrentTimeAndDate()

Tak naprawdę kod próbuje dwa razy uruchomić setinterval, ale udaje mu się raz, zmienna czyszcząca próbuje zatrzymać ten źle zadeklarowany. To tak w skrócie :D

Wszystkie podobne fragmenty

let secundes = currentTime.getSeconds()
if (secundes<10) {
    secundes = '0' + secundes
}

można zapisać w jednej linijce

const seconds = currentTime.getSeconds().toString().padStart(2, 0);

dzięki temu nie musimy korzystać z let, tylko przypisujemy to od razu do stałej const.

document.querySelector('#clock').innerHTML = hours + ':' + minutes + ':' + secundes + ' |' + ' dnia ' + day + '-' + mounth + '-' + year

 document.querySelector('#clock').textContent = `${hours}:${minutes}:${secundes} | dnia ${day} ${mounth} ${year}`

Przy dodawaniu wartości zmiennych do HTMLa bezpieczniejsze będzie textContent, ponieważ nie pozwoli na wstawienie znacznika html bez naszej wiedzy.

Event onload czeka na załadowanie całej strony w tym wszystkich zdjęć, plików css / js, elementów iframe, video itd, więc lepszym wyborem mogłoby być DOMContentLoaded.

window.addEventListener('DOMContentLoaded', () => {
  let timerId = setInterval(...) // uruchomienie zegarka po załadowaniu strony

  // ...

  // wyłączenie zegarka po kliknięciu w przycisk
  stopTimeButton.addEventListener('click', () => {
    clearInterval(timerId); 
  });

  // ponowne włączenie zegarka po kliknięciu w drugi przycisk
  startTimeButton.addEventListener('click', () => {
    timerId = setInterval(...);
  });
});

No i robisz literówki

let secundes = currentTime.getSeconds()

zobacz, że tam masz secundes i nie jest to to samo, co po drugiej stronie getSeconds, czyli poprawne słowo to seconds .

let mounth = currentTime.getMonth()

albo tutaj ^ Masz mounth, a przecież po prawej stronie masz month (więc nawet jak nie znasz tego słowa po angielsku, to możesz się przyjrzeć, jak ono jest zapisane).

setInterval(displayCurrentTimeAndDate(), 1000)

Z tym jest problem, że setInterval przyjmuje funkcję jako pierwszy argument. A jeśli masz funkcję w JS i chcesz ją przekazać, to dajesz samą nazwę funkcji (np. foo dla funkcji foo). Ponieważ funkcje to są takie obiekty, które można przekazywać. Jeśli dajesz nawiasy po nazwie funkcji, to ona się od razu odpali (np. foo()). Więc podając displayCurrentTimeAndDate() tak naprawdę odpalasz w miejscu funkcję displayCurrentTimeAndDate i przekazujesz tylko wynik funkcji. Czyli wywal nawiasy.

Dzięki kolegom za te cenne wskazówki. Teraz mam jednak inny problem. Pauza zegarka zaczęła działać, ale działa tylko raz. Prawdopodobnie, jak dobrze rozumiem, trzeba zrobić funkcję, która będzie zapamiętywała ostatni stan zegarka. Bo zapomniałem wcześniej dodać, że u mnie uruchamianie/pauzowanie obsłużone ma być na jednym divie, a nie na dwóch. Odpowiada mu selektor #button.
Obecnie, po Waszych radach, tak wygląda kod:

let displayCurrentTimeAndDate = () => {


let currentTime = new Date()

let seconds = currentTime.getSeconds()
if (seconds<10) {
    seconds = '0' + seconds
}

let minutes = currentTime.getMinutes()
if (minutes<10) {
    minutes = '0' + minutes
}

let hours = currentTime.getHours()
if (hours<10) {
    hours = '0' + hours
}

let years = currentTime.getFullYear()

let months = currentTime.getMonth()
if (months<10) {
    months = '0' + (months + 1)
}

let days = currentTime.getUTCDate()
if (days<10) {
    days = '0' + days
}


document.querySelector('#clock').innerHTML = hours + ':' + minutes + ':' + seconds + ' |' + ' dnia ' + days + '-' + months + '-' + years

// setTimeout('displayCurrentTimeAndDate()',1000)

}

// Window.onload = setInterval(displayCurrentTimeAndDate,1000)


displayCurrentTimeAndDate()


window.onload = () => {
    let timerId = setInterval(displayCurrentTimeAndDate, 1000)
    let stopStartTime = document.querySelector('#button')
    stopStartTime.addEventListener('click', () => clearInterval(timerId))
}

// let timerId = setInterval(displayCurrentTimeAndDate(), 1000)


// let stopStartTime = document.querySelector('#button')

// stopStartTime.addEventListener('click',  
//     clearInterval(timerId)

// )