Take jquery variable to javascript function

Question

I have the following javascript code:

// timer code
    var count=120;
    var pars = 0;
    var counter=setInterval(timer, 1000); // will  run it every 1 second
    var redirect="index.php";

    function timer(){
      if (count <= 0){
         window.location = redirect;
      } else{
          if(pars == 1){

          } else {
            count=count-1;
          }
      }
      document.getElementById("timer").innerHTML=":" + count ;
    }

which displays a timer in div#timer, and I need it to have a pause on some link click as follow:

$(".quad").click(function(){
    pars = 1;
});

As I am new to javascript, I think this problem is cause by global & private variables. Can you help me please?

You may want to look at using the JQuery widget factory, which manages a lot of this for you, as in jqueryui.com/widget and onallthingsweb.wordpress.com/2013/01/05/… — Brian Mains
– Brian Mains, Commented Jan 10, 2013 at 1:21
what's the actual problem that you are having? do you get an error (if so, what?), or is it just not working as you expect (what is it doing)? — GreyBeardedGeek
– GreyBeardedGeek, Commented Jan 10, 2013 at 1:23
The timer function is not pausing even if variable pars is not equal to 1 — housamz
– housamz, Commented Jan 10, 2013 at 1:24

Robin Manoli · Accepted Answer · 2013-01-10 01:28:24Z

2

Your code works for me. Make sure your .click event is registered after the document is loaded:

$(document).ready(
    $(".quad").click(function(){
        pars = 1;
    });
});

You should also do the redirect using

window.location.href = redirect;

answered Jan 10, 2013 at 1:28

Robin Manoli

2,2222 gold badges27 silver badges31 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Nick Over a year ago

The onload event is the key :)

housamz Over a year ago

Thanks dude, that gave me the solution :)

Nick Over a year ago

Ribot - Dude! You're website's a trip!!

Nick · Accepted Answer · 2013-01-10 01:36:01Z

1

It's running OK here, too. This assumes the onload function refered to by others.

It's nicer to use:

      if(pars != 1){
        count=count-1;
      }

edited Jan 10, 2013 at 1:36

answered Jan 10, 2013 at 1:30

Nick

6,04512 gold badges56 silver badges81 bronze badges

Comments

bfavaretto · Accepted Answer · 2013-01-10 01:32:04Z

0

Make sure both code snippets are working in the same scope. If you have your click handler code inside of a document.ready callback, move the timer code to the callback too.

answered Jan 10, 2013 at 1:32

bfavaretto

72k18 gold badges118 silver badges162 bronze badges

Collectives™ on Stack Overflow

Take jquery variable to javascript function

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related