Home/ Questions/Q 835783
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-15T04:52:40+00:00

Some template written using jquery is as follows . it is not working .

  • 0

Some template written using jquery is as follows . it is not working . Any suggestions to use jquery efficiently .

<html>  
<head>  
<script type="text/javascript" src="/jquerycall/"></script>  
<script type="text/javascript">  
    $(document).ready(function()  
    {  
        self.setInterval("clock()",1000);  
        $("button").click(function()  
        {  
            clock;  
        });  
        function clock()  
        {  
            clock();  
            time=new Date();  
            var s = "<p>" + time + "</p>";  
            $(s).appendTo("div");  
        }
    });  
</script>  
</head>  
<body>  
<form method="post">  
<button type="button">Click Me</button>  
<div id="someid"></div>  
</form>  
</body>  
</html>
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team
    function clock()   
{   
   //clock(); /* Remove this recursive call or you'll just get an infinite loop */
   var current = new Date(); /* Make sure to use var on variables */
   var s = "<p>" + current + "</p>";
   $('#someId').html(s); /* Assuming you want to set 'someId' to 's', not try and append all things that match s to every div */
} 

var clockStarted = false;
$("button").click(function()   
{   
   if (!clockStarted) /* You only need to set the clock to start once */
      setInterval(clock, 1000);  /* You only want the interval to start after you've clicked the button and you don't need 'self'. */
   clockStarted = true;

   //clock(); /* Said 'clock;' before */   
});

    To summarise:

    1. When you call a function, you need to call it with parentheses and optionally parameters (e.g. functionName(); instead of functionName). This applies to all JavaScript.

    2. Imagine there is one little gnome happily running through your code. When he gets to a line he reads it and does it. So let’s say you’ve called clock(). He gets there and goes “what now? Oh, I have to go to clock()”. So he goes to clock(). He gets there and goes “what now?” Oh, I have to go to clock()”. See the pattern? He’s going to keep doing this forever, but he never actually gets a chance to do the things that you want clock() to do, because the first thing he sees as soon as he reaches clock() is that he needs to go to clock(). This is called infinite recursion. What you really wanted was to say “every so often I want a little gnome to go to clock() and do the things in there” which is what setInterval does.

    3. $(s).appendTo(‘div’) selects all things that match ‘s’ (which will be nothing) and appends them to all things that match ‘div’ (which is every div). Instead, you want to select the div with ‘someId’: “$(‘#someId’)”, and append ‘s’ to that: “.append(s);”. If you don’t want to append but instead replace, you will want to use “.html” not “.append”.

    Edits

    • Includes edits as per request.
    • Includes the fixes to ‘setInterval’ and ‘var time’ which slipped past me as per Anurag’s answer. I’m not trying to take credit for these fixes, but just wanted to include as part of a comprehensive answer given the other changes to meet the OP’s comment.
    • 0