![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 260 Pomógł: 26 Dołączył: 26.01.2012 Skąd: Ci to przyszło do głowy? Ostrzeżenie: (0%) ![]() ![]() |
Cześć, chciałbym Was prosić o ocenę, wyłapanie błędów, i zasugerowanie co jeszcze można poprawić w mojej stronie, która jest moim portfolio i jednocześnie blogiem.
http://mrgod.pl |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 178 Pomógł: 49 Dołączył: 16.04.2012 Skąd: Bytom Ostrzeżenie: (0%) ![]() ![]() |
Zapomniałem jeszcze dodać, że mimo tak krótkiego kodu uznałbym to za dość dobry przykład złych praktyk (a konkretnie zwracanie różnych typów, uzależnianie funkcji/metody od parametrów z zewnątrz, zamiast przekazywania ich do metody).
No i można krócej, pousuwać parę enterów i np. nie deklarować zmiennej. Chciałem jednak zachować czytelność (IMG:style_emoticons/default/smile.gif) .
|
|
|
![]()
Post
#3
|
|
Grupa: Zarejestrowani Postów: 110 Pomógł: 6 Dołączył: 19.12.2010 Skąd: Krzyżanowice Ostrzeżenie: (0%) ![]() ![]() |
przykład złych praktyk Ja przyspawałem się do myśli, patrząc na ten obrazek, że obiekt nie "modyfikuje się sam", a czeka na modyfikację z zewnątrz. Przy innym założeniu (jak np. w Twoim kodzie) wystarczy tak: if (sad() === true) { //już jestem wesoły, nic mi tu nie trzeba z zewnątrz wciskać, pełna hermetyzacja } Niemniej jednak teza, że "to nigdy nie zadziała" jest zawsze ryzykowna i nie powinno się jej nigdy stawiać (IMG:style_emoticons/default/smile.gif) |
|
|
![]() ![]() |
![]() |
Aktualny czas: 17.10.2025 - 03:47 |