clearinterval is not working for the following code

Question

What's wrong with the code, I am unable to stop setInterval.

Error: changedHandler_BB is undefined.

...

var changedHandler_BB = setInterval(function() {

    $("#wc_room_container_BB").addClass('active_chat_block_blink_effect');

    setTimeout(function() {
        $("#wc_room_container_BB").removeClass('active_chat_block_blink_effect');
    }, 500);

}, 1000);

...

$("#wc_block_BB").click(function(){

    var newID = $("#LatestLastID_BB").val();

    $("#worldchat_last_id_BB").val(newID);

    clearInterval(changedHandler_BB);
});

Possibly because `changedHandler_BB` is not a global variable and has no meaning in the scope were `clearInterval` is being called. — huysentruitw, May 21 '13 at 12:00
Is this `var changedHandler_BB = setInterval(function()` inside another function ? — Adil Shaikh, May 21 '13 at 12:00
thanks for reply, I just copy this code from other website, can you pls help me, how to do fix it. — webfreaks, May 21 '13 at 12:01
I am new to this community...I was trying to post full code, but edition of this site is not acceptiong — webfreaks, May 21 '13 at 12:02
If you remove `var` from `changedHandler_BB` the variable will be global and you can access it from everyware. `var changedHandler_BB =` -> `changedHandler_BB =` — Andreas Louv, May 21 '13 at 12:03
http://stackoverflow.com/questions/500431/javascript-variable-scope — Ram, May 21 '13 at 12:08

score 0 · Answer 1 · answered May 21 '13 at 12:03

0

Just remove 'var':

changedHandler_BB = setInterval(function() {...}); //setting variable on global scope {window.changedHandler_BB }

answered May 21 '13 at 12:03

A. Wolff

74,033
9
94
155

thanks for reply, Now error has removed now but still clearInterval is not working – webfreaks May 21 '13 at 12:10
Yes, downvoting because that is dangerous advice. There are plenty of ways to create the variable in the correct scope without carelessly tossing it into the global scope. If nothing else, assign it directly to the `window` object, but don't just drop the `var` declaration. – Derek Henderson May 21 '13 at 12:13
I'm not the down-voter but remember, [Global variables are evil](http://yuiblog.com/blog/2006/06/01/global-domination/). – Beetroot-Beetroot May 21 '13 at 12:14
@Derek Henderson Judging by customized name of variable, i don't think it could be a problem here, anyway, you are correct, better is to use closure. – A. Wolff May 21 '13 at 12:18

score 0 · Accepted Answer · answered May 21 '13 at 12:05

As per the comment's above - you defined changedHandler_BB inside a function. You need to define that in outer scope so that it become accessible in another functions

try this -

var changedHandler_BB;

function someFunction() {
    changedHandler_BB = setInterval(function () {
        $("#wc_room_container_BB").addClass('active_chat_block_blink_effect');
        setTimeout(function () {
           $("#wc_room_container_BB").removeClass('active_chat_block_blink_effect');
        }, 500);
    }, 1000);
}
$("#wc_block_BB").click(function () {
    var newID = $("#LatestLastID_BB").val();
    $("#worldchat_last_id_BB").val(newID);
    clearInterval(changedHandler_BB);
});

oh Thanks alot, its working perfect now! thanks alot one more time :-) — webfreaks, May 21 '13 at 12:12

Derek Henderson · Answer 3 · 2013-05-21T12:23:54.273

0

changedHandler_BB is in another function. Thus, it is in that function's scope and not in scope accessible by clearInterval inside your click event handler. This is why it is undefined when you attempt to access it.

You need to make sure that either changedHandler_BB is in the same scope as your click event listener or can be otherwise accessed by it.

edited May 21 '13 at 12:23

answered May 21 '13 at 12:05

Derek Henderson

9,388
4
42
71

clearinterval is not working for the following code

3 Answers3