I've looked at other questions, but none of them have really helped me. I want to give IDs to a number of elements belonging to the same class without manually doing it. Here is my code that isn't working:
$curelem = $(".item:first");
for (var $i=0; i < $(".item").length; ++$i){
$curelem.attr("id", "item" + $i);
$curelem = $curelem.next('a');
}
Is it some small syntax error, or am I going about it entirely wrong?
If you're using jQuery you can use jQuery's each() function:
var i = 0;
$('.item').each(function(){
$(this).attr('id', 'item' + i);
i++;
});
this.id = 'item' + i; for one. Though I'm afraid the underlying issue here is much bigger. Sequential numeric IDs are the indication of bad design - usually they should be an array instead..eq with an index or better yet - use arrays. Selectors (jquery and others) are only useful at the initialization phase of the application. Once you have everything in place there is no reason to query your own presentation. OP can assign the elements of the class into an array.getElementById anyway. You can use $("#"+id) - but like I said - this is the XY problem - you can just keep the selection and then access it like an array: var items = $(".item") and then access elements through var cur = items[2]; //the third item which has the same type as what you get from getElementById anyway.
$('.item').each(function (i, el) {
el.id = "item" + i;
});
$('.item').each(function(i) {
this.setAttribute('id', "item" + i);
});
Use http://api.jquery.com/jquery.each/ to loop through elements:
$(".item").each(function( index, value ) {
$(this).attr("id", "item"+index);
});
as? Why not use an explicit binding in your code? Alternatively, a worse but simpler and still better solution, you can store adata-urlattribute on youratags that contains a selector for thedivelements you want to open.