Kalendarz w JS - błąd przy pobieraniu daty.

Question

Witam. Jakiś czas temu napisałem zegarek+kalendarz z 1 odcinka kursu JavaScript, wszystko działało. Wczoraj zrobiłem to ponownie i o ile z zegarkiem nie ma problemu, to kalendarz wyświetla datę sprzed 2 tygodni. Dzisiaj go lekko poprawiłem i teraz wyświetla datę z przyszłości (+- 2 tygodnie). Macie jakieś pomysły?

https://prnt.sc/szyk0z <- wynik

function timer(){
    let todoo = new Date();
    
    let day = todoo.getDay();
    let dayN = todoo.getDay();
    let month = todoo.getMonth()+1;
    let year = todoo.getFullYear();
    
    let hours = todoo.getHours();
    if(hours<10) hours="0"+hours;
    let minutes = todoo.getMinutes();
    if(minutes<10) minutes="0"+minutes;
    let seconds = todoo.getSeconds()
    if(seconds<10) seconds="0"+seconds;
    
    let days = new Array("niedziela","poniedziałek","wtorek","środa","czwartek","piątek","sobota");
    let months = new Array("stycznia","lutego","marca","kwietnia","maja","czerwca","lipca","sierpnia","września","października",
    "listopada","grudnia");
    
    let showDate = "Dziś jest "+days[dayN]+", "+day+" "+months[month]+" "+year+" roku. <br /> Godzina "+hours+":"+minutes+":"+seconds;    
    
    document.getElementById("watch").innerHTML = showDate;
    
    setTimeout("timer()", 1000)
}

 

Answer 1

Tablice są numerowane od 0, więc tak po prawdzie niepotrzebnie dodajesz do miesiąca 1. A zły dzień jest dlatego, że dwa razy używasz getDay, podczas gdy do pobrania dnia w miesiącu powinieneś użyć getDate. Swoją drogą można to spokojnie zastąpić przez wykorzystanie Intl.

BTW przyjęło się, że tablice tworzymy przy pomocy literałów, nie konstruktorów:

["niedziela","poniedziałek","wtorek","środa","czwartek","piątek","sobota"];