Slider OOP jquery - Nauka programowania i kodowania

5

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.

http://codepen.io/filem00n/pen/yNWNdy

0

Wygląda ładnie, choć jeśli to możliwe (a pewnie tak), to animacja etykiety mogła by zwalniać stopniowo - efekt byłby przyjemniejszy.

0

Bardzo fajny ;) Chciałbym umieć coś takiego tworzyć...

Można to użyć w swoim projekcie?

0

tak, ale to nie jest do końca poprawnie napisany projekt, nie bardzo też rozwojowy, ale jak uważasz ; P

1

popracowałbym trochę nad kodem JS.

if(this.centerArray[j].height > this.slide.height()){
			this.centerArray[j].height = this.slide.height();

Zamiast porobić jakieś zmienne i keszować wartości, to piszesz naiwnie to samo kilka razy. Ani to ładne pod kątem czytelności, ani zapewne pod kątem wydajności (jeśli np. funkcja height() wywołuje się 2 razy bez potrzeby).

To można przepisać na taką modłę mniej więcej:

var item = this.centerArray[j]; // dzieki temu tam gdzie piszemy this.centerArray[j] bedziemy mogli pisać po prostu 'item'
var slideHeight = this.slide.height(); 
if(item.height > slideHeight){
	item.height = slideHeight;
(...)
1

Ładny slajder ale w czasach "smartfonizacji" to popraw wyświetlanie przy małych ekranach

Widok przy 480
Przechwytywanie.PNG

0

Dzięki wszystkim za uwagi!
Postanowiłem jeszcze raz napisać slider, trochę inne działanie, ale dzięki temu wydaje mi się on bardziej przyjazny do edycji.
Zrezygnowałem z metod ułatwiających pracę leniom(zmniejszanie obrazka wewnątrz slidera do jego wielkości), również pozbyłem się animacji "wjeżdżania" tekstu na stronę, jest zwykła manipulacja opacity.

Jeden mankament tego slidera jest, że przy zmniejszaniu przeglądarki, slajdy się bugują, ale naprawia to odświeżenie, jak patrzy się z perspektywy urządzenia, a nie przeglądarki to takie coś się nie dzieje, więc chyba można przymknąć oko.

https://preview.c9.io/filem00n/responsive_slider_by_filem00n/

1 użytkowników online, w tym zalogowanych: 0, gości: 1