jQuery - setInterval with $.each

Question

I am about to rip my hair of the head. I'm trying to interact with my PHP function, which is measuring some times: timeSince() a post is posted.

I do NOT need a plugin for doing this in jQuery, thanks :o)

But I wan't to run the $.get() on all .post-box-time div's so therefore I am using $.each. But it's only updating the last element, but giving me the result for the other divs to. -> It's like it's not doing the $.each()..

Here is my code:

setInterval(function() {
        $('.post-box-time').each(function() {
            var time = $(this).attr('data-time');
            $this = $(this);

            $.get("sys/calls.handler.php", { do: 'timeSince', data: time },
                function(data){
                    $this.html(data);
                });
        }); // each function
    }, 1000);

If you open the console, do you see multiple outgoing request? (the same amount as the quantity of .post-box-time? — VVV, Aug 23 '12 at 18:35
For me to see that is not my problem. It's working by putting var in front of $this... $(this) doesn't work at all. Nothing is returned — skolind, Aug 23 '12 at 18:39

Draculater · Accepted Answer · 2014-01-31T16:32:45.503

6

Well you should throw a var before declaring $this.

You were declaring it as a global variable as opposed to declaring it inside the scope (which is what var does) of each function being run by each().So when your response callback was actually being fired off, your loop had already completed and set global variable $this (or window.$this) to the value of the last .post-box-time it found while looping via each()

edited Jan 31 '14 at 16:32

answered Aug 23 '12 at 18:36

Draculater

2,280
1
24
29

2

@Kolind Without the `var`, `$this` is declared as a global rather than a local variable. http://stackoverflow.com/q/1470488/ – Jonathan Lonowski Aug 23 '12 at 18:41
@JonathanLonowski Awesome. Thanks a lot for this :) Amazing that 'var' can turn the whole function to work. I love programming :) – skolind Aug 23 '12 at 18:43
2

You were declaring it as a global variable as opposed to declaring it inside the scope (which is what `var` does) of each function being run by `each()`. So when your response callback was actually being fired off, your loop had already completed and set global variable $this (or `window.$this`) to the value of the last `.post-box-time` it found while looping via `each()`. – Draculater Aug 23 '12 at 18:46

score 2 · Answer 2 · edited May 23 '17 at 10:34

2

Use async option. This Thread is related with your question.

Hope this will help you.

edited May 23 '17 at 10:34

Community

1
1

answered Aug 23 '12 at 18:35

Kundan Singh Chouhan

13,952
4
27
32

score -1 · Answer 3 · answered Aug 23 '12 at 18:38

-1

I think this within the each function is a reference to the raw DOM element. The .each gives indexes starting from 0, so you should do $('.post-box-time:eq(' + i + ')') instead.

answered Aug 23 '12 at 18:38

user1330271

2,601
3
20
26

jQuery - setInterval with $.each

3 Answers3