$.each instead of for loop in jquery

Question

I want to append li five times in body using each function. But it is not working. It should append five li in body Note: It is just a sample code

$(function(){
    var lgt= 5;
    $.each(lgt,function(i){ 
        $('body').append('<li>'+i+'</li>')
    })
})

<li> can't be the child of <body>. You need either <ul> or <ol>. — JJJ
– JJJ, Commented Sep 25, 2013 at 11:05
This is not what $.each has been designed for. $.each is working on "collections", i.e. lists of objects like jQuery objects, arrays etc. A number is not a list of objects ... — devnull69
– devnull69, Commented Sep 25, 2013 at 11:05

Gabriele Petrioli · Accepted Answer · 2013-09-25 11:08:59Z

4

The first question is Why not use a for loop ?

If you still need to go this way you can use

$(function(){
    var lgt= 5;
    $.each(new Array(lgt),function(i){ 
        $('body').append('<li>'+i+'</li>')
    });
});

but as others have mentioned you cannot add li elements directly to the body..

answered Sep 25, 2013 at 11:08

Gabriele Petrioli

197k35 gold badges275 silver badges331 bronze badges

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

Comments

Gaz Winter · Accepted Answer · 2013-09-25 11:05:52Z

0

I dont think what you are trying to do is valid.

It needs a collection to loop through, you are just passing it a single integer

answered Sep 25, 2013 at 11:05

Gaz Winter

2,9442 gold badges27 silver badges48 bronze badges

Raidri · Accepted Answer · 2013-09-25 11:17:12Z

0

$.each expects an array as first parameter, not a number.
You could use var lgt = new Array(5) instead of var lgt = 5.

answered Sep 25, 2013 at 11:05

Raidri

17.7k11 gold badges69 silver badges70 bronze badges

that's a comment on why it doesn't work but not a solution to the problem as such

The solution should be easy to get from here, or the questioner could ask for more advice if it is not clear. But I will add the solution.