Javascript clearInterval和setInterval（）行为与预期不符

Question

var timer = 0
var startInterval = function( value ) {
    timer = setInterval( "checkNewPost();", value );
}
var stopInterval = function() {
    clearInterval( timer );
}

jQuery("#centerColumn a").click(function() {
    var a_id = jQuery(this).attr("id");
    var splitValue = a_id.split("-");
    var newValue = splitValue[1];

    if( newValue == "30" ) { 
        stopInterval;
        startInterval( 10000 );
    }
    else if( newValue == "1" ) {
        stopInterval;
        startInterval( 20000 );
    }
    else if( newValue == "5" ) {
        stopInterval;
        startInterval( 30000 );
    }
    else if( newValue == "pause" )
        stopInterval;
});

As you can see in my code, the logic is pretty straight forward, when newvalue is equal to 30 it will stop the current interval and restart it with a 10000 seconds on the setInterval. And when newValue is equal to pause, it will stop all the setInterval.

The problem here is it does not act correctly, i'm not sure why? Can someone guide me on this. Your help would be greatly appreciated!

Thanks! :)

Answer 1

you need to call the stopInterval function

stopInterval();

I think it won't work without the parentheses

Answer 2

Replace

 stopInterval; // fonction simply put on stack

with

 stopInterval(); // fonction call

Answer 3

Your calls to stopInterval are missing the parentheses after them so you're currently not actually calling that method.

Try using stopInterval();

Answer 4

Everybody else is right, you should use stopInterval() . Also here's a more compact and IMO more readable version of your code:

$('#centerColumn a').click(function () {
    var id = this.id.split('-')[1];
    var value = {
        30: 10000,
        1: 20000,
        5: 30000
    };
    id ==== 'pause' && stopInterval();
    if (value[id]) {
        stopInterval();
        startInterval(value[id]);
    }
});