![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 5 Pomógł: 0 Dołączył: 16.08.2015 Ostrzeżenie: (0%) ![]() ![]() |
Witam.
Trochę poczytałem o obiektowym programowaniu i staram się stawiać pierwsze kroki, nie wiem czy robię dobrze i czy idę w dobrym kierunku więc mam do was prośbę o wskazanie mi moich błędów, przedstawiam wam mój projekt slidera. Slider Codepen |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 1 268 Pomógł: 254 Dołączył: 11.06.2009 Skąd: Świętochłowice Ostrzeżenie: (0%) ![]() ![]() |
Te animacje wydają mi się jakoś za bardzo skomplikowane. Osobiście bawiłbym się jedynie klasami elementów slidera, a całą resztę przerzucił na transition w CSS
Slider.prototype.goNext, Slider.prototype.goPrev i Slider.prototype.firstSlide (BTW rzeczowniki są dobrymi nazwami własności; metody powinny mieć "czasownikowe" nazwy) mają dużo tego samego kodu - warto to wydzielić do uniwersalnej metody Slider.prototype.changeSlide Strzałki w lewo i prawo zrobiłbym jako konfigurowalne części slidera (w końcu to de facto podstawowe elementy GUI). Nie rozumiem też czemu przed każdą zmianą slajdu jest wywoływane Slider.prototype.reset? I czemu ten reset jest średnio resetem? (IMG:style_emoticons/default/wink.gif) Środkowanie obrazka w pionie slidera na pewno da się załatwić lepiej niż przeliczać wszystko w JS (wyświetlanie tabelkowe, flexbox…) → http://howtocenterincss.com/ Osobiście przemyślałbym oparcie tego o jQuery → http://ponyfoo.com/articles/why-i-write-pl...ascript-modules |
|
|
![]() ![]() |
![]() |
Aktualny czas: 7.10.2025 - 09:23 |