• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Ocena Kodu Javascript, Jquery - Prosty slider

0 głosów
173 wizyt
pytanie zadane 3 marca 2018 w JavaScript, jQuery, AJAX przez Radekol Bywalec (2,900 p.)
$(document).ready(function () {
    
    //variables
    var slides = $('figure.slide');
    var dots = $('button.dot');
    var position = 0;
    var timer = 0;
    var current = 0;
    var flag = false;
    
    //setting initial setup (slide, image dot)
    slides.eq(position).css('display','block');
    dots.eq(position).addClass('checked');
    changeSlide(1);
    
    //function to change a slide
    function changeSlide(duration) {
        
        if (flag) {slides.eq(current).fadeOut(1000); flag = false;}
        else slides.eq(position).fadeOut(1000);
        dots.removeClass('checked');
        
        if (duration === 1) {
            position++;
            if (position > 3) position = 0;
        }
        else if (duration === 0) {
            position--; 
            if (position < 0) position = 3;
        }
        
        slides.eq(position).fadeIn(1000);
        dots.eq(position).addClass('checked');
        
        timer = setTimeout(function(){changeSlide(1)}, 4000);
        
    }
    
    //changing silde by clicking on the button
    $('button').click(function () {
        
        clearTimeout(timer);
        
        //side buttons (left or right)
        if ($(this).hasClass('side')) {
            
           if ($(this).hasClass('left')) { changeSlide(0); } 
           else { changeSlide(1); } 
            
        }
        //dots buttons
        else if ($(this).hasClass('dot')) {
            
            dots.eq(position).removeClass('checked');
            $(this).addClass('checked');

            //to fade out current slide because of change in position
            flag = true;
            current = position;
            
            //initializing dot to proper slide
            if (dots.eq(0).hasClass('checked')) {position = 3; changeSlide(1);}
            else if (dots.eq(1).hasClass('checked')) {position = 0; changeSlide(1);}
            else if (dots.eq(2).hasClass('checked')) {position = 1; changeSlide(1);}
            else if (dots.eq(3).hasClass('checked')) {position = 2; changeSlide(1);}
            
        }
        
    });
});

Ocenicie mi powyższy kod. Jakie błędy tutaj popełniam. Co bym mógł zrobić prościej i żeby to działało bardziej efektywnie. Nie oszczędzajcie mnie, najlepiej wypiszcie wszystko co zauważacie nie tak.

komentarz 3 marca 2018 przez Mariusz08 Nałogowiec (48,580 p.)

Zamiast var let lub const

1 odpowiedź

0 głosów
odpowiedź 4 marca 2018 przez MarcelWP Nowicjusz (180 p.)

Ja jako niedoświadczony programista mogę powiedzieć że możesz wkleić to w konsolę JS(about:blank +klikasz CTRL+ SHIFT +I)

Lub też zobaczyć w programie Sublime Text, Visual Studio Code itp.yes

Podobne pytania

+4 głosów
2 odpowiedzi 180 wizyt
+2 głosów
1 odpowiedź 122 wizyt
pytanie zadane 4 kwietnia 2016 w JavaScript, jQuery, AJAX przez Bartos Użytkownik (620 p.)
+2 głosów
1 odpowiedź 145 wizyt
Porady nie od parady
Nie wiesz jak poprawnie zredagować pytanie lub pragniesz poznać którąś z funkcji forum? Odwiedź podstronę Pomoc (FAQ) dostępną w menu pod ikoną apteczki.FAQ

65,738 zapytań

112,379 odpowiedzi

237,252 komentarzy

46,695 pasjonatów

Przeglądających: 241
Pasjonatów: 11 Gości: 230

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...