[javascript] Zamiana cyfr z funkcji getDay() na napisy

Question

Witam, robię sobie zegar w js i próbuję zmienić wartość zmiennej 

var weekday = getDay();

czyli dzień tygodnia,  z cyfry na napis i robię to za pomocą tego kodu

if (weekday == 0) {
 weekday = 'Poniedziałek';
} else if (weekday == 1) {
 weekday = 'Wtorek';
} else if (weekday == 2) {
 weekday = 'Środa';
} else if (weekday == 3) {
 weekday = 'Czwartek';
} else if (weekday == 4) {
 weekday = 'Piątek';
} else if (weekday == 5) {
 weekday = 'Sobota';
} else 
 weekday = 'Niedziela'; }

Cały kod skryptu zegara tutaj:

var date = new Date();

var weekday = date.getDay(); 
var day = date.getDate();
var month = date.getMonth();
var year = date.getFullYear();
var hour = date.getHours();
var minutes = date.getMinutes();
var seconds = date.getSeconds();

if (weekday == 0) {
 weekday = 'Poniedziałek';
} else if (weekday == 1) {
 weekday = 'Wtorek';
} else if (weekday == 2) {
 weekday = 'Środa';
} else if (weekday == 3) {
 weekday = 'Czwartek';
} else if (weekday == 4) {
 weekday = 'Piątek';
} else if (weekday == 5) {
 weekday = 'Sobota';
} else 
 weekday = 'Niedziela'; }

No i niestety przy realizacji tegoż skryptu w htmlu- nie działa (nie wyświetla się) żadna z powyższych zmiennych wyodrębnionych od getDate();. Problem naprawia się, gdy usuwam ten kod z instrukcjami warunkowymi (ale wtedy oczywiście przy wypisywaniu zmiennej weekday na stronie, musiałbym zadowolić się cyfrą). Moje pytanie brzmi, co jest nie tak (to mój pierwszy kod w js i zdaję sobie sprawę że błąd mogłem popełnić w czymkolwiek, choćby głupią literówkę a i tak miałbym problemy ze znalezieniem go).

Pozdrawiam.

Answer 1

Po pierwsze to nie masz jednej klamry otwartej. Ostatnie dwie linijki kodu powinny wyglądać tak:

} else {
 weekday = 'Niedziela'; }

Po drugie to dni zaczynają się od niedzieli. Czyli 0 to niedziela, 1 to poniedziałek itd.

A po trzecie to te ify dość kiepsko wyglądają do czegoś tak prostego. O tablicach słyszał?

Ja bym proponował władować dni tygodnia w tablice:

var weekdays=['niedziela', 'poniedziałek', 'wtorek', 'środa', 'czwartek', 'piątek', 'sobota'];

A później żeby pobrać jaki dzień jest wystarczy wstawić numer dnia jako numer indeksu (czyli w Twoim przypadku zmienną weekday), co spowoduje zwrócenie odpowiedniego dnia tygodnia

weekdays[weekday]

Myślę że prościej i ładniej ale rozumiem że dopiero się uczysz dlatego daję sugestię że coś takiego można zrobić inaczej ;)

Comments

Oczywiście, słyszałem o tablicach, jednak nie miałem okazji zastosować ich w praktyce w js, dlatego nie wpadłem na to rozwiązanie. Też mi się nie podobały te if'y i czułem że nadpisuję niepotrzebnie kod.

Dziękuję i pozdrawiam.

---

No rozumiem, ale nie przejmuj się z czasem będziesz samemu wpadał na coraz prostsze rozwiązania, to po prostu doświadczenie.

Nie ma za co ;)

Answer 2

Cześć, co do kodu nie podpowiem ci bo JS nie znam zbyt dobrze. Moge jedynie zaproponować inny algorytm. Mozna stworzyć tablice napisów i w niej umieścić nazwy dni tygodnia pod odpowiednimi indeksami tablicy. Zaoszczędzi to pare linijek, a i moze zadziała ? ;D

 

Pozdrawiam :)

Answer 3

http://tutorials.comandeer.pl/js-intl.html

Comments

No to tego nie znałem... Dzięki, znów się czegoś nowego od Ciebie nauczę ;)