在jQuery轮播中重置计时器

Question

I have a carousel that works fine, the only problem i have is when I click the controls the timer doesn't stop. I have a loop that is inside the timer that changes the image. When I click on of the controls I can change my current active carousel image but the timer is not reset.

Here is my jQuery:

$(document).ready(function () {
    var $slider = $('.slider');
    var $slide = 'li';
    var $transition_time = 2000;
    var $time_between_slides = 5000;
    var $btn = $('.buttons');
    var $buttonanchor = 'li';
    function slides() {
        return $slider.find($slide);
    }
    slides().fadeOut();
    slides().first().addClass('active');
    slides().first().fadeIn($transition_time);
    function button() {
        return $btn.find($buttonanchor);
    }
    button().first().addClass('btnactive');
    $interval = setInterval(
    function () {
        var $i = $slider.find($slide + '.active').index();
        slides().eq($i).removeClass('active');
        button().eq($i).removeClass('btnactive');
        button().eq($i).addClass('buttonsnotactive');
        slides().eq($i).fadeOut($transition_time);
        if (slides().length == $i + 1) $i = -1; // loop to start
        slides().eq($i + 1).fadeIn($transition_time);
        slides().eq($i + 1).addClass('active');
        button().eq($i + 1).addClass('btnactive');
    }, $transition_time + $time_between_slides);

    $('.buttons li').click(function (event) {
        var $i = $(this).index();
        $('.buttons li').removeClass('btnactive');
        $(this).addClass('btnactive');
        $('.slider li').fadeOut($transition_time);
        $('.slider li').removeClass('active');
        $('.slider li').eq($i).fadeIn($transition_time);
        $('.slider li').eq($i).addClass('active');
        event.preventDefault();

    });
});

And here is my jsFiddle for more information.

Answer 1

1) Define your $interval var first before using it. 2) Clear any previous intervals before you set one. 3) Clear interval when the controls are being clicked on.

// ....
// .... 
button().first().addClass('btnactive');
// -- Added --
var $interval = 0;
clearInterval($interval);
// -- ; --
$interval = setInterval(
// .... 

Within the click function

$('.buttons li').click(function (event) {
    // Added
    clearInterval($interval);
    // --; --

This is how you reset the timer. But, you may want an option, such as a play button, to reactivate the timer in case you'd like to init auto play again. Hence, it'd be best to move your code from within interval to a function and call it as and when you need it, like below.

// ...
// ...
button().first().addClass('btnactive');
var slideMe = function() {
    var $i = $slider.find($slide + '.active').index();
    slides().eq($i).removeClass('active');
    button().eq($i).removeClass('btnactive');
    button().eq($i).addClass('buttonsnotactive');
    slides().eq($i).fadeOut($transition_time);
    if (slides().length == $i + 1) $i = -1; // loop to start
    slides().eq($i + 1).fadeIn($transition_time);
    slides().eq($i + 1).addClass('active');
    button().eq($i + 1).addClass('btnactive');
};
// -- Added --
var $interval = 0;
clearInterval($interval);
// -- ; --
$interval = setInterval(slideMe, $transition_time + $time_between_slides);
// ...
// ...

Demo@ Fiddle

So, here is a dirty version with the play button, which would reactivate auto play when clicked. Take a look at this fiddle for the code/demo.