![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 53 Pomógł: 0 Dołączył: 19.01.2013 Ostrzeżenie: (0%) ![]() ![]() |
Witam, załączam live demo razem z kodem i proszę o sugestie co powinienem poprawić aby kod ten był napisany właściwie i zgodnie z dzisiejszymi trendami. Jeśli wypadałoby napisać to obiektowo prosiłbym o wskazanie mi jakie klasy powinienem stworzyć, gdyż nie "czuję bluesa" w obiektówce.
http://jsfiddle.net/xj7vkkgo/ Aha, wiem że można wszystko zrobić w 3 linijakch z gatDay(); ale jest to interpretacja pewnego algorytmu... Ten post edytował Matt23 23.01.2015, 19:18:59 |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 655 Pomógł: 73 Dołączył: 2.05.2014 Ostrzeżenie: (0%) ![]() ![]() |
Cytat co to znaczy monthNum? Ta tablica z numerami jest niejasna, przydałby się jakiś komentarz. Komentarze to zło (IMG:style_emoticons/default/wink.gif) można to załątwić zmieniając nazwę tablicy lub poprzez refactoring całkowity, który się tu przyda.
monthNum - odpowiednie wartości dla miesięcy.. spoko (IMG:style_emoticons/default/wink.gif) Raz masz 'month_spell' a raz 'monthNum' - brak konsekwencji w nazewnictwie. Człowiek ma bardzo potężne umiejętności nazewnictwa czegokolwiek. Korzystajmy z tego (IMG:style_emoticons/default/smile.gif) Nie bójmy się nazywać zmiennie dłuższe jak 1-2 wyrazy. Cytat Jeśli wypadałoby napisać to obiektowo prosiłbym o wskazanie mi jakie klasy powinienem stworzyć, gdyż nie "czuję bluesa" w obiektówce. Zaskoczę Cię. Javascript nie jest językiem obiektowym (IMG:style_emoticons/default/wink.gif) - zaraz polecą na mnie pewnie fale krytyki. edit: albo inaczej.. javascript nie musi być pisany "obiektowo" przy takich skryptach. Ten post edytował Forti 24.01.2015, 07:00:42 |
|
|
![]() ![]() |
![]() |
Aktualny czas: 17.10.2025 - 16:24 |